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 Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 23B8FC32789 for ; Mon, 22 Aug 2022 10:17:48 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S234596AbiHVKRq (ORCPT ); Mon, 22 Aug 2022 06:17:46 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:43430 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S234584AbiHVKRo (ORCPT ); Mon, 22 Aug 2022 06:17:44 -0400 Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.133.124]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id CBDFF2715C for ; Mon, 22 Aug 2022 03:17:43 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1661163462; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=/KtRU9aDzK4XU9Ks65egqdRwloKirfCRi6+Miz1fpKk=; b=Y2ZW/HeEIBv89LnCSdPN+0WpyeNRHbM00k3SuQA+peDc4Mm4KWHS6KvoTQ1/fQwoBqguDH lzNVdz1xKk4LWarwkx5dGAqjLkzYDVH9IJvJQyebG70a4ZXOdiP0L4f4a47P9x96qYGmWN nwQNko/SbmQZK+IZwtVw5DmjH/TFj8k= Received: from mail-ed1-f70.google.com (mail-ed1-f70.google.com [209.85.208.70]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.3, cipher=TLS_AES_128_GCM_SHA256) id us-mta-356-jmXBzTOiO56kw4YLrevBMw-1; Mon, 22 Aug 2022 06:17:41 -0400 X-MC-Unique: jmXBzTOiO56kw4YLrevBMw-1 Received: by mail-ed1-f70.google.com with SMTP id h17-20020a05640250d100b00446d1825c9fso1116447edb.14 for ; Mon, 22 Aug 2022 03:17:41 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=content-transfer-encoding:mime-version:message-id:date:references :in-reply-to:subject:cc:to:from:x-gm-message-state:from:to:cc; bh=/KtRU9aDzK4XU9Ks65egqdRwloKirfCRi6+Miz1fpKk=; b=KAKRltqTvcS0IekO5RmyVNHVeNFfPQb9iNTlXdeZjhMRAtp9B3PUCDddF5GvQEigsF b0g8+OWU/trYIV+qIKzztdV+zPedIFE0MqzZCQImV92bG44k+NqmX2p7YIKWxXfxxMSq y70NOwbMdFcKYKVZ/63Ihc/7Hw3p7zk90SPLrAbSHqO/4z4rnjEmKsyd6Lh4Hb6RRG5m ODnuWSrqxb8Fm1MMG4HKV6SnLAtzDS9VeY6bHbgFG/foUTAuIugb5hqOQ0J14xXQwRMU TDXKmQnKuzkGh9aXLNwv01vPamEi2gzazMGJ6X3ojsf+Ja8RcOpwkpJS0Ak/gk0Q5WZw 3TJg== X-Gm-Message-State: ACgBeo0FkEtuXOhvnq3uEM5QWMq5yKw9krQ5ucQRbcFj0ZNM3zzEYX9g wmzpdq65bx9eHEvaWQyjCfzJz1NcvKe7R/t04kpiOXGm1JlRP1TQ6o8YImUI15kKQa65Upup5e+ k5QBRzm5bbhXjBwYn X-Received: by 2002:a05:6402:4383:b0:446:d6ac:bddd with SMTP id o3-20020a056402438300b00446d6acbdddmr2176191edc.283.1661163459840; Mon, 22 Aug 2022 03:17:39 -0700 (PDT) X-Google-Smtp-Source: AA6agR5+l+WvEVxQS/W5H0TQ59Ekoy4qDOsUP5z/+kLkcjlvNkPB9jPMoQcOiVOt/yr2uS6SeiRsmQ== X-Received: by 2002:a05:6402:4383:b0:446:d6ac:bddd with SMTP id o3-20020a056402438300b00446d6acbdddmr2176154edc.283.1661163459114; Mon, 22 Aug 2022 03:17:39 -0700 (PDT) Received: from alrua-x1.borgediget.toke.dk ([2a0c:4d80:42:443::2]) by smtp.gmail.com with ESMTPSA id j7-20020aa7c407000000b004465d1db765sm5089367edq.89.2022.08.22.03.17.38 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 22 Aug 2022 03:17:38 -0700 (PDT) Received: by alrua-x1.borgediget.toke.dk (Postfix, from userid 1000) id A988056014F; Mon, 22 Aug 2022 12:17:37 +0200 (CEST) From: Toke =?utf-8?Q?H=C3=B8iland-J=C3=B8rgensen?= To: Jakub Kicinski Cc: "David S. Miller" , Eric Dumazet , Paolo Abeni , netdev@vger.kernel.org, bpf@vger.kernel.org Subject: Re: [PATCH bpf-next 1/3] dev: Move received_rps counter next to RPS members in softnet data In-Reply-To: <20220819155421.3ca7d6d6@kernel.org> References: <20220818165906.64450-1-toke@redhat.com> <20220818165906.64450-2-toke@redhat.com> <20220818200143.7d534a41@kernel.org> <87bksgv26h.fsf@toke.dk> <20220819155421.3ca7d6d6@kernel.org> X-Clacks-Overhead: GNU Terry Pratchett Date: Mon, 22 Aug 2022 12:17:37 +0200 Message-ID: <87a67wk2f2.fsf@toke.dk> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable Precedence: bulk List-ID: X-Mailing-List: netdev@vger.kernel.org Jakub Kicinski writes: > On Fri, 19 Aug 2022 14:38:14 +0200 Toke H=C3=B8iland-J=C3=B8rgensen wrote: >> Jakub Kicinski writes: >> > On Thu, 18 Aug 2022 18:59:03 +0200 Toke H=C3=B8iland-J=C3=B8rgensen wr= ote:=20=20 >> >> Move the received_rps counter value next to the other RPS-related mem= bers >> >> in softnet_data. This closes two four-byte holes in the structure, ma= king >> >> room for another pointer in the first two cache lines without bumping= the >> >> xmit struct to its own line.=20=20 >> > >> > What's the pointer you're making space for (which I hope will explain >> > why this patch is part of this otherwise bpf series)?=20=20 >>=20 >> The XDP queueing series adds a pointer to keep track of which interfaces >> were scheduled for transmission using the XDP dequeue hook (similar to >> how the qdisc wake code works): >>=20 >> https://lore.kernel.org/r/20220713111430.134810-12-toke@redhat.com > > I see, it makes more sense now :) > >> Note that it's still up in the air if this ends up being the way this >> will be implemented, so I'm OK with dropping this patch for now if you'd >> rather wait until it's really needed. OTOH it also seemed like a benign >> change on its own, so I figured I might as well include this patch when >> sending these out. WDYT? > > Whatever is easiest :) Alright, I'm OK with either so let's leave it up to the (BPF) maintainers if they want to drop this patch or just merge the whole series? :) -Toke