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;
Message-ID: <d30ccfb9-5632-de27-fd2e-07b93adaad42@intel.com>
Date:   Mon, 31 Jul 2023 10:20:29 -0700
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:102.0) Gecko/20100101
 Firefox/102.0 Thunderbird/102.13.0
Subject: Re: [net-next PATCH V2 1/2] octeontx2-af: Code restructure to handle
 TC outer VLAN offload
Content-Language: en-US
To:     Suman Ghosh <sumang@marvell.com>, <sgoutham@marvell.com>,
        <gakula@marvell.com>, <sbhatta@marvell.com>, <hkelam@marvell.com>,
        <davem@davemloft.net>, <edumazet@google.com>, <kuba@kernel.org>,
        <pabeni@redhat.com>, <netdev@vger.kernel.org>,
        <linux-kernel@vger.kernel.org>, <lcherian@marvell.com>,
        <jerinj@marvell.com>, <simon.horman@corigine.com>
References: <20230730143107.2845172-1-sumang@marvell.com>
 <20230730143107.2845172-2-sumang@marvell.com>
From:   Jesse Brandeburg <jesse.brandeburg@intel.com>
In-Reply-To: <20230730143107.2845172-2-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?S0pGL0Z1ajYvWmVLRzlHZFkvTDU5YlplcVVLTHp2NmFPamUxVjhuYXVVUnJM?=
 =?utf-8?B?QlA0S3hKOHhBSXQ3RUlhaGVQcENpTndJeWR2dDIzS3hlK3FkZFpFWWpqYmNz?=
 =?utf-8?B?dmVGMytySDVVVzBUOUhrOE5LZFlKckhTT3E0Zm9IWDdocXVTZSs4bTJTK2pF?=
 =?utf-8?B?MDlGeG1RTC80alVWSG9samdPZHo1OHZzQWxld2pwWStqOXQ3QlNZVkVJQ1ZQ?=
 =?utf-8?B?bkVnN1orRGtBTWxGcE84OE1pbWlqSkpmRW1pUVBFRVhrNGczd0ZZMWp4bEJG?=
 =?utf-8?B?ZXdJVERJM25XVXVNdHlYdjNhVGZYYmVXWmRQZjIxS0xGcFUwc3V3bW5UQnd3?=
 =?utf-8?B?Nnd2dk1oYkVIZk0wZlNhRHcwblQ2blV4Uk1pVUN3bDB1VlJmbG5Vdi9pd0dw?=
 =?utf-8?B?QXFLZ01hek55VDBJRUpIMlVGNTU2dkQ5TFNQak0yc0RJdDJDTU9hZ2pjZFNr?=
 =?utf-8?B?TmRHRjlSUlJka09xRUhtZ2Mvb3FHY3V0Yk5QSFJROTUrN1IzdDY0c2p4NEZW?=
 =?utf-8?B?RGtLRUFEQ0R0aG5wM2J3MW9iZkYzc1RNZHdvWXJiRUh1Q0loeDd6UVd4eWJM?=
 =?utf-8?B?amw3T0huL2FoNm53OWFmNjgzQmNnNkhjZkhXcEhuQkQwcHErYUV0WlJtTzdI?=
 =?utf-8?B?Y1dMRkYyQjBmWG90aXpJR25SNjdmS1VjSEUxbGtYejJ3UllKTVBteVpuRTNV?=
 =?utf-8?B?ZHhKTHJXUC9vTUtFOXlFQXFZWTdrQS94VDlVV2hOY0U5S1hjYWUxeG52YlFF?=
 =?utf-8?B?WENTOCtCR2E1aU0zRGV6dFRLYjByVUFwY2o1S0V2YWhNTlpJNjgrWnBzeDUy?=
 =?utf-8?B?K3VkbURzejZRRkFuMFY4WXQrTWE0RWRGbVFrcHg4Z2M5WXRBaGQvSGhKVnRs?=
 =?utf-8?B?ajQxVGtsY1N1ZTUyOEl6Tmg0NlgyUW9aSStMcG42clk5ckdPdWxTelp1NG42?=
 =?utf-8?B?b3R4cDVKYXAwRk5mUjQrUUZzWTlibXBIRHVVbm01eFFYUDRqVmN1OG9WMFFK?=
 =?utf-8?B?WHF0K3pTZDBwNTNnSzl4WndBSXNXNE1BODE5aFFvZEdLQUZJUW9Qa0d1SVRR?=
 =?utf-8?B?LzVidk5CVUZpOWxDUXpLdVkxVHNJTEZsY0hyTE96aXBaakVWdlJZODVyNjBC?=
 =?utf-8?B?OUNuNEtxbE1ZVUZ1dDcxMU9ZdlJGNjJaaXBBMzRsWFNvQ09oeU9iNkdFVUxa?=
 =?utf-8?B?aGdpbzN4VitPd2FiOVZvS056YUVibW14aDJDOGQ3UWIrUllGbkoyamt2RHRm?=
 =?utf-8?B?K1c5V2VPcTlUdW83NWh0T3VwZmh6M3pPRGJCMnlGV2lOU3dRVzFreSt0eWE0?=
 =?utf-8?B?WnIwUFM1QVl6UzB3L1c0N1prWVRUSFN5VVM4enNmTEJMcWhlZkVDUzdHelcz?=
 =?utf-8?B?c1NMSDgxR0tGZ0E3MWdLdVl5YldwcjlSOHArWjdDNUlRK0ljOU1KK0dDQmZM?=
 =?utf-8?B?aGdRRHhPanhiTExyTmV6cEZIVFZHN1FCVUhQNW1Henl5VkFWWmt4enFucDZQ?=
 =?utf-8?B?WVFvenBqc291S3hUcnlsLzlIY3dvWDI1cFZiTEFWRDk0VU5aU2hha0hEVlMr?=
 =?utf-8?B?TkFBT2hHdTBCd3VZQ2I5aTRrSUhKT2JiN25EOHpEdGRtaWhJN2hNL0YydFdz?=
 =?utf-8?B?dXoyaE5saVBQc1EzMEdUZ1RReXo0Z3dKOFdLeE5jV1pmZ1g4eWlHV1VkODJ1?=
 =?utf-8?B?TFUvbHNITmVTbU51UndPTFpQd0NNbUZWM0FKUEVYSEFxdEh5dWo4QnV6WVRT?=
 =?utf-8?B?UzVYWVFYNzl4dnEvWWxnWjB6SzhLVzdZTEx1RnI2ald4YTBvVDIrOW9nZ0tw?=
 =?utf-8?B?L2ZvT3QzVWdSSlJHUi9KNXJIZjNaV1Y2eFFrRVVSanhCMVZwT3c5ZkpOMEhh?=
 =?utf-8?B?eVlKT3hPeU4xYnJDdS9Cb29wNTV3MkdwN3V6ZlpPN3dzcU8wbVQ2M1IxbXZY?=
 =?utf-8?B?V3lTSXNheUFaelQwNTA4RWZ3dktmQzdiT3RtOERBVmxvWVJwdGFwNXhPMkxE?=
 =?utf-8?B?SW5CamdlZlFsN0VyQnpsT3dnRld1RUQyMnhmTldwZnhzNnp2N1kxdTI1RGJy?=
 =?utf-8?B?VXpjZzREK0p2bXYxQlhVMWo3VmxjS1BJVk5iVDBhck5oLy9zM0dsSVkweDBS?=
 =?utf-8?B?cmZjRzNwRXdMZnB2S2tXSUt5K3ZKSzZBa1ZYMmJIa2tIc3J2TWl6Q2xRaXU1?=
 =?utf-8?B?Ymc9PQ==?=
X-MS-Exchange-CrossTenant-Network-Message-Id: 67a97620-c318-43e7-32ec-08db91ea7114
X-MS-Exchange-CrossTenant-AuthSource: CO1PR11MB4914.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 31 Jul 2023 17:20:31.0113
 (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: e5mXtBt753HrmNxyz7woiniNn5yajrYSVawO3UjbNgfoFUb9WrFcrgQ9X4Fiyl5VSsHEFlpA/K62vDvw5DNEZoTh3iKAg7lYx9TrBefa4y8=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: LV8PR11MB8461
X-OriginatorOrg: intel.com
X-Spam-Status: No, score=-4.5 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH,
        DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,NICE_REPLY_A,
        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 7/30/2023 7:31 AM, Suman Ghosh wrote:
> Moved the TC outer VLAN offload support to a separate function.
> This change is done to handle all VLAN related changes cleanly from
> a dedicated function.
> 
> Signed-off-by: Suman Ghosh <sumang@marvell.com>

Reviewed-by: Jesse Brandeburg <jesse.brandeburg@intel.com>