Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:2 as permitted sender) client-ip=2620:137:e000::3:2;
From:   "Pucha, HimasekharX Reddy" <himasekharx.reddy.pucha@intel.com>
To:     ivecera <ivecera@redhat.com>,
        "netdev@vger.kernel.org" <netdev@vger.kernel.org>
CC:     "intel-wired-lan@lists.osuosl.org" <intel-wired-lan@lists.osuosl.org>,
        "Brandeburg, Jesse" <jesse.brandeburg@intel.com>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        Eric Dumazet <edumazet@google.com>,
        "Nguyen, Anthony L" <anthony.l.nguyen@intel.com>,
        "Keller, Jacob E" <jacob.e.keller@intel.com>,
        Jakub Kicinski <kuba@kernel.org>,
        Paolo Abeni <pabeni@redhat.com>,
        "David S. Miller" <davem@davemloft.net>
Subject: RE: [Intel-wired-lan] [PATCH net-next 2/2] i40e: Fix devlink port
 unregistering
Thread-Topic: [Intel-wired-lan] [PATCH net-next 2/2] i40e: Fix devlink port
 unregistering
Thread-Index: AQHaBnjrR+XAm/L5CUKunACDYy6s7rBmjoqg
Date:   Thu, 2 Nov 2023 05:33:25 +0000
Message-ID: <BL0PR11MB31226F590810C63CB53B151FBDA6A@BL0PR11MB3122.namprd11.prod.outlook.com>
References: <20231024125109.844045-1-ivecera@redhat.com>
 <20231024125109.844045-2-ivecera@redhat.com>
In-Reply-To: <20231024125109.844045-2-ivecera@redhat.com>
Accept-Language: en-US
Content-Language: en-US
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?WtzTJFZO0hBegvqvikw0MfFBFG1vi/n4Y8za1uKYqzOoZk/W6rLztzevY5XL?=
 =?us-ascii?Q?h0ZGIFlJGz9l8W+ooe9sUhxtLaZRpJ/X5njH6PW1iN8X2OwmHstcYtOgR9ft?=
 =?us-ascii?Q?1eJPgZ8oGJ5HtnC+lv9waNFiamlzB1Zj3GoaNIMbAH7GBPveeIHb355jLbYb?=
 =?us-ascii?Q?jbJh1yllRO+wfV+lGnEOdu7VrZtBT/Sx5TNx7/JzWcNMyJl2XpXbC8XPlSQV?=
 =?us-ascii?Q?2AvqKjBEJbcjtk+35oA4X70fSgTa5L9zdySFq/QGBVdyKUnKJUlNaUZeOTKJ?=
 =?us-ascii?Q?2VJUuVyY0pOYMc2efmmqhjsEeEbsCR7ds/94v7f6QSL3vlCDjpHCEiaOaoF3?=
 =?us-ascii?Q?VLqoXh7dIr1Z9mT4MHseXEh87JzvyGTUXcaYx6Sm0lmUkQ+he7iIRAsV3d3Y?=
 =?us-ascii?Q?BFytE98u0ZUEPhdTX7HREfoi/SQ3JLG6dko6ArHcNHOalvA0y34EWVhbhBQ4?=
 =?us-ascii?Q?OM68vDViongrkvayRZMq7nqXFXB1g1gM303bpXuLi/sCQH0c396c3Yd+syoo?=
 =?us-ascii?Q?MOSJ+wforR9TWaWZlLHfB4Fc10c6IhRMCas+BLSDdh+d56f87q9pqRuiqYES?=
 =?us-ascii?Q?T+ujehqOZSbF0mpdynNiOAZrHgbLf51LQ+YpXya+Sms2pCmNsMOAuqZiGUzt?=
 =?us-ascii?Q?UPXCOTwe8V6uRsIS1V8RJOABERZFtnyhIrtz9bX9bJB9i++ydTwXzXt0bAeh?=
 =?us-ascii?Q?pqG868sVXDxWn9u9m/IQoohI+dF482VjfH2F2tEr2wxgCV6aEBxrq7xaR8q0?=
 =?us-ascii?Q?hKh6tkj8jJqgMU540aiT+8Mk7UVMi2GFwkOLyj0ei1SZWM8uvHjghDfpowzq?=
 =?us-ascii?Q?2S18NHpKMAAuoDjIqBZYZ+T1fMVG1o8avyPb65ep+SqaK5tZqp7FCZsMynNY?=
 =?us-ascii?Q?mIN4W67bt0I4H0tcd7p0aUPl0yyS4de/PECAXLkTYxDnqOqjPFxADgTlyoLq?=
 =?us-ascii?Q?fFP5ygS4h9JfAVHGgapRpQ9BVCCjnjkNNTvv/T9qz2wRujau2HcZYgoWi9ta?=
 =?us-ascii?Q?PND5bEuLNOU6Plhd493ctRT0FQ/vRLyA/UWN1zh3RqOK2DFuA4lrY7xRe334?=
 =?us-ascii?Q?byCT7vGinfsdTvba7Nv2mzGgmaam7sijISV+OdKBM31Hq51VWsUbKGTJMUaS?=
 =?us-ascii?Q?hQ1otg7c2LkEfN5IZLlJG36+OvZonoPX8pH9e2uQvPKwuTuEJxPjmlIXi4ls?=
 =?us-ascii?Q?QEnbTLBQ+MycU/uoZlpocl0JMQRwGfggQ0Cshnv+NwXhEPZI++mDysD8boTK?=
 =?us-ascii?Q?lt8EFMyrIPD2HAJPgbiCnBo/kJnanT/N1uoNL87VqwU359Wmtpca5ko+wDCy?=
 =?us-ascii?Q?IMfBHpL6bSmWQ5qpAH0/6JkU6FVKuc5XpF42qIkFqpyuOabQczRu0o9A4ZzZ?=
 =?us-ascii?Q?MyDIC7Cw3f1UiC+4oIXb9EKrrvYkiSepI/kfu6HcoZIMmdM3u/QxCHKuyA2p?=
 =?us-ascii?Q?Ira8VmR8YI1r1f0Efp6FM4BNPyPE9lQ7Mvy/HMQ3YN/eT1SKc8LwmRU5WyWt?=
 =?us-ascii?Q?zLeNfQffwwImZYmddKzSS4bM4aaRK69hglADIkYa9c0hiByfXLkEpDb6hXzP?=
 =?us-ascii?Q?fIqSfgJZSGsQ/qU7lNlhJhqq6OgmDdPgrFQTZuW5+uyp0QlL44q935WAUDm4?=
 =?us-ascii?Q?TA=3D=3D?=
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: BL0PR11MB3122.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 2df011f1-ee22-4aa5-c1a0-08dbdb653c91
X-MS-Exchange-CrossTenant-originalarrivaltime: 02 Nov 2023 05:33:25.6897
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 46c98d88-e344-4ed4-8496-4ed7712e255d
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: 25rCP4B1fm/AbcdMN2UmlkRdc1uG95L+R7ZJMQN+vNCMlTC17FH9w1mXmbNS0OkXYx4h7K1le+kI5btl8iaRC190zpYYX3ZaRLxsxed10Yc+joyoH6Zfv/gFQ2txMUd8
X-MS-Exchange-Transport-CrossTenantHeadersStamped: SJ2PR11MB8515
X-OriginatorOrg: intel.com
X-Spam-Status: No, score=-1.3 required=5.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED,
	DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,
	MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,T_SCC_BODY_TEXT_LINE,
	URIBL_BLOCKED autolearn=unavailable autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on agentk.vger.email
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org
X-Greylist: Sender passed SPF test, not delayed by milter-greylist-4.6.4 (agentk.vger.email [0.0.0.0]); Wed, 01 Nov 2023 22:33:44 -0700 (PDT)

