> -----Original Message----- > From: Kuniyuki Iwashima <[email protected]> > Sent: Saturday, July 19, 2025 4:48 PM > To: Jakub Kicinski <[email protected]> > Cc: Haiyang Zhang <[email protected]>; linux- > [email protected]; [email protected]; Haiyang Zhang > <[email protected]>; KY Srinivasan <[email protected]>; > [email protected]; [email protected]; [email protected]; > [email protected]; [email protected]; [email protected]; > [email protected]; [email protected]; linux- > [email protected] > Subject: [EXTERNAL] Re: [PATCH net] net: core: Fix the loop in > default_device_exit_net() > > [You don't often get email from [email protected]. Learn why this is > important at https://aka.ms/LearnAboutSenderIdentification ] > > On Fri, Jul 18, 2025 at 4:37 PM Jakub Kicinski <[email protected]> wrote: > > > > On Fri, 18 Jul 2025 13:20:14 -0700 Haiyang Zhang wrote: > > > The loop in default_device_exit_net() won't be able to properly detect > the > > > head then stop, and will hit NULL pointer, when a driver, like > hv_netvsc, > > > automatically moves the slave device together with the master device. > > > > > > To fix this, add a helper function to return the first migratable > netdev > > > correctly, no matter one or two devices were removed from this net's > list > > > in the last iteration. > > > > FTR I think that what the driver is trying to do is way too hacky, and > > it should be fixed instead. But I defer to Kuniyuki for the final word, > > maybe this change is useful for other reasons.. > > I agree that it should be fixed on the driver side. I don't > think of a good reason for the change.
Kuniyuki and Jakub: Thanks for the reviews. I'm working on a patch that will fix the driver side. - Haiyang
