From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753777AbeAQWA5 (ORCPT ); Wed, 17 Jan 2018 17:00:57 -0500 Received: from mga01.intel.com ([192.55.52.88]:6875 "EHLO mga01.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753702AbeAQWA4 (ORCPT ); Wed, 17 Jan 2018 17:00:56 -0500 X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.46,374,1511856000"; d="scan'208";a="166956644" Subject: Re: [mm 4.15-rc8] Random oopses under memory pressure. To: Linus Torvalds , Tetsuo Handa References: <201801160115.w0G1FOIG057203@www262.sakura.ne.jp> <201801170233.JDG21842.OFOJMQSHtOFFLV@I-love.SAKURA.ne.jp> <201801172008.CHH39543.FFtMHOOVSQJLFO@I-love.SAKURA.ne.jp> Cc: "Kirill A. Shutemov" , Andrew Morton , Johannes Weiner , Joonsoo Kim , Mel Gorman , Tony Luck , Vlastimil Babka , Michal Hocko , Ingo Molnar , Linux Kernel Mailing List , linux-mm , the arch/x86 maintainers From: Dave Hansen Message-ID: <87aea433-ba6b-8543-d925-3ef36911f124@linux.intel.com> Date: Wed, 17 Jan 2018 14:00:54 -0800 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.5.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 01/17/2018 01:39 PM, Linus Torvalds wrote: > > So maybe something like this to test the theory? > > diff --git a/mm/page_alloc.c b/mm/page_alloc.c > index 76c9688b6a0a..f919a5548943 100644 > --- a/mm/page_alloc.c > +++ b/mm/page_alloc.c > @@ -756,6 +756,8 @@ static inline void rmv_page_order(struct page *page) > static inline int page_is_buddy(struct page *page, struct page *buddy, > unsigned int order) > { > + if (WARN_ON_ONCE(page_zone(page) != page_zone(buddy))) > + return 0; > if (page_is_guard(buddy) && page_order(buddy) == order) { > if (page_zone_id(page) != page_zone_id(buddy)) > return 0; I thought that page_zone_id() stuff was there to prevent this kind of cross-zone stuff from happening.