On Thu, Mar 08, 2012 at 07:26:08AM -0800, Pravin B Shelar wrote: > Fixed according to comments from Ben. > v1-v2: > - 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 <[email protected]>
Thanks. 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? Modulo that question, I don't see any problem with the patch. _______________________________________________ dev mailing list [email protected] http://openvswitch.org/mailman/listinfo/dev
