[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 32/32] Relax IDE CHS limits from 16383, 16, 63 t
From: |
Alexander Graf |
Subject: |
Re: [Qemu-devel] [PATCH 32/32] Relax IDE CHS limits from 16383, 16, 63 to 65535, 16, 255 |
Date: |
Fri, 6 Jul 2012 16:50:37 +0200 |
On 05.07.2012, at 18:20, Markus Armbruster wrote:
> Kevin Wolf <address@hidden> writes:
>
>> Am 29.06.2012 17:34, schrieb Markus Armbruster:
>>> New limits straight from ATA4 6.2 Register delivered data transfer
>>> command sector addressing.
>>>
>>> I figure the old sector limit 63 was blindly copied from the BIOS
>>> int 13 limit. Doesn't apply to the hardware. No idea where the old
>>> cylinder limit comes from.
>>>
>>> Signed-off-by: Markus Armbruster <address@hidden>
>>
>> Now I think we have the very same thing in IDE, SCSI and virtio-blk.
>> Would it make sense to have a helper function in hd-geometry.c that
>> takes and validates the geometry from a BlockConf, applies defaults and
>> puts the result into device state fields passed by reference?
>
> I can look into this, but I'm afraid we'd need two helpers, because of
> IDE complications.
>
>
> When it comes to block device models, IDE is *always* the troublemaker.
> And a big reason for that is the messy data structures that are
> impractical to clean up while we still support non-qdevified IDE. Which
> we do almost three years after IDE qdevification.
>
> The laggards are:
>
> * mac99, g3beige
> Alexander Graf <address@hidden>
>
> * spitz, borzoi, terrier
> Andrzej Zaborowski <address@hidden>
>
> * tosa
> unmaintained
>
> * r2d
> Magnus Damm <address@hidden>
>
> Any plans to drag these boards into the current decade already?
Got a pointer to a board that does it right?
Alex
- Re: [Qemu-devel] [PATCH 32/32] Relax IDE CHS limits from 16383, 16, 63 to 65535, 16, 255, Kevin Wolf, 2012/07/05
- Re: [Qemu-devel] [PATCH 32/32] Relax IDE CHS limits from 16383, 16, 63 to 65535, 16, 255, Markus Armbruster, 2012/07/05
- Re: [Qemu-devel] [PATCH 32/32] Relax IDE CHS limits from 16383, 16, 63 to 65535, 16, 255, Markus Armbruster, 2012/07/05
- Re: [Qemu-devel] [PATCH 32/32] Relax IDE CHS limits from 16383, 16, 63 to 65535, 16, 255, Kevin Wolf, 2012/07/06
- Re: [Qemu-devel] [PATCH 32/32] Relax IDE CHS limits from 16383, 16, 63 to 65535, 16, 255, Markus Armbruster, 2012/07/11
- Re: [Qemu-devel] [PATCH 32/32] Relax IDE CHS limits from 16383, 16, 63 to 65535, 16, 255,
Alexander Graf <=