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:     Colin Foster <colin.foster@in-advantage.com>
CC:     Christian Marangi <ansuelsmth@gmail.com>,
        "netdev@vger.kernel.org" <netdev@vger.kernel.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        "David S. Miller" <davem@davemloft.net>,
        Eric Dumazet <edumazet@google.com>,
        Jakub Kicinski <kuba@kernel.org>,
        Paolo Abeni <pabeni@redhat.com>,
        Florian Fainelli <f.fainelli@gmail.com>,
        Vivien Didelot <vivien.didelot@gmail.com>,
        Andrew Lunn <andrew@lunn.ch>,
        Vladimir Oltean <olteanv@gmail.com>,
        Claudiu Manoil <claudiu.manoil@nxp.com>,
        Alexandre Belloni <alexandre.belloni@bootlin.com>,
        "UNGLinuxDriver@microchip.com" <UNGLinuxDriver@microchip.com>,
        Roopa Prabhu <roopa@nvidia.com>,
        Nikolay Aleksandrov <razor@blackwall.org>,
        Tobias Waldekranz <tobias@waldekranz.com>,
        =?Windows-1252?Q?Marek_Beh=FAn?= <kabel@kernel.org>,
        DENG Qingfang <dqfext@gmail.com>,
        =?Windows-1252?Q?Alvin_=8Aipraga?= <alsi@bang-olufsen.dk>,
        Linus Walleij <linus.walleij@linaro.org>,
        Luiz Angelo Daros de Luca <luizluca@gmail.com>,
        Felix Fietkau <nbd@nbd.name>, John Crispin <john@phrozen.org>,
        Sean Wang <sean.wang@mediatek.com>
Subject: Re: [PATCH net-next 0/9] DSA changes for multiple CPU ports (part 4)
Thread-Topic: [PATCH net-next 0/9] DSA changes for multiple CPU ports (part 4)
Thread-Index: AQHYvKsLmg9+Fvc+3E2TksPGMZq+1a3MfsSAgAMyhYCAAuvOgIAAAkMA
Date:   Tue, 6 Sep 2022 16:18:49 +0000
Message-ID: <20220906161848.sbmz6x7xsfdgdx3g@skbuf>
References: <20220830195932.683432-1-vladimir.oltean@nxp.com>
 <63124f17.170a0220.80d35.2d31@mx.google.com>
 <20220904193413.zmjaognji4s4gedt@skbuf> <YxdxA1yrWOTwAZhZ@colin-ia-desktop>
