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>

Reply via email to