Re: [PATCH v9 0/7] QEMU CXL Provide mock CXL events and irq support

From: Yuquan Wang <wangyuquan1236@phytium.com.cn>
To: jonathan.cameron@huawei.com
Cc: dave.jiang@intel.com, fan.ni@samsung.com, ira.weiny@intel.com,
	linux-cxl@vger.kernel.org, linuxarm@huawei.com,
	qemu-devel@nongnu.org
Subject: Re: [PATCH v9 0/7]  QEMU CXL Provide mock CXL events and irq support
Date: Fri, 15 Mar 2024 09:52:28 +0800	[thread overview]
Message-ID: <20240315015228.307512-1-wangyuquan1236@phytium.com.cn> (raw)
In-Reply-To: <20230530133603.16934-1-Jonathan.Cameron@huawei.com>

Hello, Jonathan

When during the test of qmps of CXL events like "cxl-inject-general-media-event", 
I am confuesd about the argument "flags". According to "qapi/cxl.json" in qemu, 
this argument represents "Event Record Flags" in Common Event Record Format.
However, it seems like the specific 'Event Record Severity' in this field can be
different from the value of 'Event Status' in "Event Status Register". 

For instance (take an injection example in the coverlatter):

{ "execute": "cxl-inject-general-media-event",
    "arguments": {
        "path": "/machine/peripheral/cxl-mem0",
        "log": "informational",
        "flags": 1,
        "dpa": 1000,
        "descriptor": 3,
        "type": 3,
        "transaction-type": 192,
        "channel": 3,
        "device": 5,
        "component-id": "iras mem"
    }}

In my understanding, the 'Event Status' is informational and the 
'Event Record Severity' is Warning event, which means these two arguments are
independent of each other. Is my understanding correct?

Many thanks
Yuquan