Re: [PATCH v2] xfs_io: support -c "open foo" command

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

 



On 12/5/16 7:03 AM, Amir Goldstein wrote:
> There is an undocumented and possibly unused feature in xfs_io
> where all commands are executed per file in the file args list.
> 
> This feature creates ambiguity when trying to execute commands
> such as "open" and "file" from command line and result in an
> endless loop in the command loop code.

I may well be dense, but it's not immediately obvious what the
old and/or problematic behavior is.  Can you spell it out a bit
more?  What is an example of this loop?
 
> Also, when running xfs_io -c <cmd> without any file args, xfs_io
> exits without doing anything. This behavior is also undocumented
> and makes very little sense.

What /should/ it do if there is no file specified?  Is this just for
cmds that don't require a file, such as "-c sync?"

> Change the behavior of xfs_io to execute each command specified
> with -c <cmd> exactly once, regardless of the number of files in
> the file args list.

So then which file does it act on?  I'm not saying this is incorrect,
I'm just not quite understanding what this fixes, and what the new
behavior is, sorry.

Thanks,
-Eric

> This enables writing proper xfs_io scripts in command line, which
> include "open" and "file" commands.
> 
> This change does not modify the behavior of xfs_io in the most
> commonly used case of single file argument in command line and
> no "open" and "file" commands in command line.
> 
> Signed-off-by: Amir Goldstein <amir73il@xxxxxxxxx>
> ---
>  io/init.c | 9 +++++----
>  1 file changed, 5 insertions(+), 4 deletions(-)
> 
> v2:
> - Fix the case of multiple file args
> 
> v1:
> - Fix the case of zero file args
> 
> diff --git a/io/init.c b/io/init.c
> index a9191cf..82b6700 100644
> --- a/io/init.c
> +++ b/io/init.c
> @@ -90,14 +90,15 @@ init_commands(void)
>  	cowextsize_init();
>  }
>  
> +/*
> + * Return true for first call and false for the next call,
> + * to execute each command once.
> + */
>  static int
>  init_args_command(
>  	int	index)
>  {
> -	if (index >= filecount)
> -		return 0;
> -	file = &filetable[index++];
> -	return index;
> +	return !index;
>  }
>  
>  static int
> 
--
To unsubscribe from this list: send the line "unsubscribe linux-xfs" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [XFS Filesystem Development (older mail)]     [Linux Filesystem Development]     [Linux Audio Users]     [Yosemite Trails]     [Linux Kernel]     [Linux RAID]     [Linux SCSI]


  Powered by Linux