Re: [PATCH] sparc64: Expose mdesc to sysfs

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



From: Eric Saint Etienne <eric.saint.etienne@xxxxxxxxxx>
Date: Sat, 16 Sep 2017 05:36:03 +0100

> To get a snapshot as complete as possible of what the system is
> composed of,
> this commit adds the ability to browse machine description from sysfs.
> 
> This commit eases porting of user-space utilities that exist on other
> architectures
> like lshw for instance.
> 
> Whenever the HV changes machine description content, sysfs will be
> autopmatically
> re-populated. This is done almost atomically by swapping the old mdesc
> sysfs folder
> with the new one. While the tree under the new folder is being
> created, reading from
> /sys/firmware/mdesc will return EAGAIN. That way userspace programs
> will not see
> stale content in /sys/firmware/mdesc
> 
> Signed-off-by: Eric Saint Etienne <eric.saint.etienne@xxxxxxxxxx>
> Reviewed-by: Dave Aldridge <david.j.aldridge@xxxxxxxxxx>

Please explain why /dev/mdesc could not be extended to do what you
need, such as sending an event to userspace.

There is absolutely no reason to present a formatted version of
mdesc under sysfs.

That is entirely a userspace problem, where the userspace component
grabs the mdesc via /dev/mdesc and does whatever it wants to do
with that image.

Your patch was also whitespace damaged by your email client.

Thank you.
--
To unsubscribe from this list: send the line "unsubscribe sparclinux" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Kernel Development]     [DCCP]     [Linux ARM Development]     [Linux]     [Photo]     [Yosemite Help]     [Linux ARM Kernel]     [Linux SCSI]     [Linux x86_64]     [Linux Hams]

  Powered by Linux