Re: sysfs filenames with spaces
From: Pavel Machek
Date: Mon Oct 19 2020 - 14:27:26 EST
On Tue 2020-10-13 12:48:49, Joe Perches wrote:
> On Tue, 2020-10-13 at 19:17 +0200, Pavel Machek wrote:
> > On Mon 2020-10-05 19:41:15, Joe Perches wrote:
> > > This doesn't seem like a great idea to me.
> > >
> > > For my system I've got:
> > >
> > > /sys/devices/platform/Fixed MDIO bus.0/
> > > /sys/bus/platform/drivers/int3401 thermal/
> > > /sys/bus/platform/drivers/int3403 thermal/
> > > /sys/bus/platform/drivers/int3400 thermal/
> > > /sys/bus/mdio_bus/drivers/Generic PHY/
> > > /sys/bus/mdio_bus/drivers/Generic Clause 45 PHY/
> > > /sys/bus/pnp/drivers/i8042 aux/
> > > /sys/bus/pnp/drivers/i8042 kbd/
> > > /sys/bus/i2c/drivers/CHT Whiskey Cove PMIC/
> > >
> > > Could these filenames be avoided in the future or
> > > even renamed today?
> >
> > Does not look like great idea to me, either. Hmm. Is there filename
> > with "/" in it? :-)
> >
> > But I guess you'd need to cc relevant maintainers and that this is
> > going to be a bit of whack-a-mole.
>
> An option might be to convert any invalid filename
> via an alloc and substitution in sysfs_add_file
> and similar free in sysfs_remove_file.
>
> Emitting a logging message describing any new name
> would be useful too.
I believe that would be dangerous... renaming existing entries and
risking duplicities.
But ... some kind of warning when such entry is created might be good,
we'd then fix them manually.
Best regards,
Pavel
--
http://www.livejournal.com/~pavelmachek
Attachment:
signature.asc
Description: PGP signature