Received-SPF: pass (google.com: domain of linux-kernel+bounces-195773-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45d1:ec00::1 as permitted sender) client-ip=2604:1380:45d1:ec00::1;
Message-ID: <12ef3572-ba28-4abe-b891-26fa4f6fbe0d@intel.com>
Date: Thu, 30 May 2024 11:20:36 -0700
User-Agent: Mozilla Thunderbird
Subject: Re: [net PATCH] octeontx2-af: Always allocate PF entries from low
 prioriy zone
To: Subbaraya Sundeep <sbhatta@marvell.com>, <netdev@vger.kernel.org>,
	<linux-kernel@vger.kernel.org>
CC: Sunil Goutham <sgoutham@marvell.com>, Linu Cherian <lcherian@marvell.com>,
	Geetha sowjanya <gakula@marvell.com>, Jerin Jacob <jerinj@marvell.com>,
	hariprasad <hkelam@marvell.com>, "David S. Miller" <davem@davemloft.net>,
	Eric Dumazet <edumazet@google.com>, Jakub Kicinski <kuba@kernel.org>, "Paolo
 Abeni" <pabeni@redhat.com>
References: <1716996584-14470-1-git-send-email-sbhatta@marvell.com>
Content-Language: en-US
From: Jacob Keller <jacob.e.keller@intel.com>
In-Reply-To: <1716996584-14470-1-git-send-email-sbhatta@marvell.com>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: 7bit
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?TVU3bktVeDlhMlNEZDQwNVB5a21Sc0hPcVI1OURUQ3Zad0k5Vld6OGhRY3Bo?=
 =?utf-8?B?dkM3cXF4OVRPemdoa2lpK2p5KzNhNU8vSGlHdXFQNkVWaW9PS3FiZHBIRzZQ?=
 =?utf-8?B?UEpubitmVTBrV2FrVEpYQ3hBUFpJOHNZYVNiUVhPU0RJalVvdXVBT2RKN21S?=
 =?utf-8?B?bUprSTNEMXRQQ0htbTIxYmRrM0h5Z09LdCtpRkpMUi9uODJodGkrUndRUU8x?=
 =?utf-8?B?cHNlaFBHS2w1bmdDdkE1NmxUMk1GRXY4UDlXR0sxQjFwVUFZUCsyTVVyZFFF?=
 =?utf-8?B?OER2bHBhbVI3ZVltVFUybVZzdWlTNWdEWHIwRUVLOWdRUTF1NHNtYWxmbC9Y?=
 =?utf-8?B?MzNzU21CM2dybTBuWWJHY05xVWM3NnY2SEF0a0YzSWZXWkNzOU11cGQ2Qng0?=
 =?utf-8?B?MFN4UGJrVUlWNDNkRWNGTkNxUDlyVVVRTTVWQmNDNnM0M2h4T0dlZHQzZUhK?=
 =?utf-8?B?Z09yZld0cVA1KzFMN1JsY2RRU0dGS2NQdnNtNmlPc2RCMHNrbXVMUFlnaGc0?=
 =?utf-8?B?K1hGMFhjb1ZjcXA3M2J5OER0Y3V2RUpCN1RObTJyZEtZc2loY1lDYUVTUXFQ?=
 =?utf-8?B?VHNuZGNtM1NzTHQrTVh4SGpvZlhnVXpqeHh3UW41SDU3dVpOM2VIdVpYWDU4?=
 =?utf-8?B?d21UMVkxb2R5bW92TnlwR0w0NTQyNHIxbVJMV3NpVjduYmZ4MWZIUVEyejd4?=
 =?utf-8?B?RFlCa01rc1N5NWtYWm8ramlrT1lBbytmamJ0bER2S1RLZmI1b3NpN2c2ckU5?=
 =?utf-8?B?RCsxUWZRUXFwajR5TnFQYUgzdlNScEtuVk5sMGlRVEJiQ1ptNkVmTFNwRnpr?=
 =?utf-8?B?dllxUFR2VDV5Q0RVSHc2bTgzVkxqcDZWT09XN2FLdmd6aUxreXUxd0NiQW5P?=
 =?utf-8?B?MUNnNVVoWHdTUmhUck9lS2pMSXlIVXBVd3Y1clo5cnRHckwyblhJMFRKa3RC?=
 =?utf-8?B?N3F6MjRvQm0rOFdnTlVzODVYMWR1L3Z2Z3BsSGJrTStyM0F6TDdYSWlvMURT?=
 =?utf-8?B?ZEp5R2JNVFhiUmdhWjNlOTQ5SWVVbEtyMWJyWDdVOWszaE9JcVFoTXlEOG43?=
 =?utf-8?B?NllpYUI3WFBJRDdaTkcvOUJjTlYyRlB6VytMcWFUcDFLd0pLWnh4K0k4R1Fs?=
 =?utf-8?B?Qm84YUpDVVJpd202OTRUVlRFUVZNZ3NlMkZNSGpSdmlRVzVQZ2o4OHdJYzAy?=
 =?utf-8?B?ZE5DNkZJaHZLVkZHVE9FM2pzMi9QL2F3Sy9LZ2NXcWRWM1k5dUlQVDFQN1lF?=
 =?utf-8?B?Nyt4UHhYTCt2b25IZEF3TUp5aUFqSlBYeFV3M0FhaTVMemF6dkdZZkd1emtC?=
 =?utf-8?B?OXVCY1ZkbHVpZmNPT1k1MDkzRk9aeGdEd3BvRFJYazZ1Q2RkMzVMbktFbjdm?=
 =?utf-8?B?SFVrdHQwdWY2ZDg4YlhybU41M25EeVh3SHVZU20vT3hvcFRmK0prUkxFMXRx?=
 =?utf-8?B?eGRKYzVCdk9Qc01ST1VaNjVSWm16amhTZWFmNjR1YnQxSEY5VlhiS3dZMzJN?=
 =?utf-8?B?NGRDOTRud1RBcHhicnNBZWRFS09HaitMclowUDRXTGVSbERhbW1aVVRqSFI3?=
 =?utf-8?B?c1l1WkRzdVRUZXdHV01hb3RIK3BUd1c0aXMvS0Rzc3FvVmw2amJJbUtHOWo3?=
 =?utf-8?B?eDJBYUVja29LZFk0QnNLLzZsVlNKNWU2Q2dsMW8vcExKN1NVVjQ2QksyNVlC?=
 =?utf-8?B?WThZQlo3TXdJR00veXFNZnQvRW1ZQzBWYThYOVlOL0tjOTI2OCtGQUdVRDNI?=
 =?utf-8?B?YXozallpMitzRHZBZFNzSUZxR3paKzFCcVRucnRTUUw2L0dyZXBwK3Q0bHNF?=
 =?utf-8?B?aVpZL29yQTZyNmYzQ2Vsd3V4MjU5dUZodEdrUDN6QVlYUUR6SHloVU5hdy80?=
 =?utf-8?B?Z1BTS3ZUNTM0QTVabWpGcFBGRDhFT0dCMkI2cmtPeksxZERTMjBaTVU2U05h?=
 =?utf-8?B?Z2hpSjJUYVNPYm5pdXZsMjhPWllZKzMxK1BtSnJGZHZiZGFCWkZ2Z2c5VERF?=
 =?utf-8?B?Ukw2L2pMQWpkK2p1c0tuY0RWL3NEV0JGcnJpWmlTRDJXYlN1OGxsYkRpa1l3?=
 =?utf-8?B?VXNEMjFMMnFwclY0QVdQS01oNjExWkIyN3p4eFdnWTlHVFhkN04rNCtVOWVI?=
 =?utf-8?B?N3Y0c2VqOUwrQUkwSDNUdEJjY0dFMVpTaFRUMTgyczNIQU5nOXRMbVNaOEVo?=
 =?utf-8?B?MUE9PQ==?=
X-MS-Exchange-CrossTenant-Network-Message-Id: 05b0bda8-0e5f-4913-a36d-08dc80d534a1
X-MS-Exchange-CrossTenant-AuthSource: CO1PR11MB5089.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 30 May 2024 18:20:37.9873
 (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: 0uLrUWn8NsGvqOpVeYVFLqnOJDz/WP4BO3+CdEvRTVBUWxiacTk47ZaFYHkMtOqfCBNudWozqkoLEASG8kJxYOrCXfGydgkmLSGkxyH9OyE=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DS7PR11MB6062
X-OriginatorOrg: intel.com


On 5/29/2024 8:29 AM, Subbaraya Sundeep wrote:
> PF mcam entries has to be at low priority always so that VF
> can install longest prefix match rules at higher priority.
> This was taken care currently but when priority allocation
> wrt reference entry is requested then entries are allocated
> from mid-zone instead of low priority zone. Fix this and
> always allocate entries from low priority zone for PFs.
> 
> Fixes: 7df5b4b260dd ("octeontx2-af: Allocate low priority entries for PF")
> Signed-off-by: Subbaraya Sundeep <sbhatta@marvell.com>
> ---

Reviewed-by: Jacob Keller <jacob.e.keller@intel.com>