[PATCH v2] docs: fix vhost-user protocol doc

2024-09-10 Thread luzhixing12345
Add a ref link to Memory region description and Multiple Memory region description. Desciptions about memory regions are merged into one line. Add extra type(64 bits) to Log description structure fields Fix ’s to 's >On Sun, Sep 08, 2024 at 10:49:54PM +0800, luzhixing12345 wrote: &g

[PATCH] cxl: update kernel config requirement

2024-09-09 Thread luzhixing12345
add CXL_PMEM config dependence LIBNVIDIMM CXL_REGION config is not enabled in 5.18, and it was introduced into kernel in 6.0 by commit 779dd20c for dynamic provisioning of new memory region Signed-off-by: luzhixing12345 --- docs/system/devices/cxl.rst | 5 - 1 file changed, 4 insertions

[PATCH] docs: fix vhost-user protocol doc

2024-09-08 Thread luzhixing12345
>On Fri, Sep 06, 2024 at 10:10:45AM +0800, luzhixing12345 wrote: >> Hi, can someone help review this patch? >> >> Signed-off-by: luzhixing12345 > >You got comments Aug 5, pls address them. ok, the comments are addressed. >On Sun, Aug 04, 2024 at 01:04:20PM GMT,

Re: [PATCH v2] docs: fix vhost-user protocol doc

2024-09-05 Thread luzhixing12345
Hi, can someone help review this patch? Signed-off-by: luzhixing12345

Re: [PATCH] vhost-user: add NEED_REPLY flag

2024-09-05 Thread luzhixing12345
Signed-off-by: luzhixing12345 >On Mon, Aug 12, 2024 at 12:53:19PM GMT, 陆知行 wrote: >>Hi, can someone review this patch? >>I find requests which call vhost_user_get_u64 does not set NEED_REPLY flag > >Can you provide an example to trigger this issue? > >Also, wi

Re: [PATCH] vhost-user: rewrite vu_dispatch with if-else

2024-08-05 Thread luzhixing12345
Signed-off-by: luzhixing12345 >On Sun, Aug 04, 2024 at 10:23:53PM GMT, luzhixing12345 wrote: >>rewrite with if-else instead of goto > >Why? > >IMHO was better before this patch with a single error path. I think this if-else version is more clear for me, and it's good t

[PATCH v2] docs: fix vhost-user protocol doc

2024-08-05 Thread luzhixing12345
add a ref link to Memory region description, improve type description add extra type(64 bits) to Log description structure fields fix ’s to 's Signed-off-by: luzhixing12345 --- docs/interop/vhost-user.rst | 24 +--- 1 file changed, 13 insertions(+), 11 deletions(-)

[PATCH] vhost-user: add NEED_REPLY flag

2024-08-04 Thread luzhixing12345
Front-end message requests which need reply should set NEED_REPLY_MASK in flag, and response from slave need clear NEED_REPLY_MASK flag. --- hw/virtio/vhost-user.c| 2 +- subprojects/libvhost-user/libvhost-user.c | 1 + 2 files changed, 2 insertions(+), 1 deletion(-) diff --g

[PATCH] vhost-user: rewrite vu_dispatch with if-else

2024-08-04 Thread luzhixing12345
rewrite with if-else instead of goto and I have a question, in two incorrent cases - need reply but no reply_requested - no need reply but has reply_requested should we call vu_panic or print warning message? --- subprojects/libvhost-user/libvhost-user.c | 39 +-- subprojec

[PATCH] docs: fix vhost-user protocol doc

2024-08-03 Thread luzhixing12345
add a ref link to Memory region description add extra type(64 bits) to Log description structure fields fix ’s to 's --- docs/interop/vhost-user.rst | 22 +- 1 file changed, 13 insertions(+), 9 deletions(-) diff --git a/docs/interop/vhost-user.rst b/docs/interop/vhost-user.

[PATCH] docs: add more information about CXL2.0 device type

2024-07-19 Thread luzhixing12345
Add more information with CXL type1 and type2 devices. Original doc says "May also have device private memory accessible via means such as PCI memory reads and writes to BARs.", but actually CXL type1 devices doesn't have device memory. Signed-off-by: luzhixing12345 --- docs