From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 84634C4167B for ; Mon, 30 Oct 2023 12:42:35 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S233240AbjJ3Mmg (ORCPT ); Mon, 30 Oct 2023 08:42:36 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:51952 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S233276AbjJ3Mmf (ORCPT ); Mon, 30 Oct 2023 08:42:35 -0400 Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 8916AC9; Mon, 30 Oct 2023 05:42:33 -0700 (PDT) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 6DD8AC433C9; Mon, 30 Oct 2023 12:42:32 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=linuxfoundation.org; s=korg; t=1698669753; bh=iv2Zh6aHWtUQa1gLirjeb4yEXxfSeJpOFe0DZhCw6fo=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=st2vt2djSVdWHoDfTnUny++9RQr/FefLfjBP6n8Bzv7qXLZ9o238RhD+NsU7eNI82 srcbGBTLz9CbC3rybz0nhpmYhRPRWglS+GX3r0BNrWygp6uzJuMrH3VUkZfvFdCVTV E2U1G3DJnJpUeglFOAit5e67y4wtDQyifli9tIIg= Date: Mon, 30 Oct 2023 13:42:29 +0100 From: Greg KH To: Oliver Hartkopp Cc: stable@vger.kernel.org, sashal@kernel.org, linux-can@vger.kernel.org, lukas.magel@posteo.net, patches@lists.linux.dev, maxime.jayat@mobile-devices.fr, mkl@pengutronix.de, michal.sojka@cvut.cz Subject: Re: [PATCH] can: isotp: upgrade 5.15 LTS to latest 6.6 mainline code base Message-ID: <2023103038-spinning-uncooked-b608@gregkh> References: <20231030113110.3404-1-socketcan@hartkopp.net> <2023103048-riverside-giving-e44d@gregkh> <9bf2b7c9-fe80-4509-b023-c406f2fff994@hartkopp.net> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <9bf2b7c9-fe80-4509-b023-c406f2fff994@hartkopp.net> Precedence: bulk List-ID: X-Mailing-List: stable@vger.kernel.org On Mon, Oct 30, 2023 at 01:09:41PM +0100, Oliver Hartkopp wrote: > Hello Greg, > > On 30.10.23 12:38, Greg KH wrote: > > On Mon, Oct 30, 2023 at 12:31:10PM +0100, Oliver Hartkopp wrote: > > > The backport of commit 9c5df2f14ee3 ("can: isotp: isotp_ops: fix poll() to > > > not report false EPOLLOUT events") introduced a new regression where the > > > fix could potentially introduce new side effects. > > > > > > To reduce the risk of other unmet dependencies and missing fixes and checks > > > the latest mainline code base is ported back to the 5.15 LTS tree. > > > > > > To meet the former Linux 5.15 API these commits have been reverted: > > > f4b41f062c42 ("net: remove noblock parameter from skb_recv_datagram()") > > > 96a7457a14d9 ("can: skb: unify skb CAN frame identification helpers") > > > dc97391e6610 ("sock: Remove ->sendpage*() in favour of sendmsg(MSG_SPLICE_PAGES)") > > > 0145462fc802 ("can: isotp: isotp_recvmsg(): use sock_recv_cmsgs() to get SOCK_RXQ_OVFL infos") > > > > > > New features and communication stability measures: > > > 9f39d36530e5 ("can: isotp: add support for transmission without flow control") > > > 96d1c81e6a04 ("can: isotp: add module parameter for maximum pdu size") > > > 4b7fe92c0690 ("can: isotp: add local echo tx processing for consecutive frames") > > > 530e0d46c613 ("can: isotp: set default value for N_As to 50 micro seconds") > > > > Please send these as individual patches, reverts and then the new ones > > added, not as one huge commit that we can't review properly at all. > > That would be around 20 patches including fixes and fixes of fixes of fixes. > For that reason I simply copied the 6.6 code and made the code work in 5.x > by adapting it to the old 5.x kernel APIs. 20 patches is trivial for us to handle, please do it that way as it ensures we keep the proper history, AND we know what to backport when in the future. When you try to crunch patches together, or do non-upstream changes, 90%+ of the time they end up being wrong or impossible to maintain over time. > > But why just 5.15? What about 6.1.y and 6.5.y? > > I have posted 5.10 and 5.15 for now. > 6.1 would be only this patch > 96d1c81e6a04 ("can: isotp: add module parameter for maximum pdu size") > I can also provide. Why add new features to older kernels? That's not going to be ok, sorry. And why is can adding module parameters? This isn't the 1990's, we have proper ways of doing this correctly (hint, module parameters are not the way to do that...) thanks, greg k-h