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: <559918e5-0c06-4962-a2d0-05b377aae2db@intel.com>
Date:   Thu, 23 Nov 2023 11:55:34 +0100
User-Agent: Mozilla Thunderbird
Subject: Re: [net PATCH 4/5] octeontx2-af: Add missing mcs flr handler call
Content-Language: en-US
To:     Geetha sowjanya <gakula@marvell.com>, <netdev@vger.kernel.org>,
        <linux-kernel@vger.kernel.org>
CC:     <kuba@kernel.org>, <davem@davemloft.net>, <pabeni@redhat.com>,
        <edumazet@google.com>, <sgoutham@marvell.com>,
        <sbhatta@marvell.com>, <hkelam@marvell.com>
References: <20231123055941.19430-1-gakula@marvell.com>
 <20231123055941.19430-5-gakula@marvell.com>
From:   Wojciech Drewek <wojciech.drewek@intel.com>
In-Reply-To: <20231123055941.19430-5-gakula@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?K1hIOUI4YkM0ZzVkWkh5d0tGdEZocjRDa3JSM25NSXdhS1ZVT3JLQlppdGNK?=
 =?utf-8?B?NHBNRDQ3aXJJa3ZudGVQZFpkbFJZYklBZ0tMQkZYZW1vVmQ4aVRkT3VkRmxs?=
 =?utf-8?B?MWIrZWRmK1FySGh2QzNqck5lb3BGcHFPU0tDZGJZOGtVU25KU0dFdURRVjQx?=
 =?utf-8?B?a1NJdUJXakdjU3pmMk1IVUxOMHpic2xnbFk5UzFxYkdFeEV2Sit1aGp4RGJu?=
 =?utf-8?B?RXM5dXpnZUFRSDIxM0pleTMxczNESjhtKzJKaW5jODRDMXAwN1FBYkNGRGY0?=
 =?utf-8?B?N0N3Q0h4SDFsaWM2eXhxL04vMEI3cDNPUHFibUhPdWNENFloMnUybW16SlNh?=
 =?utf-8?B?NTdqS1FnNEhnZDR5cWRldE9tQnliTDEreVJZaDdiV1dEVTV0RURFdXloVEFN?=
 =?utf-8?B?ejgxSlNtRmpUTGgrZDhXS25TdXdoMmpUWDMzRUtyOEo3VWFHekx4TTIwRWNt?=
 =?utf-8?B?cUJIQWR2blgwU3YreWpnMU5qK0dBRWl5N0tOSzF3Y3dDd2tPNkJ2TnZIMmRr?=
 =?utf-8?B?RktXdkFOZGxzOWdGYWJoNWZQL2tjQ082Mi9YWHlVQ2Fzd2tVWGpVUDNlTXYz?=
 =?utf-8?B?OUhITkJIOXZ3MVhIeVRkTHhjcGZDK3dGWTFUdEQ0blRJRWluQU9HSndJTEkx?=
 =?utf-8?B?K3RLR2F6b0dGMzB3S3phemowcVFwTlRHVnozRDZRYnNieFZwVGtSeXNuSU5J?=
 =?utf-8?B?UTNwc2oySmlScUVoSXJNU2RFL3YyTzYzMUF0OHJmTmIwc0MrUlJZTVBCSGtZ?=
 =?utf-8?B?ajVjZ01mNDdtZ0ljZ1NBNSt5M1haSXVVWmt2MlB6cmprNStpK3B5d3ptZkJ3?=
 =?utf-8?B?WjU4aEFOelhhRzNBeVJrd0FlZTQ1dG5jUGd2N2ZBeVBZYjdiSVJlV1gyYnUw?=
 =?utf-8?B?b29BQU85RzBTZXlhZmNvRVFFcE1LNzlCaDBnV3F2QTJ5bVdZU1U5Qk5MTi8y?=
 =?utf-8?B?QXJ0UW03dThVdzFWczQ4ZmRpbGZGQXZpbnk1MDV0NjYyckVjVVNGR1UzdHY2?=
 =?utf-8?B?UVFDR2NDWWJVT2JCbktqYnMxTG1LMjZkVmVDVFU2WXBKL3gyUU53djFXT2tv?=
 =?utf-8?B?aEFNV3hidnJyak8wY2VTWVZ4ckNtMlJxemhwL3JBa29PZUdwTk1hdFpxNW8y?=
 =?utf-8?B?VUJzRHJjR2NGSWV6cExxTm9KV0NsN1c1dFFxVDdvTnNnUHRYKzN3VnRxOHBi?=
 =?utf-8?B?cUdFNk8xVkZTSkxpUy9VV2xwSFpzYTkxZVJsalhQUTZkSXJETzBCRGR3YTEr?=
 =?utf-8?B?WWxjV0UwalJJZ2RGN3o1a3hWeTNoTExneVEvMGh6ODMrVzZ1R1dXWnZIOTNG?=
 =?utf-8?B?eG1TdE5CaFZBVkI3c0E2S3BGK0pESVpPWlJiNzAwUm9DcVFZY09hVDUvdzBR?=
 =?utf-8?B?K3V0TFd0aytvdTNJRENBRGhBbWxBeCt5eXlGVDM1bGRlVzU4c3I2eUdJT1hY?=
 =?utf-8?B?T1JMYzZydkMyUloyTzFjQjV6aXl2WVRqdnArd29UVHJ2VTk1bC9QSVJEZkNl?=
 =?utf-8?B?V3k1RGF3bXJGNlQrdFdmc0dFa2EwVUh1aTNUNmtncEp0Yy85M3dMRWlhSyty?=
 =?utf-8?B?bFZoaTZ1bk5SNUtmOUovR0oyVHh6cWNKMFdaVjVGMnNEVzhVY3B2VFR4RzBw?=
 =?utf-8?B?UUZPd1QvZnR3MkM5cGZFREVwNnkvczQza1dVSllDYmtCRXVadVp2RC9hVnRv?=
 =?utf-8?B?ZG05bnNoazVTWUtIK3dvQWxXblNzcm1PbTBZdXE1U1BNOGVlTnB5QWFrcVZ2?=
 =?utf-8?B?MmpSWktoMW5hNmM2S1FuUVVsRjdxemlCaTBlWTRlWGdnangxenp6dVRxLzR0?=
 =?utf-8?B?T0dMTmZwRlliSTZqRjNFR2VBTDlwQVRsNlZEcjQrTDFJS0dHdytTWjJHb0pt?=
 =?utf-8?B?VzFWdEVyOUs0YVVmOTVjYlcwaHdjVjV4VmUxaXlmeEw5Q2t0OUpGSEFFa01u?=
 =?utf-8?B?UHhDUlpSR3FoQmdLbHcvL0oyM2dtaEM2SWtad08ySlZDWDVOUjJnTE5wTHlp?=
 =?utf-8?B?Z1V4dlhzUGtsL3B3OVZHSU1SeWgxNG0zOHpDdk9qbnBwQWpTR0o2bnZtdXRY?=
 =?utf-8?B?Q1dtKzFvWU9xaUZRRm9Gem01SjVzSzVtMXZudWdXczV1VnppSUtnVXBsUFMw?=
 =?utf-8?B?UUpMZldTempMTmk1U3JiTUU2clkrZE4wdGR2aXJWRlEvemx1TzZuSm1KYUJs?=
 =?utf-8?B?U3c9PQ==?=
