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;
From:   Vladimir Oltean <vladimir.oltean@nxp.com>
To:     netdev@vger.kernel.org
Cc:     Andrew Lunn <andrew@lunn.ch>,
        Florian Fainelli <f.fainelli@gmail.com>,
        "David S. Miller" <davem@davemloft.net>,
        Eric Dumazet <edumazet@google.com>,
        Jakub Kicinski <kuba@kernel.org>,
        Paolo Abeni <pabeni@redhat.com>, linux-kernel@vger.kernel.org
Subject: [PATCH net 0/2] Fix mangled link-local MAC DAs with SJA1105 DSA
Date:   Thu, 29 Jun 2023 17:14:51 +0300
Message-Id: <20230629141453.1112919-1-vladimir.oltean@nxp.com>
Content-Transfer-Encoding: 8bit
Content-Type: text/plain
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?o/7mQbN2s7py8cfozXJHYLFo308/HtqV+GsWVPDt3E50o5URAJYsXZHLMSIu?=
 =?us-ascii?Q?jphTiC4HJmxdXhbDdi1jeBRfOPm7iqnUu+a/I+3Rg3zjHw+4YglftJjsdZNV?=
 =?us-ascii?Q?4eQ855PXnnqjxq4ngcsUla4grQ15Iy7t4mXZF/kVex4mJXIT6HIIe+9CV6CV?=
 =?us-ascii?Q?pZ+ubni2c92ZSh+LG4HQurOKu7z5Sl7DA1I/FFrRVMoBvx6OiyiwM5asQqG0?=
 =?us-ascii?Q?HYT68DkrJl3ftRioW5Y9ZbU/OoE8nQ2iSh9e0MAzSF+SjO9R+obxqGZiImjB?=
 =?us-ascii?Q?QncvaZgY/lGQbCgyuRxFA6/x4vtqvtBX3251gVdSEwvo5NOulmq6PvyLzOc7?=
 =?us-ascii?Q?CVGueOUbpXC3JALyy/UfZNCsJd3jRNiQviiU7wIEJlo1VhWxYk/qb82mP0Bm?=
 =?us-ascii?Q?AklkytLXUUMZDeC8UaOQHqh2IQAMUj0I+55epHCso2BMH8iw1WAXLGDQMftr?=
 =?us-ascii?Q?EpYrMJn45RfMOqpLtLn7/AS5bKU1KW7ntnqGZN8vX3EQEC36jtjBUbIKxslR?=
 =?us-ascii?Q?CrJ2D27wrp52mQPXhW5g0yXrh/gPktsWIF7y90pbPqtxY9Vbf/75Hw1lpNKU?=
 =?us-ascii?Q?1DwZr3a7Fx+ZLzjnwNkDKcQpRqIW6I/3IW97+3IKQ8bPO/Bne1mj83ixlscq?=
 =?us-ascii?Q?fMVE7pIOPyH7KoxjpQLyjBoV0sCYrlxa2ZwM6i7Zon3KJ0j/joAyEC+60QNn?=
 =?us-ascii?Q?/itPlNcgIrlE0Rdg2BE+kqaT9O++WQcoQ6dMZ5urGi5FeSM1JxML+Q6I/Rsh?=
 =?us-ascii?Q?B7gw53AkArJola42SmGPi1d/NRji6LcR5Zgdd45ouJ9sH7vBN4EyssNLa8kC?=
 =?us-ascii?Q?bP4vp52cs1EyELcsQ8zVLuMdI6UKzuLfFJ57imLhNSHzw07XAzXmv5vqik5M?=
 =?us-ascii?Q?6PswVYFABsPBKzt8476rV9gy6LrynCS++ycxSIW9fovcvDbPT9WDlzzVm1lc?=
 =?us-ascii?Q?5CA5WScXqHayqKNEO+U7LQxc0snDkz2l87QuIdnNOzAiZbnRdSXp7KiIjdpr?=
 =?us-ascii?Q?Lzd5bF/58Ao8ztp4sWgIRjyxUyWtDJmt1OLMwzC47SqsftYo17z7ULTEfN/A?=
 =?us-ascii?Q?d+NTXCCg7DSsY4KPZmisKs572y/NfvwB9+M3rIxjD4ah5sWXMwP51ZTdvMIE?=
 =?us-ascii?Q?fRVYX5M8XcMEqo3Hyr7KRSeaQAS6VtFxPvYlRWRwWQa5W9kroyqLZhU1jNyh?=
 =?us-ascii?Q?8g80RLgiicv/UxXGYyoXgkm63TyLQOlTRdiJWIyNir+SNhq+y00EWqjtqBqP?=
 =?us-ascii?Q?Ci44cQUbSmQd6FXwKpa8vMXqExPc2P4CYEfsZ3VYEtj+evRAyE/w2jZOajyT?=
 =?us-ascii?Q?T77X4uOMgipE+xHXBIa6yKf7NOQXgkd5YJ+UUe0eqlOPR9cf7yStVd9gCkf8?=
 =?us-ascii?Q?ZJ5o9nJnf813KHF7TLNyEKrfrTuHn40RTlrtbXZpsb0eZKXzUvJ5+a/+sdrT?=
 =?us-ascii?Q?2OpHumw5EdidHEVWKUmDZR4lWJoBJItb1vBbTwNR5+caAFksnSWlqR1Tg+JS?=
 =?us-ascii?Q?Kv6ERFeh7KljfLEy7bjeMRsSLNYlt13HV6R5cz1670te6nbY955RX7+/BQfq?=
 =?us-ascii?Q?WhyuEdjuE79IHbnNVz770FWTlCLbmIJzL+sFthkzoFBoXkuBPcnPRX+HUJJC?=
 =?us-ascii?Q?hw=3D=3D?=
X-OriginatorOrg: nxp.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 34e65408-0d1c-4269-5828-08db78ab3c40
X-MS-Exchange-CrossTenant-AuthSource: AM0PR04MB6452.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 29 Jun 2023 14:15:05.0249
 (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: V/GLNK4EjzQZB8TfWrcKxt2QeMZV1kWt2bF47FWwUoAWlUU7Q07nj77udtpS1j2d1m9NnBfM6KItccP+LKfipg==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: PAXPR04MB8406
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

The SJA1105 hardware tagging protocol is weird and will put DSA
information (source port, switch ID) in the MAC DA of the packets sent
to the CPU, and then send some additional (meta) packets which contain
the original bytes from the previous packet's MAC DA.

The tagging protocol driver contains logic to handle this, but the meta
frames are optional functionality, and there are configurations when
they aren't received (no PTP RX timestamping). Thus, the MAC DA from
packets sent to the stack is not correct in all cases.

Also, during testing it was found that the MAC DA patching procedure was
incorrect.

The investigation comes as a result of this discussion with Paolo:
https://lore.kernel.org/netdev/f494387c8d55d9b1d5a3e88beedeeb448f2e6cc3.camel@redhat.com/

Vladimir Oltean (2):
  net: dsa: tag_sja1105: fix MAC DA patching from meta frames
  net: dsa: sja1105: always enable the send_meta options

 drivers/net/dsa/sja1105/sja1105.h      |  2 +-
 drivers/net/dsa/sja1105/sja1105_main.c |  5 ++-
 drivers/net/dsa/sja1105/sja1105_ptp.c  | 48 +++----------------------
 include/linux/dsa/sja1105.h            |  4 ---
 net/dsa/tag_sja1105.c                  | 49 ++------------------------
 5 files changed, 9 insertions(+), 99 deletions(-)

-- 
2.34.1