[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] e1000: Pad short frames to minimum size (60 byt
From: |
Stefan Hajnoczi |
Subject: |
Re: [Qemu-devel] [PATCH] e1000: Pad short frames to minimum size (60 bytes) |
Date: |
Sun, 19 Sep 2010 07:36:51 +0100 |
On Sat, Sep 18, 2010 at 10:27 PM, Edgar E. Iglesias
<address@hidden> wrote:
> This doesn't look right. AFAIK, MAC's dont pad on receive.
I agree. NICs that do padding will do it on transmit, not receive.
Anything coming in on the wire should already have the minimum length.
In QEMU that isn't true today and that's why rtl8139, pcnet, and
ne2000 already do this same padding. This patch is the smallest
change to cover e1000.
> IMO this kind of padding should somehow be done by the bridge that forwards
> packets into the qemu vlan (e.g slirp or the generic tap bridge).
That should work and we can then drop the padding code from existing
NICs. I'll take a look.
Stefan
- [Qemu-devel] [PATCH] e1000: Pad short frames to minimum size (60 bytes), Stefan Hajnoczi, 2010/09/18
- Re: [Qemu-devel] [PATCH] e1000: Pad short frames to minimum size (60 bytes), Edgar E. Iglesias, 2010/09/18
- Re: [Qemu-devel] [PATCH] e1000: Pad short frames to minimum size (60 bytes),
Stefan Hajnoczi <=
- Re: [Qemu-devel] [PATCH] e1000: Pad short frames to minimum size (60 bytes), Kevin Wolf, 2010/09/20
- Re: [Qemu-devel] [PATCH] e1000: Pad short frames to minimum size (60 bytes), Edgar E. Iglesias, 2010/09/20
- Re: [Qemu-devel] [PATCH] e1000: Pad short frames to minimum size (60 bytes), Stefan Hajnoczi, 2010/09/20
- Re: [Qemu-devel] [PATCH] e1000: Pad short frames to minimum size (60 bytes), Michael S. Tsirkin, 2010/09/20
- Re: [Qemu-devel] [PATCH] e1000: Pad short frames to minimum size (60 bytes), Anthony Liguori, 2010/09/20
- Re: [Qemu-devel] [PATCH] e1000: Pad short frames to minimum size (60 bytes), Edgar E. Iglesias, 2010/09/20
- Re: [Qemu-devel] [PATCH] e1000: Pad short frames to minimum size (60 bytes), Michael S. Tsirkin, 2010/09/20