Re: [PATCH net-next 1/6] r8169: change driver data type

[Heiner Kallweit]

On 31.05.2020 07:17, Stephen Hemminger wrote:
> On Sat, 30 May 2020 23:54:36 +0200
> Heiner Kallweit <hkallwe...@gmail.com> wrote:
> 
>> Change driver private data type to struct rtl8169_private * to avoid
>> some overhead.
>>
>> Signed-off-by: Heiner Kallweit <hkallwe...@gmail.com>
>> ---
> 
> Are you sure about this. Using netdev_priv() is actually at a fixed
> offset from netdev, and almost always the compiler can optimize and
> use one register.  Look at the assembly code difference of what you
> did.
> 
When saying overhead I had the source code in mind. With regard to
the generated code you're right, however no hot path is touched here.