In-Reply-To: <YxdxA1yrWOTwAZhZ@colin-ia-desktop>
Accept-Language: en-US
Content-Language: en-US
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?Windows-1252?Q?tf81wxzDLILkriutC2/DgqOgZIIWyIGohQCE209N98y6RCAHHuL/XlvT?=
 =?Windows-1252?Q?h0qorVUiH9hjl6ZZf4KaEZYa2+vAVULsFMFWqclMYADUv3VqPNx0Lkqs?=
 =?Windows-1252?Q?shmdAXZccOSfTjfdzr8dmbYwQnUTwqj/Hc6wdoNplauEe37eGYOXZb+3?=
 =?Windows-1252?Q?k8/vLOhZj7TuufX9hyCwkt0S6dtoxENaRKmGTTlC5+hBeQpfLDwEHskr?=
 =?Windows-1252?Q?GIXL1/b/KGJBvcmNcfaZjqfko2mDY3R6rz8kcma44MuVu3Uv5fndHX+x?=
 =?Windows-1252?Q?Q7dJdp743MrJLagYB2EO1uJ5P4IgUyr5jV+JNK+RNszCvHwdLNG9CuVW?=
 =?Windows-1252?Q?xNy4pwYaMOOsw8z6pgS4vEMynO2jfCRwRufxrfJRC8W1LDSXqO34zT+O?=
 =?Windows-1252?Q?p793FfXwlU3LFG5XZqQF8e9c0BZwescHKBWix3oH7reuxVn4/59AHiog?=
 =?Windows-1252?Q?K6+0NB6M8guQKPtNkWLFZaZNc5x/bh/1SPl2CdSHY30clqfz+2nDFtzM?=
 =?Windows-1252?Q?1Zo8pNJtCh3nhMMOzNeA6TvJehMRBFO5uWLYlhyhEFZNlXgH1X8V7XT0?=
 =?Windows-1252?Q?/ZElYbny5WdXqRNAouCJM2XNjvUMLJKzuRJs2QqbxlXw00h92Kbj1SPG?=
 =?Windows-1252?Q?LgREUEra5zXafjvnIomaOFuusO6ak2y8OYD3DbKC5L/ZkJ8+EJ4HyBak?=
 =?Windows-1252?Q?S2ljA2oziBgimgI80ZtaJCO359wT7kr3xJ2QmuzQVMj1aGnPknL1fxOu?=
 =?Windows-1252?Q?yMZ65LYdOoRKLGKYmzkPxVszQjBOpBUTWyA0/M2fierBUpxHcTqrIlwB?=
 =?Windows-1252?Q?36Wpmex3JyhPrUyIO9QYq26bp46OzOoSit8WBmFmdUKbKIGT0wFjzAvM?=
 =?Windows-1252?Q?lMnXkbjDjai+7usUyLP/Cnby0XmROlT72Skb4h4Xo3FdZWKR0YJn3L3r?=
 =?Windows-1252?Q?TPRoZQlQOd60Z3PghQcCtaPEk9XsfsAWQkX9rQf7UiXe6KKj5SoN5ZpN?=
 =?Windows-1252?Q?1w67bzrQ/2Tp/DHF61AO9MfRh98tZ8kjH41PtZCwCn0MrglF7UHk9ujP?=
 =?Windows-1252?Q?XjtHJ8oInMFRn4fKC/9tRX6KkXqGYcwBe2VimQv97YCq0XDB8Q8oOpjH?=
 =?Windows-1252?Q?fs/qMFqkPiaFKLS7pKAU+cy6p66ElhBWw8kiL3iBG4TrFNsbkulD5RCw?=
 =?Windows-1252?Q?jLXSjZ5pBPKKrv6WZ2Bl+rvDTF4ze03vUIAVRFkczdrBmSU872hDajJl?=
 =?Windows-1252?Q?PlkCDCRO0VGbUaNtSXv+P8tDMXDqHqmAHx4sqM7f8wBv1GCigEBEnOSl?=
 =?Windows-1252?Q?wqufV8+SaRTRHwKY8S8iNE+8lLZLd37s+gBT+4S5rWEmRdiP5XS241NB?=
 =?Windows-1252?Q?w9AQ6sRxF8D492mebXpLixbpBBeX/+gtyWCWH+X2yZQtTMTuqispB/ak?=
 =?Windows-1252?Q?peyL6sBDbY7+var1F0wLEkyw1/gljsL4L2BDVBUEXtcr9jZh6mHypLyf?=
 =?Windows-1252?Q?Q4eCvRYVCgsr1ItYmqvEbP7WRmapAr4npECwN8doYqJkRfoTBXmQ/Tln?=
 =?Windows-1252?Q?8JBEqOPjOl+O+xOFdNePO0vu4DqWM80u06q5EDYaUQavfl/UhPpAhAXM?=
 =?Windows-1252?Q?/4FQ35HZ4jMaiYO6jdiRhZBRHFyHH63e/2GwBAc8Ka/NFBjO3+c59Qnz?=
 =?Windows-1252?Q?AQ0Gw1fLhAuBIjbicp1Ohv+uI+c0On/gUFiSM31EC15eYu9vwpCSbg?=
 =?Windows-1252?Q?=3D=3D?=
Content-Type: text/plain; charset="Windows-1252"
Content-ID: <621A42126499004B97604A6A6DB03665@eurprd04.prod.outlook.com>
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-OriginatorOrg: nxp.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: VI1PR04MB5136.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 744ca57c-39a9-4af2-a25b-08da90237b41
X-MS-Exchange-CrossTenant-originalarrivaltime: 06 Sep 2022 16:18:49.2188
 (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: eFXbhZYMeEw7NoHtav6lleN21YUv0IaPU8jsBXYzHzSSacj8dg2x81pJDv01ho6+CXmWDNiK1yxeJ0CEpnRNtQ==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: HE1PR0402MB3418
X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIM_SIGNED,
        DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_NONE,
        RCVD_IN_MSPIKE_H2,SPF_HELO_PASS,SPF_PASS,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 Tue, Sep 06, 2022 at 09:10:43AM -0700, Colin Foster wrote:
> On Sun, Sep 04, 2022 at 07:34:14PM +0000, Vladimir Oltean wrote:
> > On Fri, Sep 02, 2022 at 08:44:37PM +0200, Christian Marangi wrote:
> > For transparency, here is the commit list I used to produce the backpor=
t (top-most is most recent):
>=20
> Tangentially related: how did you come up with this list?
>=20
> I can only assume this is a manual process based on intricate knowledge
> of net / DSA as a whole. I just want to make sure there isn't a "git
> backport net/dsa origin/master v5.10" sort of thing ;-)

Years of practice ;)

Although I found a tool called git-deps and I ran it in a few places,
I found it to be fundamentally lacking for correct kernel backporting
work. The way that tool works is it finds patches related to the current
one based on what other lines from this patch's context they touch.
This obviously isn't going to work too well. There may very well be no
connection obvious to this tool between, say, a function prototype added
to include/net/dsa.h and its first use in drivers/net/dsa/mv88e6xxx/chip.c.
So it may not know it has to backport the include/net/dsa.h change.

I suspect one way in which git-deps may be modified in order to be much
more useful for kernel work is for it to backport the entire patch
series from which the dependency commit came. But (1) I don't know
python even enough to be able to manage a pip virtual environment, let
alone code up such a change, and (2) I'm not sure, from a kernel devel
process perspective, that we offer enough machine-parsable hints such
that a python program can just figure out "yes, these are the patches
that all belong to the same patch set as commit X".=