Re: [RFC PATCH 0/3] cxl: Fix background operation handling

[Dave Jiang <dave.jiang@intel.com>]

On 9/22/23 06:05, Jeongtae Park wrote:
> Hello,
> 
> We are currently trying to add background commands like
> fw-related and sanitize operation. However, as we modified 
> the code, we found that there was a lot of duplication of
> code, so we would like to modify the existing background 
> command handling a little bit. We want to make the most of
> the existing background command handling so that we can
> easily add other background commands in the future.
> 
> The first patch modifies the existing sanitize operation
> handling to allow other background commands to be used 
> with it, and the second patch modifies sanitize to work
> with the modified code. Finally, the third patch adds a
> sysfs file related to the background operation. 
> This sysfs entry is select/poll capable from userspace 
> to notify upon completion of a background operation.
> 
> Thanks,
> /jtpark

Hi Jeongtae,
For the next version of the series you send out, can you please make sure the series is threaded? Currently all your series are coming as individual mails. Thanks!


> 
> Jeongtae Park (3):
>   cxl: Modify background cmd handling for the others
>   cxl/mbox: Update sanitize opcode support
>   cxl/memdev: Add background command sysfs file
> 
>  Documentation/ABI/testing/sysfs-bus-cxl | 10 ++++
>  drivers/cxl/core/mbox.c                 |  5 +-
>  drivers/cxl/core/memdev.c               | 71 +++++++++++++++++++++++--
>  drivers/cxl/cxlmem.h                    | 33 ++++++++++--
>  drivers/cxl/pci.c                       | 65 +++++++++++-----------
>  5 files changed, 141 insertions(+), 43 deletions(-)
> 
> 
> base-commit: fe77cc2e5a6a7c85f5c6ef8a39d7694ffc7f41c9