Re: [PATCH v12 7/7] nvmem: core: Expose cells through sysfs

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

 



Hi Srinivas,

srinivas.kandagatla@xxxxxxxxxx wrote on Wed, 11 Oct 2023 10:26:43 +0100:

> On 11/10/2023 09:58, Miquel Raynal wrote:
> > Hi Srinivas,
> > 
> > srinivas.kandagatla@xxxxxxxxxx wrote on Wed, 11 Oct 2023 09:45:11 +0100:
> >   
> >> On 11/10/2023 09:33, Miquel Raynal wrote:  
> >>> Hi Srinivas,
> >>>
> >>> srinivas.kandagatla@xxxxxxxxxx wrote on Wed, 11 Oct 2023 09:27:20 +0100:  
> >>>    >>>> On 11/10/2023 08:15, Miquel Raynal wrote:
> >>>>>>> +
> >>>>>>> +	nvmem_cells_group.bin_attrs = cells_attrs;
> >>>>>>> +
> >>>>>>> +	ret = devm_device_add_groups(&nvmem->dev, nvmem_cells_groups);
> >>>>>>> +	if (ret)
> >>>>>>> +		goto unlock_mutex;  
> >>>>>> This is going to create groups after the nvmem device is added, isn't this going to be problem with user space notifications?  
> >>>>> Greg said it was not. I hope I understood correctly 😄
> >>>>>
> >>>>> And anyway, cells have never been available to userspace, so there is
> >>>>> nothing userspace might expect yet?  
> >>>> I agree, but once we add sysfs uapi then this is going to change.  
> >>>
> >>> Can you elaborate? I'm not sure I follow you here. Is there still a
> >>> problem you fear or you think it's okay?  
> >>>    >> Now that we add cells to sysfs.  
> >> AFAIU, By the time the userspace sees the udev event from this device we might not have cells populated.  
> > 
> > Yes, but why would this be a problem?
> >   
> It will be problem if the userspace is using things like libudev to act on these events. There seems to be some caching of attributes in udev during event more info http://www.kroah.com/log/blog/2013/06/26/how-to-create-a-sysfs-file-correctly/

I am already using these attributes, right? The problem here is that we
always attach cells sysfs attributes to the nvmem device, but in some
cases (when using layout devices/drivers) the probe of these devices
will happen after the main nvmem device has been announced to userspace
and thus these attributes might not be populated yet (but Greg said it
was "supported" and I assumed it was fine).

So what is your idea here to overcome this?

Thanks,
Miquèl





[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]


  Powered by Linux