[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20230519163758.000044dc@Huawei.com>
Date: Fri, 19 May 2023 16:37:58 +0100
From: Jonathan Cameron <Jonathan.Cameron@...wei.com>
To: Ira Weiny <ira.weiny@...el.com>
CC: Davidlohr Bueso <dave@...olabs.net>, LiuLele <liu.lele@...com>,
<dave.jiang@...el.com>, <alison.schofield@...el.com>,
<bhelgaas@...gle.com>, <bwidawsk@...nel.org>,
<dan.j.williams@...el.com>, <helgaas@...nel.org>,
<linux-acpi@...r.kernel.org>, <linux-cxl@...r.kernel.org>,
<linux-kernel@...r.kernel.org>, <linux-pci@...r.kernel.org>,
<rostedt@...dmis.org>, <vishal.l.verma@...el.com>
Subject: Re: CXL memory device not created correctly
On Fri, 19 May 2023 08:20:44 -0700
Ira Weiny <ira.weiny@...el.com> wrote:
> Davidlohr Bueso wrote:
> > On Fri, 19 May 2023, LiuLele wrote:
> >
> > >In my testing CXL device /sys/bus/cxl/devices/mem0 not created, and the get error messages :
> > >
> > >```
> > >cxl_pci 0000:0d:00.0: Failed to get interrupt for event Info log
> > >```
> > >
> > >My test environment is a qemu CXL emulator with qemu v8.0.0, Linux kernel v6.3.0.
> > >While with kernel 5.9.13, /sys/bus/cxl/devices/mem0 can be created.
> >
> > Yes, this can be annoying and would argue the probe should not error out.
>
> I had to double check. Events are mandatory on devices. On checking
> again interrupt support is mandatory as well. So that is why I errored
> out here. With real HW this should not be an issue.
>
> > Regardless, the actual qemu support is in Jonathan's tree:
> >
> > https://gitlab.com/jic23/qemu/-/commit/a04e6476df363d1f6bc160577b30dda6564d3f67
>
> That is the commit you need but it is probably best to use one of
> Jonathans 'official' branches. Looks like he just pushed a new one today.
>
> https://gitlab.com/jic23/qemu/-/tree/cxl-2023-05-19
Leave that one for now. It was to get the CI tests to run. I need to tidy up
a bit and will announce when I have a clean one...
>
> I've not run that one yet. So if you have issues try his previous one it
> is working well for me.
>
> https://gitlab.com/jic23/qemu/-/tree/cxl-2023-04-19
That one should be good to go still I think
Jonathan
>
> Ira
>
> >
> > Thanks,
> > Davidlohr
>
>
Powered by blists - more mailing lists