All of lore.kernel.org
 help / color / mirror / Atom feed
From: Anthony Iliopoulos <ailiop@suse.com>
To: Eric Sandeen <sandeen@sandeen.net>
Cc: linux-xfs@vger.kernel.org
Subject: [PATCH 1/2] xfsdump: remove obsolete code for handling mountpoint inodes
Date: Wed,  7 Oct 2020 00:07:03 +0200	[thread overview]
Message-ID: <20201006220704.31157-2-ailiop@suse.com> (raw)
In-Reply-To: <20201006220704.31157-1-ailiop@suse.com>

The S_IFMNT file type was never supported in Linux, remove the last
vestiges.

Signed-off-by: Anthony Iliopoulos <ailiop@suse.com>
---
 doc/xfsdump.html | 5 ++---
 dump/content.c   | 3 ---
 2 files changed, 2 insertions(+), 6 deletions(-)

diff --git a/doc/xfsdump.html b/doc/xfsdump.html
index d4d157fa62c5..9d06129a5e1d 100644
--- a/doc/xfsdump.html
+++ b/doc/xfsdump.html
@@ -102,9 +102,8 @@ or stdout. The dump includes all the filesystem objects of:
 <li>named pipes (S_FIFO)
 <li>XENIX named pipes (S_IFNAM) 
 </ul>
-but not mount point types (S_IFMNT).
-It also does not dump files from <i>/var/xfsdump</i> which
-is where the xfsdump inventory is located.
+It does not dump files from <i>/var/xfsdump</i> which is where the
+xfsdump inventory is located.
 Other data which is stored:
 <ul>
 <li> file attributes (stored in stat data) of owner, group, permissions,
diff --git a/dump/content.c b/dump/content.c
index 30232d422206..7637fe89609e 100644
--- a/dump/content.c
+++ b/dump/content.c
@@ -3913,9 +3913,6 @@ dump_file(void *arg1,
 			contextp->cc_stat_lastino = statp->bs_ino;
 		}
 		return RV_OK;
-	/* not yet implemented
-	case S_IFMNT:
-	 */
 	}
 
 	if (rv == RV_OK
-- 
2.28.0


  reply	other threads:[~2020-10-06 22:06 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-10-06 22:07 [PATCH 0/2] xfsdump: obsolete file type handling cleanups Anthony Iliopoulos
2020-10-06 22:07 ` Anthony Iliopoulos [this message]
2020-10-06 22:09   ` [PATCH 1/2] xfsdump: remove obsolete code for handling mountpoint inodes Darrick J. Wong
2020-10-06 22:07 ` [PATCH 2/2] xfsdump: remove obsolete code for handling xenix named pipes Anthony Iliopoulos
2020-10-06 22:09   ` Darrick J. Wong

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20201006220704.31157-2-ailiop@suse.com \
    --to=ailiop@suse.com \
    --cc=linux-xfs@vger.kernel.org \
    --cc=sandeen@sandeen.net \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.