Am 06.09.2011 18:58, schrieb Markus Armbruster:
> No functional change.
>
> It would be nice to have handler functions in the table, like commit
> e1a064f9 did for ATAPI. Left for another day.
>
> Signed-off-by: Markus Armbruster <arm...@redhat.com>
> ---
> hw/ide/core.c | 105 +++++++++++++++++++++++++++++++++++++++++++-------------
> 1 files changed, 80 insertions(+), 25 deletions(-)
> + [IBM_SENSE_CONDITION] = CFA_OK,
> + [CFA_WEAR_LEVEL] = CFA_OK,
> + [WIN_READ_NATIVE_MAX] = ALL_OK,
> +};
> +
> +static bool ide_cmd_permitted(IDEState *s, uint32_t cmd)
> +{
> + return cmd <= ARRAY_SIZE(ide_cmd_table)
Shouldn't it be < instead of <= ?
Kevin
