* [PATCH RESEND 1/1] multi-process: fix usage information
@ 2021-07-13 0:47 Dongli Zhang
2021-09-16 10:06 ` Laurent Vivier
2021-09-23 15:16 ` Laurent Vivier
0 siblings, 2 replies; 3+ messages in thread
From: Dongli Zhang @ 2021-07-13 0:47 UTC (permalink / raw)
To: qemu-trivial, qemu-devel
Cc: elena.ufimtseva, jag.raman, mjt, laurent, john.g.johnson
From source code, the 'devid' of x-remote-object should be one of devices
in remote QEMU process.
Signed-off-by: Dongli Zhang <dongli.zhang@oracle.com>
Reviewed-by: Jagannathan Raman <jag.raman@oracle.com>
---
Resend to be applied as trivial patch.
I have verified by reading the code and playing with below orchestrator.
https://github.com/finallyjustice/sample/blob/master/kvm/multiprocess/orchestrator.py
docs/system/multi-process.rst | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/docs/system/multi-process.rst b/docs/system/multi-process.rst
index 46bb0cafc2..210531ee17 100644
--- a/docs/system/multi-process.rst
+++ b/docs/system/multi-process.rst
@@ -45,7 +45,7 @@ Following is a description of command-line used to launch mpqemu.
-device lsi53c895a,id=lsi0 \
-drive id=drive_image2,file=/build/ol7-nvme-test-1.qcow2 \
-device scsi-hd,id=drive2,drive=drive_image2,bus=lsi0.0,scsi-id=0 \
- -object x-remote-object,id=robj1,devid=lsi1,fd=4,
+ -object x-remote-object,id=robj1,devid=lsi0,fd=4,
* QEMU:
--
2.17.1
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH RESEND 1/1] multi-process: fix usage information
2021-07-13 0:47 [PATCH RESEND 1/1] multi-process: fix usage information Dongli Zhang
@ 2021-09-16 10:06 ` Laurent Vivier
2021-09-23 15:16 ` Laurent Vivier
1 sibling, 0 replies; 3+ messages in thread
From: Laurent Vivier @ 2021-09-16 10:06 UTC (permalink / raw)
To: Dongli Zhang, qemu-trivial, qemu-devel
Cc: elena.ufimtseva, mjt, jag.raman, john.g.johnson
Le 13/07/2021 à 02:47, Dongli Zhang a écrit :
> From source code, the 'devid' of x-remote-object should be one of devices
> in remote QEMU process.
>
> Signed-off-by: Dongli Zhang <dongli.zhang@oracle.com>
> Reviewed-by: Jagannathan Raman <jag.raman@oracle.com>
> ---
> Resend to be applied as trivial patch.
>
> I have verified by reading the code and playing with below orchestrator.
> https://github.com/finallyjustice/sample/blob/master/kvm/multiprocess/orchestrator.py
>
> docs/system/multi-process.rst | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/docs/system/multi-process.rst b/docs/system/multi-process.rst
> index 46bb0cafc2..210531ee17 100644
> --- a/docs/system/multi-process.rst
> +++ b/docs/system/multi-process.rst
> @@ -45,7 +45,7 @@ Following is a description of command-line used to launch mpqemu.
> -device lsi53c895a,id=lsi0 \
> -drive id=drive_image2,file=/build/ol7-nvme-test-1.qcow2 \
> -device scsi-hd,id=drive2,drive=drive_image2,bus=lsi0.0,scsi-id=0 \
> - -object x-remote-object,id=robj1,devid=lsi1,fd=4,
> + -object x-remote-object,id=robj1,devid=lsi0,fd=4,
>
> * QEMU:
>
>
Reviewed-by: Laurent Vivier <laurent@vivier.eu>
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH RESEND 1/1] multi-process: fix usage information
2021-07-13 0:47 [PATCH RESEND 1/1] multi-process: fix usage information Dongli Zhang
2021-09-16 10:06 ` Laurent Vivier
@ 2021-09-23 15:16 ` Laurent Vivier
1 sibling, 0 replies; 3+ messages in thread
From: Laurent Vivier @ 2021-09-23 15:16 UTC (permalink / raw)
To: Dongli Zhang, qemu-trivial, qemu-devel
Cc: elena.ufimtseva, mjt, jag.raman, john.g.johnson
Le 13/07/2021 à 02:47, Dongli Zhang a écrit :
> From source code, the 'devid' of x-remote-object should be one of devices
> in remote QEMU process.
>
> Signed-off-by: Dongli Zhang <dongli.zhang@oracle.com>
> Reviewed-by: Jagannathan Raman <jag.raman@oracle.com>
> ---
> Resend to be applied as trivial patch.
>
> I have verified by reading the code and playing with below orchestrator.
> https://github.com/finallyjustice/sample/blob/master/kvm/multiprocess/orchestrator.py
>
> docs/system/multi-process.rst | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/docs/system/multi-process.rst b/docs/system/multi-process.rst
> index 46bb0cafc2..210531ee17 100644
> --- a/docs/system/multi-process.rst
> +++ b/docs/system/multi-process.rst
> @@ -45,7 +45,7 @@ Following is a description of command-line used to launch mpqemu.
> -device lsi53c895a,id=lsi0 \
> -drive id=drive_image2,file=/build/ol7-nvme-test-1.qcow2 \
> -device scsi-hd,id=drive2,drive=drive_image2,bus=lsi0.0,scsi-id=0 \
> - -object x-remote-object,id=robj1,devid=lsi1,fd=4,
> + -object x-remote-object,id=robj1,devid=lsi0,fd=4,
>
> * QEMU:
>
>
Applied to my trivial-patches branch.
Thanks,
Laurent
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2021-09-23 15:18 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-07-13 0:47 [PATCH RESEND 1/1] multi-process: fix usage information Dongli Zhang
2021-09-16 10:06 ` Laurent Vivier
2021-09-23 15:16 ` Laurent Vivier
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).