[PATCH v3 0/2] docs: show how to spawn qemu-storage-daemon with fd passing

Stefan Hajnoczi posted 2 patches 3 years, 2 months ago
Test checkpatch passed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20210301172728.135331-1-stefanha@redhat.com
docs/tools/qemu-storage-daemon.rst | 49 +++++++++++++++++++++++++++---
1 file changed, 44 insertions(+), 5 deletions(-)
[PATCH v3 0/2] docs: show how to spawn qemu-storage-daemon with fd passing
Posted by Stefan Hajnoczi 3 years, 2 months ago
v3:
 * Explain how to detect launch errors and that the listen socket must be
   closed in the parent process in order for this to work [Daniel]

v2:
 * Use /var/run/qmp.sock instead of /tmp/qmp-$PID.sock to prevent security
   issues with world-writeable directories [Rich, Daniel]
 * Add Patch 2 to fix insecure examples in the documentation [Rich, Daniel]

Add an example of how to spawn qemu-storage-daemon with fd passing. This
approach eliminates the need to busy wait for the QMP, NBD, or vhost-user
socket to become available.

Stefan Hajnoczi (2):
  docs: show how to spawn qemu-storage-daemon with fd passing
  docs: replace insecure /tmp examples in qsd docs

 docs/tools/qemu-storage-daemon.rst | 49 +++++++++++++++++++++++++++---
 1 file changed, 44 insertions(+), 5 deletions(-)

-- 
2.29.2

Re: [PATCH v3 0/2] docs: show how to spawn qemu-storage-daemon with fd passing
Posted by Kevin Wolf 3 years, 2 months ago
Am 01.03.2021 um 18:27 hat Stefan Hajnoczi geschrieben:
> v3:
>  * Explain how to detect launch errors and that the listen socket must be
>    closed in the parent process in order for this to work [Daniel]
> 
> v2:
>  * Use /var/run/qmp.sock instead of /tmp/qmp-$PID.sock to prevent security
>    issues with world-writeable directories [Rich, Daniel]
>  * Add Patch 2 to fix insecure examples in the documentation [Rich, Daniel]
> 
> Add an example of how to spawn qemu-storage-daemon with fd passing. This
> approach eliminates the need to busy wait for the QMP, NBD, or vhost-user
> socket to become available.

Thanks, applied to the block branch.

Kevin