Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:6 as permitted sender) client-ip=2620:137:e000::3:6;
Message-ID: <bb7d635a-8cea-4fcd-9d27-fb9e3d82cab9@intel.com>
Date:   Fri, 1 Dec 2023 11:44:45 +0100
User-Agent: Mozilla Thunderbird
Subject: Re: [net PATCH v3] octeontx2-pf: consider both Rx and Tx packet stats
 for adaptive interrupt coalescing
Content-Language: en-US
To:     Suman Ghosh <sumang@marvell.com>, <sgoutham@marvell.com>,
        <gakula@marvell.com>, <sbhatta@marvell.com>, <hkelam@marvell.com>,
        <lcherian@marvell.com>, <jerinj@marvell.com>,
        <davem@davemloft.net>, <edumazet@google.com>, <kuba@kernel.org>,
        <pabeni@redhat.com>, <netdev@vger.kernel.org>,
        <linux-kernel@vger.kernel.org>, <horms@kernel.org>,
        <naveenm@marvell.com>
References: <20231201053330.3903694-1-sumang@marvell.com>
From:   Wojciech Drewek <wojciech.drewek@intel.com>
In-Reply-To: <20231201053330.3903694-1-sumang@marvell.com>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: 7bit
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?b1kzYmFVbENvUHh6UE55UzR5SThPeHZ3OFowamF5QTZBMitMRUdMaCthZmFB?=
 =?utf-8?B?MzNrUHMvNEVBbE5HWkg4d3dybnpvZ2dlc0R1TTBJV2V4bFVQV09Oc2JPTUZQ?=
 =?utf-8?B?N1VGeHk4MEZ5SThCbkErS2VpKzYrV0V4akREamMxQ3NucFBHR0VRbnpkNVJo?=
 =?utf-8?B?bWx0NFRFa1NRMHhyVE5sVHB3QTFTTWdGTmM0eFF6VDZHOVhhMUg1M0Z1SDhM?=
 =?utf-8?B?akU5N1RyaklSdnI3dnJlMzN0VVpRZjRKTjRqdjd4WGNaL05yUE1Ed3Nkb3Ba?=
 =?utf-8?B?ZnpwenRUaFFET3VPcklnM3VwMHo0cWJlNUlzWmdTYmpkVVBhOStWTE14NVAx?=
 =?utf-8?B?VExxNHQ3NCtidGt4TlduVFBqek1PZUJteHpiNGtmdzZ6THlFV3ZXUW5GZ2Fo?=
 =?utf-8?B?Y2FHNnZHcGN0dGNlL3V4S3hxMlJjdVQ3MXJhQnJ5QnU1OENYdnJmZndVS2Rz?=
 =?utf-8?B?cm14ckd4cmNzd0RVOWJaUFM2ZVBXS24zVXptYlU4OFZ3UlZUd3VZUEpWTnhv?=
 =?utf-8?B?MW5NWDV5K1RBNFBERWxSdEh6MUx3NnhVeUViUFBXaTdnT1IzM3p5cmZhc0Q2?=
 =?utf-8?B?b1ZVZVpsOUxmODFqK1Y4dEpEbHhZNENQSnVnWmxrMzRRWFp6ejl2WWUxV21o?=
 =?utf-8?B?S0RNWERGS0JjcEhzdEEybHFNbW03TXI2a2dkWDJLbEdoZ25Ja2t0a2JScGoz?=
 =?utf-8?B?Vncwa3JwZHdkVFVLci8vNXJoSFBERVBiZERUb0xkNGpXcDEvVm9tbVA3ZHpa?=
 =?utf-8?B?aVdBR0dnc2cxcnQ1YUNJUXBVckN6UFV5VTRzV3RjTXpTcXhQMS84bEVKbWdN?=
 =?utf-8?B?Y2lsZ0dlUzd0MzFTdUhjd3RjNGVkTzRmanBFUVFZU080TkhaWXJTYnlCeVRp?=
 =?utf-8?B?YzZPazBtc1BHT1RuL09mbmZEeW1FbWxpcUdNUEtxNlZaODFIQ0RPamUxdEdV?=
 =?utf-8?B?TDVZeVZiMWt4WExaNnR2MlhZY2NncVNIZ0JkUDlKNnJKMFZHV2pWbFBjRVBh?=
 =?utf-8?B?cjlqRUdTL0RXS0JqUVhyeTkvVTdkUWFYUnVGdnI3MFZmcEpLZnpndXMxVEdk?=
 =?utf-8?B?clU4ckRCU3dwa1QwTW15TG5YbjgrVXJiZ3ZEQ3V0THMzWmtiSVRiSzdLNkNG?=
 =?utf-8?B?SC9YQ2twcG1PclZYb2UvZVpGRUJaN0Yra3I4a3FsOUgzQ2tVcUNwTTJJaXJk?=
 =?utf-8?B?YkFhR0ZRMmlqYzd2ZFphbm9oQ3huTkhjQ1Y2anFZMUM1cC92UEdWdDVmbzd3?=
 =?utf-8?B?dmxLVDdDNUZsU2VnN0kvek9vdmcxZXRrdEZqaVpENE5UN2UvY0xmL0haNzhB?=
 =?utf-8?B?TDJFWlEvZDlXOFhrcEduaEdCZmNlVVd2SjlMNXlDakszMzVBbnRqZk1hNFhi?=
 =?utf-8?B?RytZWGF5dkRPVjNKaHA3Y0piMUNPMkhZVUhUdXpMc3hmSU5BMVpUai9NRzBk?=
 =?utf-8?B?QjczaXFod1lYMmtuZzk5R29UZXN2WVZkSWpOcU5pTWVsbTJGSHRYZ1FBdmwx?=
 =?utf-8?B?Rk9qeUlsVVFUVHdEY3AwN0t2RWNvV1BlQ2toR1RCOWwvaFBXb0lqMVhIc0Va?=
 =?utf-8?B?dDJLZCtmbmdOcXFHUEs3MTlFdTdscEN4U1Qzd29Kb0t3RGhtNHBUWTFOSjE4?=
 =?utf-8?B?MXhpV1U3Q3pZS0Y2TmhrWnB4RDJXNGRZT0w2UDBlN0twako3dVVmV3k2UE0z?=
 =?utf-8?B?Wk5lZjkzeXFXd2t4V2w0SGEwcVBqZmVBQmFoZk5IcmwxT0tHdC9BY3EzTDRS?=
 =?utf-8?B?Wkt4Sk91K1Y3UUNaajhNUDg3SU1seXYzWW9RQWtrZms2dVNVSnJRSEZxZU1U?=
 =?utf-8?B?K1RSUkd3WlZ4dkhxWThJamhrU3p1aEwvNXdON3hsTzJQOGd0VUJ4dmkwa3FY?=
 =?utf-8?B?ZlZyenpBb2F2QklBbzdHanZOU0JibVNJOWtKN0Z0TitTVjMvUWlUdlVqZmNZ?=
 =?utf-8?B?ZlkvL1BLSlpBVGFMdzVGYVJPb1d4eUFNMXNnRU5HYmZUeVZpVnFPWDFsb0Iy?=
 =?utf-8?B?N0NFVGZmTnlFdmhRMERseUJEYmlMM2NxeGRIcTV3NnJXakRZOWUyQmlYT1NM?=
 =?utf-8?B?NU5od0s5REJRYVlBcHZPRC9xVi9jTFliUENpWDV4TTRqc240NVFxVUhPNktq?=
 =?utf-8?B?QzlMMWt4bmh6cWFqOTBicTVjQllyamJLUmYwck92ZGpIQXJweUF0NFNTRE5R?=
 =?utf-8?B?UFE9PQ==?=
