* [Intel-wired-lan] [PATCH] add one parameter wro_enable to enable relaxed ordering for IXGBE @ 2016-10-29 7:08 ` Mao Wenan 0 siblings, 0 replies; 8+ messages in thread From: Mao Wenan @ 2016-10-29 7:08 UTC (permalink / raw) To: intel-wired-lan This patch provides a way to enable relaxed ordering, where it helps with performance in some architecture. The default value of wro_enable is 0, if you want to enable relaxed ordering, please set wro_enable=1. Mao Wenan (1): add one parameter wro_enable for IXGBE drivers/net/ethernet/intel/ixgbe/ixgbe.h | 1 + drivers/net/ethernet/intel/ixgbe/ixgbe_82598.c | 29 ++++++++++++++----------- drivers/net/ethernet/intel/ixgbe/ixgbe_common.c | 28 +++++++++++++----------- drivers/net/ethernet/intel/ixgbe/ixgbe_main.c | 9 ++++++++ 4 files changed, 41 insertions(+), 26 deletions(-) -- 2.5.0 ^ permalink raw reply [flat|nested] 8+ messages in thread
* [PATCH] add one parameter wro_enable to enable relaxed ordering for IXGBE @ 2016-10-29 7:08 ` Mao Wenan 0 siblings, 0 replies; 8+ messages in thread From: Mao Wenan @ 2016-10-29 7:08 UTC (permalink / raw) To: jeffrey.t.kirsher, intel-wired-lan, netdev, linux-kernel This patch provides a way to enable relaxed ordering, where it helps with performance in some architecture. The default value of wro_enable is 0, if you want to enable relaxed ordering, please set wro_enable=1. Mao Wenan (1): add one parameter wro_enable for IXGBE drivers/net/ethernet/intel/ixgbe/ixgbe.h | 1 + drivers/net/ethernet/intel/ixgbe/ixgbe_82598.c | 29 ++++++++++++++----------- drivers/net/ethernet/intel/ixgbe/ixgbe_common.c | 28 +++++++++++++----------- drivers/net/ethernet/intel/ixgbe/ixgbe_main.c | 9 ++++++++ 4 files changed, 41 insertions(+), 26 deletions(-) -- 2.5.0 ^ permalink raw reply [flat|nested] 8+ messages in thread
* [Intel-wired-lan] [PATCH] add one parameter wro_enable for IXGBE 2016-10-29 7:08 ` Mao Wenan @ 2016-10-29 7:08 ` Mao Wenan -1 siblings, 0 replies; 8+ messages in thread From: Mao Wenan @ 2016-10-29 7:08 UTC (permalink / raw) To: intel-wired-lan --- drivers/net/ethernet/intel/ixgbe/ixgbe.h | 1 + drivers/net/ethernet/intel/ixgbe/ixgbe_82598.c | 29 ++++++++++++++----------- drivers/net/ethernet/intel/ixgbe/ixgbe_common.c | 28 +++++++++++++----------- drivers/net/ethernet/intel/ixgbe/ixgbe_main.c | 9 ++++++++ 4 files changed, 41 insertions(+), 26 deletions(-) diff --git a/drivers/net/ethernet/intel/ixgbe/ixgbe.h b/drivers/net/ethernet/intel/ixgbe/ixgbe.h index b06e32d..9bc0be5 100644 --- a/drivers/net/ethernet/intel/ixgbe/ixgbe.h +++ b/drivers/net/ethernet/intel/ixgbe/ixgbe.h @@ -1027,4 +1027,5 @@ netdev_tx_t ixgbe_xmit_frame_ring(struct sk_buff *skb, struct ixgbe_ring *tx_ring); u32 ixgbe_rss_indir_tbl_entries(struct ixgbe_adapter *adapter); void ixgbe_store_reta(struct ixgbe_adapter *adapter); +bool ixgbe_wro_enable(void); #endif /* _IXGBE_H_ */ diff --git a/drivers/net/ethernet/intel/ixgbe/ixgbe_82598.c b/drivers/net/ethernet/intel/ixgbe/ixgbe_82598.c index fb51be7..c312aaa 100644 --- a/drivers/net/ethernet/intel/ixgbe/ixgbe_82598.c +++ b/drivers/net/ethernet/intel/ixgbe/ixgbe_82598.c @@ -186,20 +186,23 @@ static s32 ixgbe_start_hw_82598(struct ixgbe_hw *hw) ret_val = ixgbe_start_hw_generic(hw); #ifndef CONFIG_SPARC - /* Disable relaxed ordering */ - for (i = 0; ((i < hw->mac.max_tx_queues) && - (i < IXGBE_DCA_MAX_QUEUES_82598)); i++) { - regval = IXGBE_READ_REG(hw, IXGBE_DCA_TXCTRL(i)); - regval &= ~IXGBE_DCA_TXCTRL_DESC_WRO_EN; - IXGBE_WRITE_REG(hw, IXGBE_DCA_TXCTRL(i), regval); - } + if(likely(!ixgbe_wro_enable())) { + + /* Disable relaxed ordering */ + for (i = 0; ((i < hw->mac.max_tx_queues) && + (i < IXGBE_DCA_MAX_QUEUES_82598)); i++) { + regval = IXGBE_READ_REG(hw, IXGBE_DCA_TXCTRL(i)); + regval &= ~IXGBE_DCA_TXCTRL_DESC_WRO_EN; + IXGBE_WRITE_REG(hw, IXGBE_DCA_TXCTRL(i), regval); + } - for (i = 0; ((i < hw->mac.max_rx_queues) && - (i < IXGBE_DCA_MAX_QUEUES_82598)); i++) { - regval = IXGBE_READ_REG(hw, IXGBE_DCA_RXCTRL(i)); - regval &= ~(IXGBE_DCA_RXCTRL_DATA_WRO_EN | - IXGBE_DCA_RXCTRL_HEAD_WRO_EN); - IXGBE_WRITE_REG(hw, IXGBE_DCA_RXCTRL(i), regval); + for (i = 0; ((i < hw->mac.max_rx_queues) && + (i < IXGBE_DCA_MAX_QUEUES_82598)); i++) { + regval = IXGBE_READ_REG(hw, IXGBE_DCA_RXCTRL(i)); + regval &= ~(IXGBE_DCA_RXCTRL_DATA_WRO_EN | + IXGBE_DCA_RXCTRL_HEAD_WRO_EN); + IXGBE_WRITE_REG(hw, IXGBE_DCA_RXCTRL(i), regval); + } } #endif if (ret_val) diff --git a/drivers/net/ethernet/intel/ixgbe/ixgbe_common.c b/drivers/net/ethernet/intel/ixgbe/ixgbe_common.c index 77d3039..7115dc0 100644 --- a/drivers/net/ethernet/intel/ixgbe/ixgbe_common.c +++ b/drivers/net/ethernet/intel/ixgbe/ixgbe_common.c @@ -347,22 +347,24 @@ s32 ixgbe_start_hw_gen2(struct ixgbe_hw *hw) IXGBE_WRITE_FLUSH(hw); #ifndef CONFIG_SPARC - /* Disable relaxed ordering */ - for (i = 0; i < hw->mac.max_tx_queues; i++) { - u32 regval; - - regval = IXGBE_READ_REG(hw, IXGBE_DCA_TXCTRL_82599(i)); - regval &= ~IXGBE_DCA_TXCTRL_DESC_WRO_EN; - IXGBE_WRITE_REG(hw, IXGBE_DCA_TXCTRL_82599(i), regval); - } + if(likely(!ixgbe_wro_enable())) { + /* Disable relaxed ordering */ + for (i = 0; i < hw->mac.max_tx_queues; i++) { + u32 regval; + + regval = IXGBE_READ_REG(hw, IXGBE_DCA_TXCTRL_82599(i)); + regval &= ~IXGBE_DCA_TXCTRL_DESC_WRO_EN; + IXGBE_WRITE_REG(hw, IXGBE_DCA_TXCTRL_82599(i), regval); + } - for (i = 0; i < hw->mac.max_rx_queues; i++) { - u32 regval; + for (i = 0; i < hw->mac.max_rx_queues; i++) { + u32 regval; - regval = IXGBE_READ_REG(hw, IXGBE_DCA_RXCTRL(i)); - regval &= ~(IXGBE_DCA_RXCTRL_DATA_WRO_EN | - IXGBE_DCA_RXCTRL_HEAD_WRO_EN); + regval = IXGBE_READ_REG(hw, IXGBE_DCA_RXCTRL(i)); + regval &= ~(IXGBE_DCA_RXCTRL_DATA_WRO_EN | + IXGBE_DCA_RXCTRL_HEAD_WRO_EN); IXGBE_WRITE_REG(hw, IXGBE_DCA_RXCTRL(i), regval); + } } #endif return 0; diff --git a/drivers/net/ethernet/intel/ixgbe/ixgbe_main.c b/drivers/net/ethernet/intel/ixgbe/ixgbe_main.c index a244d9a..79ebce3 100644 --- a/drivers/net/ethernet/intel/ixgbe/ixgbe_main.c +++ b/drivers/net/ethernet/intel/ixgbe/ixgbe_main.c @@ -171,11 +171,20 @@ static int debug = -1; module_param(debug, int, 0); MODULE_PARM_DESC(debug, "Debug level (0=none,...,16=all)"); +static bool wro_enable = 0; +module_param(wro_enable, bool, 0); +MODULE_PARM_DESC(wro_enable, "enable 82599 TX and RX's WRO attribute"); + MODULE_AUTHOR("Intel Corporation, <linux.nics@intel.com>"); MODULE_DESCRIPTION("Intel(R) 10 Gigabit PCI Express Network Driver"); MODULE_LICENSE("GPL"); MODULE_VERSION(DRV_VERSION); +bool ixgbe_wro_enable(void) +{ + return wro_enable; +} + static struct workqueue_struct *ixgbe_wq; static bool ixgbe_check_cfg_remove(struct ixgbe_hw *hw, struct pci_dev *pdev); -- 2.5.0 ^ permalink raw reply related [flat|nested] 8+ messages in thread
* [PATCH] add one parameter wro_enable for IXGBE @ 2016-10-29 7:08 ` Mao Wenan 0 siblings, 0 replies; 8+ messages in thread From: Mao Wenan @ 2016-10-29 7:08 UTC (permalink / raw) To: jeffrey.t.kirsher, intel-wired-lan, netdev, linux-kernel --- drivers/net/ethernet/intel/ixgbe/ixgbe.h | 1 + drivers/net/ethernet/intel/ixgbe/ixgbe_82598.c | 29 ++++++++++++++----------- drivers/net/ethernet/intel/ixgbe/ixgbe_common.c | 28 +++++++++++++----------- drivers/net/ethernet/intel/ixgbe/ixgbe_main.c | 9 ++++++++ 4 files changed, 41 insertions(+), 26 deletions(-) diff --git a/drivers/net/ethernet/intel/ixgbe/ixgbe.h b/drivers/net/ethernet/intel/ixgbe/ixgbe.h index b06e32d..9bc0be5 100644 --- a/drivers/net/ethernet/intel/ixgbe/ixgbe.h +++ b/drivers/net/ethernet/intel/ixgbe/ixgbe.h @@ -1027,4 +1027,5 @@ netdev_tx_t ixgbe_xmit_frame_ring(struct sk_buff *skb, struct ixgbe_ring *tx_ring); u32 ixgbe_rss_indir_tbl_entries(struct ixgbe_adapter *adapter); void ixgbe_store_reta(struct ixgbe_adapter *adapter); +bool ixgbe_wro_enable(void); #endif /* _IXGBE_H_ */ diff --git a/drivers/net/ethernet/intel/ixgbe/ixgbe_82598.c b/drivers/net/ethernet/intel/ixgbe/ixgbe_82598.c index fb51be7..c312aaa 100644 --- a/drivers/net/ethernet/intel/ixgbe/ixgbe_82598.c +++ b/drivers/net/ethernet/intel/ixgbe/ixgbe_82598.c @@ -186,20 +186,23 @@ static s32 ixgbe_start_hw_82598(struct ixgbe_hw *hw) ret_val = ixgbe_start_hw_generic(hw); #ifndef CONFIG_SPARC - /* Disable relaxed ordering */ - for (i = 0; ((i < hw->mac.max_tx_queues) && - (i < IXGBE_DCA_MAX_QUEUES_82598)); i++) { - regval = IXGBE_READ_REG(hw, IXGBE_DCA_TXCTRL(i)); - regval &= ~IXGBE_DCA_TXCTRL_DESC_WRO_EN; - IXGBE_WRITE_REG(hw, IXGBE_DCA_TXCTRL(i), regval); - } + if(likely(!ixgbe_wro_enable())) { + + /* Disable relaxed ordering */ + for (i = 0; ((i < hw->mac.max_tx_queues) && + (i < IXGBE_DCA_MAX_QUEUES_82598)); i++) { + regval = IXGBE_READ_REG(hw, IXGBE_DCA_TXCTRL(i)); + regval &= ~IXGBE_DCA_TXCTRL_DESC_WRO_EN; + IXGBE_WRITE_REG(hw, IXGBE_DCA_TXCTRL(i), regval); + } - for (i = 0; ((i < hw->mac.max_rx_queues) && - (i < IXGBE_DCA_MAX_QUEUES_82598)); i++) { - regval = IXGBE_READ_REG(hw, IXGBE_DCA_RXCTRL(i)); - regval &= ~(IXGBE_DCA_RXCTRL_DATA_WRO_EN | - IXGBE_DCA_RXCTRL_HEAD_WRO_EN); - IXGBE_WRITE_REG(hw, IXGBE_DCA_RXCTRL(i), regval); + for (i = 0; ((i < hw->mac.max_rx_queues) && + (i < IXGBE_DCA_MAX_QUEUES_82598)); i++) { + regval = IXGBE_READ_REG(hw, IXGBE_DCA_RXCTRL(i)); + regval &= ~(IXGBE_DCA_RXCTRL_DATA_WRO_EN | + IXGBE_DCA_RXCTRL_HEAD_WRO_EN); + IXGBE_WRITE_REG(hw, IXGBE_DCA_RXCTRL(i), regval); + } } #endif if (ret_val) diff --git a/drivers/net/ethernet/intel/ixgbe/ixgbe_common.c b/drivers/net/ethernet/intel/ixgbe/ixgbe_common.c index 77d3039..7115dc0 100644 --- a/drivers/net/ethernet/intel/ixgbe/ixgbe_common.c +++ b/drivers/net/ethernet/intel/ixgbe/ixgbe_common.c @@ -347,22 +347,24 @@ s32 ixgbe_start_hw_gen2(struct ixgbe_hw *hw) IXGBE_WRITE_FLUSH(hw); #ifndef CONFIG_SPARC - /* Disable relaxed ordering */ - for (i = 0; i < hw->mac.max_tx_queues; i++) { - u32 regval; - - regval = IXGBE_READ_REG(hw, IXGBE_DCA_TXCTRL_82599(i)); - regval &= ~IXGBE_DCA_TXCTRL_DESC_WRO_EN; - IXGBE_WRITE_REG(hw, IXGBE_DCA_TXCTRL_82599(i), regval); - } + if(likely(!ixgbe_wro_enable())) { + /* Disable relaxed ordering */ + for (i = 0; i < hw->mac.max_tx_queues; i++) { + u32 regval; + + regval = IXGBE_READ_REG(hw, IXGBE_DCA_TXCTRL_82599(i)); + regval &= ~IXGBE_DCA_TXCTRL_DESC_WRO_EN; + IXGBE_WRITE_REG(hw, IXGBE_DCA_TXCTRL_82599(i), regval); + } - for (i = 0; i < hw->mac.max_rx_queues; i++) { - u32 regval; + for (i = 0; i < hw->mac.max_rx_queues; i++) { + u32 regval; - regval = IXGBE_READ_REG(hw, IXGBE_DCA_RXCTRL(i)); - regval &= ~(IXGBE_DCA_RXCTRL_DATA_WRO_EN | - IXGBE_DCA_RXCTRL_HEAD_WRO_EN); + regval = IXGBE_READ_REG(hw, IXGBE_DCA_RXCTRL(i)); + regval &= ~(IXGBE_DCA_RXCTRL_DATA_WRO_EN | + IXGBE_DCA_RXCTRL_HEAD_WRO_EN); IXGBE_WRITE_REG(hw, IXGBE_DCA_RXCTRL(i), regval); + } } #endif return 0; diff --git a/drivers/net/ethernet/intel/ixgbe/ixgbe_main.c b/drivers/net/ethernet/intel/ixgbe/ixgbe_main.c index a244d9a..79ebce3 100644 --- a/drivers/net/ethernet/intel/ixgbe/ixgbe_main.c +++ b/drivers/net/ethernet/intel/ixgbe/ixgbe_main.c @@ -171,11 +171,20 @@ static int debug = -1; module_param(debug, int, 0); MODULE_PARM_DESC(debug, "Debug level (0=none,...,16=all)"); +static bool wro_enable = 0; +module_param(wro_enable, bool, 0); +MODULE_PARM_DESC(wro_enable, "enable 82599 TX and RX's WRO attribute"); + MODULE_AUTHOR("Intel Corporation, <linux.nics@intel.com>"); MODULE_DESCRIPTION("Intel(R) 10 Gigabit PCI Express Network Driver"); MODULE_LICENSE("GPL"); MODULE_VERSION(DRV_VERSION); +bool ixgbe_wro_enable(void) +{ + return wro_enable; +} + static struct workqueue_struct *ixgbe_wq; static bool ixgbe_check_cfg_remove(struct ixgbe_hw *hw, struct pci_dev *pdev); -- 2.5.0 ^ permalink raw reply related [flat|nested] 8+ messages in thread
* [Intel-wired-lan] [PATCH] add one parameter wro_enable to enable relaxed ordering for IXGBE 2016-10-29 7:08 ` Mao Wenan @ 2016-10-29 7:42 ` Jeff Kirsher -1 siblings, 0 replies; 8+ messages in thread From: Jeff Kirsher @ 2016-10-29 7:42 UTC (permalink / raw) To: intel-wired-lan On Sat, 2016-10-29 at 15:08 +0800, Mao Wenan wrote: > This patch provides a way to enable relaxed ordering, where it helps with > performance in some architecture. > The default value of wro_enable is 0, if you want to enable relaxed > ordering, please set wro_enable=1. > > Mao Wenan (1): > ? add one parameter wro_enable for IXGBE > > ?drivers/net/ethernet/intel/ixgbe/ixgbe.h??????? |? 1 + > ?drivers/net/ethernet/intel/ixgbe/ixgbe_82598.c? | 29 ++++++++++++++----- > ------ > ?drivers/net/ethernet/intel/ixgbe/ixgbe_common.c | 28 +++++++++++++---- > ------- > ?drivers/net/ethernet/intel/ixgbe/ixgbe_main.c?? |? 9 ++++++++ > ?4 files changed, 41 insertions(+), 26 deletions(-) Why have a title patch for only one patch? ?Better yet, the one patch does not have a patch description. ?Get rid of the title patch and add the above information into the patches description. In addition, module parameters are not kindly looked upon, one reason is that it cannot be standardized and enforced. I am also confused because you are stating that on some architectures, yet this code is only compiled in when SPARC is defined and that there are times when you want relaxed ordering enabled and other times disabled? ?Your gonna have to provide more data on why, because the code as is was resolving serious performance issues on SPARC when relaxed ordering was enabled. -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 819 bytes Desc: This is a digitally signed message part URL: <http://lists.osuosl.org/pipermail/intel-wired-lan/attachments/20161029/8b4f7b65/attachment.asc> ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] add one parameter wro_enable to enable relaxed ordering for IXGBE @ 2016-10-29 7:42 ` Jeff Kirsher 0 siblings, 0 replies; 8+ messages in thread From: Jeff Kirsher @ 2016-10-29 7:42 UTC (permalink / raw) To: Mao Wenan, intel-wired-lan, netdev, linux-kernel [-- Attachment #1: Type: text/plain, Size: 1417 bytes --] On Sat, 2016-10-29 at 15:08 +0800, Mao Wenan wrote: > This patch provides a way to enable relaxed ordering, where it helps with > performance in some architecture. > The default value of wro_enable is 0, if you want to enable relaxed > ordering, please set wro_enable=1. > > Mao Wenan (1): > add one parameter wro_enable for IXGBE > > drivers/net/ethernet/intel/ixgbe/ixgbe.h | 1 + > drivers/net/ethernet/intel/ixgbe/ixgbe_82598.c | 29 ++++++++++++++----- > ------ > drivers/net/ethernet/intel/ixgbe/ixgbe_common.c | 28 +++++++++++++---- > ------- > drivers/net/ethernet/intel/ixgbe/ixgbe_main.c | 9 ++++++++ > 4 files changed, 41 insertions(+), 26 deletions(-) Why have a title patch for only one patch? Better yet, the one patch does not have a patch description. Get rid of the title patch and add the above information into the patches description. In addition, module parameters are not kindly looked upon, one reason is that it cannot be standardized and enforced. I am also confused because you are stating that on some architectures, yet this code is only compiled in when SPARC is defined and that there are times when you want relaxed ordering enabled and other times disabled? Your gonna have to provide more data on why, because the code as is was resolving serious performance issues on SPARC when relaxed ordering was enabled. [-- Attachment #2: This is a digitally signed message part --] [-- Type: application/pgp-signature, Size: 819 bytes --] ^ permalink raw reply [flat|nested] 8+ messages in thread
* [Intel-wired-lan] 答复: [PATCH] add one parameter wro_enable to enable relaxed ordering for IXGBE 2016-10-29 7:42 ` Jeff Kirsher @ 2016-11-09 9:43 ` maowenan -1 siblings, 0 replies; 8+ messages in thread From: maowenan @ 2016-11-09 9:43 UTC (permalink / raw) To: intel-wired-lan I have verified that the performance will be enhanced certainly when I enabled Relax Ordering on SPARC, but think it is not very flexible to disable or enable Relax Ordering feature using CONFIG_SPARC currently, So I want to use module parameter to set RO instead of "#ifndef CONFIG_SPARC", no need to rebuild the whole kernel. -----????----- ???: Jeff Kirsher [mailto:jeffrey.t.kirsher at intel.com] ????: 2016?10?29? 15:42 ???: maowenan; intel-wired-lan at lists.osuosl.org; netdev at vger.kernel.org; linux-kernel at vger.kernel.org ??: Re: [PATCH] add one parameter wro_enable to enable relaxed ordering for IXGBE On Sat, 2016-10-29 at 15:08 +0800, Mao Wenan wrote: > This patch provides a way to enable relaxed ordering, where it helps > with performance in some architecture. > The default value of wro_enable is 0, if you want to enable relaxed > ordering, please set wro_enable=1. > > Mao Wenan (1): > ? add one parameter wro_enable for IXGBE > > ?drivers/net/ethernet/intel/ixgbe/ixgbe.h??????? |? 1 + > ?drivers/net/ethernet/intel/ixgbe/ixgbe_82598.c? | 29 > ++++++++++++++----- > ------ > ?drivers/net/ethernet/intel/ixgbe/ixgbe_common.c | 28 > +++++++++++++---- > ------- > ?drivers/net/ethernet/intel/ixgbe/ixgbe_main.c?? |? 9 ++++++++ > ?4 files changed, 41 insertions(+), 26 deletions(-) Why have a title patch for only one patch? ?Better yet, the one patch does not have a patch description. ?Get rid of the title patch and add the above information into the patches description. In addition, module parameters are not kindly looked upon, one reason is that it cannot be standardized and enforced. I am also confused because you are stating that on some architectures, yet this code is only compiled in when SPARC is defined and that there are times when you want relaxed ordering enabled and other times disabled? ?Your gonna have to provide more data on why, because the code as is was resolving serious performance issues on SPARC when relaxed ordering was enabled. ^ permalink raw reply [flat|nested] 8+ messages in thread
* 答复: [PATCH] add one parameter wro_enable to enable relaxed ordering for IXGBE @ 2016-11-09 9:43 ` maowenan 0 siblings, 0 replies; 8+ messages in thread From: maowenan @ 2016-11-09 9:43 UTC (permalink / raw) To: Jeff Kirsher, intel-wired-lan@lists.osuosl.org, netdev@vger.kernel.org, linux-kernel@vger.kernel.org Cc: Dingtianhong, weiyongjun (A) I have verified that the performance will be enhanced certainly when I enabled Relax Ordering on SPARC, but think it is not very flexible to disable or enable Relax Ordering feature using CONFIG_SPARC currently, So I want to use module parameter to set RO instead of "#ifndef CONFIG_SPARC", no need to rebuild the whole kernel. -----邮件原件----- 发件人: Jeff Kirsher [mailto:jeffrey.t.kirsher@intel.com] 发送时间: 2016年10月29日 15:42 收件人: maowenan; intel-wired-lan@lists.osuosl.org; netdev@vger.kernel.org; linux-kernel@vger.kernel.org 主题: Re: [PATCH] add one parameter wro_enable to enable relaxed ordering for IXGBE On Sat, 2016-10-29 at 15:08 +0800, Mao Wenan wrote: > This patch provides a way to enable relaxed ordering, where it helps > with performance in some architecture. > The default value of wro_enable is 0, if you want to enable relaxed > ordering, please set wro_enable=1. > > Mao Wenan (1): > add one parameter wro_enable for IXGBE > > drivers/net/ethernet/intel/ixgbe/ixgbe.h | 1 + > drivers/net/ethernet/intel/ixgbe/ixgbe_82598.c | 29 > ++++++++++++++----- > ------ > drivers/net/ethernet/intel/ixgbe/ixgbe_common.c | 28 > +++++++++++++---- > ------- > drivers/net/ethernet/intel/ixgbe/ixgbe_main.c | 9 ++++++++ > 4 files changed, 41 insertions(+), 26 deletions(-) Why have a title patch for only one patch? Better yet, the one patch does not have a patch description. Get rid of the title patch and add the above information into the patches description. In addition, module parameters are not kindly looked upon, one reason is that it cannot be standardized and enforced. I am also confused because you are stating that on some architectures, yet this code is only compiled in when SPARC is defined and that there are times when you want relaxed ordering enabled and other times disabled? Your gonna have to provide more data on why, because the code as is was resolving serious performance issues on SPARC when relaxed ordering was enabled. ^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2016-11-09 9:45 UTC | newest] Thread overview: 8+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2016-10-29 7:08 [Intel-wired-lan] [PATCH] add one parameter wro_enable to enable relaxed ordering for IXGBE Mao Wenan 2016-10-29 7:08 ` Mao Wenan 2016-10-29 7:08 ` [Intel-wired-lan] [PATCH] add one parameter wro_enable " Mao Wenan 2016-10-29 7:08 ` Mao Wenan 2016-10-29 7:42 ` [Intel-wired-lan] [PATCH] add one parameter wro_enable to enable relaxed ordering " Jeff Kirsher 2016-10-29 7:42 ` Jeff Kirsher 2016-11-09 9:43 ` [Intel-wired-lan] 答复: " maowenan 2016-11-09 9:43 ` maowenan
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.