[ndctl,V4,0/2] add support for CCI Get Alert Configuration

Message

Jonathan Zhang Nov. 7, 2022, 11:38 p.m. UTC

CXL spec 3.0 section 8.2.9.8.3.2 defines CCI command
Get Alert Configuration.

This patchset adds support for it.
  
v4:
- Fix the typo of LIEF instead of LIFE (Alison).

v3:
- Update man page and JSON output to use "alert-config" (Alison).

v2:
- Change the command parameter from "alert" to "alert-config" (Vishal).
- Update the format based on clang-formt (Vishal).
- Updated the names to be consistent with the existing code (Vishal). Such as:
-- pers_mem --> pmem
-- temp --> temperature
-- drop "get_" from some function names.

Jonathan Zhang (2):
  libcxl: add accessors for Get Alert Configuration CCI output
  cxl: display alert configuration fields in list command

 Documentation/cxl/cxl-list.txt   |  33 ++++++
 Documentation/cxl/lib/libcxl.txt |   1 +
 cxl/filter.c                     |   2 +
 cxl/filter.h                     |   1 +
 cxl/json.c                       | 185 +++++++++++++++++++++++++++++++
 cxl/lib/libcxl.c                 | 163 +++++++++++++++++++++++++++
 cxl/lib/libcxl.sym               |  23 ++++
 cxl/lib/private.h                |  38 +++++++
 cxl/libcxl.h                     |  35 ++++++
 cxl/list.c                       |   6 +-
 util/json.h                      |   1 +
 11 files changed, 486 insertions(+), 2 deletions(-)

Comments

Verma, Vishal L Nov. 10, 2022, 8 p.m. UTC | #1

On Mon, 2022-11-07 at 15:38 -0800, Jonathan Zhang wrote:
> CXL spec 3.0 section 8.2.9.8.3.2 defines CCI command
> Get Alert Configuration.
> 
> This patchset adds support for it.
>   
> v4:
> - Fix the typo of LIEF instead of LIFE (Alison).
> 
> v3:
> - Update man page and JSON output to use "alert-config" (Alison).
> 
> v2:
> - Change the command parameter from "alert" to "alert-config" (Vishal).
> - Update the format based on clang-formt (Vishal).
> - Updated the names to be consistent with the existing code (Vishal). Such as:
> -- pers_mem --> pmem
> -- temp --> temperature
> -- drop "get_" from some function names.
> 
> Jonathan Zhang (2):
>   libcxl: add accessors for Get Alert Configuration CCI output
>   cxl: display alert configuration fields in list command

Thanks for the updates Jonathan, I've queued these up for v75 with a
couple of really minor fixups.
I've pushed a branch out: for-75/jz/alert-config

> 
>  Documentation/cxl/cxl-list.txt   |  33 ++++++
>  Documentation/cxl/lib/libcxl.txt |   1 +
>  cxl/filter.c                     |   2 +
>  cxl/filter.h                     |   1 +
>  cxl/json.c                       | 185 +++++++++++++++++++++++++++++++
>  cxl/lib/libcxl.c                 | 163 +++++++++++++++++++++++++++
>  cxl/lib/libcxl.sym               |  23 ++++
>  cxl/lib/private.h                |  38 +++++++
>  cxl/libcxl.h                     |  35 ++++++
>  cxl/list.c                       |   6 +-
>  util/json.h                      |   1 +
>  11 files changed, 486 insertions(+), 2 deletions(-)
>