[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <EDC0E76513226749BFBC9C3FB031318F016F501D8D@orsmsx508.amr.corp.intel.com>
Date: Tue, 21 Jun 2011 10:23:05 -0700
From: "Wyborny, Carolyn" <carolyn.wyborny@...el.com>
To: David Miller <davem@...emloft.net>
CC: "netdev@...r.kernel.org" <netdev@...r.kernel.org>,
"bhutchings@...arflare.com" <bhutchings@...arflare.com>
Subject: RE: [RFC 2/2] ethtool: Add support for DMA Coalescing feature
config to ethtool.
>-----Original Message-----
>From: David Miller [mailto:davem@...emloft.net]
>Sent: Friday, June 17, 2011 11:54 AM
>To: Wyborny, Carolyn
>Cc: netdev@...r.kernel.org; bhutchings@...arflare.com
>Subject: Re: [RFC 2/2] ethtool: Add support for DMA Coalescing feature
>config to ethtool.
>
>From: "Wyborny, Carolyn" <carolyn.wyborny@...el.com>
>Date: Fri, 17 Jun 2011 08:50:11 -0700
>
>> I will add a fuller description of the feature in my updated patch.
>> I thought the feature was more well known. Quick description is that
>> it's a power saving feature that causes the adapter to coalesce its
>> DMA writes at low traffic times to save power on the platform by
>> reducing wakeups. The parameter is intended as a simple u32 value,
>> not just an on or off, but also to allow a variety of configuration
>> by adapter vendors, with validation of the input on the driver side.
>> Since I left out the implementation in my patch, this wasn't clear.
>> I will also fix this in my next submission.
>
>The value cannot have adapter specific meaning, you must define it
>precisely and in a generic manner, such that the user can specify the
>same setting across different card types.
Ok, good point. I will refine the definition of the parameter in the next submission, once the dust clears on the major revisions in progress.
Thanks,
Carolyn
Carolyn Wyborny
Linux Development
LAN Access Division
Intel Corporation
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists