[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <8c67377e-888e-4c90-85a6-33983b323029@lunn.ch>
Date: Sun, 16 Jun 2024 16:59:22 +0200
From: Andrew Lunn <andrew@...n.ch>
To: FUJITA Tomonori <fujita.tomonori@...il.com>
Cc: hfdevel@....net, kuba@...nel.org, netdev@...r.kernel.org,
horms@...nel.org, jiri@...nulli.us, pabeni@...hat.com,
linux@...linux.org.uk, naveenm@...vell.com, jdamato@...tly.com
Subject: Re: [PATCH net-next v10 4/7] net: tn40xx: add basic Tx handling
On Sun, Jun 16, 2024 at 09:46:22PM +0900, FUJITA Tomonori wrote:
> On Sat, 15 Jun 2024 20:33:04 +0200
> Andrew Lunn <andrew@...n.ch> wrote:
>
> > I did wounder if calculating the value as needed would be any
> > slower/faster than doing a table access. Doing arithmetic is very
> > cheap compared to a cache miss for a table lookup. Something which
> > could be bench marked and optimised later.
>
> Indeed, that might be faster. I'll drop the table in the next
> version. I'll put that experiment on my to-do list.
I would generally recommend getting something merged, and then
optimise it. This could be in the noise, it makes no real difference.
By getting code merged, you make it easier for others to contribute to
this driver, and there does appear to be others who would like to work
on this code.
Andrew
Powered by blists - more mailing lists