X-MS-Exchange-CrossTenant-Network-Message-Id: e4d7d021-b979-4ca6-ce08-08dbec12b9be
X-MS-Exchange-CrossTenant-AuthSource: MW4PR11MB5776.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 23 Nov 2023 10:55:37.5179
 (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: 3PQqNI6WoaOD14qxS+EouMz+fgsFUKN1L/SFYScoHvQp+Ti+VAAxN+aiQw8GjBFSqlYUE+F09R7QNUXLA6iZ1bOL9buZTukioT9Qvr5A5ig=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: SA2PR11MB5019
X-OriginatorOrg: intel.com
X-Spam-Status: No, score=-0.9 required=5.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED,
	DKIM_VALID,DKIM_VALID_AU,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]); Thu, 23 Nov 2023 03:02:59 -0800 (PST)


On 23.11.2023 06:59, Geetha sowjanya wrote:
> If mcs resources are attached to PF/VF. These resources need
> to be freed on FLR. This patch add mssing mcs flr call on PF FLR.
> 
> Fixes: bd69476e86fc ("octeontx2-af: cn10k: mcs: Install a default TCAM for normal traffic")
> Signed-off-by: Geetha sowjanya <gakula@marvell.com>
> ---

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

>  drivers/net/ethernet/marvell/octeontx2/af/rvu.c | 3 +++
>  1 file changed, 3 insertions(+)
> 
> diff --git a/drivers/net/ethernet/marvell/octeontx2/af/rvu.c b/drivers/net/ethernet/marvell/octeontx2/af/rvu.c
> index 22c395c7d040..731bb82b577c 100644
> --- a/drivers/net/ethernet/marvell/octeontx2/af/rvu.c
> +++ b/drivers/net/ethernet/marvell/octeontx2/af/rvu.c
> @@ -2631,6 +2631,9 @@ static void __rvu_flr_handler(struct rvu *rvu, u16 pcifunc)
>  	rvu_npc_free_mcam_entries(rvu, pcifunc, -1);
>  	rvu_mac_reset(rvu, pcifunc);
>  
> +	if (rvu->mcs_blk_cnt)
> +		rvu_mcs_flr_handler(rvu, pcifunc);
> +
>  	mutex_unlock(&rvu->flr_lock);
>  }
>