* [RESEND] CXL_MEM_QUERY_COMMANDS returning commands not supported by the hardware.
@ 2022-10-10 12:59 Jonathan Cameron
2022-10-21 22:36 ` Dan Williams
0 siblings, 1 reply; 2+ messages in thread
From: Jonathan Cameron @ 2022-10-10 12:59 UTC (permalink / raw)
To: linux-cxl, Dan Williams, Vishal Verma, Ben Widawsky,
Alison Schofield
Resent because I can't type email addresses so this didn't got to the list..
Hi All,
Noticed, whilst implementing a switch CCI mailbox version of the same...
UAPI documentation include/uapi/linux/cxl_mem.h says:
struct cxl_mem_query_commands...
Allow userspace to query the available commands supported by both the
driver, and the hardware.
Currently it seem to report all commands without checking they are in the CEL.
Easy to tell given I just added a bunch of switch commands which are
apparently supported by the type 3 devices (which would be novel).
How to fix? Make the code do what the docs suggest, or change the
docs?
Or am I missing something?
Thanks,
Jonathan
^ permalink raw reply [flat|nested] 2+ messages in thread
* RE: [RESEND] CXL_MEM_QUERY_COMMANDS returning commands not supported by the hardware.
2022-10-10 12:59 [RESEND] CXL_MEM_QUERY_COMMANDS returning commands not supported by the hardware Jonathan Cameron
@ 2022-10-21 22:36 ` Dan Williams
0 siblings, 0 replies; 2+ messages in thread
From: Dan Williams @ 2022-10-21 22:36 UTC (permalink / raw)
To: Jonathan Cameron, linux-cxl, Dan Williams, Vishal Verma,
Ben Widawsky, Alison Schofield
Jonathan Cameron wrote:
>
> Resent because I can't type email addresses so this didn't got to the list..
>
> Hi All,
>
> Noticed, whilst implementing a switch CCI mailbox version of the same...
>
> UAPI documentation include/uapi/linux/cxl_mem.h says:
>
> struct cxl_mem_query_commands...
>
> Allow userspace to query the available commands supported by both the
> driver, and the hardware.
>
> Currently it seem to report all commands without checking they are in the CEL.
>
> Easy to tell given I just added a bunch of switch commands which are
> apparently supported by the type 3 devices (which would be novel).
>
> How to fix? Make the code do what the docs suggest, or change the
> docs?
>
> Or am I missing something?
No, you are not missing anything. This topic has come up before. Why
have a query if it does not reflect the state of the CEL. I would be in
favor of the enabled state in the CEL being communicated in the
CXL_MEM_QUERY_COMMANDS output payload.
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2022-10-21 22:36 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2022-10-10 12:59 [RESEND] CXL_MEM_QUERY_COMMANDS returning commands not supported by the hardware Jonathan Cameron
2022-10-21 22:36 ` Dan Williams
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox