Re: [PATCH net-next] net: ppp: remove ppp->closing check

From: Qingfang Deng
Date: Fri Nov 08 2024 - 01:10:20 EST


Hi Paolo,

On Thu, Nov 7, 2024 at 8:08 PM Paolo Abeni <pabeni@xxxxxxxxxx> wrote:
>
>
>
> On 11/4/24 10:24, Qingfang Deng wrote:
> > ppp->closing was used to test if an interface is closing down. But upon
> > .ndo_uninit() where ppp->closing is set to 1, dev_close() is already
> > called to bring down an interface and a synchronize_net() guarantees
> > that no pending TX/RX can take place, so the check is unnecessary.
> > Remove the check.
>
> I'm unsure we can remote such check. The TX callback can be triggered
> even from a write on the controlling file, and it looks like such file
> will be untouched by uninit.

ppp_release (when the file is closed) calls unregister_netdevice, and
no more writes can happen after that.

>
> Cheers,
>
> Paolo
>