Paolo Bonzini <pbonz...@redhat.com> writes: > Sure, CHS translation is an obscure topic, and legacy options for > hard-disk geometries are obscure as well. But since QEMU does nothing > with it except telling the BIOS, and since there "large" and "rechs" > are listed in the enums, parsing them seems to be the bare minimum. > > Signed-off-by: Paolo Bonzini <pbonz...@redhat.com>
This is semantically on top of "[PATCH 00/12] qdev: cleanup legacy properties", which adds these two values to isa-hd property bios-chs-trans. Here, you add them to the two obsolete ways to specify the property. While I don't think that's necessary, I don't object to it either. Despite my doubts on the need for the patch: Reviewed-by: Markus Armbruster <arm...@redhat.com>