Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Date:   Thu, 6 Jul 2023 11:51:59 +0200
From:   Michal Kubiak <michal.kubiak@intel.com>
To:     Ratheesh Kannoth <rkannoth@marvell.com>
CC:     <netdev@vger.kernel.org>, <linux-kernel@vger.kernel.org>,
        <sgoutham@marvell.com>, <davem@davemloft.net>,
        <edumazet@google.com>, <kuba@kernel.org>, <pabeni@redhat.com>,
        <sbhatta@marvell.com>, <gakula@marvell.com>, <schalla@marvell.com>,
        <hkelam@marvell.com>
Subject: Re: [PATCH net v1] octeontx2-af: Promisc enable/disable through mbox
Message-ID: <ZKaOv6t8+9epGWra@localhost.localdomain>
References: <20230706042705.3235990-1-rkannoth@marvell.com>
Content-Type: text/plain; charset="us-ascii"
Content-Disposition: inline
In-Reply-To: <20230706042705.3235990-1-rkannoth@marvell.com>
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?/rq+7CZ3rNEIlotBboiX5edg01g3BnpafyMHUywwUhueZmt5pWwrTLFTP5MB?=
 =?us-ascii?Q?lR7KCDryS2a+34rDGYGOHvbaCvKn+ZvGNR2mFNqcSDNmo0x9PFzl/PSf3JKb?=
 =?us-ascii?Q?cFC9Xp2lFK0jBIBhUe5XufZxFr/ATKpzAhPAUqVK/LCNKoeDiTrqdKuu+Tm1?=
 =?us-ascii?Q?OK/D/pev0y140MDrjBcHufd2fWGo4UwiKgXjfvVBOJFQxR+dxshSY8SW8T33?=
 =?us-ascii?Q?wEfjhB9a/CpEDhZwK86gFh/csICe69TIJ0O0vuXJwmGNViAdZyXOy9Ko+W0a?=
 =?us-ascii?Q?cPjmkzBf9Rdztmh9zn35baKnR+aPMeUBI81AfEniqENksLtMdzCgFMlVew4D?=
 =?us-ascii?Q?FRBiewW0Yhyh8bM4EPecYHjfdTUdhcbhiBpqW1KeyKjJxPGIx4rT/GxZJmlx?=
 =?us-ascii?Q?5YxCDk+jq6ipEoIWXqRbxeaHCcrlto65KFlc9wBsDidBHbFxYoC5xTh8fou1?=
 =?us-ascii?Q?Wv2G8e4KTuNAljqHYdLdTu1fOjcj02duQhLdKzT6PqoJnfbqPC3iERY2Ljij?=
 =?us-ascii?Q?PESv2WXZHsgnwPyIfrERuS6uU2Nmx+3mPbTMvbZToJ399Hrdd1ZQ1lmCnh16?=
 =?us-ascii?Q?WEo1vrMaPtMuH6JewlLawURPewiNn4WgOZs6rS4tw6DA4rjQFfMFjgrsK+Z5?=
 =?us-ascii?Q?qVpvjI0pMT2iSQziAOnLMsz8KJq4CyRqSI3/Hq0QZL+yZFuNKwy9LCpxkSq1?=
 =?us-ascii?Q?jhv73oHVMxXw07/YITJqKOXXLOSnQhtG5BRD5qRtpYOINY8Ex8o3qAreIzuU?=
 =?us-ascii?Q?Gi44MO+fbfKj3oIaQclY9KMZ8DIgrTKDDWQ/ZLVg7+xUAkk66yuIRyeQCkA+?=
 =?us-ascii?Q?uGKDrsoDdt6cCZF+0tCkHyjbNfkPZvD+ftZeL98SvGMc4ZhblggIta0+PIFl?=
 =?us-ascii?Q?FaaqnQAia1tbJTGaEP+5+2CWfm1qajGV4YKmcrejt5fglnPgEWr9JF53EdxO?=
 =?us-ascii?Q?mKrTQ4dQLw/CHqEbP1puMxLvb/2alEKU+AyEpybOI5qNRrpxrvgnBz+wXfrN?=
 =?us-ascii?Q?i1xIYGMPHO/OM/BoWWSnJnDPKeUNAfUnSmw+UKYez6VAxKEdWNVfLxVlcG2d?=
 =?us-ascii?Q?dP4KuTOvTzMjXNqSEnygVETmSFA6jgcFIU1gWWyG+Fg2ASNONziz9Tq22kW9?=
 =?us-ascii?Q?LLF0OL8aCZWPTT1eobZQ4CjCGClR018t5t8XOpPhTFOD6PrT2wVScPJ3bS/A?=
 =?us-ascii?Q?M7sLbpkhOHsvTjxFcJbs/RqBdtfwnUBOHv/6qnL6LpkMyJAQCkDUD6DJsZE7?=
 =?us-ascii?Q?bBWv2m/vYmkGvL4hEYJzFqkSE1/cWz6Lr/ApxB7oEuUU3GZ0Tr0r6KT3XJZ0?=
 =?us-ascii?Q?a/BStkelDcdQZUuGmXX7jZz2Y61Q/ozIAdSNPOZrtMOAQjTuQqTxIOy40UbA?=
 =?us-ascii?Q?n8Rv41bYofS/DwL2dRVZuYDPIST3iJ/0281k8hoHBo9SjXyn9iSStqt/bweu?=
 =?us-ascii?Q?yP+oJftc13EavOpMPwrIp5cvYlYVRmYrBTc2ksvcQ1C/KYw4EyKZdviGbb6T?=
 =?us-ascii?Q?+jrNpa8I+gf5ok2Wy7tvrV8GDzLmbxaQTT7qYy4AZs7cjEAF7Tgl8u36Yez4?=
 =?us-ascii?Q?i71qoev0g2xeSlaOtCYxhCrFvED9uzlMYIu0tRGK8/buTb/4RLnJGMsgR0W5?=
 =?us-ascii?Q?2w=3D=3D?=
X-MS-Exchange-CrossTenant-Network-Message-Id: 61770922-9df7-485d-2bf6-08db7e06a7ee
X-MS-Exchange-CrossTenant-AuthSource: DM6PR11MB2937.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 06 Jul 2023 09:52:05.7714
 (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: jhKpgOIwUrvWw+m2ZJzWRwbj+qgTu1jy3rKA+lXs9V0xS7zBatxTSSqdIIIS+y4QQpttZcZoox51cBgWgyv99w==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: BN9PR11MB5241
X-OriginatorOrg: intel.com
X-Spam-Status: No, score=-4.4 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH,
        DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_MED,
        SPF_HELO_NONE,SPF_NONE,T_SCC_BODY_TEXT_LINE autolearn=ham
        autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On Thu, Jul 06, 2023 at 09:57:05AM +0530, Ratheesh Kannoth wrote:
> In legacy silicon, promiscuous mode is only modified
> through CGX mbox messages. In CN10KB silicon, it is modified
> from CGX mbox and NIX. This breaks legacy application
> behaviour. Fix this by removing call from NIX.
> 
> Fixes: d6c9784baf59 ("octeontx2-af: Invoke exact match functions if supported")
> Signed-off-by: Ratheesh Kannoth <rkannoth@marvell.com>
> 
> ---
> ChangeLog
> 
> v0 -> v1: Fix 80 lines checkpatch warnings

Thanks for fixing this!

Reviewed-by: Michal Kubiak <michal.kubiak@intel.com>