From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-14.3 required=3.0 tests=DKIMWL_WL_MED,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH, MAILING_LIST_MULTI,SIGNED_OFF_BY,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED, USER_IN_DEF_DKIM_WL autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 230E0C43331 for ; Thu, 26 Mar 2020 15:48:58 +0000 (UTC) Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by mail.kernel.org (Postfix) with ESMTP id BAA6920737 for ; Thu, 26 Mar 2020 15:48:57 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=google.com header.i=@google.com header.b="S/kSykVL" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org BAA6920737 Authentication-Results: mail.kernel.org; dmarc=fail (p=reject dis=none) header.from=google.com Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=owner-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix) id 2453F6B000A; Thu, 26 Mar 2020 11:48:57 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 1CE866B000C; Thu, 26 Mar 2020 11:48:57 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 0BDD26B000D; Thu, 26 Mar 2020 11:48:57 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from forelay.hostedemail.com (smtprelay0143.hostedemail.com [216.40.44.143]) by kanga.kvack.org (Postfix) with ESMTP id E5AA96B000A for ; Thu, 26 Mar 2020 11:48:56 -0400 (EDT) Received: from smtpin15.hostedemail.com (10.5.19.251.rfc1918.com [10.5.19.251]) by forelay03.hostedemail.com (Postfix) with ESMTP id F1296824805A for ; Thu, 26 Mar 2020 15:48:56 +0000 (UTC) X-FDA: 76637946672.15.women14_6b316eccf631c X-HE-Tag: women14_6b316eccf631c X-Filterd-Recvd-Size: 5261 Received: from mail-wm1-f65.google.com (mail-wm1-f65.google.com [209.85.128.65]) by imf47.hostedemail.com (Postfix) with ESMTP for ; Thu, 26 Mar 2020 15:48:56 +0000 (UTC) Received: by mail-wm1-f65.google.com with SMTP id z18so6999410wmk.2 for ; Thu, 26 Mar 2020 08:48:55 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20161025; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc:content-transfer-encoding; bh=tnn7KHY4eWHTo8zdroLBrOpWxVoG9chHd2LE7rb+g4c=; b=S/kSykVLIzOW+bMtz28j9SL0IboXQaDEabZEPluAQ/jdTHKWYG9FhojaUl0Fmq4B1P r3AOSY0O8LRBGxIut4Bs4gZXoJioXnt3PtNqTi1ystbqUw6iLFXcpAkXhOpGcigUixPy S3ncz6t/Ez10hkmDVMtK4taf2eL2L8Yc7N/mCu2LeWk2BMUaqKLjXQbdxJxM3t7p01vm DO4BwnJI29K3mRmETdN4mZCqxfiDdPigj1O7hrw7LtCoGHzH9Iu/dAy3ZkDdaeonu93N 4HIrT4yneg3HIAFtgQYxod4eIlZMNQiUgoqjN9wVmdZ8fghCzcV0kAEEByfR9Je738JZ MdNQ== 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=tnn7KHY4eWHTo8zdroLBrOpWxVoG9chHd2LE7rb+g4c=; b=ue4P+avPKFtHZqGEp2GeCljSJ1T4KYVRCz0JjZd29Lw0bhIjUvDEKhPGXDYBoUcBJ8 /R21KJhRwkszFJVGbhTy95VVt4omSC78X3hXnxZ/MYWTaEvBFRL54NWFx72IvS0kztxX sYI60vzhiev1qPQRHGDILvc0kzbT9GB7E2ohbRvUUdCukd9q/r8oq6XSvHw90ntnluXb DmUlg1RrnUniFMY265DW9gJGjHIMK43al8PQpmkUtv2xZtLNGb7/fYHqOFcWBtQ1EXdO jP2I3c5/Tn8tMxdKHApN+3+vGjNzgawFKZVy5Aa7wZo4sQYVIsVLQyLUhRcMjroOnhbD 73Eg== X-Gm-Message-State: ANhLgQ3JkeiJ5WUtUbKZfulPUhHQP2uJkE+R1MvboxQPQhmFQktycvl8 STNseV1tDCsPibQWHouOda+Nh5zoV8zC3Sjhvg/IVQ== X-Google-Smtp-Source: ADFU+vv6nSdJKaQuPkAMrcUtKm8kmac0YVBKRQ4HyVrhEfsZEMMr7tcXI64Kt8igxgOVmda0hFaoxYzxjJIM+XsxhHA= X-Received: by 2002:a05:600c:22c1:: with SMTP id 1mr518314wmg.29.1585237733584; Thu, 26 Mar 2020 08:48:53 -0700 (PDT) MIME-Version: 1.0 References: <20200324203231.64324-1-keescook@chromium.org> <20200324203231.64324-3-keescook@chromium.org> In-Reply-To: <20200324203231.64324-3-keescook@chromium.org> From: Alexander Potapenko Date: Thu, 26 Mar 2020 16:48:41 +0100 Message-ID: Subject: Re: [PATCH v2 2/5] init_on_alloc: Unpessimize default-on builds To: Kees Cook Cc: Thomas Gleixner , Elena Reshetova , "the arch/x86 maintainers" , Andy Lutomirski , Peter Zijlstra , Catalin Marinas , Will Deacon , Mark Rutland , Ard Biesheuvel , Jann Horn , "Perla, Enrico" , Kernel Hardening , linux-arm-kernel@lists.infradead.org, Linux Memory Management List , LKML Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable X-Bogosity: Ham, tests=bogofilter, spamicity=0.000000, version=1.2.4 Sender: owner-linux-mm@kvack.org Precedence: bulk X-Loop: owner-majordomo@kvack.org List-ID: On Tue, Mar 24, 2020 at 9:32 PM Kees Cook wrote: > > Right now, the state of CONFIG_INIT_ON_ALLOC_DEFAULT_ON (and > ...ON_FREE...) did not change the assembly ordering of the static branch > tests. Use the new jump_label macro to check CONFIG settings to default > to the "expected" state, unpessimizes the resulting assembly code. > > Signed-off-by: Kees Cook > --- > include/linux/mm.h | 6 ++++-- > 1 file changed, 4 insertions(+), 2 deletions(-) > > diff --git a/include/linux/mm.h b/include/linux/mm.h > index 059658604dd6..64e911159ffa 100644 > --- a/include/linux/mm.h > +++ b/include/linux/mm.h > @@ -2665,7 +2665,8 @@ static inline void kernel_poison_pages(struct page = *page, int numpages, > DECLARE_STATIC_KEY_MAYBE(CONFIG_INIT_ON_ALLOC_DEFAULT_ON, init_on_alloc)= ; > static inline bool want_init_on_alloc(gfp_t flags) > { > - if (static_branch_unlikely(&init_on_alloc) && > + if (static_branch_maybe(CONFIG_INIT_ON_ALLOC_DEFAULT_ON, > + &init_on_alloc) && > !page_poisoning_enabled()) > return true; > return flags & __GFP_ZERO; > @@ -2674,7 +2675,8 @@ static inline bool want_init_on_alloc(gfp_t flags) > DECLARE_STATIC_KEY_MAYBE(CONFIG_INIT_ON_FREE_DEFAULT_ON, init_on_free); > static inline bool want_init_on_free(void) > { > - return static_branch_unlikely(&init_on_free) && > + return static_branch_maybe(CONFIG_INIT_ON_FREE_DEFAULT_ON, > + &init_on_free) && > !page_poisoning_enabled(); > } > > -- > 2.20.1 > Reviewed-by: Alexander Potapenko --=20 Alexander Potapenko Software Engineer Google Germany GmbH Erika-Mann-Stra=C3=9Fe, 33 80636 M=C3=BCnchen Gesch=C3=A4ftsf=C3=BChrer: Paul Manicle, Halimah DeLaine Prado Registergericht und -nummer: Hamburg, HRB 86891 Sitz der Gesellschaft: Hamburg