[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Qemu-devel] Re: [Qemu-commits] [COMMIT 5c6c3a6] raw-posix: add Linux na

From: Riku Voipio
Subject: [Qemu-devel] Re: [Qemu-commits] [COMMIT 5c6c3a6] raw-posix: add Linux native AIO support
Date: Fri, 28 Aug 2009 15:07:33 +0300
User-agent: Mutt/1.5.18 (2008-05-17)

On Fri, Aug 28, 2009 at 02:38:57PM +0400, malc wrote:
> -        } else if (s->use_aio) {
> +        }
> +        else if (s->use_aio) {
>              return laio_submit(bs, s->aio_ctx, s->fd, sector_num, qiov,
> -                            nb_sectors, cb, opaque, type);
> +                               nb_sectors, cb, opaque, type);
> +#else
> +            abort();
> +#endif

use_aio is only ever set in CONFIG_LINUX_AIO protected section, so the #else
section is not needed. Even if that were not the case, falling back to 
feels like a better idea than aborting emulation.

fwiw, s->use_aio is a bad name as well, as it refers to linux aio, while the
file also supports posix aio...

reply via email to

[Prev in Thread] Current Thread [Next in Thread]