* [PATCH v2 1/2] hw/cxl: Pass CXLComponentState to cache_mem_ops
@ 2023-10-18 8:24 Li Zhijian
2023-10-18 8:24 ` [PATCH v2 2/2] hw/cxl: Pass NULL for a NULL MemoryRegionOps Li Zhijian
2023-10-19 10:50 ` Jonathan Cameron
0 siblings, 2 replies; 5+ messages in thread
From: Li Zhijian @ 2023-10-18 8:24 UTC (permalink / raw)
To: jonathan.cameron, fan.ni; +Cc: qemu-devel, philmd, Li Zhijian
cache_mem_ops.{read,write}() interprets opaque as
CXLComponentState(cxl_cstate) instead of ComponentRegisters(cregs).
Fortunately, cregs is the first member of cxl_cstate, so their values are
the same.
Fixes: 9e58f52d3f8 ("hw/cxl/component: Introduce CXL components (8.1.x, 8.2.5)")
Signed-off-by: Li Zhijian <lizhijian@fujitsu.com>
---
V2: change the source side since cache_mem_ops.{read,write}() will use
cxl_cstate.
---
hw/cxl/cxl-component-utils.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/hw/cxl/cxl-component-utils.c b/hw/cxl/cxl-component-utils.c
index f3bbf0fd131..6214dcdcc12 100644
--- a/hw/cxl/cxl-component-utils.c
+++ b/hw/cxl/cxl-component-utils.c
@@ -179,7 +179,7 @@ void cxl_component_register_block_init(Object *obj,
/* io registers controls link which we don't care about in QEMU */
memory_region_init_io(&cregs->io, obj, NULL, cregs, ".io",
CXL2_COMPONENT_IO_REGION_SIZE);
- memory_region_init_io(&cregs->cache_mem, obj, &cache_mem_ops, cregs,
+ memory_region_init_io(&cregs->cache_mem, obj, &cache_mem_ops, cxl_cstate,
".cache_mem", CXL2_COMPONENT_CM_REGION_SIZE);
memory_region_add_subregion(&cregs->component_registers, 0, &cregs->io);
--
2.41.0
^ permalink raw reply related [flat|nested] 5+ messages in thread
* [PATCH v2 2/2] hw/cxl: Pass NULL for a NULL MemoryRegionOps
2023-10-18 8:24 [PATCH v2 1/2] hw/cxl: Pass CXLComponentState to cache_mem_ops Li Zhijian
@ 2023-10-18 8:24 ` Li Zhijian
2023-10-19 10:50 ` Jonathan Cameron
1 sibling, 0 replies; 5+ messages in thread
From: Li Zhijian @ 2023-10-18 8:24 UTC (permalink / raw)
To: jonathan.cameron, fan.ni; +Cc: qemu-devel, philmd, Li Zhijian
a NULL parameter is enough for a NULL MemoryRegionOps
Signed-off-by: Li Zhijian <lizhijian@fujitsu.com>
---
hw/cxl/cxl-component-utils.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/hw/cxl/cxl-component-utils.c b/hw/cxl/cxl-component-utils.c
index 6214dcdcc12..010ed82edab 100644
--- a/hw/cxl/cxl-component-utils.c
+++ b/hw/cxl/cxl-component-utils.c
@@ -177,7 +177,7 @@ void cxl_component_register_block_init(Object *obj,
CXL2_COMPONENT_BLOCK_SIZE);
/* io registers controls link which we don't care about in QEMU */
- memory_region_init_io(&cregs->io, obj, NULL, cregs, ".io",
+ memory_region_init_io(&cregs->io, obj, NULL, NULL, ".io",
CXL2_COMPONENT_IO_REGION_SIZE);
memory_region_init_io(&cregs->cache_mem, obj, &cache_mem_ops, cxl_cstate,
".cache_mem", CXL2_COMPONENT_CM_REGION_SIZE);
--
2.41.0
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH v2 1/2] hw/cxl: Pass CXLComponentState to cache_mem_ops
@ 2023-10-19 10:50 ` Jonathan Cameron
0 siblings, 0 replies; 5+ messages in thread
From: Jonathan Cameron via @ 2023-10-19 10:50 UTC (permalink / raw)
To: Li Zhijian; +Cc: fan.ni, qemu-devel, philmd, mst
On Wed, 18 Oct 2023 16:24:07 +0800
Li Zhijian <lizhijian@fujitsu.com> wrote:
> cache_mem_ops.{read,write}() interprets opaque as
> CXLComponentState(cxl_cstate) instead of ComponentRegisters(cregs).
>
> Fortunately, cregs is the first member of cxl_cstate, so their values are
> the same.
>
> Fixes: 9e58f52d3f8 ("hw/cxl/component: Introduce CXL components (8.1.x, 8.2.5)")
> Signed-off-by: Li Zhijian <lizhijian@fujitsu.com>
Both these lgtm. I'll carry them on my CXL tree and add them to the next
series I send out for general cleaup etc, but if you send them again
cc Michael Tsirkin so he has the option to pick them up directly if he wishes
(all CXL changes got through Michael currently).
> ---
> V2: change the source side since cache_mem_ops.{read,write}() will use
> cxl_cstate.
> ---
> hw/cxl/cxl-component-utils.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/hw/cxl/cxl-component-utils.c b/hw/cxl/cxl-component-utils.c
> index f3bbf0fd131..6214dcdcc12 100644
> --- a/hw/cxl/cxl-component-utils.c
> +++ b/hw/cxl/cxl-component-utils.c
> @@ -179,7 +179,7 @@ void cxl_component_register_block_init(Object *obj,
> /* io registers controls link which we don't care about in QEMU */
> memory_region_init_io(&cregs->io, obj, NULL, cregs, ".io",
> CXL2_COMPONENT_IO_REGION_SIZE);
> - memory_region_init_io(&cregs->cache_mem, obj, &cache_mem_ops, cregs,
> + memory_region_init_io(&cregs->cache_mem, obj, &cache_mem_ops, cxl_cstate,
> ".cache_mem", CXL2_COMPONENT_CM_REGION_SIZE);
>
> memory_region_add_subregion(&cregs->component_registers, 0, &cregs->io);
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH v2 1/2] hw/cxl: Pass CXLComponentState to cache_mem_ops
@ 2023-10-19 10:50 ` Jonathan Cameron
0 siblings, 0 replies; 5+ messages in thread
From: Jonathan Cameron @ 2023-10-19 10:50 UTC (permalink / raw)
To: Li Zhijian; +Cc: fan.ni, qemu-devel, philmd, mst
On Wed, 18 Oct 2023 16:24:07 +0800
Li Zhijian <lizhijian@fujitsu.com> wrote:
> cache_mem_ops.{read,write}() interprets opaque as
> CXLComponentState(cxl_cstate) instead of ComponentRegisters(cregs).
>
> Fortunately, cregs is the first member of cxl_cstate, so their values are
> the same.
>
> Fixes: 9e58f52d3f8 ("hw/cxl/component: Introduce CXL components (8.1.x, 8.2.5)")
> Signed-off-by: Li Zhijian <lizhijian@fujitsu.com>
Both these lgtm. I'll carry them on my CXL tree and add them to the next
series I send out for general cleaup etc, but if you send them again
cc Michael Tsirkin so he has the option to pick them up directly if he wishes
(all CXL changes got through Michael currently).
> ---
> V2: change the source side since cache_mem_ops.{read,write}() will use
> cxl_cstate.
> ---
> hw/cxl/cxl-component-utils.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/hw/cxl/cxl-component-utils.c b/hw/cxl/cxl-component-utils.c
> index f3bbf0fd131..6214dcdcc12 100644
> --- a/hw/cxl/cxl-component-utils.c
> +++ b/hw/cxl/cxl-component-utils.c
> @@ -179,7 +179,7 @@ void cxl_component_register_block_init(Object *obj,
> /* io registers controls link which we don't care about in QEMU */
> memory_region_init_io(&cregs->io, obj, NULL, cregs, ".io",
> CXL2_COMPONENT_IO_REGION_SIZE);
> - memory_region_init_io(&cregs->cache_mem, obj, &cache_mem_ops, cregs,
> + memory_region_init_io(&cregs->cache_mem, obj, &cache_mem_ops, cxl_cstate,
> ".cache_mem", CXL2_COMPONENT_CM_REGION_SIZE);
>
> memory_region_add_subregion(&cregs->component_registers, 0, &cregs->io);
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH v2 1/2] hw/cxl: Pass CXLComponentState to cache_mem_ops
2023-10-19 10:50 ` Jonathan Cameron
(?)
@ 2023-10-23 8:20 ` Zhijian Li (Fujitsu)
-1 siblings, 0 replies; 5+ messages in thread
From: Zhijian Li (Fujitsu) @ 2023-10-23 8:20 UTC (permalink / raw)
To: Jonathan Cameron; +Cc: fan.ni, qemu-devel, philmd, mst
On 19/10/2023 18:50, Jonathan Cameron wrote:
> On Wed, 18 Oct 2023 16:24:07 +0800
> Li Zhijian <lizhijian@fujitsu.com> wrote:
>
>> cache_mem_ops.{read,write}() interprets opaque as
>> CXLComponentState(cxl_cstate) instead of ComponentRegisters(cregs).
>>
>> Fortunately, cregs is the first member of cxl_cstate, so their values are
>> the same.
>>
>> Fixes: 9e58f52d3f8 ("hw/cxl/component: Introduce CXL components (8.1.x, 8.2.5)")
>> Signed-off-by: Li Zhijian <lizhijian@fujitsu.com>
>
> Both these lgtm. I'll carry them on my CXL tree and add them to the next
> series I send out for general cleaup etc,
Well, I'm fine with this.
but if you send them again
> cc Michael Tsirkin so he has the option to pick them up directly if he wishes
> (all CXL changes got through Michael currently).
Good to know this
Thanks
Zhijian
>
>> ---
>> V2: change the source side since cache_mem_ops.{read,write}() will use
>> cxl_cstate.
>> ---
>> hw/cxl/cxl-component-utils.c | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/hw/cxl/cxl-component-utils.c b/hw/cxl/cxl-component-utils.c
>> index f3bbf0fd131..6214dcdcc12 100644
>> --- a/hw/cxl/cxl-component-utils.c
>> +++ b/hw/cxl/cxl-component-utils.c
>> @@ -179,7 +179,7 @@ void cxl_component_register_block_init(Object *obj,
>> /* io registers controls link which we don't care about in QEMU */
>> memory_region_init_io(&cregs->io, obj, NULL, cregs, ".io",
>> CXL2_COMPONENT_IO_REGION_SIZE);
>> - memory_region_init_io(&cregs->cache_mem, obj, &cache_mem_ops, cregs,
>> + memory_region_init_io(&cregs->cache_mem, obj, &cache_mem_ops, cxl_cstate,
>> ".cache_mem", CXL2_COMPONENT_CM_REGION_SIZE);
>>
>> memory_region_add_subregion(&cregs->component_registers, 0, &cregs->io);
>
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2023-10-23 8:22 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-10-18 8:24 [PATCH v2 1/2] hw/cxl: Pass CXLComponentState to cache_mem_ops Li Zhijian
2023-10-18 8:24 ` [PATCH v2 2/2] hw/cxl: Pass NULL for a NULL MemoryRegionOps Li Zhijian
2023-10-19 10:50 ` [PATCH v2 1/2] hw/cxl: Pass CXLComponentState to cache_mem_ops Jonathan Cameron via
2023-10-19 10:50 ` Jonathan Cameron
2023-10-23 8:20 ` Zhijian Li (Fujitsu)
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.