Re: [PATCH] [PATCH] Firmware security information in SYSFS
From: Greg Kroah-Hartman
Date: Fri Jul 17 2020 - 02:28:58 EST
On Thu, Jul 16, 2020 at 07:36:27PM -0300, Daniel Gutson wrote:
> +What: /sys/kernel/firmware-security/bioswe
Ick, I stopped reading right here.
No, this is not where this belongs.
We already have /sys/firmware/, right? And firmware-specific
subdirectories below that.
We also have /sys/devices/system/ and I think that would be a much
better place for this, as it is easier to work with a real 'struct
device' than a "raw" kobject any day. Bonus is you get full support of
userspace libraries when you do that, unlike when dealing with kobjects.
Also, this really is a _SPECIFIC_ type of firmware that supports these
features, right? Why not call that out too? This is not generic by any
means.
thanks,
greg k-h