> -----Original Message-----
> From: Intel-wired-lan <intel-wired-lan-bounces@osuosl.org> On Behalf Of I=
van Vecera
> Sent: Tuesday, October 24, 2023 6:21 PM
> To: netdev@vger.kernel.org
> Cc: intel-wired-lan@lists.osuosl.org; Brandeburg, Jesse <jesse.brandeburg=
@intel.com>; linux-kernel@vger.kernel.org; Eric Dumazet <edumazet@google.co=
m>; Nguyen, Anthony L <anthony.l.nguyen@intel.com>; Keller, Jacob E <jacob.=
e.keller@intel.com>; Jakub Kicinski <kuba@kernel.org>; Paolo Abeni <pabeni@=
redhat.com>; David S. Miller <davem@davemloft.net>
> Subject: [Intel-wired-lan] [PATCH net-next 2/2] i40e: Fix devlink port un=
registering
>
> Ensure that devlink port is unregistered after unregistering
> of net device.
>
> Reproducer:
> [root@host ~]# rmmod i40e
> [ 4742.939386] i40e 0000:02:00.1: i40e_ptp_stop: removed PHC on enp2s0f1n=
p1
> [ 4743.059269] ------------[ cut here ]------------
> [ 4743.063900] WARNING: CPU: 21 PID: 10766 at net/devlink/port.c:1078 dev=
l_port_unregister+0x69/0x80
> ...
>
> Fixes: 9e479d64dc58 ("i40e: Add initial devlink support")
> Signed-off-by: Ivan Vecera <ivecera@redhat.com>
> ---
>  drivers/net/ethernet/intel/i40e/i40e_main.c | 10 ++++++----
>  1 file changed, 6 insertions(+), 4 deletions(-)
>

Tested-by: Pucha Himasekhar Reddy <himasekharx.reddy.pucha@intel.com> (A Co=
ntingent worker at Intel)