From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932069AbaFBUJO (ORCPT ); Mon, 2 Jun 2014 16:09:14 -0400 Received: from mail-ig0-f179.google.com ([209.85.213.179]:42574 "EHLO mail-ig0-f179.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751828AbaFBUJM (ORCPT ); Mon, 2 Jun 2014 16:09:12 -0400 Date: Mon, 2 Jun 2014 13:09:08 -0700 (PDT) From: David Rientjes X-X-Sender: rientjes@chino.kir.corp.google.com To: Vlastimil Babka cc: Fabio Estevam , Andrew Morton , Shawn Guo , Kevin Hilman , Rik van Riel , Stephen Warren , Minchan Kim , Bartlomiej Zolnierkiewicz , Hugh Dickins , LKML , Michal Nazarewicz , linux-mm@kvack.org, Mel Gorman , Olof Johansson , Greg Thelen , Joonsoo Kim , Christoph Lameter , linux-arm-kernel Subject: Re: [PATCH -mm] mm, compaction: properly signal and act upon lock and need_sched() contention - fix In-Reply-To: <538C8B45.6070803@suse.cz> Message-ID: References: <1399904111-23520-1-git-send-email-vbabka@suse.cz> <1400233673-11477-1-git-send-email-vbabka@suse.cz> <537F082F.50501@suse.cz> <538C8B45.6070803@suse.cz> User-Agent: Alpine 2.02 (DEB 1266 2009-07-14) MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, 2 Jun 2014, Vlastimil Babka wrote: > compact_should_abort() returns true instead of false and vice versa > due to changes between v1 and v2 of the patch. This makes both async > and sync compaction abort with high probability, and has been reported > to cause e.g. soft lockups on some ARM boards, or drivers calling > dma_alloc_coherent() fail to probe with CMA enabled on different boards. > > This patch fixes the return value to match comments and callers expecations. > > Reported-and-tested-by: Kevin Hilman > Reported-and-tested-by: Shawn Guo > Tested-by: Stephen Warren > Signed-off-by: Vlastimil Babka Acked-by: David Rientjes From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-ig0-f178.google.com (mail-ig0-f178.google.com [209.85.213.178]) by kanga.kvack.org (Postfix) with ESMTP id 5CADD6B0036 for ; Mon, 2 Jun 2014 16:09:12 -0400 (EDT) Received: by mail-ig0-f178.google.com with SMTP id hl10so3889388igb.5 for ; Mon, 02 Jun 2014 13:09:12 -0700 (PDT) Received: from mail-ig0-x235.google.com (mail-ig0-x235.google.com [2607:f8b0:4001:c05::235]) by mx.google.com with ESMTPS id dq1si27145889icb.23.2014.06.02.13.09.11 for (version=TLSv1 cipher=ECDHE-RSA-RC4-SHA bits=128/128); Mon, 02 Jun 2014 13:09:11 -0700 (PDT) Received: by mail-ig0-f181.google.com with SMTP id h3so3884418igd.14 for ; Mon, 02 Jun 2014 13:09:11 -0700 (PDT) Date: Mon, 2 Jun 2014 13:09:08 -0700 (PDT) From: David Rientjes Subject: Re: [PATCH -mm] mm, compaction: properly signal and act upon lock and need_sched() contention - fix In-Reply-To: <538C8B45.6070803@suse.cz> Message-ID: References: <1399904111-23520-1-git-send-email-vbabka@suse.cz> <1400233673-11477-1-git-send-email-vbabka@suse.cz> <537F082F.50501@suse.cz> <538C8B45.6070803@suse.cz> MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Sender: owner-linux-mm@kvack.org List-ID: To: Vlastimil Babka Cc: Fabio Estevam , Andrew Morton , Shawn Guo , Kevin Hilman , Rik van Riel , Stephen Warren , Minchan Kim , Bartlomiej Zolnierkiewicz , Hugh Dickins , LKML , Michal Nazarewicz , linux-mm@kvack.org, Mel Gorman , Olof Johansson , Greg Thelen , Joonsoo Kim , Christoph Lameter , linux-arm-kernel On Mon, 2 Jun 2014, Vlastimil Babka wrote: > compact_should_abort() returns true instead of false and vice versa > due to changes between v1 and v2 of the patch. This makes both async > and sync compaction abort with high probability, and has been reported > to cause e.g. soft lockups on some ARM boards, or drivers calling > dma_alloc_coherent() fail to probe with CMA enabled on different boards. > > This patch fixes the return value to match comments and callers expecations. > > Reported-and-tested-by: Kevin Hilman > Reported-and-tested-by: Shawn Guo > Tested-by: Stephen Warren > Signed-off-by: Vlastimil Babka Acked-by: David Rientjes -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@kvack.org. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: email@kvack.org From mboxrd@z Thu Jan 1 00:00:00 1970 From: rientjes@google.com (David Rientjes) Date: Mon, 2 Jun 2014 13:09:08 -0700 (PDT) Subject: [PATCH -mm] mm, compaction: properly signal and act upon lock and need_sched() contention - fix In-Reply-To: <538C8B45.6070803@suse.cz> References: <1399904111-23520-1-git-send-email-vbabka@suse.cz> <1400233673-11477-1-git-send-email-vbabka@suse.cz> <537F082F.50501@suse.cz> <538C8B45.6070803@suse.cz> Message-ID: To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Mon, 2 Jun 2014, Vlastimil Babka wrote: > compact_should_abort() returns true instead of false and vice versa > due to changes between v1 and v2 of the patch. This makes both async > and sync compaction abort with high probability, and has been reported > to cause e.g. soft lockups on some ARM boards, or drivers calling > dma_alloc_coherent() fail to probe with CMA enabled on different boards. > > This patch fixes the return value to match comments and callers expecations. > > Reported-and-tested-by: Kevin Hilman > Reported-and-tested-by: Shawn Guo > Tested-by: Stephen Warren > Signed-off-by: Vlastimil Babka Acked-by: David Rientjes