xfs
[Top] [All Lists]

Re: [PATCH] xfs_io: allow -F in open args

To: Eric Sandeen <sandeen@xxxxxxxxxx>
Subject: Re: [PATCH] xfs_io: allow -F in open args
From: Mark Tinguely <tinguely@xxxxxxx>
Date: Thu, 08 Mar 2012 16:39:52 -0600
Cc: xfs-oss <xfs@xxxxxxxxxxx>, Christoph Hellwig <hch@xxxxxxxxxxxxx>
In-reply-to: <4F5934BF.9060808@xxxxxxxxxx>
References: <4F2AC948.1050909@xxxxxxxxxx> <4F592728.9060706@xxxxxxx> <4F5930BC.7040106@xxxxxxxxxx> <4F593493.6010106@xxxxxxx> <4F5934BF.9060808@xxxxxxxxxx>
User-agent: Mozilla/5.0 (X11; FreeBSD amd64; rv:9.0) Gecko/20120122 Thunderbird/9.0
On 03/08/12 16:37, Eric Sandeen wrote:
On 3/8/12 4:37 PM, Mark Tinguely wrote:
On 03/08/12 16:20, Eric Sandeen wrote:
Now that -F ("foreign") is automagic, we should no longer list
it in the help output for open, but we should still accept
it for compatibility; esp. as it is still in the case statement.
Oops.

Reported-by: Mark Tinguely<tinguely@xxxxxxx>
Signed-off-by: Eric Sandeen<sandeen@xxxxxxxxxx>
---

diff --git a/io/open.c b/io/open.c
index ac39ccc..46538ba 100644
--- a/io/open.c
+++ b/io/open.c
@@ -240,7 +240,6 @@ open_help(void)
   "\n"
   " Opens a file for subsequent use by all of the other xfs_io commands.\n"
   " With no arguments, open uses the stat command to show the current file.\n"
-" -F -- foreign filesystem file, disallow XFS-specific commands\n"
   " -a -- open with the O_APPEND flag (append-only mode)\n"
   " -d -- open with O_DIRECT (non-buffered IO, note alignment constraints)\n"
   " -f -- open with O_CREAT (create the file if it doesn't exist)\n"
@@ -274,7 +273,7 @@ open_f(
           return 0;
       }

-    while ((c = getopt(argc, argv, "Racdfm:nrstx")) != EOF) {
+    while ((c = getopt(argc, argv, "FRacdfm:nrstx")) != EOF) {
           switch (c) {
           case 'F':
               /* Ignored / deprecated now, handled automatically */



Thank-you. Probably could take it out of the man page under the open I/O 
command now too.

Gah.  With your blessing I'll commit that as well?

Reviewed-by: Mark Tinguely




Yes, and I should have saw it before too. Good idea to remove the flag.

--Mark Tinguely

<Prev in Thread] Current Thread [Next in Thread>