From: James Chapman <jchapman@katalix.com>
To: Mandeep Singh Baines <mandeep.baines@gmail.com>
Cc: Daniele Venzano <venza@brownhat.org>,
hadi@cyberus.ca, davem@davemloft.net, rick.jones2@hp.com,
msb@google.com, netdev@vger.kernel.org, grundler@google.com,
robert.olsson@its.uu.se, jeff@garzik.org, nhorman@tuxdriver.com
Subject: Re: [PATCH] [sis900] convert to NAPI, WAS Re: pktgen terminating condition
Date: Wed, 05 Sep 2007 13:03:21 +0100 [thread overview]
Message-ID: <46DE9B09.7060603@katalix.com> (raw)
In-Reply-To: <20070905074411.GA4815@ludhiana>
Mandeep Singh Baines wrote:
> Daniele Venzano (venza@brownhat.org) wrote:
>> The patch looks good and I think it can be pushed higher (-mm ?) for some wider
>> testing. I don't have the hardware available to do some tests myself,
>> unfortunately, but it would be similar to yours anyway.
>>
>> I'd like to know how this works for people with less memory and slower CPU, but any
>> kind of test run will be appreciated.
>
> Hi Daniele,
>
> I tested the driver under different setting of CPU clock. Under a lower clock,
> I get less interrupts (what I was hoping for) and slightly less performance.
> Under higher clock, I get better performance and almost 1 interrupt per packet.
I have a patch that solves the high interrupt rate problem by keeping
the driver in polled mode longer. It's written for the latest NAPI
version that DaveM posted recently. I'll try to get some time to write
it up and post it for comment.
--
James Chapman
Katalix Systems Ltd
http://www.katalix.com
Catalysts for your Embedded Linux software development
next prev parent reply other threads:[~2007-09-05 12:03 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-09-01 10:47 pktgen terminating condition Daniele Venzano
2007-09-04 3:20 ` [PATCH] [sis900] convert to NAPI, WAS " Mandeep Singh Baines
2007-09-04 13:03 ` jamal
2007-09-04 17:21 ` Mandeep Baines
2007-09-04 16:56 ` Daniele Venzano
2007-09-04 17:24 ` Mandeep Baines
2007-09-05 7:44 ` Mandeep Singh Baines
2007-09-05 12:03 ` James Chapman [this message]
2007-09-05 12:33 ` jamal
2007-09-05 13:55 ` James Chapman
2007-09-05 14:21 ` jamal
2007-09-06 5:06 ` Mandeep Singh Baines
2007-09-06 13:22 ` jamal
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=46DE9B09.7060603@katalix.com \
--to=jchapman@katalix.com \
--cc=davem@davemloft.net \
--cc=grundler@google.com \
--cc=hadi@cyberus.ca \
--cc=jeff@garzik.org \
--cc=mandeep.baines@gmail.com \
--cc=msb@google.com \
--cc=netdev@vger.kernel.org \
--cc=nhorman@tuxdriver.com \
--cc=rick.jones2@hp.com \
--cc=robert.olsson@its.uu.se \
--cc=venza@brownhat.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).