From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752852AbcKIJpH (ORCPT ); Wed, 9 Nov 2016 04:45:07 -0500 Received: from szxga02-in.huawei.com ([119.145.14.65]:14769 "EHLO szxga02-in.huawei.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752163AbcKIJpB (ORCPT ); Wed, 9 Nov 2016 04:45:01 -0500 From: maowenan To: Jeff Kirsher , "intel-wired-lan@lists.osuosl.org" , "netdev@vger.kernel.org" , "linux-kernel@vger.kernel.org" CC: Dingtianhong , "weiyongjun (A)" Subject: =?utf-8?B?562U5aSNOiBbUEFUQ0hdIGFkZCBvbmUgcGFyYW1ldGVyIHdyb19lbmFibGUg?= =?utf-8?Q?to_enable_relaxed_ordering_for_IXGBE?= Thread-Topic: [PATCH] add one parameter wro_enable to enable relaxed ordering for IXGBE Thread-Index: AQHSMbN+Jh8RHKFihUyvKV0Xm00txaC+hnOAgBHpwsA= Date: Wed, 9 Nov 2016 09:43:36 +0000 Message-ID: References: <1477724932-5440-1-git-send-email-maowenan@huawei.com> <1477726941.2309.9.camel@intel.com> In-Reply-To: <1477726941.2309.9.camel@intel.com> Accept-Language: zh-CN, en-US Content-Language: zh-CN X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [10.177.96.96] Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 X-CFilter-Loop: Reflected Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Transfer-Encoding: 8bit X-MIME-Autoconverted: from base64 to 8bit by mail.home.local id uA99jBSW013391 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.