Messages in this thread | | | Date | Thu, 21 Aug 2014 10:23:05 +0100 | From | Mark Einon <> | Subject | Re: [PATCH 8/8] staging: et131x: Implement NAPI support |
| |
On Wed, Aug 20, 2014 at 08:25:01PM -0700, Stephen Hemminger wrote: > On Wed, 20 Aug 2014 23:17:58 +0100 > Mark Einon <mark.einon@gmail.com> wrote: > > > > > + 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
| |