RE: [PATCH net-next 1/2] virtio_net: Fix short frame length check

2023-01-14 Thread Parav Pandit via Virtualization
> From: Alexander Duyck > Sent: Friday, January 13, 2023 7:24 PM > > On Fri, Jan 13, 2023 at 3:37 PM Parav Pandit wrote: > > > > > > > From: Alexander H Duyck > > > Sent: Friday, January 13, 2023 6:24 PM > > > > > > On Sat, 2023-01-14 at 00:36 +0200, Parav Pandit wrote: > > > > A smallest Ethe

Re: [PATCH net-next 1/2] virtio_net: Fix short frame length check

2023-01-13 Thread Alexander Duyck
On Fri, Jan 13, 2023 at 4:23 PM Alexander Duyck wrote: > > On Fri, Jan 13, 2023 at 3:37 PM Parav Pandit wrote: > > > > > > > From: Alexander H Duyck > > > Sent: Friday, January 13, 2023 6:24 PM > > > > > > On Sat, 2023-01-14 at 00:36 +0200, Parav Pandit wrote: > > > > A smallest Ethernet frame d

Re: [PATCH net-next 1/2] virtio_net: Fix short frame length check

2023-01-13 Thread Alexander Duyck
On Fri, Jan 13, 2023 at 3:37 PM Parav Pandit wrote: > > > > From: Alexander H Duyck > > Sent: Friday, January 13, 2023 6:24 PM > > > > On Sat, 2023-01-14 at 00:36 +0200, Parav Pandit wrote: > > > A smallest Ethernet frame defined by IEEE 802.3 is 60 bytes without > > > any preemble and CRC. > > >

RE: [PATCH net-next 1/2] virtio_net: Fix short frame length check

2023-01-13 Thread Parav Pandit via Virtualization
> From: Alexander H Duyck > Sent: Friday, January 13, 2023 6:24 PM > > On Sat, 2023-01-14 at 00:36 +0200, Parav Pandit wrote: > > A smallest Ethernet frame defined by IEEE 802.3 is 60 bytes without > > any preemble and CRC. > > > > Current code only checks for minimal 14 bytes of Ethernet header

Re: [PATCH net-next 1/2] virtio_net: Fix short frame length check

2023-01-13 Thread Alexander H Duyck
On Sat, 2023-01-14 at 00:36 +0200, Parav Pandit wrote: > A smallest Ethernet frame defined by IEEE 802.3 is 60 bytes without any > preemble and CRC. > > Current code only checks for minimal 14 bytes of Ethernet header length. > Correct it to consider the minimum Ethernet frame length. > > Fixes: