Google Git
Sign in
qemu / qemu / 66647ed4591572e7928cb5865909bb6c572d7e40
commit66647ed4591572e7928cb5865909bb6c572d7e40[log] [tgz]
authorMarkus Armbruster <armbru@redhat.com>Tue Jul 20 14:54:04 2021 +0200
committerMarkus Armbruster <armbru@redhat.com>Thu Aug 26 17:15:28 2021 +0200
treed388696d682aae8efe8ffc1c17d89fe074cfbb3f
parent0d9a6654512a6b7ab18c53fc3f5a6bba8de2dde8 [diff]
vhost: Clean up how VhostOpts method vhost_get_config() fails

vhost_user_get_config() can fail without setting an error.  Unclean.
Its caller vhost_dev_get_config() compensates by substituting a
generic error then.  Goes back to commit 50de51387f "vhost:
Distinguish errors in vhost_dev_get_config()".

Clean up by moving the generic error from vhost_dev_get_config() to
all the failure paths that neglect to set an error.

Cc: Kevin Wolf <kwolf@redhat.com>
Cc: Michael S. Tsirkin <mst@redhat.com>
Signed-off-by: Markus Armbruster <armbru@redhat.com>
Message-Id: <20210720125408.387910-13-armbru@redhat.com>
Acked-by: Michael S. Tsirkin <mst@redhat.com>
Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>
[Sign of error_setg_errno()'s second argument fixed in both calls]
2 files changed
tree: d388696d682aae8efe8ffc1c17d89fe074cfbb3f
  1. .github/
  2. .gitlab/
  3. .gitlab-ci.d/
  4. accel/
  5. audio/
  6. authz/
  7. backends/
  8. block/
  9. bsd-user/
  10. chardev/
  11. configs/
  12. contrib/
  13. crypto/
  14. disas/
  15. docs/
  16. dump/
  17. ebpf/
  18. fpu/
  19. fsdev/
  20. gdb-xml/
  21. hw/
  22. include/
  23. io/
  24. libdecnumber/
  25. linux-headers/
  26. linux-user/
  27. migration/
  28. monitor/
  29. nbd/
  30. net/
  31. pc-bios/
  32. plugins/
  33. po/
  34. python/
  35. qapi/
  36. qga/
  37. qobject/
  38. qom/
  39. replay/
  40. roms/
  41. scripts/
  42. scsi/
  43. semihosting/
  44. softmmu/
  45. storage-daemon/
  46. stubs/
  47. subprojects/
  48. target/
  49. tcg/
  50. tests/
  51. tools/
  52. trace/
  53. ui/
  54. util/
  55. capstone
  56. dtc
  57. meson
  58. slirp
  59. .cirrus.yml
  60. .dir-locals.el
  61. .editorconfig
  62. .exrc
  63. .gdbinit
  64. .gitattributes
  65. .gitignore
  66. .gitlab-ci.yml
  67. .gitmodules
  68. .gitpublish
  69. .mailmap
  70. .patchew.yml
  71. .readthedocs.yml
  72. .travis.yml
  73. block.c
  74. blockdev-nbd.c
  75. blockdev.c
  76. blockjob.c
  77. configure
  78. COPYING
  79. COPYING.LIB
  80. cpu.c
  81. cpus-common.c
  82. disas.c
  83. gdbstub.c
  84. gitdm.config
  85. hmp-commands-info.hx
  86. hmp-commands.hx
  87. iothread.c
  88. job-qmp.c
  89. job.c
  90. Kconfig
  91. Kconfig.host
  92. LICENSE
  93. MAINTAINERS
  94. Makefile
  95. memory_ldst.c.inc
  96. meson.build
  97. meson_options.txt
  98. module-common.c
  99. os-posix.c
  100. os-win32.c
  101. page-vary-common.c
  102. page-vary.c
  103. qemu-bridge-helper.c
  104. qemu-edid.c
  105. qemu-img-cmds.hx
  106. qemu-img.c
  107. qemu-io-cmds.c
  108. qemu-io.c
  109. qemu-keymap.c
  110. qemu-nbd.c
  111. qemu-options.hx
  112. qemu.nsi
  113. qemu.sasl
  114. README.rst
  115. replication.c
  116. thunk.c
  117. trace-events
  118. VERSION
  119. version.rc