From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43) id 1G03AR-0002aT-82 for qemu-devel@nongnu.org; Mon, 10 Jul 2006 17:19:47 -0400 Received: from exim by lists.gnu.org with spam-scanned (Exim 4.43) id 1G03AO-0002ZR-2k for qemu-devel@nongnu.org; Mon, 10 Jul 2006 17:19:46 -0400 Received: from [199.232.76.173] (helo=monty-python.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1G03AN-0002ZO-W2 for qemu-devel@nongnu.org; Mon, 10 Jul 2006 17:19:44 -0400 Received: from [64.233.182.185] (helo=nf-out-0910.google.com) by monty-python.gnu.org with esmtp (Exim 4.52) id 1G03BY-0007qO-EX for qemu-devel@nongnu.org; Mon, 10 Jul 2006 17:20:56 -0400 Received: by nf-out-0910.google.com with SMTP id k26so8917nfc for ; Mon, 10 Jul 2006 14:19:42 -0700 (PDT) Message-ID: Date: Tue, 11 Jul 2006 01:19:42 +0400 From: "Igor Kovalenko" Subject: Re: [Qemu-devel] qemu/hw rtl8139.c In-Reply-To: MIME-Version: 1.0 Content-Type: multipart/mixed; boundary="----=_Part_9003_14987149.1152566382436" References: Reply-To: qemu-devel@nongnu.org List-Id: qemu-devel.nongnu.org List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org ------=_Part_9003_14987149.1152566382436 Content-Type: multipart/alternative; boundary="----=_Part_9004_29452283.1152566382436" ------=_Part_9004_29452283.1152566382436 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Content-Disposition: inline On 7/9/06, Igor Kovalenko wrote: > > Hi! > > This updated patch against current CVS implements TCP segmentation > offloading for RTL8139 in C+ mode. > I fixed a couple of problems in implementation (wrong sequence number > calculation), and now TCP performance seem to be normal. > > Dependency on slirp.h header is now gone. > > Again tested with linux (ethtool -K eth0 tx on sg on tso on); please > apply. > > It turned out that a couple of variable types were left dependent on slirp.h and transmit/receive pointers were not reset in right places (required for freebsd re driver) This is an incremental update to fix these problems. ------=_Part_9004_29452283.1152566382436 Content-Type: text/html; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Content-Disposition: inline On 7/9/06, Igor Kovalenko wrote:
Hi!

This updated patch against current CVS implements TCP segmentation offloading for RTL8139 in C+ mode.
I fixed a couple of problems in implementation (wrong sequence number calculation), and now TCP performance seem to be normal.

Dependency on slirp.h header is now gone.

Again tested with linux (ethtool -K eth0 tx on sg on tso on); please apply.


It turned out that a couple of variable types were left dependent on slirp.h
and transmit/receive pointers were not reset in right places (required for freebsd re driver)

This is an incremental update to fix these problems.

------=_Part_9004_29452283.1152566382436-- ------=_Part_9003_14987149.1152566382436 Content-Type: text/x-patch; name=rtl8139-20060711-3.diff; charset=ANSI_X3.4-1968 Content-Transfer-Encoding: base64 X-Attachment-Id: f_ephc8q5a Content-Disposition: attachment; filename="rtl8139-20060711-3.diff" LS0tIGh3L3J0bDgxMzktMjAwNjA3MDktd2l0aC13b3JraW5nLXRzby0yLmMJMjAwNi0wNy0wOSAx Mjo0MTo0Ny4wMDAwMDAwMDAgKzA0MDAKKysrIGh3L3J0bDgxMzkuYwkyMDA2LTA3LTExIDAxOjE1 OjI3LjAwMDAwMDAwMCArMDQwMApAQCAtNDAsNiArNDAsNyBAQAogICogIDIwMDYtSnVsLTA5ICBJ Z29yIEtvdmFsZW5rbyA6ICAgRml4ZWQgVENQIGhlYWRlciBsZW5ndGggY2FsY3VsYXRpb24gd2hp bGUgcHJvY2Vzc2luZwogICogICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgc2VnbWVu dGF0aW9uIG9mZmxvYWRpbmcKICAqICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgIFJl bW92ZWQgc2xpcnAuaCBkZXBlbmRlbmN5CisgKiAgICAgICAgICAgICAgICAgICAgICAgICAgICAg ICAgICBBZGRlZCByeC90eCBidWZmZXIgcmVzZXQgd2hlbiBlbmFibGluZyByeC90eCBvcGVyYXRp b24KICAqLwogCiAjaW5jbHVkZSAidmwuaCIKQEAgLTEzNjgsMTAgKzEzNjksMTQgQEAKICAgICBp ZiAodmFsICYgQ21kUnhFbmIpCiAgICAgewogICAgICAgICBERUJVR19QUklOVCgoIlJUTDgxMzk6 IENoaXBDbWQgZW5hYmxlIHJlY2VpdmVyXG4iKSk7CisKKyAgICAgICAgcy0+Y3VyckNQbHVzUnhE ZXNjID0gMDsKICAgICB9CiAgICAgaWYgKHZhbCAmIENtZFR4RW5iKQogICAgIHsKICAgICAgICAg REVCVUdfUFJJTlQoKCJSVEw4MTM5OiBDaGlwQ21kIGVuYWJsZSB0cmFuc21pdHRlclxuIikpOwor CisgICAgICAgIHMtPmN1cnJDUGx1c1R4RGVzYyA9IDA7CiAgICAgfQogCiAgICAgLyogbWFzayB1 bndyaXRlYWJsZSBiaXRzICovCkBAIC0yMDc4LDggKzIwODMsOCBAQAogICAgICAgICAgICAgLyog aXAgcGFja2V0IGhlYWRlciAqLwogICAgICAgICAgICAgaXBfaGVhZGVyICppcCA9IDA7CiAgICAg ICAgICAgICBpbnQgaGxlbiA9IDA7Ci0gICAgICAgICAgICB1X2ludDhfdCAgaXBfcHJvdG9jb2wg PSAwOwotICAgICAgICAgICAgdV9pbnQxNl90IGlwX2RhdGFfbGVuID0gMDsKKyAgICAgICAgICAg IHVpbnQ4X3QgIGlwX3Byb3RvY29sID0gMDsKKyAgICAgICAgICAgIHVpbnQxNl90IGlwX2RhdGFf bGVuID0gMDsKIAogICAgICAgICAgICAgdWludDhfdCAqZXRoX3BheWxvYWRfZGF0YSA9IDA7CiAg ICAgICAgICAgICBzaXplX3QgICBldGhfcGF5bG9hZF9sZW4gID0gMDsKQEAgLTI0NTAsOCArMjQ1 NSw2IEBACiAgICAgREVCVUdfUFJJTlQoKCJSVEw4MTM5OiBUeEFkZHIgd3JpdGUgb2Zmc2V0PTB4 JXggdmFsPTB4JTA4eFxuIiwgdHhBZGRyT2Zmc2V0LCB2YWwpKTsKIAogICAgIHMtPlR4QWRkclt0 eEFkZHJPZmZzZXQvNF0gPSBsZTMyX3RvX2NwdSh2YWwpOwotCi0gICAgcy0+Y3VyckNQbHVzVHhE ZXNjID0gMDsKIH0KIAogc3RhdGljIHVpbnQzMl90IHJ0bDgxMzlfVHhBZGRyX3JlYWQoUlRMODEz OVN0YXRlICpzLCB1aW50MzJfdCB0eEFkZHJPZmZzZXQpCg== ------=_Part_9003_14987149.1152566382436--