Re: [ovs-dev] [PATCH ovn 2/2] northd.c: Avoid sending ICMP time exceeded for multicast packets.

2023-04-14 Thread Han Zhou
On Fri, Apr 14, 2023 at 2:16 AM Dumitru Ceara wrote: > > On 4/14/23 04:15, Han Zhou wrote: > > On Thu, Apr 13, 2023 at 12:44 PM Han Zhou wrote: > >> > >> > >> > >> On Thu, Apr 13, 2023 at 12:35 PM Dumitru Ceara wrote: > >>> > >>> On 4/13/23 21:21, Han Zhou wrote: > On Thu, Apr 13, 2023 at

Re: [ovs-dev] [PATCH ovn 2/2] northd.c: Avoid sending ICMP time exceeded for multicast packets.

2023-04-14 Thread Dumitru Ceara
On 4/14/23 04:15, Han Zhou wrote: > On Thu, Apr 13, 2023 at 12:44 PM Han Zhou wrote: >> >> >> >> On Thu, Apr 13, 2023 at 12:35 PM Dumitru Ceara wrote: >>> >>> On 4/13/23 21:21, Han Zhou wrote: On Thu, Apr 13, 2023 at 8:37 AM Dumitru Ceara > wrote: > > On 4/13/23 17:34, Han Zhou

Re: [ovs-dev] [PATCH ovn 2/2] northd.c: Avoid sending ICMP time exceeded for multicast packets.

2023-04-13 Thread Han Zhou
On Thu, Apr 13, 2023 at 12:44 PM Han Zhou wrote: > > > > On Thu, Apr 13, 2023 at 12:35 PM Dumitru Ceara wrote: > > > > On 4/13/23 21:21, Han Zhou wrote: > > > On Thu, Apr 13, 2023 at 8:37 AM Dumitru Ceara wrote: > > >> > > >> On 4/13/23 17:34, Han Zhou wrote: > > >>> On Thu, Apr 13, 2023 at

Re: [ovs-dev] [PATCH ovn 2/2] northd.c: Avoid sending ICMP time exceeded for multicast packets.

2023-04-13 Thread Han Zhou
On Thu, Apr 13, 2023 at 12:35 PM Dumitru Ceara wrote: > > On 4/13/23 21:21, Han Zhou wrote: > > On Thu, Apr 13, 2023 at 8:37 AM Dumitru Ceara wrote: > >> > >> On 4/13/23 17:34, Han Zhou wrote: > >>> On Thu, Apr 13, 2023 at 12:54 AM Dumitru Ceara > > wrote: > > On 4/13/23 07:07, Han

Re: [ovs-dev] [PATCH ovn 2/2] northd.c: Avoid sending ICMP time exceeded for multicast packets.

2023-04-13 Thread Dumitru Ceara
On 4/13/23 21:21, Han Zhou wrote: > On Thu, Apr 13, 2023 at 8:37 AM Dumitru Ceara wrote: >> >> On 4/13/23 17:34, Han Zhou wrote: >>> On Thu, Apr 13, 2023 at 12:54 AM Dumitru Ceara > wrote: On 4/13/23 07:07, Han Zhou wrote: > In RFC1812 section 5.3.1, it is mentioned that: >

Re: [ovs-dev] [PATCH ovn 2/2] northd.c: Avoid sending ICMP time exceeded for multicast packets.

2023-04-13 Thread Han Zhou
On Thu, Apr 13, 2023 at 8:37 AM Dumitru Ceara wrote: > > On 4/13/23 17:34, Han Zhou wrote: > > On Thu, Apr 13, 2023 at 12:54 AM Dumitru Ceara wrote: > >> > >> On 4/13/23 07:07, Han Zhou wrote: > >>> In RFC1812 section 5.3.1, it is mentioned that: > >>> > >>>If the TTL is reduced to zero (or

Re: [ovs-dev] [PATCH ovn 2/2] northd.c: Avoid sending ICMP time exceeded for multicast packets.

2023-04-13 Thread Dumitru Ceara
On 4/13/23 17:34, Han Zhou wrote: > On Thu, Apr 13, 2023 at 12:54 AM Dumitru Ceara wrote: >> >> On 4/13/23 07:07, Han Zhou wrote: >>> In RFC1812 section 5.3.1, it is mentioned that: >>> >>>If the TTL is reduced to zero (or less), the packet MUST be >>>discarded, and if the destination is

Re: [ovs-dev] [PATCH ovn 2/2] northd.c: Avoid sending ICMP time exceeded for multicast packets.

2023-04-13 Thread Han Zhou
On Thu, Apr 13, 2023 at 12:54 AM Dumitru Ceara wrote: > > On 4/13/23 07:07, Han Zhou wrote: > > In RFC1812 section 5.3.1, it is mentioned that: > > > >If the TTL is reduced to zero (or less), the packet MUST be > >discarded, and if the destination is not a multicast address the > >

Re: [ovs-dev] [PATCH ovn 2/2] northd.c: Avoid sending ICMP time exceeded for multicast packets.

2023-04-13 Thread Dumitru Ceara
On 4/13/23 07:07, Han Zhou wrote: > In RFC1812 section 5.3.1, it is mentioned that: > >If the TTL is reduced to zero (or less), the packet MUST be >discarded, and if the destination is not a multicast address the >router MUST send an ICMP Time Exceeded message ... > The code itself

[ovs-dev] [PATCH ovn 2/2] northd.c: Avoid sending ICMP time exceeded for multicast packets.

2023-04-12 Thread Han Zhou
In RFC1812 section 5.3.1, it is mentioned that: If the TTL is reduced to zero (or less), the packet MUST be discarded, and if the destination is not a multicast address the router MUST send an ICMP Time Exceeded message ... So if the destionation is a multicast address the route