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 smtp4.osuosl.org (smtp4.osuosl.org [140.211.166.137]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id D5D18EB64DB for ; Thu, 15 Jun 2023 12:38:01 +0000 (UTC) Received: from localhost (localhost [127.0.0.1]) by smtp4.osuosl.org (Postfix) with ESMTP id 4F25841DDE; Thu, 15 Jun 2023 12:38:01 +0000 (UTC) DKIM-Filter: OpenDKIM Filter v2.11.0 smtp4.osuosl.org 4F25841DDE DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=osuosl.org; s=default; t=1686832681; bh=rgBjeGQR9gAuQkAoE1OoOGOmwsQW8hN5274/4yCatPU=; h=From:To:In-Reply-To:References:Date:Subject:List-Id: List-Unsubscribe:List-Archive:List-Post:List-Help:List-Subscribe: Cc:From; b=8g9RvHQxuwjLNEaqRILGy8IYqh+ZBT06q0vgGAN84tWXlX360OZT/Odl0pii+WRA4 f+GukFkuB13VV6gGxP9eNH9SoDy2TUpCeibAbslsu/tIva+ykv5Kdt8kZYsXCLAJst GElDgcmFGqUErQ3NqFPho0FWtQIJgAbVWsZYTwurbpbUHsEH8OrohiF8duAWCfNGnr LFfWRyo7szsSNmTQIudtszqUls5+81gIcE0tqKTuu6kdoeJPR+LJ7/d+u3hdKM6VN5 fyWdVlthYYN6pl7Yo0wRkdb1DMZXcDlPzLhekU8ENWwTpa/gGAsp20/MW9t4Lvg5Jf w05F2vyAC8UWw== X-Virus-Scanned: amavisd-new at osuosl.org Received: from smtp4.osuosl.org ([127.0.0.1]) by localhost (smtp4.osuosl.org [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id Tag2OYzCAqkG; Thu, 15 Jun 2023 12:38:00 +0000 (UTC) Received: from ash.osuosl.org (ash.osuosl.org [140.211.166.34]) by smtp4.osuosl.org (Postfix) with ESMTP id 2744E41E94; Thu, 15 Jun 2023 12:38:00 +0000 (UTC) DKIM-Filter: OpenDKIM Filter v2.11.0 smtp4.osuosl.org 2744E41E94 Received: from smtp4.osuosl.org (smtp4.osuosl.org [140.211.166.137]) by ash.osuosl.org (Postfix) with ESMTP id C12551BF831 for ; Thu, 15 Jun 2023 12:37:58 +0000 (UTC) Received: from localhost (localhost [127.0.0.1]) by smtp4.osuosl.org (Postfix) with ESMTP id A4DC141E94 for ; Thu, 15 Jun 2023 12:37:58 +0000 (UTC) DKIM-Filter: OpenDKIM Filter v2.11.0 smtp4.osuosl.org A4DC141E94 X-Virus-Scanned: amavisd-new at osuosl.org Received: from smtp4.osuosl.org ([127.0.0.1]) by localhost (smtp4.osuosl.org [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id 5v1PC1zys3jP for ; Thu, 15 Jun 2023 12:37:57 +0000 (UTC) X-Greylist: domain auto-whitelisted by SQLgrey-1.8.0 DKIM-Filter: OpenDKIM Filter v2.11.0 smtp4.osuosl.org B231A41DDE Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.133.124]) by smtp4.osuosl.org (Postfix) with ESMTPS id B231A41DDE for ; Thu, 15 Jun 2023 12:37:57 +0000 (UTC) Received: from mail-ej1-f72.google.com (mail-ej1-f72.google.com [209.85.218.72]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.3, cipher=TLS_AES_256_GCM_SHA384) id us-mta-418-5uDaxK5dNY-NLY-ZX_kYrw-1; Thu, 15 Jun 2023 08:37:54 -0400 X-MC-Unique: 5uDaxK5dNY-NLY-ZX_kYrw-1 Received: by mail-ej1-f72.google.com with SMTP id a640c23a62f3a-94a348facbbso172874566b.1 for ; Thu, 15 Jun 2023 05:37:53 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1686832672; x=1689424672; h=content-transfer-encoding:mime-version:message-id:date:references :in-reply-to:subject:cc:to:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=gTIAUs6tWTx1JxWTYhYUIaHzGBgz21d+mfQ6B/1Bob8=; b=kI0DnOEIFJF36vygipjWTQrJHHfSzjubT5Bv6Y5W5Wtufb6Xunm2Z6j5/7dH1HCRqW 84Ia6goMuAj73nADnMVIvWRQ6ol9LcoF7lG0KdHl7qmOh+OmvfLazoTN3ZYJHI7dXgoY mzqhRu2CDwHfFc/QPS3IiW/XV4LEYOHIpN7D+RhFTladYXGt0tgrsMYV42JDNFxTkOoC goKueOq4B+u+wQpHqXK4MCoenu6LVj0eTuMjulsmkJ99AnWUJLQME0tRCAC1VzGoDU7h QsJRkGwA1cLtWky6kludZeq/ZYCbsw3mQnzSkiSi6QyQ42K27Ec3q0QeSnnYAlfEvcLW iOHg== X-Gm-Message-State: AC+VfDxu9XeWifv+3CSweRWRgpMaLCx1YGIV2C66PBtps1cATDGjX8HR NGJjXC6lamwqtEhCT3G92YkiW1KNYigOZAtN87wCEXB596T9IDRdXx7y4qj6y47+j4CJ3Tesh3s PBB8aAQ5S/iaiAk2xU93YpEXVZcS3LQ== X-Received: by 2002:a17:906:58c8:b0:96f:e7cf:5015 with SMTP id e8-20020a17090658c800b0096fe7cf5015mr20449710ejs.17.1686832672180; Thu, 15 Jun 2023 05:37:52 -0700 (PDT) X-Google-Smtp-Source: ACHHUZ4yLdrC3Ns33uRvMP1BQdUvRNhpaN9fVe272nbI+nzoGeNbjGmBXTp3+HDcp0B9u0AVYvhhYg== X-Received: by 2002:a17:906:58c8:b0:96f:e7cf:5015 with SMTP id e8-20020a17090658c800b0096fe7cf5015mr20449690ejs.17.1686832671834; Thu, 15 Jun 2023 05:37:51 -0700 (PDT) Received: from alrua-x1.borgediget.toke.dk ([45.145.92.2]) by smtp.gmail.com with ESMTPSA id q15-20020a170906940f00b00977eb9957e9sm9378146ejx.128.2023.06.15.05.37.51 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 15 Jun 2023 05:37:51 -0700 (PDT) Received: by alrua-x1.borgediget.toke.dk (Postfix, from userid 1000) id 68D0ABBEC27; Thu, 15 Jun 2023 14:29:54 +0200 (CEST) From: Toke =?utf-8?Q?H=C3=B8iland-J=C3=B8rgensen?= To: Maciej Fijalkowski , intel-wired-lan@lists.osuosl.org In-Reply-To: <20230615113326.347770-1-maciej.fijalkowski@intel.com> References: <20230615113326.347770-1-maciej.fijalkowski@intel.com> X-Clacks-Overhead: GNU Terry Pratchett Date: Thu, 15 Jun 2023 14:29:54 +0200 Message-ID: <87v8fogb7h.fsf@toke.dk> MIME-Version: 1.0 X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com X-Mailman-Original-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1686832676; 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=gTIAUs6tWTx1JxWTYhYUIaHzGBgz21d+mfQ6B/1Bob8=; b=NtxGeqGaysaPGDRrOe3w/ZEYhnlb/eCZW55qR04dGOp3S5PyEdxBWZwn7b5jFfiEFGtAoS xnQjw+6G7CJcPrTeyLOwJJt2Jf9yG8jiQZIQSaQDDhqx7RkbV4SX3FAyxzf1R1aVH/nmob E6X/bzB2WXGiMyHfbe0oLuO+DiOtY6I= X-Mailman-Original-Authentication-Results: smtp4.osuosl.org; dkim=pass (1024-bit key) header.d=redhat.com header.i=@redhat.com header.a=rsa-sha256 header.s=mimecast20190719 header.b=NtxGeqGa Subject: Re: [Intel-wired-lan] [PATCH v3 iwl-next] ice: allow hot-swapping XDP programs X-BeenThere: intel-wired-lan@osuosl.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Intel Wired Ethernet Linux Kernel Driver Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: netdev@vger.kernel.org, anthony.l.nguyen@intel.com, magnus.karlsson@intel.com, fred@cloudflare.com Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Errors-To: intel-wired-lan-bounces@osuosl.org Sender: "Intel-wired-lan" TWFjaWVqIEZpamFsa293c2tpIDxtYWNpZWouZmlqYWxrb3dza2lAaW50ZWwuY29tPiB3cml0ZXM6 Cgo+IEN1cnJlbnRseSBpY2UgZHJpdmVyJ3MgLm5kb19icGYgY2FsbGJhY2sgYnJpbmdzIGludGVy ZmFjZSBkb3duIGFuZCB1cAo+IGluZGVwZW5kZW50bHkgb2YgWERQIHJlc291cmNlcycgcHJlc2Vu Y2UuIFRoaXMgaXMgb25seSBuZWVkZWQgd2hlbgo+IGVpdGhlciB0aGVzZSByZXNvdXJjZXMgaGF2 ZSB0byBiZSBjb25maWd1cmVkIG9yIHJlbW92ZWQuIEl0IG1lYW5zIHRoYXQKPiBpZiBvbmUgaXMg c3dpdGNoaW5nIFhEUCBwcm9ncmFtcyBvbi10aGUtZmx5IHdpdGggcnVubmluZyB0cmFmZmljLAo+ IHBhY2tldHMgd2lsbCBiZSBkcm9wcGVkLgo+Cj4gVG8gYXZvaWQgdGhpcywgY29tcGFyZSBlYXJs eSBvbiBpY2VfeGRwX3NldHVwX3Byb2coKSBzdGF0ZSBvZiBpbmNvbWluZwo+IGJwZl9wcm9nIHBv aW50ZXIgdnMgdGhlIGJwZl9wcm9nIHBvaW50ZXIgdGhhdCBpcyBhbHJlYWR5IGFzc2lnbmVkIHRv Cj4gVlNJLiBEbyB0aGUgc3dhcCBpbiBjYXNlIFZTSSBoYXMgYnBmX3Byb2cgYW5kIGluY29taW5n IG9uZSBhcmUgbm9uLU5VTEwuCj4KPiBMYXN0bHksIHdoaWxlIGF0IGl0LCBwdXQgb2xkIGJwZl9w cm9nICphZnRlciogdGhlIHVwZGF0ZSBvZiBSeCByaW5nJ3MKPiBicGZfcHJvZyBwb2ludGVyLiBJ biB0aGVvcnkgcHJldmlvdXMgY29kZSBjb3VsZCBleHBvc2UgdXMgdG8gYSBzdGF0ZQo+IHdoZXJl IFJ4IHJpbmcncyBicGZfcHJvZyB3b3VsZCBzdGlsbCBiZSByZWZlcnJpbmcgdG8gb2xkX3Byb2cg dGhhdCBnb3QKPiByZWxlYXNlZCB3aXRoIGVhcmxpZXIgYnBmX3Byb2dfcHV0KCkuCj4KPiBTaWdu ZWQtb2ZmLWJ5OiBNYWNpZWogRmlqYWxrb3dza2kgPG1hY2llai5maWphbGtvd3NraUBpbnRlbC5j b20+CgpBY2tlZC1ieTogVG9rZSBIw7hpbGFuZC1Kw7hyZ2Vuc2VuIDx0b2tlQHJlZGhhdC5jb20+ CgpfX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fXwpJbnRlbC13 aXJlZC1sYW4gbWFpbGluZyBsaXN0CkludGVsLXdpcmVkLWxhbkBvc3Vvc2wub3JnCmh0dHBzOi8v bGlzdHMub3N1b3NsLm9yZy9tYWlsbWFuL2xpc3RpbmZvL2ludGVsLXdpcmVkLWxhbgo= From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from lindbergh.monkeyblade.net (lindbergh.monkeyblade.net [23.128.96.19]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 11FCD6106 for ; Thu, 15 Jun 2023 12:38:02 +0000 (UTC) Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.129.124]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id C26F81FDB for ; Thu, 15 Jun 2023 05:38:01 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1686832681; 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=gTIAUs6tWTx1JxWTYhYUIaHzGBgz21d+mfQ6B/1Bob8=; b=NQLN7iFng6hFlqLz1FJO5bY7JNngeDT72Y01bJtI3QTtrKLUAnFNV3K7PIqmjp3NxLejqB 3/m/2AsHVRJtcO26o9EFDDfMtH628zJawz6aBWWZan+RPcEueqPCSdfWD5EOwrV0D3VuTY gyFFvdrPX2HjxTtUT0NDlmXiVRdu+vg= Received: from mail-ej1-f71.google.com (mail-ej1-f71.google.com [209.85.218.71]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.3, cipher=TLS_AES_256_GCM_SHA384) id us-mta-381-caIo-0gqOCqRqzkvrhflww-1; Thu, 15 Jun 2023 08:37:53 -0400 X-MC-Unique: caIo-0gqOCqRqzkvrhflww-1 Received: by mail-ej1-f71.google.com with SMTP id a640c23a62f3a-9829a080268so80996366b.3 for ; Thu, 15 Jun 2023 05:37:53 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1686832672; x=1689424672; h=content-transfer-encoding:mime-version:message-id:date:references :in-reply-to:subject:cc:to:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=gTIAUs6tWTx1JxWTYhYUIaHzGBgz21d+mfQ6B/1Bob8=; b=Z2JxKjcpuBORc2idnL3HrmfAVdANeCkisTpQpdIhZdVqFq3NyW9mdrCLwfnIq8XcRH EoHkGyxKM+KQwm6ZcI3VdTQ3Xr3YogQATNv/MYxGE478YARHPBZVmFiBdKvs954Y5G+7 hEnjDi/R5cmRTw/Iw96EvXm4SgERtppfexZAZQ+zZyb9nb0+JWdSc/uuyqSrgfL3g5bt rZlDV8ZQP6iIZO13riWJPueoIZWAcfiFG/Bf///mSNlHdYWFVH16PvyiYSyb2cW4Pblx aRrRDmMOJ2A1eiaAqNMb4WsaiBBkXKHU1DAJB7sa6FpPC30s53T40Bu0fX0cuDpbxlnZ QCIw== X-Gm-Message-State: AC+VfDwo7Fgb1sE96npiKOqNOixPzzxmgptRLEx/sGaFP80+2HT+NJad DuOs4XIX3mCKzjcY92J5E/tj5xk41xBRfOMP++JwXl0VuRdKreie88tCyAYQD/5kXfkuNKKgBBl Ax1xg49dZZexPTDWP X-Received: by 2002:a17:906:58c8:b0:96f:e7cf:5015 with SMTP id e8-20020a17090658c800b0096fe7cf5015mr20449709ejs.17.1686832672180; Thu, 15 Jun 2023 05:37:52 -0700 (PDT) X-Google-Smtp-Source: ACHHUZ4yLdrC3Ns33uRvMP1BQdUvRNhpaN9fVe272nbI+nzoGeNbjGmBXTp3+HDcp0B9u0AVYvhhYg== X-Received: by 2002:a17:906:58c8:b0:96f:e7cf:5015 with SMTP id e8-20020a17090658c800b0096fe7cf5015mr20449690ejs.17.1686832671834; Thu, 15 Jun 2023 05:37:51 -0700 (PDT) Received: from alrua-x1.borgediget.toke.dk ([45.145.92.2]) by smtp.gmail.com with ESMTPSA id q15-20020a170906940f00b00977eb9957e9sm9378146ejx.128.2023.06.15.05.37.51 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 15 Jun 2023 05:37:51 -0700 (PDT) Received: by alrua-x1.borgediget.toke.dk (Postfix, from userid 1000) id 68D0ABBEC27; Thu, 15 Jun 2023 14:29:54 +0200 (CEST) From: Toke =?utf-8?Q?H=C3=B8iland-J=C3=B8rgensen?= To: Maciej Fijalkowski , intel-wired-lan@lists.osuosl.org Cc: netdev@vger.kernel.org, anthony.l.nguyen@intel.com, magnus.karlsson@intel.com, fred@cloudflare.com, aleksander.lobakin@intel.com, Maciej Fijalkowski Subject: Re: [PATCH v3 iwl-next] ice: allow hot-swapping XDP programs In-Reply-To: <20230615113326.347770-1-maciej.fijalkowski@intel.com> References: <20230615113326.347770-1-maciej.fijalkowski@intel.com> X-Clacks-Overhead: GNU Terry Pratchett Date: Thu, 15 Jun 2023 14:29:54 +0200 Message-ID: <87v8fogb7h.fsf@toke.dk> Precedence: bulk X-Mailing-List: netdev@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 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,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_NONE, SPF_HELO_NONE,SPF_NONE,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 Maciej Fijalkowski writes: > Currently ice driver's .ndo_bpf callback brings interface down and up > independently of XDP resources' presence. This is only needed when > either these resources have to be configured or removed. It means that > if one is switching XDP programs on-the-fly with running traffic, > packets will be dropped. > > To avoid this, compare early on ice_xdp_setup_prog() state of incoming > bpf_prog pointer vs the bpf_prog pointer that is already assigned to > VSI. Do the swap in case VSI has bpf_prog and incoming one are non-NULL. > > Lastly, while at it, put old bpf_prog *after* the update of Rx ring's > bpf_prog pointer. In theory previous code could expose us to a state > where Rx ring's bpf_prog would still be referring to old_prog that got > released with earlier bpf_prog_put(). > > Signed-off-by: Maciej Fijalkowski Acked-by: Toke H=C3=B8iland-J=C3=B8rgensen