Re: [Qemu-devel] [PATCH] ide: IDENTIFY word 86 bit 14 is reserved

[Stefan Hajnoczi]

On Tue, Mar 13, 2012 at 1:49 PM, Kevin Wolf <address@hidden> wrote:
> Reserved bits should be cleared to zero.
>
> Signed-off-by: Kevin Wolf <address@hidden>
> ---
>  hw/ide/core.c |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/hw/ide/core.c b/hw/ide/core.c
> index 6f06d28..771811c 100644
> --- a/hw/ide/core.c
> +++ b/hw/ide/core.c
> @@ -150,7 +150,7 @@ static void ide_identify(IDEState *s)
>     else
>          put_le16(p + 85, (1 << 14) | 1);
>     /* 13=flush_cache_ext,12=flush_cache,10=lba48 */
> -    put_le16(p + 86, (1 << 14) | (1 << 13) | (1 <<12) | (1 << 10));
> +    put_le16(p + 86, (1 << 13) | (1 <<12) | (1 << 10));

Looks good.  This seems like a copy-paste bug in "LBA48 support (Jens
Axboe)" (c2ff060fd4c7389de2016442e3327538a66696bd) where words 83 and
86 were modified in the same way.

Reviewed-by: Stefan Hajnoczi <address@hidden>