Hi Philippe, On Mon, Mar 15, 2021 at 5:13 PM Philippe Mathieu-Daudé <phi...@redhat.com> wrote: > > On 3/15/21 8:57 AM, Bin Meng wrote: > > Add a new macro ETH_ZLEN which represents the minimum size of an > > Ethernet frame without FCS. > > > > Signed-off-by: Bin Meng <bmeng...@gmail.com> > > --- > > > > include/net/eth.h | 1 + > > 1 file changed, 1 insertion(+) > > > > diff --git a/include/net/eth.h b/include/net/eth.h > > index 0671be6916..7c825ecb2f 100644 > > --- a/include/net/eth.h > > +++ b/include/net/eth.h > > @@ -31,6 +31,7 @@ > > > > #define ETH_ALEN 6 > > #define ETH_HLEN 14 > > +#define ETH_ZLEN 60 /* Min. octets in frame sans FCS */ > > What means 'sans'?
sans-serif font? Does that sound familiar? :) Please check: https://www.dictionary.com/browse/sans This comment was not invented by me, but was just a copy from the one used in Linux kernel: https://github.com/torvalds/linux/blob/d635a69dd4981cc51f90293f5f64268620ed1565/include/uapi/linux/if_ether.h#L35 > > Otherwise: > Reviewed-by: Philippe Mathieu-Daudé <phi...@redhat.com> > Regards, Bin
