[PATCH 0/8] Implement NAPI in et131x · Mark Einon <mark.einon@gmail.com> · 2014-08-20
[PATCH 3/8] staging: et131x: Use for loop to initialise contiguous registers to zero · Mark Einon <mark.einon@gmail.com> · 2014-08-20
Re: [PATCH 3/8] staging: et131x: Use for loop to initialise contiguous registers to zero · Greg KH <gregkh@linuxfoundation.org> · 2014-08-30
Re: [PATCH 3/8] staging: et131x: Use for loop to initialise contiguous registers to zero · Mark Einon <mark.einon@gmail.com> · 2014-08-31
Re: [PATCH 3/8] staging: et131x: Use for loop to initialise contiguous registers to zero · Greg KH <gregkh@linuxfoundation.org> · 2014-08-31
[PATCH 2/8] staging: et131x: Don't handle rx/tx packets when changing mtu · Mark Einon <mark.einon@gmail.com> · 2014-08-20
[PATCH 8/8] staging: et131x: Implement NAPI support · Mark Einon <mark.einon@gmail.com> · 2014-08-20
Re: [PATCH 8/8] staging: et131x: Implement NAPI support · Stephen Hemminger <stephen@networkplumber.org> · 2014-08-21
Re: [PATCH 8/8] staging: et131x: Implement NAPI support · Mark Einon <mark.einon@gmail.com> · 2014-08-21
Re: [PATCH 8/8] staging: et131x: Implement NAPI support · Stephen Hemminger <stephen@networkplumber.org> · 2014-08-21
Re: [PATCH 8/8] staging: et131x: Implement NAPI support · Mark Einon <mark.einon@gmail.com> · 2014-08-21
[PATCH 7/8] staging: et131x: Fix ET_INTR_TXDMA_ISR register name typo · Mark Einon <mark.einon@gmail.com> · 2014-08-20
[PATCH 6/8] staging: et131x: Rename NUM_PACKETS_HANDLED to MAX_PACKETS_HANDLED · Mark Einon <mark.einon@gmail.com> · 2014-08-20
[PATCH 5/8] staging: et131x: Remove unnecessary i2c_wack variable · Mark Einon <mark.einon@gmail.com> · 2014-08-20
Re: [PATCH 5/8] staging: et131x: Remove unnecessary i2c_wack variable · Fabio Estevam <festevam@gmail.com> · 2014-08-20
Re: [PATCH 5/8] staging: et131x: Remove unnecessary i2c_wack variable · Mark Einon <mark.einon@gmail.com> · 2014-08-21
Re: [PATCH 5/8] staging: et131x: Remove unnecessary i2c_wack variable · Fabio Estevam <festevam@gmail.com> · 2014-08-21
Re: [PATCH 5/8] staging: et131x: Remove unnecessary i2c_wack variable · Mark Einon <mark.einon@gmail.com> · 2014-08-21
[PATCH 4/8] staging: et131x: Use for loop to initialise contiguous macstat registers to zero · Mark Einon <mark.einon@gmail.com> · 2014-08-20
RE: [PATCH 4/8] staging: et131x: Use for loop to initialise contiguous macstat registers to zero · David Laight <hidden> · 2014-08-21
Re: [PATCH 4/8] staging: et131x: Use for loop to initialise contiguous macstat registers to zero · Mark Einon <mark.einon@gmail.com> · 2014-08-21
[PATCH 1/8] staging: et131x: Use eth_mac_addr() instead of duplicating the functionality · Mark Einon <mark.einon@gmail.com> · 2014-08-20

Re: [PATCH 8/8] staging: et131x: Implement NAPI support

From: Mark Einon <mark.einon@gmail.com>
Date: 2014-08-21 09:23:12
Also in: lkml

On Wed, Aug 20, 2014 at 08:25:01PM -0700, Stephen Hemminger wrote:

On Wed, 20 Aug 2014 23:17:58 +0100
Mark Einon [off-list ref] wrote:

quoted

 
+	if (budget > MAX_PACKETS_HANDLED)
+		limit = MAX_PACKETS_HANDLED;

Why this artificial restriction?

Hi Stephen, thanks for the review.

It's a restriction that was in the original driver code, and I'm being
cautious. I don't have much documentation for the device, and I haven't
yet figured a way to test the limit so I can play with removing it. If
you have any suggestions on how to do that, I'd be happy to hear them.

Cheers,

Mark

`h`	back out one level
`j`	next message in thread
`k`	previous message in thread
`l`	drill in
`Esc`	close help / fold thread tree
`?`	toggle this help