Re: [PATCH] xfs_io: deprecate the "-F" foreign flag

[Eric Sandeen <sandeen@sandeen.net>]

On 3/8/12 3:39 PM, Mark Tinguely wrote:
> On 02/02/12 11:35, Eric Sandeen wrote:

...

>> @@ -282,10 +274,10 @@ open_f(
>>           return 0;
>>       }
>>
>> -    while ((c = getopt(argc, argv, "FRacdfm:nrstx")) != EOF) {
>> +    while ((c = getopt(argc, argv, "Racdfm:nrstx")) != EOF) {
>>           switch (c) {
>>           case 'F':
>> -            flags |= IO_FOREIGN;
>> +            /* Ignored / deprecated now, handled automatically */
>>               break;
> 
> 
> The "-F" is still in the command line open's help in open.
> 
> cxfs_io> help open
> open [-acdrstx] [path] -- open the file specified by path
> 
>  opens a new file in the requested mode
> 
>  Example:
>  'open -cd /tmp/data' - creates/opens data file read-write for direct IO
> 
>  Opens a file for subsequent use by all of the other xfs_io commands.
>  With no arguments, open uses the stat command to show the current file.
>  -F -- foreign filesystem file, disallow XFS-specific commands
>  -a -- open with the O_APPEND flag (append-only mode)
>  -d -- open with O_DIRECT (non-buffered IO, note alignment constraints)
>  -f -- open with O_CREAT (create the file if it doesn't exist)
>  -m -- permissions to use in case a new file is created (default 0600)
>  -n -- open with O_NONBLOCK
>  -r -- open with O_RDONLY, the default is O_RDWR
>  -s -- open with O_SYNC
>  -t -- open with O_TRUNC (truncate the file to zero length if it exists)
>  -R -- mark the file as a realtime XFS file immediately after opening it
>  Note1: usually read/write direct IO requests must be blocksize aligned;
>         some kernels, however, allow sectorsize alignment for direct IO.
>  Note2: the bmap for non-regular files can be obtained provided the file
>         was opened correctly (in particular, must be opened read-only).
> 
> 
> The "F" was taken out of the command line's getopt(), so it can't be
> specified. I guess either put the "F" back in the the open.c getopt()
> or take it complete from the command line.

Ah crud, sorry.  Patch shortly.

-Eric

> xfs_io> open -F somefile
> open: invalid option -- 'F'
> open [-acdrstx] [path] -- open the file specified by path
> 
> Thanks,
> 
> --Mark Tinguely
> tinguely@sgi.com
> 
> _______________________________________________
> xfs mailing list
> xfs@oss.sgi.com
> http://oss.sgi.com/mailman/listinfo/xfs
> 

_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs