Re: [PATCH] net: neigh: disallow state transition DELAY->STALE in neigh_update()

From: Julian Anastasov
Date: Mon Jul 25 2016 - 14:58:06 EST



Hello,

On Mon, 25 Jul 2016, åèèæ wrote:

> OK, following blocks are "no-op" and we will get same result.
>
> Well, please do not try changing several things at the same time and
> you could say:
>
> if (ladder == neigh->ha && new == NUD_STALE &&
> !(flags & NUD_UPDATE_F_ADMIN))
> new = old;

OK, lets do it with 2 patches then.

Chunhui He, can you modify your patch to delete the
both lines and explain that we prefer to resolve the
remote address, even while remote packets try to set NUD_STALE
state. If your patch is accepted, I'll post second patch that
adds the line with the ADMIN check. As result, the code will
look like the example from Yoshifuji Hideaki above.

Regards

--
Julian Anastasov <ja@xxxxxx>