[PATCH for 8.7.0 00/80] Remove pre-blockdev disk setup code

Ján Tomko jtomko at redhat.com
Thu Aug 4 23:33:44 UTC 2022


On a Tuesday in 2022, Peter Krempa wrote:
>Starting with libvirt-8.7.0 we'll support qemu-4.2 as minimum and thus
>we'll only use -blockdev now. Remove the old cruft.
>
>This series applies on top of the patches bumping minimum qemu version
>to qemu-4.2 I've posted earlier.
>
>The whole branch can be fetched at:
>
> git fetch https://gitlab.com/pipo.sk/libvirt.git blockdev-remove-2
>
>Peter Krempa (80):
>  qemu: Allways assume QEMU_CAPS_CHANGE_BACKING_FILE
>  qemu: capabilities: Retire QEMU_CAPS_CHANGE_BACKING_FILE
>  qemu: Always assume support for QEMU_CAPS_ACTIVE_COMMIT
>  qemu: capabilities: Retire QEMU_CAPS_ACTIVE_COMMIT
>  qemu: Always assume support for QEMU_CAPS_GLUSTER_DEBUG_LEVEL
>  qemu: capabilities: Retire QEMU_CAPS_GLUSTER_DEBUG_LEVEL
>  qemu: Always assume support for QEMU_CAPS_DRIVE_DISCARD and
>    QEMU_CAPS_DRIVE_DETECT_ZEROES
>  qemu: capabilities: Retire QEMU_CAPS_DRIVE_DISCARD and
>    QEMU_CAPS_DRIVE_DETECT_ZEROES
>  qemuxml2argvtest: Use DO_TEST_CAPS_LATEST for all 'user-aliases' tests
>  qemuxml2xmltest: Modernize all test cases using QEMU_CAPS_QCOW2_LUKS
>  qemu: Always assume support for QEMU_CAPS_QCOW2_LUKS
>  qemu: capabilities: Retire QEMU_CAPS_QCOW2_LUKS
>  qemu: capabilities: Unconditionally set
>    QEMU_CAPS_BLOCKDEV/QEMU_CAPS_BLOCKDEV_HOSTDEV_SCSI
>  tests: Force QEMU_CAPS_BLOCKDEV(_HOSTDEV_SCSI) in fake caps tests
>  qemu: Refactor access to 'qomName' field of the qemu disk private data
>  qemu: command: Always assume support for QEMU_CAPS_SCSI_DISK_DEVICE_ID
>  qemu: capabilities: Retire capabilities used for -blockdev detection
>  qemu: Remove code conditional on QEMU_CAPS_BLOCKDEV_HOSTDEV_SCSI
>  qemu: capabilities: Retire QEMU_CAPS_BLOCKDEV_HOSTDEV_SCSI
>  qemu: block: Remove legacy spellings for InetSocketAddress
>  qemu: migration: Remove pre-blockdev code paths
>  qemuMigrationSrcNBDStorageCopyBlockdev: Remove some arguments
>  qemu: migration: Assume support for QEMU_CAPS_BLOCKDEV_DEL
>  qemu: capabilities: Retire QEMU_CAPS_BLOCKDEV_DEL
>  qemu: snapshot: Remove pre-blockdev code paths
>  qemu: Remove unused code for pre-blockdev disk snapshot monitor
>    interaction
>  qemu: block: Remove pre-blockdev code paths
>  qemu: migration_cookie: Remove pre-blockdev code paths
>  qemu: validate: Remove impossible validation checks
>  qemu: process: Remove pre-blockdev code paths
>  qemu: blockjob: Remove legacy block job completion code
>  qemu: Remove legacy block job event processing handler
>  qemu: monitor: Remove infrastructure for BLOCK_JOB_* events
>  qemuDomainSetBlockThreshold: Remove pre-blockdev node name detection
>  qemu: block: Remove 'qemuBlockNodeNamesDetect' and related
>    infrastructure
>  qemu: blockjob: Remove pre-blockdev blockjob setup
>  qemu: alias: Remove check for QEMU_CAPS_BLOCKDEV
>  qemuDomainGetStatsBlock: Remove pre-blockdev code paths
>  qemu: block: Remove unused qemuBlockGetNodeData
>  qemu: monitor: Remove unused qemuMonitorQueryNamedBlockNodes and clean
>    up
>  qemu: driver: Remove pre-blockdev code paths from
>    qemuDomainBlockCopyCommon
>  qemu: monitor: Remove unused qemuMonitorDriveMirror
>  qemu: driver: Remove pre-blockdev code paths from
>    qemuDomainBlockCommit
>  qemu: monitor: Remove unused arguments of qemuMonitorBlockCommit
>  qemu: driver: Remove pre-blockdev code paths from
>    qemuDomainBlockPullCommon
>  qemu: monitor: Remove unused arguments of qemuMonitorBlockStream
>  qemu: monitor: Remove unused qemuMonitorDiskNameLookup
>  qemu: driver: Remove pre-blockdev code paths from qemuDomainBlockPivot
>  qemu: monitor: Remove unused qemuMonitorDrivePivot
>  qemu: command: Remove pre-blockdev floppy setup
>  qemu: Remove pre-blockdev PFLASH setup
>  qemuDomainBlockResize: Always refuse resize of empty/readonly disks
>  qemuDomainBlocksStatsGather: Remove pre-blockdev code paths
>  qemu: monitor: Remove unused qemuMonitorBlockStatsUpdateCapacity
>  qemu: driver: Remove remaining QEMU_CAPS_BLOCKDEV capability checks
>  qemu: hotplug: Remove pre-blockdev code path from
>    qemuDomainChangeEjectableMedia
>  qemu: monitor: Remove unused qemuMonitor(Eject|Change)Media
>  qemuhotplugtest: Switch to -blockdev mode
>  qemuDomainRemoveDiskDevice: Remove special casing for disks on 'sd'
>    bus
>  qemu: hotplug: Remove pre-blockdev code paths
>  qemu: block: Remove unused
>    qemuBlockStorageSourceChainDetachPrepareLegacy
>  qemu: Remove 'driveAlias' argument of
>    qemuBlockStorageSourceDetachPrepare
>  qemu: block: Remove support for legacy disk hotplug/unplug
>  qemu: monitor: Remove unused qemuMonitorAddDrive/qemuMonitorDriveDel
>  qemu: command: Delegate qemuBuildDriveStr to SD cards only
>  qemu: command: Generate -drive for SD cards via JSON props
>  qemu: command: Remove unused qemuGetDriveSourceString
>  qemu: domain: Assume QEMU_CAPS_BLOCKDEV for the domain private code
>  qemuDomainSecretStorageSourcePrepare: Remove cabapbility check when
>    preparing cookies
>  qemuDomainDiskBlockJobIsSupported: Assume QEMU_CAPS_BLOCKDEV is
>    supported
>  qemuDomainDiskGetBackendAlias: Assume QEMU_CAPS_BLOCKDEV is supported
>  qemuDomainValidateStorageSource: Remove QEMU_CAPS_BLOCKDEV validation
>  qemuDomainValidateStorageSource: Remove 'maskBlockdev' argument
>  qemuDomainDetermineDiskChain: Assume QEMU_CAPS_BLOCKDEV
>  qemu: Retire QEMU_CAPS_BLOCKDEV
>  qemu: Setup disk throttling for SD cards via monitor
>  qemu: capabilities: Retire QEMU_CAPS_QUERY_NAMED_BLOCK_NODES
>  qemu: Assume support and retire QEMU_CAPS_BLOCK_WRITE_THRESHOLD
>  qemu: Assume and retire QEMU_CAPS_BLOCKDEV_BACKUP
>  qemuDomainDetermineDiskChain: Remove 'report_broken' argument
>



Reviewed-by: Ján Tomko <jtomko at redhat.com>

Jano
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 488 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20220805/bcf0dbf3/attachment.sig>


More information about the libvir-list mailing list