X-MS-Exchange-CrossTenant-Network-Message-Id: 420cbb20-e3ce-4f5c-0c82-08dbf25a8cb8
X-MS-Exchange-CrossTenant-AuthSource: MW4PR11MB5776.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 01 Dec 2023 10:44:52.7214
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 46c98d88-e344-4ed4-8496-4ed7712e255d
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: MYSc/oQfEeyuGdLKOUc4FvLA/Y1p5AGWxk6CDwkX7wOJIhz431HA4ueMDFjznmircHXjON0i8W89ockECVnHWYyT+oRGctQ0eEUylY6ieVM=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DS0PR11MB8739
X-OriginatorOrg: intel.com
X-Spam-Status: No, score=-0.6 required=5.0 tests=DKIM_INVALID,DKIM_SIGNED,
	HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,SPF_HELO_NONE,
	SPF_PASS,T_SCC_BODY_TEXT_LINE autolearn=unavailable autolearn_force=no
	version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on pete.vger.email
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org
X-Greylist: Sender passed SPF test, not delayed by milter-greylist-4.6.4 (pete.vger.email [0.0.0.0]); Fri, 01 Dec 2023 02:45:07 -0800 (PST)


On 01.12.2023 06:33, Suman Ghosh wrote:
> From: Naveen Mamindlapalli <naveenm@marvell.com>
> 
> The current adaptive interrupt coalescing code updates only rx
> packet stats for dim algorithm. This patch also updates tx packet
> stats which will be useful when there is only tx traffic.
> Also moved configuring hardware adaptive interrupt setting to
> driver dim callback.
> 
> Fixes: 6e144b47f560 ("octeontx2-pf: Add support for adaptive interrupt coalescing")
> Signed-off-by: Naveen Mamindlapalli <naveenm@marvell.com>
> Signed-off-by: Suman Ghosh <sumang@marvell.com>
> ---

Reviewed-by: Wojciech Drewek <wojciech.drewek@intel.com>

> v3 changes:
> - Updated signed-off-by signature.
> 
> v2 changes:
> - Missed adding the fixes tag in v1. Added the same in v2.
> 
>  .../ethernet/marvell/octeontx2/nic/otx2_pf.c  |  9 +++++++++
>  .../marvell/octeontx2/nic/otx2_txrx.c         | 20 +++++++++----------
>  2 files changed, 19 insertions(+), 10 deletions(-)
>