[PATCH 0/3] Minor cleanups: eliminate redundant code in QOM, block-backend and vl.c

Bin Guo posted 3 patches 1 month ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20260429062004.36582-1-guobin@linux.alibaba.com
Maintainers: Kevin Wolf <kwolf@redhat.com>, Hanna Reitz <hreitz@redhat.com>, Paolo Bonzini <pbonzini@redhat.com>, "Daniel P. Berrangé" <berrange@redhat.com>
block/block-backend.c |  8 +-------
qom/object.c          |  7 -------
system/vl.c           | 37 +++++++++++++++----------------------
3 files changed, 16 insertions(+), 36 deletions(-)
[PATCH 0/3] Minor cleanups: eliminate redundant code in QOM, block-backend and vl.c
Posted by Bin Guo 1 month ago
This series contains three independent cleanups that remove redundant
code and improve consistency across different subsystems.

Patch 1 merges a double hash-table traversal in object_property_del_child()
into a single pass, halving the number of hash table operations in the
common case.

Patch 2 makes blk_co_preadv() a one-liner wrapper around
blk_co_preadv_part(), matching the existing pattern already used by
the write side (blk_co_pwritev / blk_co_pwritev_part).

Patch 3 inlines qemu_opts_parse_noisily() return-value checks directly
into if-conditions for options that do not use the returned pointer,
matching the style already used by QEMU_OPTION_action.

All three patches are independent and can be applied in any order.

Bin Guo (3):
  qom/object: merge double hash table traversal in
    object_property_del_child
  block/block-backend: delegate blk_co_preadv to blk_co_preadv_part
  system/vl.c: inline qemu_opts_parse_noisily() result checks

 block/block-backend.c |  8 +-------
 qom/object.c          |  7 -------
 system/vl.c           | 37 +++++++++++++++----------------------
 3 files changed, 16 insertions(+), 36 deletions(-)

-- 
2.50.1 (Apple Git-155)