From patchwork Wed Mar 22 23:38:19 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Vladimir Oltean X-Patchwork-Id: 73701 Return-Path: Delivered-To: ouuuleilei@gmail.com Received: by 2002:a5d:604a:0:0:0:0:0 with SMTP id j10csp2633183wrt; Wed, 22 Mar 2023 16:54:26 -0700 (PDT) X-Google-Smtp-Source: AK7set9D226SoVpW0m5wuBUEVY4aYKFAnt69kYaQhO86xeP1gURsrK+V865WA0yDzPp6xVT6oM5a X-Received: by 2002:a17:907:a090:b0:93b:68a8:a0e0 with SMTP id hu16-20020a170907a09000b0093b68a8a0e0mr4582905ejc.16.1679529266431; Wed, 22 Mar 2023 16:54:26 -0700 (PDT) ARC-Seal: i=2; a=rsa-sha256; t=1679529266; cv=pass; d=google.com; s=arc-20160816; b=gZGDWvGJ27Bbjg6u4v+tbFp/tdMuwxZ94OzQXuME8cW4otyEI6cbTPngZ4/RtdbNqK gzXdQfZmNZ9vcd/+hyG6X6UbISFwScfspnFpyeR9BuS+zu+fOaqnEbSbM8ZXEvDdRPo3 2O2oEK1c0V5mxPRQOHu457WsD1+RlH25OkJ3e403+0Tobw5I80DvwCHfapAw2tadhphf UedOqOu9kZMjcTkxhcY7Xhc54O2dwoCs6FB7yfjtOspsaRhR2xqdvX2JnQew1iov1KCy VnYrdYaRoGwj7U1dPYuie15ymO/cg9jXleI5C3moMqJV3YF/jydAuMTS+auyXuWZGbYm Yxtg== ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:mime-version:content-transfer-encoding :references:in-reply-to:message-id:date:subject:cc:to:from :dkim-signature; bh=hd/5oYgaxvpHfDjM/kbGG6yPnjs27J4T7oS5QnTeMXA=; b=NGgZZD5qiiCGbIn5Zn/gSMtjJ8YYyIemKTaUcO0WIPTdIMk1vWjS+wpJHlysAQsAO5 LtznsMk+z6GFj+CavKJ3h4Z/POBI0t5M1OjNZ2r5JWXzd4567I8CLuZlBDMtrzUvbbDC YuH8/pPbwW+n2J0Rd/vpNgatMoUBUkS02+kOQnTXdTbcffSAT7i1mlDM/OYWJwFSHbDV IFz6Oj57AROwT/X0u7TsCS0S7FMZ3diIT31MdlNVQF5DIr0OGvZierkgXurEseyoiMri 0ILJo+hrmWm97rKO2llvh1k756s7RQYSJsgAYhGTYmrxek6WHK9ZJUKL3xTLM3Mgs22s 2LAw== ARC-Authentication-Results: i=2; mx.google.com; dkim=pass header.i=@nxp.com header.s=selector2 header.b=N6uNlSoT; arc=pass (i=1 spf=pass spfdomain=nxp.com dkim=pass dkdomain=nxp.com dmarc=pass fromdomain=nxp.com); spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=nxp.com Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id p25-20020a1709061b5900b0092e8abc18a4si1629149ejg.965.2023.03.22.16.54.03; Wed, 22 Mar 2023 16:54:26 -0700 (PDT) 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; Authentication-Results: mx.google.com; dkim=pass header.i=@nxp.com header.s=selector2 header.b=N6uNlSoT; arc=pass (i=1 spf=pass spfdomain=nxp.com dkim=pass dkdomain=nxp.com dmarc=pass fromdomain=nxp.com); spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=nxp.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230157AbjCVXjI (ORCPT + 99 others); Wed, 22 Mar 2023 19:39:08 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:56082 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S230146AbjCVXiw (ORCPT ); Wed, 22 Mar 2023 19:38:52 -0400 Received: from EUR04-VI1-obe.outbound.protection.outlook.com (mail-vi1eur04on2048.outbound.protection.outlook.com [40.107.8.48]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 901B82B9E1; Wed, 22 Mar 2023 16:38:45 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=b8ZIpwmGpyTRR9b/4DI/oT/EU1Lt01Sde/O1h0JJ+ZWdSshJPuVEF0PCawQAQikTa/5MT3XwRkH8cpL3n/GoPgGhRW5ZeQ795CWGgA3VKu6FCtvhy3xSEQLkriJ0L6fGD8BKfVc+2k9Kv6vMJC+TjI4P7hdHWoLSHMM/vTwGYz1QprMjsKWpmPHxUCnBx5BGexzBqt6a1+MU7AajckF+RykQUwqcOlx+YcjQQNLL2LV98hJNz/cj3Jt2M+1gzOrC+liThsrsnh2EernCSPGMjVl/CDd8DMsRRDA4qnDU6igT/scFHtw2Fg2EXNvOFI/EKRMxlBC7YGtzvw0dBHI4fA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=hd/5oYgaxvpHfDjM/kbGG6yPnjs27J4T7oS5QnTeMXA=; b=IlY16sRmBkKY9DrZPw367QbdvQuxPzlerUk1CwJQ8tgKDdVJbdcUNNQjH553sgy+0b0tyxafJZwnipzjGFHEsfo9gwAO55XT9ZAho8B5c0u7+gpfrFKFuyPsA/rDB0jpSEZCkJUUH3U+8Qc+5lGCx9tj9WfFqSlywcnC5ZLwCB+z5X8CbWU9EwvVwF5vE/A/CerRb4WX6VFVsKUTNm2RQajY6WUdKO17Y4cKEd1sTTOecqpoZeyrBKzC0c4b9p4pZyjMxs8RsOI31UKjFmMTrFEBXbzjkXnGVRgdQSPoL02AUmzA9YaCC1ziR0o+LPbBL8f8f8EqtUZTGMdyR1jjtw== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=nxp.com; dmarc=pass action=none header.from=nxp.com; dkim=pass header.d=nxp.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=nxp.com; s=selector2; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=hd/5oYgaxvpHfDjM/kbGG6yPnjs27J4T7oS5QnTeMXA=; b=N6uNlSoT1ztnE/6GPg13MyG1WjzTjWDmxfIxHvWK0ekJTVIhQ2a5iV0ysq9TsfQbxI/mKp0xAkoFWooZ7ZhjFvWXOD7+lmf2oTWScLQ+lqD+3X/i3wVRF5O6UxCz414wGGH7OrsiZCeicDwu6K9msF0tcLwOpjvAT3Hj0TtAxvc= Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=nxp.com; Received: from AM0PR04MB6452.eurprd04.prod.outlook.com (2603:10a6:208:16d::21) by VE1PR04MB7263.eurprd04.prod.outlook.com (2603:10a6:800:1af::21) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.6178.37; Wed, 22 Mar 2023 23:38:40 +0000 Received: from AM0PR04MB6452.eurprd04.prod.outlook.com ([fe80::29a3:120c:7d42:3ca8]) by AM0PR04MB6452.eurprd04.prod.outlook.com ([fe80::29a3:120c:7d42:3ca8%7]) with mapi id 15.20.6178.038; Wed, 22 Mar 2023 23:38:40 +0000 From: Vladimir Oltean To: netdev@vger.kernel.org Cc: "David S. Miller" , Eric Dumazet , Jakub Kicinski , Paolo Abeni , Andrew Lunn , Florian Fainelli , linux-kernel@vger.kernel.org Subject: [PATCH net-next 5/9] net: dsa: tag_ksz: do not rely on skb_mac_header() in TX paths Date: Thu, 23 Mar 2023 01:38:19 +0200 Message-Id: <20230322233823.1806736-6-vladimir.oltean@nxp.com> X-Mailer: git-send-email 2.34.1 In-Reply-To: <20230322233823.1806736-1-vladimir.oltean@nxp.com> References: <20230322233823.1806736-1-vladimir.oltean@nxp.com> X-ClientProxiedBy: FR2P281CA0081.DEUP281.PROD.OUTLOOK.COM (2603:10a6:d10:9a::8) To AM0PR04MB6452.eurprd04.prod.outlook.com (2603:10a6:208:16d::21) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: AM0PR04MB6452:EE_|VE1PR04MB7263:EE_ X-MS-Office365-Filtering-Correlation-Id: 798b6d7f-d903-4ce5-0f91-08db2b2e90a0 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: k21tdOphDvuzhUxRrHBsXGh/TDj/qsz0XInkHqEld271PflsLswgqT14ayPzyY9PwpGMT8W60Ksgu8euk25yaEYdtWkhRWNvr6/2sw3ubovE2+Y4MToQy/rDSuamakJU4AJ170TqlXxB3WeXGRMEsngTUtPF1cEioY9F8LLoy7lO73ZKOUvw4QMdByRTQt1zrskKfzgKpgS5U8HZvpkQg/mkm10B4Sa/o5jY2P9s9d2ls8mqn0H+6/r27ietyUF1KLD6w5VqfXfdGjcqykJ3CYDN4hNgnhxQ9ZzKcezmC1PDTIYVrzgxmYc/OTtyWzxbnm324fTMntAVqurAM6ZxLM+RxDc83567+eBty4pls2me8vAs4DE+G9tlbjhThDyDS+GFJqhSg7ywLDHq0bfbFRQastP/WCIe+vvWjn4vpQ+UU5nPPOHcunm0IiyTDJ3j7GiDLO3WReqsXhY4sm0pzMANIpgORGaKrDAgKWb4cB+Xq7G0eCfv3+i4g1AhCQYFONjXASYONKE5djwtfmA0nXrnnyrTpnM70OjTyN+0g1/b1wgqGDxNDbaXG8jRN77Uwnb5vOGNXfPq2q6ksGuaHt+aiCL8Rw8DElFGXoEIAtpKfj0ESUVvQxcC5njkiPcqBFl6jHTYbR/hroYI3Rsykbo3aJ7xCt9klxYshKnTPzqVYrUQG5Tl2hvF6xSZLCNc7NsfK82OGUftaJqCD+zGAI+tsWLCkpiq3YhMbd+r7TxKelX2kpIEL+IEE69sJc0P X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:AM0PR04MB6452.eurprd04.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230025)(4636009)(346002)(376002)(39860400002)(136003)(366004)(396003)(451199018)(54906003)(66556008)(6486002)(6916009)(316002)(66476007)(8676002)(66946007)(478600001)(6666004)(4326008)(52116002)(41300700001)(6506007)(1076003)(6512007)(26005)(8936002)(186003)(44832011)(5660300002)(2906002)(2616005)(36756003)(83380400001)(38100700002)(38350700002)(86362001);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: BSQNfs9N8YCoC/EPAiNueapYRO7JCAbUo6YWnGwQNoTLWrn54Nt2yCFgtq7DlFNKCv2stwB6nX1ddm1L7sn8OS2hVAGG3D6DUBItAWrkGXnNTEIqOCiYVcRKno77ntYu6YdJ42LXD7JgHZfMk1SHf60ZQ7+sLYykXPJ9jsobuG7fHwOsfs2HCS7TR0TkhQVGOPAjjqfw7kTvAVqnfTYDWzIjlpGZttwIgRy1Dt5akW8xVTRXLiEiY25FTjRiTRqwo13q2ACbJsxIzjMAHhfHiGT9vLRTWbzJGjY4j/b9ufeZ/IwmFbaiPN9D8ZeK8+/sizJWXUD6hiWCeplVZyPdV1JVutAmzlHX0d3DLzG/26+3VWSRK8jQekwvnPBrMCpvqsWNL0z1MCwY+m3RzOKw/V5CmAhpqWnMRf2Q6NZOcWXgjayJ9i8UhC15hPth0q74vKaP2z6RZvu0vUsfJ+oFRpF4JBVTiya78Xt8d8nlR8caHIowGzo+83WyD7us5zLg8E/ec/3ap7eIeqPi8y2uESCZbQUTAtPXQPgREbcvqD3k6Nl2aRWAo9TYdxXiX9iylwh8u4pp/oYaMpo6WLJnnTQqYKRV9W/Ku9qbbaYKBOZgM4XgKbRHerRV+rCnpriUV6U7hr7z2FxBvKcF8fNVN6Kzdw7GaZfC+sv8aNPUFB2tWDtsGxGqsuHJNprT4+g6PwnPXMK23uENQJIGt+tS5oXFnnBZTpc6BtW95i6LxBOrsBulx0K1MSNai66x/d57CGyU847UVwz7a8weMg0aZb419su7U5+QjrA3MJOmRK+mOl7hCCSOaxN4EVIUs6UFzb0gwS7vHscxgVsQPHgvJO7Ev+cYErEnLxuW/mvSvM5+mOaaCgKla9TQ92pZlYTfLIbHW3DAGg0eTiRNcTUz4+WVT8bDssyYyLBULeCBkCqDB611S8Mx5EnFWC8zzjZRF/VW+8JB0oHwwC4JsrtTAAGorg59lrTK4VfNUZrKx/eJD8h3idcEIWgQBFyMO9z1Ghj79JH7Lv3+wB9jF9j57bW0LnslmtGMFFaJTaP3FSkWn9wQvYyrzzxnkJydBoqg0fUu1S7blYcmzLkhSn7gCZNqTfUy2r9zMqjP18RhHxtNjhH02G9Amg8sp4BDX8Wu0KmBeU8gsKrXsySymJwi1EISTuQhwfqfCp5xepTCz7++ouMuRYjZ/2m0UoWW+heG/0OYqQfGjznnAw2n+wd3uWi1bUVYbv/T+mJIRqPfUd6R7dO9Wh/i5EW8RNsNjFLvOQOKNhEs6jj3KBA6lJnUUFn+oqU556Zinuukt5jAZbohnMvcTaG8rVnklajh0wLT7IGQPXVZyXCL//AzHN/KViKfqnxE36iHXxWvzB9v2XK8jOTROYE9sgP6UghomqpidjCTlIqeAjY+WjfoPS0srzsWpaE2Xc7e6uHhCf5yBE++u64D62/zKs1806QbVeah8dPEwg+19P5MSL36zxgFphDcJrzjOpgqxujCOTVUaJ38FVQaIMyCnbJcb/9j2Jo5HqDjbdUWQXtU3DF2/3cjoZPwa9d6MrWPICprKWSKIqN7uuRm0n7a6rb6mFuRWZ78QkEbYfjoy4vfCldA6H0R4w== X-OriginatorOrg: nxp.com X-MS-Exchange-CrossTenant-Network-Message-Id: 798b6d7f-d903-4ce5-0f91-08db2b2e90a0 X-MS-Exchange-CrossTenant-AuthSource: AM0PR04MB6452.eurprd04.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 22 Mar 2023 23:38:39.9518 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635 X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: nQx29gAAWjr7iRt5JqVzAz3r4XGqMzS74O6sFbFPafJP20ug/OpuTnQYgF691cT+IQaLMrffEPUJ5zSdYB1PNg== X-MS-Exchange-Transport-CrossTenantHeadersStamped: VE1PR04MB7263 X-Spam-Status: No, score=-0.2 required=5.0 tests=DKIM_SIGNED,DKIM_VALID, DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_NONE,RCVD_IN_MSPIKE_H2, SPF_HELO_PASS,SPF_PASS,URIBL_BLOCKED autolearn=unavailable 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: X-Mailing-List: linux-kernel@vger.kernel.org X-getmail-retrieved-from-mailbox: =?utf-8?q?INBOX?= X-GMAIL-THRID: =?utf-8?q?1761114080233491223?= X-GMAIL-MSGID: =?utf-8?q?1761114080233491223?= skb_mac_header() will no longer be available in the TX path when reverting commit 6d1ccff62780 ("net: reset mac header in dev_start_xmit()"). As preparation for that, let's use skb_eth_hdr() to get to the Ethernet header's MAC DA instead, helper which assumes this header is located at skb->data (assumption which holds true here). Signed-off-by: Vladimir Oltean Reviewed-by: Simon Horman Reviewed-by: Florian Fainelli --- net/dsa/tag_ksz.c | 18 +++++++++--------- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/net/dsa/tag_ksz.c b/net/dsa/tag_ksz.c index 0eb1c7784c3d..ea100bd25939 100644 --- a/net/dsa/tag_ksz.c +++ b/net/dsa/tag_ksz.c @@ -120,18 +120,18 @@ static struct sk_buff *ksz_common_rcv(struct sk_buff *skb, static struct sk_buff *ksz8795_xmit(struct sk_buff *skb, struct net_device *dev) { struct dsa_port *dp = dsa_slave_to_port(dev); + struct ethhdr *hdr; u8 *tag; - u8 *addr; if (skb->ip_summed == CHECKSUM_PARTIAL && skb_checksum_help(skb)) return NULL; /* Tag encoding */ tag = skb_put(skb, KSZ_INGRESS_TAG_LEN); - addr = skb_mac_header(skb); + hdr = skb_eth_hdr(skb); *tag = 1 << dp->index; - if (is_link_local_ether_addr(addr)) + if (is_link_local_ether_addr(hdr->h_dest)) *tag |= KSZ8795_TAIL_TAG_OVERRIDE; return skb; @@ -273,8 +273,8 @@ static struct sk_buff *ksz9477_xmit(struct sk_buff *skb, u16 queue_mapping = skb_get_queue_mapping(skb); u8 prio = netdev_txq_to_tc(dev, queue_mapping); struct dsa_port *dp = dsa_slave_to_port(dev); + struct ethhdr *hdr; __be16 *tag; - u8 *addr; u16 val; if (skb->ip_summed == CHECKSUM_PARTIAL && skb_checksum_help(skb)) @@ -284,13 +284,13 @@ static struct sk_buff *ksz9477_xmit(struct sk_buff *skb, ksz_xmit_timestamp(dp, skb); tag = skb_put(skb, KSZ9477_INGRESS_TAG_LEN); - addr = skb_mac_header(skb); + hdr = skb_eth_hdr(skb); val = BIT(dp->index); val |= FIELD_PREP(KSZ9477_TAIL_TAG_PRIO, prio); - if (is_link_local_ether_addr(addr)) + if (is_link_local_ether_addr(hdr->h_dest)) val |= KSZ9477_TAIL_TAG_OVERRIDE; *tag = cpu_to_be16(val); @@ -337,7 +337,7 @@ static struct sk_buff *ksz9893_xmit(struct sk_buff *skb, u16 queue_mapping = skb_get_queue_mapping(skb); u8 prio = netdev_txq_to_tc(dev, queue_mapping); struct dsa_port *dp = dsa_slave_to_port(dev); - u8 *addr; + struct ethhdr *hdr; u8 *tag; if (skb->ip_summed == CHECKSUM_PARTIAL && skb_checksum_help(skb)) @@ -347,13 +347,13 @@ static struct sk_buff *ksz9893_xmit(struct sk_buff *skb, ksz_xmit_timestamp(dp, skb); tag = skb_put(skb, KSZ_INGRESS_TAG_LEN); - addr = skb_mac_header(skb); + hdr = skb_eth_hdr(skb); *tag = BIT(dp->index); *tag |= FIELD_PREP(KSZ9893_TAIL_TAG_PRIO, prio); - if (is_link_local_ether_addr(addr)) + if (is_link_local_ether_addr(hdr->h_dest)) *tag |= KSZ9893_TAIL_TAG_OVERRIDE; return ksz_defer_xmit(dp, skb);