Re: [Qemu-devel] [PATCH] e1000 emulation code

andrzej zaborowski
2008-01-10T16:58:26Z

Hi,

On 09/01/2008, Dor Laor <dor.laor@gmail.com> wrote:
> It which supports TCP/UDP and IP transmit checksum, as well as TSO.
> It has been tested with Linux (2.6.18|22|23|24)++ and Windows XP (using
> the driver supplied at the intel download site).
> Windows Vista recognizes it but there is no traffic.
>
> Checksum calculation is currently naïve and unoptimized (the host
> kernel does it better). But when working in conjuction to tso
> the performance is drastically better.
>
> Some figures: Linux rx 350Mbps, tx 150bps, Windows rx 700mbps, tx 100 mbps.

That will be great to have in qemu.

>
> The e1000_hw.h is copied from Linux kernel.

Would it be possible to pick only the useful stuff from the header and
put it into the same file, rather than maintaining a copy of a linux
header (which is 3.5 bigger than the actual emulation in kLoC)? Qemu
has avoided that until now.

> qemu/hw/e1000.c | 1000 +++++++++++++++
> qemu/hw/e1000_hw.h | 3411 ++++++++++++++++++++++++++++++++++++++++++++++++++

Regards.
-------------------------------------------------------------------------
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace
_______________________________________________
kvm-devel mailing list
kvm-devel@lists.sourceforge.net