Re: [PATCH 35/46] cxl/region: Add a 'uuid' attribute

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

 



Jonathan Cameron wrote:
> On Thu, 23 Jun 2022 21:19:39 -0700
> Dan Williams <dan.j.williams@xxxxxxxxx> wrote:
> 
> > From: Ben Widawsky <bwidawsk@xxxxxxxxxx>
> > 
> > The process of provisioning a region involves triggering the creation of
> > a new region object, pouring in the configuration, and then binding that
> > configured object to the region driver to start is operation. For
> > persistent memory regions the CXL specification mandates that it
> > identified by a uuid. Add an ABI for userspace to specify a region's
> > uuid.
> > 
> > Signed-off-by: Ben Widawsky <bwidawsk@xxxxxxxxxx>
> > [djbw: simplify locking]
> > Signed-off-by: Dan Williams <dan.j.williams@xxxxxxxxx>
> 
> I think this needs to be a little less restrictive as it currently errors
> out on trying to write the same UUID to the same region twice.
> 
> Short cut that case and just return 0 if the UUID is same as already set.

Sure, fixed that up locally.



[Index of Archives]     [DMA Engine]     [Linux Coverity]     [Linux USB]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Greybus]

  Powered by Linux