Received: by 2002:a25:e74b:0:0:0:0:0 with SMTP id e72csp1833896ybh; Thu, 23 Jul 2020 20:07:04 -0700 (PDT) X-Google-Smtp-Source: ABdhPJz2HBl0hIq6yLOXb2WBQ6EyP2oi4VMP0NCqV0ls7OuPkp6Xae7h5GbUFLe4vjISlKfM4jjS X-Received: by 2002:a17:906:2b50:: with SMTP id b16mr7485962ejg.410.1595560024521; Thu, 23 Jul 2020 20:07:04 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1595560024; cv=none; d=google.com; s=arc-20160816; b=lpGkKUwqMDYCISn9L5cJFMe0BNAVEf90a6zSUpzK+plmFkRpVXY5ATVFcgqWJ8IgME M2cQbpU/XObC5r7Z5G1NJVUQEqUF0klqtF/rRkLvEE3+lp5B4EjmAPzYhFhfOgYPW6dz qv+9a3BAtvLnjyur0WO11t5yqkEBGA48050mIWk5NQWYuIrCtOFOI7zP/T8+SqP314F+ EQ3J+jsjQxIWH973ADE4Od9FfPi/d7n6CFoAV2MIoU64k9lXFS2EKSGyvl42y7FxGgtj 1PDM4eWWvVdBgHy7f1vfwS6GysVcLYtLgOR0rf3OA/HQscBnmk+koQs3AkpnVEOoUaiN cz5g== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:content-transfer-encoding:cc:to:subject :message-id:date:from:in-reply-to:references:mime-version :dkim-signature; bh=Pq5OZv+1Cdv1nhK/D4C8Dz963Kl7Du6n7kqUvhsIG9o=; b=uhmSoJRNe4jh56ZFnxxSh5dCFgYUVSGNeHDf9uwhHFKJR2J31cIFJreSqH3QiUFT3u jW7OgYkymZ8U44AHoosMx0M0O8C1eVKI+T2shEE85cwu4YL9dlfsbFyJ9dROgVG6b6/W oY1stAghos8ss5RAnnEtAumdx7v7YJ42LqeOB1uxATgnAVaMqv1Zez2wn54a5M+eayAi GfH3MO2VhiUNb8eBb7uiNbs378xZWGCRnNDktKvMAtVMULqSXIZLS/eM3glrY7gkMeuK s3GfEsEzfSiSb+TnTmBXBje4To9s1DIEkBYDxHWvUJCUwisH+m6E8zBuUmYGa0V/kLlj Kxmw== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@gmail.com header.s=20161025 header.b=qsGf5Y+e; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id 12si3067329edx.70.2020.07.23.20.06.41; Thu, 23 Jul 2020 20:07:04 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=pass header.i=@gmail.com header.s=20161025 header.b=qsGf5Y+e; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726780AbgGXDEP (ORCPT + 99 others); Thu, 23 Jul 2020 23:04:15 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:53414 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726437AbgGXDEO (ORCPT ); Thu, 23 Jul 2020 23:04:14 -0400 Received: from mail-qv1-xf41.google.com (mail-qv1-xf41.google.com [IPv6:2607:f8b0:4864:20::f41]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id A344AC0619D3; Thu, 23 Jul 2020 20:04:14 -0700 (PDT) Received: by mail-qv1-xf41.google.com with SMTP id u8so3515995qvj.12; Thu, 23 Jul 2020 20:04:14 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc:content-transfer-encoding; bh=Pq5OZv+1Cdv1nhK/D4C8Dz963Kl7Du6n7kqUvhsIG9o=; b=qsGf5Y+eAHB2nfvmMACk9RPMbSNLUk3wCIuPP0VRSEch3lQ01wDuEuqsN/3wbSre4n h+hryRYytPrwoB3pf2nnN/OJzTS9VnnL0daQ5L4b+GuC7l9etql0GSlZRagIqz00WqOL 6CsD5xOvOA577EZxSjSkqLbsEUXYO63r60YNioeEPaJ+tWqfI9yy6EwrqFWt+uNitbHT R8EEiiUNREX6zswIU3QlVOAlAA4edHBllH9GwyLn+3DbrXXQsaQplONQQ/5F+eGOAJcy s3RGVSiuB4zPBhq8JJjE/HXJa/stf5lKdhzi8Gs7X+0JNDQFdZxHsPo/Pg2xdxHE8UqX OPBg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc:content-transfer-encoding; bh=Pq5OZv+1Cdv1nhK/D4C8Dz963Kl7Du6n7kqUvhsIG9o=; b=QAmXhYfFylLSoM+A2e5PkoUyceRmOMRCnGZLH4CjjnW4R1mMWylp7Rug0azlmI07Th mwB4SMdo6E+8MbtIy0zjiG0Fxb8YxyWQL0pdIN0Zcc8TInMHoc40UgtStA3aBWTtYL/k O0QZxqxYS0cM5pcY+kxWTJCOWQO7sqMQxpoYJLvmnpcaYcg7z18YSBbkpfN+q6PmX7nn RnnMAbCGT4ZcA+dIOVrh83KbjKsGHVQnL5z2O+s4h9fp4MNtO5Tor5fRnZetI5G+ZHDY DSkkSjWaN2hNmOXYjEuyxnZ8HVFLgjOBqY1Gg5eFPn6ruVsYICHccwD/SPx+l/wsjMTY aI0Q== X-Gm-Message-State: AOAM531FPt4ld6bAaC3Ms1GH81uuaTlXpGviOTk40dWULl6ECjoagSt7 DJbjWUsa/NMDr0GjTtoVbBW1RmIrwDYy3ILonsdAQi7S X-Received: by 2002:a0c:a306:: with SMTP id u6mr7860949qvu.88.1595559853706; Thu, 23 Jul 2020 20:04:13 -0700 (PDT) MIME-Version: 1.0 References: <1595468942-29687-1-git-send-email-iamjoonsoo.kim@lge.com> <20200723180814.acde28b92ce6adc785a79120@linux-foundation.org> <20200723193600.28e3eedd00925b22f7ca9780@linux-foundation.org> In-Reply-To: <20200723193600.28e3eedd00925b22f7ca9780@linux-foundation.org> From: Joonsoo Kim Date: Fri, 24 Jul 2020 12:04:02 +0900 Message-ID: Subject: Re: [PATCH v2] mm/page_alloc: fix memalloc_nocma_{save/restore} APIs To: Andrew Morton Cc: Linux Memory Management List , LKML , kernel-team@lge.com, Vlastimil Babka , Christoph Hellwig , Roman Gushchin , Mike Kravetz , Naoya Horiguchi , Michal Hocko , "Aneesh Kumar K . V" , Joonsoo Kim , stable@vger.kernel.org Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org 2020=EB=85=84 7=EC=9B=94 24=EC=9D=BC (=EA=B8=88) =EC=98=A4=EC=A0=84 11:36, = Andrew Morton =EB=8B=98=EC=9D=B4 =EC=9E=91=EC=84= =B1: > > On Fri, 24 Jul 2020 11:23:52 +0900 Joonsoo Kim wrote: > > > > > Second, clearing __GFP_MOVABLE in current_gfp_context() has a side = effect > > > > to exclude the memory on the ZONE_MOVABLE for allocation target. > > > > > > More whoops. > > > > > > Could we please have a description of the end-user-visible effects of > > > this change? Very much needed when proposing a -stable backport, I t= hink. > > > > In fact, there is no noticeable end-user-visible effect since the fallb= ack would > > cover the problematic case. It's mentioned in the commit description. P= erhap, > > performance would be improved due to reduced retry and more available m= emory > > (we can use ZONE_MOVABLE with this patch) but it would be neglectable. > > > > > d7fefcc8de9147c is over a year old. Why did we only just discover > > > this? This makes one wonder how serious those end-user-visible effec= ts > > > are? > > > > As mentioned above, there is no visible problem to the end-user. > > OK, thanks. In that case, I don't believe that a stable backport is > appropriate? > > (Documentation/process/stable-kernel-rules.rst) Thanks for the pointer! Hmm... I'm not sure the correct way to handle this patch. I thought that memalloc_nocma_{save,restore} is an API that is callable from the module. If it is true, it's better to regard this patch as the stable candidate sin= ce out-of-tree modules could use it without the fallback and it would cause a problem. But, yes, there is no visible problem to the end-user, at least, within the mainline so it is possibly not a stable candidate. Please share your opinion about this situation. Thanks.