Skip to content

Commit b46c5fa

Browse files
stellarhopperdjbw
authored andcommitted
tools/testing/cxl: Fix command effects for inject/clear poison
The CXL spec (3.0, section 8.2.9.8.4) Lists Inject Poison and Clear Poison as having the effects of "Immediate Data Change". Fix this in the mock driver so that the command effect log is populated correctly. Fixes: 371c161 ("tools/testing/cxl: Mock the Inject Poison mailbox command") Cc: Alison Schofield <alison.schofield@intel.com> Cc: Dan Williams <dan.j.williams@intel.com> Reviewed-by: Alison Schofield <alison.schofield@intel.com> Reviewed-by: Jonathan Cameron <Jonathan.Cameron@huawei.com> Reviewed-by: Dave Jiang <dave.jiang@intel.com> Signed-off-by: Vishal Verma <vishal.l.verma@intel.com> Link: https://lore.kernel.org/r/20230602-vv-fw_update-v4-2-c6265bd7343b@intel.com Signed-off-by: Dan Williams <dan.j.williams@intel.com>
1 parent 9521875 commit b46c5fa

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

  • tools/testing/cxl/test

tools/testing/cxl/test/mem.c

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -52,11 +52,11 @@ static struct cxl_cel_entry mock_cel[] = {
5252
},
5353
{
5454
.opcode = cpu_to_le16(CXL_MBOX_OP_INJECT_POISON),
55-
.effect = cpu_to_le16(0),
55+
.effect = cpu_to_le16(EFFECT(2)),
5656
},
5757
{
5858
.opcode = cpu_to_le16(CXL_MBOX_OP_CLEAR_POISON),
59-
.effect = cpu_to_le16(0),
59+
.effect = cpu_to_le16(EFFECT(2)),
6060
},
6161
};
6262

0 commit comments

Comments
 (0)