A couple of virsocket related cleanups

[PATCH 0/5] A couple of virsocket related cleanups

Michal Privoznik posted 5 patches 2 months, 3 weeks ago

Download series mbox

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/libvirt tags/patchew/cover.1706883582.git.mprivozn@redhat.com

po/POTFILES             |  1 -
src/ch/ch_process.c     | 11 ++++++---
src/qemu/qemu_monitor.c | 27 ++------------------
src/util/virsocket.c    | 55 ++++++++++-------------------------------
src/util/virsocket.h    |  4 +--
5 files changed, 25 insertions(+), 73 deletions(-)

Expand all Fold all

[PATCH 0/5] A couple of virsocket related cleanups

Posted by Michal Privoznik 2 months, 3 weeks ago

After I've merged some patches mingw fails to build. The first patch
fixes the issue, and the rest is just a cleanup.

Michal Prívozník (5):
  virsocket: Drop unused #include and #define
  virSocketSendMsgWithFDs: Don't report errors, just set errno
  virSocketSendMsgWithFDs: Introduce @payload_len argument
  virsocket: Simplify virSocketSendFD()
  qemu_monitor: Simplify qemuMonitorIOWriteWithFD()

 po/POTFILES             |  1 -
 src/ch/ch_process.c     | 11 ++++++---
 src/qemu/qemu_monitor.c | 27 ++------------------
 src/util/virsocket.c    | 55 ++++++++++-------------------------------
 src/util/virsocket.h    |  4 +--
 5 files changed, 25 insertions(+), 73 deletions(-)

-- 
2.43.0
_______________________________________________
Devel mailing list -- devel@lists.libvirt.org
To unsubscribe send an email to devel-leave@lists.libvirt.org

Re: [PATCH 0/5] A couple of virsocket related cleanups

Posted by Praveen Paladugu 2 months, 3 weeks ago

On Fri, Feb 02, 2024 at 03:20:29PM +0100, Michal Privoznik wrote:
> After I've merged some patches mingw fails to build. The first patch
> fixes the issue, and the rest is just a cleanup.
> 
> Michal Pr??vozn??k (5):
>   virsocket: Drop unused #include and #define
>   virSocketSendMsgWithFDs: Don't report errors, just set errno
>   virSocketSendMsgWithFDs: Introduce @payload_len argument
>   virsocket: Simplify virSocketSendFD()
>   qemu_monitor: Simplify qemuMonitorIOWriteWithFD()
> 
>  po/POTFILES             |  1 -
>  src/ch/ch_process.c     | 11 ++++++---
>  src/qemu/qemu_monitor.c | 27 ++------------------
>  src/util/virsocket.c    | 55 ++++++++++-------------------------------
>  src/util/virsocket.h    |  4 +--
>  5 files changed, 25 insertions(+), 73 deletions(-)
>
Thanks for the cleanup patches Michal.
LGTM!

Regards,
Praveen

> -- 
> 2.43.0
> _______________________________________________
> Devel mailing list -- devel@lists.libvirt.org
> To unsubscribe send an email to devel-leave@lists.libvirt.org
_______________________________________________
Devel mailing list -- devel@lists.libvirt.org
To unsubscribe send an email to devel-leave@lists.libvirt.org