> From: netdev-ow...@vger.kernel.org [mailto:netdev- > ow...@vger.kernel.org] On Behalf Of Willem de Bruijn > Sent: Friday, August 25, 2017 8:06 AM > To: Kirsher, Jeffrey T <jeffrey.t.kirs...@intel.com> > Cc: intel-wired-...@lists.osuosl.org; netdev@vger.kernel.org; Brandeburg, > Jesse <jesse.brandeb...@intel.com>; Willem de Bruijn > <will...@google.com> > Subject: [PATCH] e1000e: apply burst mode settings only on default > > From: Willem de Bruijn <will...@google.com> > > Devices that support FLAG2_DMA_BURST have different default values > for RDTR and RADV. Apply burst mode default settings only when no > explicit value was passed at module load. > > The RDTR default is zero. If the module is loaded for low latency > operation with RxIntDelay=0, do not override this value with a burst > default of 32. > > Move the decision to apply burst values earlier, where explicitly > initialized module variables can be distinguished from defaults. > > Signed-off-by: Willem de Bruijn <will...@google.com> > --- > drivers/net/ethernet/intel/e1000e/e1000.h | 4 ---- > drivers/net/ethernet/intel/e1000e/netdev.c | 8 -------- > drivers/net/ethernet/intel/e1000e/param.c | 16 +++++++++++++++- > 3 files changed, 15 insertions(+), 13 deletions(-)
Tested-by: Aaron Brown <aaron.f.br...@intel.com>
