Re: [PATCH 2/2] hw/cxl: Allow tracing component I/O accesses

["Philippe Mathieu-Daudé" <philmd@linaro.org>]

On 24/1/25 17:20, Jonathan Cameron wrote:
> On Thu, 23 Jan 2025 09:51:51 +0000
> Jonathan Cameron via <qemu-devel@nongnu.org> wrote:
> 
>> On Wed, 22 Jan 2025 07:56:24 +0100
>> Philippe Mathieu-Daudé <philmd@linaro.org> wrote:
>>
>>> Map the component I/O region as UnimplementedDevice
>>> to be able to trace guest I/O accesses with '-d unimp'.
>>>
>>> Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
>> I'm not that familiar with this infrastructure but seems
>> fine to me.
>>
>> I'd definitely be curious if anything is touching this space so
>> tracing may be helpful for that!
> Hi Philippe
> 
>>
>> Reviewed-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
> Retract that.
> 
> Can't instantiate a CXL device with this patch because:
> 
> Device unimplemented-device can not be dynamically instantiated.

Hmm the qtests using CXL devices pass, how do you trigger that?

> 
> Reverting this patch on my tree fixes that.
> 
>>> ---
>>>   include/hw/cxl/cxl_component.h |  3 ++-
>>>   hw/cxl/cxl-component-utils.c   | 14 +++++++++++---
>>>   hw/cxl/Kconfig                 |  1 +
>>>   3 files changed, 14 insertions(+), 4 deletions(-)