Received: by 2002:a05:6358:3188:b0:123:57c1:9b43 with SMTP id q8csp2875829rwd; Fri, 16 Jun 2023 09:14:51 -0700 (PDT) X-Google-Smtp-Source: ACHHUZ6yXduRUP3+qoiN7kNXbIu+zZCt7GxVZk3kxS4HS+RXnJGoWBSTjFuNiO6qhcjpozu8RpWl X-Received: by 2002:a05:6808:1444:b0:396:40e8:689 with SMTP id x4-20020a056808144400b0039640e80689mr2998416oiv.53.1686932091599; Fri, 16 Jun 2023 09:14:51 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1686932091; cv=none; d=google.com; s=arc-20160816; b=J2vZxXhh61KCxhzmQBJDk82doHVorxkws/5yDxRRpmL37UWWMq5PWCLPR0y44mU7Bw AxZA39s95xRrkY9hQ2Adeg+NkV1hVfyCb0vgRYwhpp/nfOX9T+21p3cLEf244tJIZ9o5 q0bRbFM8rerIDIQdzoJQOIiW/dC4Q/gWdhYCpp5/Cf//8bw3xd+2evvuZscS427s2wiy NSmxkBNfQmC924bby8Kj+tBsWdPpTbgW/Om/mQ5L3A1ffcRyiETB+D0Z7fAgk3M1GtlF Odkd3vauq3h3OI5lSREg7X91PCrcU+/g+qenvfy9ab8RQid9pdCTvoXgHM8z49j8lSfg 7RJA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:cc:to:subject :message-id:date:from:in-reply-to:references:mime-version :dkim-signature; bh=i0G7Semk72J4t3p4kyn3yyN8DMzzZ6FPSFQ8xY4ugkA=; b=Ke/6EOtkDlDNbZF/KbyMQJpfsq84/lD6Yjcqlh3ZXRnMC4oc3k0fQsqHPhno+WpDDY AbVG8MJ0A61nc4JmtfcW2Epd/pjtihdHeNCBBHBEtcvM3t1bBOXH34qfsy5Fo47QD20i v9XDMVJiPxqvlTMRv/kGdSTYLAcryuVLCf+9nTHC9Lw/ZeSZDzYD3hslkBPZ+/0ZYxj+ g+3rYlFKq863fHtasleSLs4iprlJd+HdylpmCpFWXDEivWhQLPG/zJKYKCvlo6cnBNWp G2sV6tqhR/xMqQs0R0pJOL7mNa1WZ2e7HOdefR/zCuHRJklwTHl4qva8rb/4sSOJJlk9 JPuA== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@gmail.com header.s=20221208 header.b=QJxa1vOc; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 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 out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id h15-20020a17090a470f00b0025bdaaf0d17si2004119pjg.33.2023.06.16.09.14.37; Fri, 16 Jun 2023 09:14:51 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; dkim=pass header.i=@gmail.com header.s=20221208 header.b=QJxa1vOc; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 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 S230148AbjFPQG7 (ORCPT + 99 others); Fri, 16 Jun 2023 12:06:59 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:36214 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229470AbjFPQG6 (ORCPT ); Fri, 16 Jun 2023 12:06:58 -0400 Received: from mail-pj1-x102c.google.com (mail-pj1-x102c.google.com [IPv6:2607:f8b0:4864:20::102c]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id D72692119 for ; Fri, 16 Jun 2023 09:06:56 -0700 (PDT) Received: by mail-pj1-x102c.google.com with SMTP id 98e67ed59e1d1-25ec0b27e4fso336428a91.0 for ; Fri, 16 Jun 2023 09:06:56 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20221208; t=1686931616; x=1689523616; h=content-transfer-encoding:cc:to:subject:message-id:date:from :in-reply-to:references:mime-version:from:to:cc:subject:date :message-id:reply-to; bh=i0G7Semk72J4t3p4kyn3yyN8DMzzZ6FPSFQ8xY4ugkA=; b=QJxa1vOcE8rQhWgizO0XuACCRpq0Uppp+jXsxnRuMncxkb5yrQAJI7qMt9LkqwDd7R PV4tqzr1iPuNsJDpvBqlHB3nUCCP+5vPD4k/JFpxC6D2gIqRxggYDkFesv3ZaS0K5JWR /wStYTXp5//TcktuJR64Fxd4pr2R0sEWY0m47BO9hDIuN63Ec8bwdE+rThvAM1ZuwIYB T4D7AS/ndymzYcYgPRbSXFzXppRmJq+lUstVt85ocmqnAIrsGY2pYWGxCV5+ba5qqfVe 7asrMrMMPg+hNewml/KmL5O24wKjWcrKsI4SumLxsurmw4hkhn74w6QWTdZc48isQ1H9 oVbg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1686931616; x=1689523616; h=content-transfer-encoding:cc:to:subject:message-id:date:from :in-reply-to:references:mime-version:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=i0G7Semk72J4t3p4kyn3yyN8DMzzZ6FPSFQ8xY4ugkA=; b=XnJZzc9zOk0+DDzjbMbQFVrUq9AKlMl5w30SCYpzA+9eMfLOfaJsF6OD+AdldalzvM R1I/5X2RFfhVcAo+PQMYl7r5rCkSzFltVFEI1upQa48gjWuLnBdPPrsrJ2QWub+SyGNk GgZ7ffBUtLMc0N96GUh/bzPahYOxpsL6LB8rgA43CEL67rODNKH3jqRhqFioYxwufpwm w1QDNgQXVprD5vQ/s7hh//yVM3ybG15RXLsmErnNXOojJ0UDAR0etYt83O8f6CXfqcPj 61twcSt086j3f8jPKwUq2ehluT4bKieKAimv6cCGjb7Kma3WUMAk46Jd7zg8V8Kq2+Km wG4A== X-Gm-Message-State: AC+VfDyQiXlSW/d2khioWUqgWHLpY6hYRUqmaGxES0O+cJE8D0BV/BPz c5+phDoKTDujvCD1xuqkIfLmNIxMR3GAZBP54H8= X-Received: by 2002:a17:90a:6c63:b0:25b:fbdc:dd8 with SMTP id x90-20020a17090a6c6300b0025bfbdc0dd8mr1991263pjj.24.1686931616191; Fri, 16 Jun 2023 09:06:56 -0700 (PDT) MIME-Version: 1.0 References: <20230524153311.3625329-1-dhowells@redhat.com> <20230524153311.3625329-4-dhowells@redhat.com> <51161740e832334594960ed43430b868a6f892c3.camel@gmail.com> <617059.1686929309@warthog.procyon.org.uk> In-Reply-To: <617059.1686929309@warthog.procyon.org.uk> From: Alexander Duyck Date: Fri, 16 Jun 2023 09:06:19 -0700 Message-ID: Subject: Re: [PATCH net-next 03/12] mm: Make the page_frag_cache allocator alignment param a pow-of-2 To: David Howells Cc: "David S. Miller" , Eric Dumazet , Jakub Kicinski , Paolo Abeni , Willem de Bruijn , David Ahern , Matthew Wilcox , Jens Axboe , linux-mm@kvack.org, linux-kernel@vger.kernel.org, Jeroen de Borst , Catherine Sullivan , Shailend Chand , Felix Fietkau , John Crispin , Sean Wang , Mark Lee , Lorenzo Bianconi , Matthias Brugger , AngeloGioacchino Del Regno , Keith Busch , Jens Axboe , Christoph Hellwig , Sagi Grimberg , Chaitanya Kulkarni , Andrew Morton , linux-arm-kernel@lists.infradead.org, linux-mediatek@lists.infradead.org, linux-nvme@lists.infradead.org Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,FREEMAIL_FROM, RCVD_IN_DNSWL_NONE,SPF_HELO_NONE,SPF_PASS,T_SCC_BODY_TEXT_LINE autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, Jun 16, 2023 at 8:28=E2=80=AFAM David Howells = wrote: > > Alexander H Duyck wrote: > > > The requirement should only be cache alignment, not power of 2 > > alignment. > > Sure, but, upstream, page_frag_alloc_align() allows the specification of = an > alignment and applies that alignment by: > > offset &=3D align_mask; > > which doesn't really make sense unless the alignment boils down to being = a > power of two. Now, it might make sense to kill off the align_mask parame= ter > and just go with SMP_CACHE_BYTES (which will be a power of two). > > Note, though, that most users seem to use an align_mask of ~0u which feel= s a > bit dodgy (it's not an unsigned long), but is probably fine. Yeah, now that I think about it this should be fine. I must have been responding to this without enough sleep. When I originally read it I saw it as limiting it to a power of 2 allocation, not limiting the alignment mask to a power of 2.