Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1503829

Re: [Intel-wired-lan] [PATCH] igb: add missing fields to TXDCTL-register

Show all headers | View raw

[Multipart message — attachments visible in raw view] - view raw

On Wed, Oct 19, 2016 at 07:25:10AM -0700, Jesse Brandeburg wrote:
> On Wed, 19 Oct 2016 14:37:59 +0200
> Henrik Austad <henrik@austad.us> wrote:
> 
> > The current list of E1000_TXDCTL-registers is incomplete. This adds
> > the missing parts for the Transmit Descriptor Control (TXDCTL)
> > register.
> > 
> > The rest of these values (threshold for descriptor read/write) for
> > TXDCTL seems to be defined in igb/igb.h, not sure why this is split
> > though.
> 
> Hi Henrik, thanks for helping with our code.
> 
> While totally correct, having defines added to the kernel that are not
> being used anywhere in the code isn't really very useful.  Often the
> upstream maintainers/reviewers will reject a patch like this that just
> adds to a .h file, because there are no actual users of the defines.

Yes, I agree, best to avoid bloat whenever possible.

> If the transmit or ethtool code were to use these (via the same patch)
> or something like that, then the patch would be more likely to be
> accepted.

Ah, good to know. I am in the process of spinning out a new set of 
TSN-patches (previous version: see [1]) and setting the priority-bit for 
the Tx-queues is required. This means that I'm hacking more at igb_main.c.

So this was more about laying the groundwork for the series.

I'll leave this patch in the tsn-series then, and resend once I'm ready and 
hope you can provide some feedback on the rest of the series then :)

> Jesse
> 
> PS In the future no need to copy linux-kernel for patches going to our
> submaintainer list.

Ok, I'll remember that, thanks!

1) https://lkml.org/lkml/2016/6/11/187

-- 
Henrik Austad

Back to linux.kernel | Previous | Next — Previous in thread | Find similar | Unroll thread

Thread

Re: [Intel-wired-lan] [PATCH] igb: add missing fields to  TXDCTL-register Jesse Brandeburg <jesse.brandeburg@intel.com> - 2016-10-19 16:30 +0200
  Re: [Intel-wired-lan] [PATCH] igb: add missing fields to  TXDCTL-register Henrik Austad <henrik@austad.us> - 2016-10-19 17:10 +0200

csiph-web