* [PATCH v3] neighbour: confirm neigh entries when ARP packet is received
@ 2018-09-13 18:12 Vasily Khoruzhick
2018-09-13 18:26 ` Sergei Shtylyov
0 siblings, 1 reply; 4+ messages in thread
From: Vasily Khoruzhick @ 2018-09-13 18:12 UTC (permalink / raw)
To: David S. Miller, Roopa Prabhu, Alexey Dobriyan, Eric Dumazet,
Stephen Hemminger, Jim Westfall, Wolfgang Bumiller,
Vasily Khoruzhick, Kees Cook, Ihar Hrachyshka, netdev,
linux-kernel, Sergei Shtylyov
Cc: Vasily Khoruzhick
Update 'confirmed' timestamp when ARP packet is received. It shouldn't
affect locktime logic and anyway entry can be confirmed by any higher-layer
protocol. Thus it makes sense to confirm it when ARP packet is received.
Fixes: 77d7123342 ("neighbour: update neigh timestamps iff update is effective")
Signed-off-by: Vasily Khoruzhick <vasilykh@arista.com>
---
v2: - update comment to match new code.
v3: - fix wording in commit message, make 'Fixes' tag one line, remove extra new line
between 'Fixes' and 'Signed-off-by'.
net/core/neighbour.c | 13 ++++++++-----
1 file changed, 8 insertions(+), 5 deletions(-)
diff --git a/net/core/neighbour.c b/net/core/neighbour.c
index aa19d86937af..91592fceeaad 100644
--- a/net/core/neighbour.c
+++ b/net/core/neighbour.c
@@ -1180,6 +1180,12 @@ int neigh_update(struct neighbour *neigh, const u8 *lladdr, u8 new,
lladdr = neigh->ha;
}
+ /* Update confirmed timestamp for neighbour entry after we
+ * received ARP packet even if it doesn't change IP to MAC binding.
+ */
+ if (new & NUD_CONNECTED)
+ neigh->confirmed = jiffies;
+
/* If entry was valid and address is not changed,
do not change entry state, if new one is STALE.
*/
@@ -1201,15 +1207,12 @@ int neigh_update(struct neighbour *neigh, const u8 *lladdr, u8 new,
}
}
- /* Update timestamps only once we know we will make a change to the
+ /* Update timestamp only once we know we will make a change to the
* neighbour entry. Otherwise we risk to move the locktime window with
* noop updates and ignore relevant ARP updates.
*/
- if (new != old || lladdr != neigh->ha) {
- if (new & NUD_CONNECTED)
- neigh->confirmed = jiffies;
+ if (new != old || lladdr != neigh->ha)
neigh->updated = jiffies;
- }
if (new != old) {
neigh_del_timer(neigh);
--
2.18.0
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH v3] neighbour: confirm neigh entries when ARP packet is received
2018-09-13 18:12 [PATCH v3] neighbour: confirm neigh entries when ARP packet is received Vasily Khoruzhick
@ 2018-09-13 18:26 ` Sergei Shtylyov
2018-09-13 19:02 ` David Miller
0 siblings, 1 reply; 4+ messages in thread
From: Sergei Shtylyov @ 2018-09-13 18:26 UTC (permalink / raw)
To: Vasily Khoruzhick, David S. Miller, Roopa Prabhu,
Alexey Dobriyan, Eric Dumazet, Stephen Hemminger, Jim Westfall,
Wolfgang Bumiller, Vasily Khoruzhick, Kees Cook, Ihar Hrachyshka,
netdev, linux-kernel
Hello!
On 09/13/2018 09:12 PM, Vasily Khoruzhick wrote:
> Update 'confirmed' timestamp when ARP packet is received. It shouldn't
> affect locktime logic and anyway entry can be confirmed by any higher-layer
> protocol. Thus it makes sense to confirm it when ARP packet is received.
>
> Fixes: 77d7123342 ("neighbour: update neigh timestamps iff update is effective")
Need at least 12 hex digits... Perhaps could be fixed when applying?
> Signed-off-by: Vasily Khoruzhick <vasilykh@arista.com>
[...]
MBR, Sergei
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH v3] neighbour: confirm neigh entries when ARP packet is received
2018-09-13 18:26 ` Sergei Shtylyov
@ 2018-09-13 19:02 ` David Miller
2018-09-13 19:10 ` Vasiliy Khoruzhick
0 siblings, 1 reply; 4+ messages in thread
From: David Miller @ 2018-09-13 19:02 UTC (permalink / raw)
To: sergei.shtylyov
Cc: vasilykh, roopa, adobriyan, edumazet, stephen, jwestfall,
w.bumiller, anarsoul, keescook, ihrachys, netdev, linux-kernel
From: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>
Date: Thu, 13 Sep 2018 21:26:42 +0300
> Hello!
>
> On 09/13/2018 09:12 PM, Vasily Khoruzhick wrote:
>
>> Update 'confirmed' timestamp when ARP packet is received. It shouldn't
>> affect locktime logic and anyway entry can be confirmed by any higher-layer
>> protocol. Thus it makes sense to confirm it when ARP packet is received.
>>
>> Fixes: 77d7123342 ("neighbour: update neigh timestamps iff update is effective")
>
> Need at least 12 hex digits... Perhaps could be fixed when applying?
Yeah I took care of that.
>
>> Signed-off-by: Vasily Khoruzhick <vasilykh@arista.com>
> [...]
Applied and queued up for -stable.
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH v3] neighbour: confirm neigh entries when ARP packet is received
2018-09-13 19:02 ` David Miller
@ 2018-09-13 19:10 ` Vasiliy Khoruzhick
0 siblings, 0 replies; 4+ messages in thread
From: Vasiliy Khoruzhick @ 2018-09-13 19:10 UTC (permalink / raw)
To: David Miller
Cc: sergei.shtylyov, Roopa Prabhu, Alexey Dobriyan, Eric Dumazet,
Stephen Hemminger, Jim Westfall, Wolfgang Bumiller,
Vasily Khoruzhick, Kees Cook, Ihar Hrachyshka, netdev,
linux-kernel
On Thu, Sep 13, 2018 at 12:02 PM, David Miller <davem@davemloft.net> wrote:
> From: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>
> Date: Thu, 13 Sep 2018 21:26:42 +0300
>
>> Hello!
>>
>> On 09/13/2018 09:12 PM, Vasily Khoruzhick wrote:
>>
>>> Update 'confirmed' timestamp when ARP packet is received. It shouldn't
>>> affect locktime logic and anyway entry can be confirmed by any higher-layer
>>> protocol. Thus it makes sense to confirm it when ARP packet is received.
>>>
>>> Fixes: 77d7123342 ("neighbour: update neigh timestamps iff update is effective")
>>
>> Need at least 12 hex digits... Perhaps could be fixed when applying?
>
> Yeah I took care of that.
>
>>
>>> Signed-off-by: Vasily Khoruzhick <vasilykh@arista.com>
>> [...]
>
> Applied and queued up for -stable.
Thanks!
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2018-09-13 19:10 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-09-13 18:12 [PATCH v3] neighbour: confirm neigh entries when ARP packet is received Vasily Khoruzhick
2018-09-13 18:26 ` Sergei Shtylyov
2018-09-13 19:02 ` David Miller
2018-09-13 19:10 ` Vasiliy Khoruzhick
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).