On Tue, Aug 26, 2025 at 4:03 PM Jason Gunthorpe <jgg@xxxxxxxxxx> wrote: > > On Thu, Aug 07, 2025 at 01:44:25AM +0000, Pasha Tatashin wrote: > > Introduce a sysfs interface for the Live Update Orchestrator > > under /sys/kernel/liveupdate/. This interface provides a way for > > userspace tools and scripts to monitor the current state of the LUO > > state machine. > > Now that you have a cdev these files may be more logically placed > under the cdev's sysfs and not under kernel? This can be done easially > using the attribute mechanisms in the struct device. > > Again sort of back to my earlier point that everything should be > logically linked to the cdev as though there could be many cdevs, even > though there are not. It just keeps the code design more properly > layered and understanble rather than doing something unique.. I am going to drop this patch entirely, and only rely on "luoctl state" (see https://tinyurl.com/luoddesign) to query the state from "/dev/liveupdate" Pasha