[v2] qdev: Consolidate qdev_get_human_name() into qdev_get_printable_name()

[PATCH v2 0/2] qdev: Consolidate qdev_get_human_name() into qdev_get_printable_name()

Alessandro Ratti posted 2 patches 3 weeks, 5 days ago

Diff against v1
Download series mbox

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20260311215003.664815-1-alessandro@0x65c.net

Maintainers: John Snow <jsnow@redhat.com>, Kevin Wolf <kwolf@redhat.com>, Hanna Reitz <hreitz@redhat.com>, Paolo Bonzini <pbonzini@redhat.com>, "Daniel P. Berrangé" <berrange@redhat.com>, Eduardo Habkost <eduardo@habkost.net>

hw/block/block.c       |  5 ++---
hw/core/qdev.c         | 37 ++++++++++++-------------------------
include/hw/core/qdev.h | 25 +++++--------------------
3 files changed, 19 insertions(+), 48 deletions(-)

Expand all Fold all

[PATCH v2 0/2] qdev: Consolidate qdev_get_human_name() into qdev_get_printable_name()

Posted by Alessandro Ratti 3 weeks, 5 days ago

v2 addresses Markus's review of v1:

- Split into two patches: improve qdev_get_printable_name() first,
  then remove qdev_get_human_name()
- Fix comments to accurately describe qdev_get_dev_path() behavior
- Drop unnecessary comments and the "<unknown device>" placeholder
- Rename @vdev to @dev, add g_assert(dev) precondition
- Update doc comment in qdev.h to reflect the new fallback chain
- Rebase on current master now that the prerequisite
  qdev_get_printable_name() changes have landed

Formatting bus-specific paths as "<bus> device <dev-path>" (e.g.
"PCI device 0000:00:00.0") is left for a follow-up series.

v1: https://lore.kernel.org/qemu-devel/20260308160040.354186-2-alessandro@0x65c.net/


Alessandro Ratti (2):
  hw/qdev: Clarify fallback order in qdev_get_printable_name()
  hw/qdev: Remove qdev_get_human_name()

 hw/block/block.c       |  5 ++---
 hw/core/qdev.c         | 37 ++++++++++++-------------------------
 include/hw/core/qdev.h | 25 +++++--------------------
 3 files changed, 19 insertions(+), 48 deletions(-)

-- 
2.53.0