Received-SPF: pass (google.com: domain of linux-kernel+bounces-71937-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:4601:e00::3 as permitted sender) client-ip=2604:1380:4601:e00::3;
Date: Mon, 19 Feb 2024 21:02:54 +0100
From: Michael Grzeschik <mgr@pengutronix.de>
To: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Cc: Eric Van Hensbergen <ericvh@kernel.org>,
	Latchesar Ionkov <lucho@ionkov.net>,
	Dominique Martinet <asmadeus@codewreck.org>,
	Christian Schoenebeck <linux_oss@crudebyte.com>,
	Jonathan Corbet <corbet@lwn.net>, v9fs@lists.linux.dev,
	linux-doc@vger.kernel.org, linux-kernel@vger.kernel.org,
	linux-usb@vger.kernel.org, kernel@pengutronix.de
Subject: Re: [PATCH v2 3/4] usb: gadget: legacy: add 9pfs multi gadget
Message-ID: <ZdOz7mc-NbiEe2Ei@pengutronix.de>
References: <20240116-ml-topic-u9p-v2-0-b46cbf592962@pengutronix.de>
 <20240116-ml-topic-u9p-v2-3-b46cbf592962@pengutronix.de>
 <2024021757-geography-hacksaw-3022@gregkh>
 <ZdKze80oFj0PRkkZ@pengutronix.de>
 <2024021911-facelift-graveyard-0760@gregkh>
Precedence: bulk
MIME-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha512;
	protocol="application/pgp-signature"; boundary="+zGieLIMQqtXbity"
Content-Disposition: inline
In-Reply-To: <2024021911-facelift-graveyard-0760@gregkh>


--+zGieLIMQqtXbity
Content-Type: text/plain; charset=us-ascii; format=flowed
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

On Mon, Feb 19, 2024 at 09:13:03AM +0100, Greg Kroah-Hartman wrote:
>On Mon, Feb 19, 2024 at 02:48:43AM +0100, Michael Grzeschik wrote:
>> On Sat, Feb 17, 2024 at 04:59:28PM +0100, Greg Kroah-Hartman wrote:
>> > On Fri, Feb 02, 2024 at 01:05:12AM +0100, Michael Grzeschik wrote:
>> > > Add the newly introduced 9pfs transport gadget interface with an new
>> > > multi composed gadget together with acm and eem.
>> > >
>> > > When using this legacy module, it is also possible to
>> > > mount the 9PFS usb dir as root filesystem. Just follow the
>> > > instrucitons from Documentation/filesystems/9p.rst
>> >
>> > Why are we adding new "legacy" gadgets?  What's wrong with the "correc=
t"
>> > api instead?  You need a lot of justification here to add something to
>> > an api we want to one day just delete.
>>
>> Without the legacy gadget there is no real solution to mount
>> the 9pfs via the gadget as rootfs. The "correct" api is configfs
>> which will need the user to have some filesystem to mount it to.
>
>That's what your initramfs is for.  Why can't you just use that?

Because out of convenience and to make it just work as simple as
nfsrootfs. With this we now need to maintain an full initramfs,
that has to come from somewhere.

>> There is the relatively new concept of bootconfig which sounds
>> promising to describe an complete configfs tree from system boot.
>
>Great, but until that happens, again, just use initramfs.

Hmm, okay.

>> However this is some future talk for now, so we would like to
>> stick with the legacy setup to be able to mount the 9pfs rootfs.
>
>I'd prefer to NOT add new legacy gadget drivers, and do everything
>possible to delete them all from the tree "soon".

When you mean this "soon" is in the near future, you are probably right
then.

>> > > +/*
>> > > + * Gadget usb9pfs only needs two bulk endpoints, and will use the u=
sb9pfs usb
>> > > + * transport to mount host filesystem via usb gadget. This driver w=
ill
>> > > + * also add one ACM and NCM interface.
>> >
>> > Why "also"?  What are those interfaces going to be used for and what do
>> > they have to do with 9pfs?
>>
>> They are not necessary to be used with 9pfs. But since we introduce an
>> new legacy module which is fully claiming the UDC, it would make sense
>> to leave the other endpoints unavailable but instead add some common
>> interfaces like ecm and acm.
>
>But if no one needs/wants them, why make this complex?  Again, configfs
>can handle the composition of this if you need it, which is why that
>"new" interface was created.


Okay, What about the rest of the series? Can you just skip this patch
then for? Or do you want me to send the series again without this
legacy driver. There are no dependencies to this in that series.

Michael

--=20
Pengutronix e.K.                           |                             |
Steuerwalder Str. 21                       | http://www.pengutronix.de/  |
31137 Hildesheim, Germany                  | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

--+zGieLIMQqtXbity
Content-Type: application/pgp-signature; name="signature.asc"

-----BEGIN PGP SIGNATURE-----

iQIzBAABCgAdFiEElXvEUs6VPX6mDPT8C+njFXoeLGQFAmXTs+oACgkQC+njFXoe
LGRwLA/+LrNLPbGGYFlAzMGpno4wxMgOLzvDHPdnuBGsFQ0aAzm7SWtjv92xdbJs
iv+j4zQNjpshdqmWXLtwGO6thupVI5zT7A1Upqt3K5SVe0LGnwlfeqYWqa9aHIwS
uwmiBXMR6DtPutwduaFHCJeEMC9j1+MtSe2M6UxMDzQaXujOtQ4K9yHU6/XOyn/H
1L+7IHrW2z40WvbtfHRxjAiySUaoIkHyn1x0q7ui4QD/UTssbrg+2smB0uMtHxZX
bWNyrFbh0iTEeCXPVfd5LcVUSPCFcqtqag8dOwGPqevmCTkOSFiL4RhS4pZKrCrh
kFOdfPW5tfh+dLvfIV5RkMqzKKvlg1wh2mzqh5gyGiJsQsipurTAvo5cnJFujbRc
Gy5wztK56jnnXehPyVDGYmxt4ORJf51PHBi5+adp975RRP2yPq82nSDMAoEN6hkh
IcS9po3h5N59UuA38WcPO1qCBg00QYnJJto0S9nyYicrVwzZnvz6/HScdfVKv+U1
YQ0ChpBj5ijfelb3HHb5jchLvQ449C+/7iZijn5m4g7zE7Jz5VyYiXmLrGOETisQ
4iW6d156+OH18SaWODWRQQZnioVdEmPSv84SMSxgLawqKfLlkziCi7ozqXZE79pO
ZBvK+LDllbO8mZfAOsQSmLy/LXTrjZNaz1Ew3nW4Kd3bd/Ut1pE=
=n31+
-----END PGP SIGNATURE-----

--+zGieLIMQqtXbity--