[ovs-dev] [PATCH v2 1/6] netdev-linux: Cache drv-info for net device.
pshelar at nicira.com
Thu Mar 8 15:07:41 PST 2012
On Thu, Mar 8, 2012 at 2:57 PM, Ben Pfaff <blp at nicira.com> wrote:
> On Thu, Mar 08, 2012 at 07:26:08AM -0800, Pravin B Shelar wrote:
>> Fixed according to comments from Ben.
>> - Added comment for netdev_internal_open
>> - Removed get-stat call from status check.
>> --8<--------------------------cut here-------------------------->8--
>> Netdev-linux calls ETHTOOL_GDRVINFO on every netdev_linux_get_status()
>> which is not optimal as drv-info does not change for given device.
>> So following patch changes netdev_linux_get_status() to read drv-info at
>> device initialization and cache it.
>> Signed-off-by: Pravin B Shelar <pshelar at nicira.com>
> This series has a number of patches with the similar purposes of
> increasing the effectiveness of caching in netdev-linux. They tend to
> follow the same pattern. But this patch stands out as the only one
> that populates the cache proactively, before the client asks for it.
> Why is drvinfo different? That is, if we have to populate drvinfo
> proactively, why don't we have to populate the other values
> proactively too?
I didn't do it as we always cache drv-info info. If you want I can add
caching for error code.
More information about the dev