Re: A typo in io_u.c::get_next_file()?

[Jens Axboe <axboe@kernel.dk>]

On 2014-06-27 18:54, Andrey Kuzmin wrote:
> diff --git a/io_u.c b/io_u.c
> index e132fd9..f487b0a 100644
> --- a/io_u.c
> +++ b/io_u.c
> @@ -1100,7 +1100,7 @@ out:
>
>   static struct fio_file *get_next_file(struct thread_data *td)
>   {
> -       if (!(td->flags & TD_F_PROFILE_OPS)) {
> +       if ((td->flags & TD_F_PROFILE_OPS)) {
>                  struct prof_io_ops *ops = &td->prof_io_ops;
>
>                  if (ops->get_next_file)

Hah indeed, good catch! I was wondering why it would not segfault, but I 
guess we fall through normally anyway. So it only impacts profiles that 
have get_next_file set, which makes it a minor issue.

Thanks for the patch! Applied.

-- 
Jens Axboe