* [PATCH 0/2] xfsdump: obsolete file type handling cleanups
@ 2020-10-06 22:07 Anthony Iliopoulos
2020-10-06 22:07 ` [PATCH 1/2] xfsdump: remove obsolete code for handling mountpoint inodes Anthony Iliopoulos
2020-10-06 22:07 ` [PATCH 2/2] xfsdump: remove obsolete code for handling xenix named pipes Anthony Iliopoulos
0 siblings, 2 replies; 5+ messages in thread
From: Anthony Iliopoulos @ 2020-10-06 22:07 UTC (permalink / raw)
To: Eric Sandeen; +Cc: linux-xfs
Just a couple of code and doc removals for file types that were never
supported in Linux (S_IFMNT, S_IFNAM).
Anthony Iliopoulos (2):
xfsdump: remove obsolete code for handling mountpoint inodes
xfsdump: remove obsolete code for handling xenix named pipes
doc/files.obj | 2 +-
doc/xfsdump.html | 6 ++----
dump/content.c | 6 ------
dump/inomap.c | 3 ---
restore/content.c | 8 --------
5 files changed, 3 insertions(+), 22 deletions(-)
--
2.28.0
^ permalink raw reply [flat|nested] 5+ messages in thread
* [PATCH 1/2] xfsdump: remove obsolete code for handling mountpoint inodes
2020-10-06 22:07 [PATCH 0/2] xfsdump: obsolete file type handling cleanups Anthony Iliopoulos
@ 2020-10-06 22:07 ` Anthony Iliopoulos
2020-10-06 22:09 ` Darrick J. Wong
2020-10-06 22:07 ` [PATCH 2/2] xfsdump: remove obsolete code for handling xenix named pipes Anthony Iliopoulos
1 sibling, 1 reply; 5+ messages in thread
From: Anthony Iliopoulos @ 2020-10-06 22:07 UTC (permalink / raw)
To: Eric Sandeen; +Cc: linux-xfs
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
^ permalink raw reply related [flat|nested] 5+ messages in thread
* [PATCH 2/2] xfsdump: remove obsolete code for handling xenix named pipes
2020-10-06 22:07 [PATCH 0/2] xfsdump: obsolete file type handling cleanups Anthony Iliopoulos
2020-10-06 22:07 ` [PATCH 1/2] xfsdump: remove obsolete code for handling mountpoint inodes Anthony Iliopoulos
@ 2020-10-06 22:07 ` Anthony Iliopoulos
2020-10-06 22:09 ` Darrick J. Wong
1 sibling, 1 reply; 5+ messages in thread
From: Anthony Iliopoulos @ 2020-10-06 22:07 UTC (permalink / raw)
To: Eric Sandeen; +Cc: linux-xfs
We can safely drop support for XENIX named pipes (S_IFNAM) at this
point, since this was never implemented in Linux.
Signed-off-by: Anthony Iliopoulos <ailiop@suse.com>
---
doc/files.obj | 2 +-
doc/xfsdump.html | 1 -
dump/content.c | 3 ---
dump/inomap.c | 3 ---
restore/content.c | 8 --------
5 files changed, 1 insertion(+), 16 deletions(-)
diff --git a/doc/files.obj b/doc/files.obj
index 4f4653ac56fc..098620e356da 100644
--- a/doc/files.obj
+++ b/doc/files.obj
@@ -295,7 +295,7 @@ minilines(486,15,0,0,0,0,0,[
mini_line(486,12,3,0,0,0,[
str_block(0,486,12,3,0,-4,0,0,0,[
str_seg('black','Courier',0,80640,486,12,3,0,-4,0,0,0,0,0,
- "Other File (S_IFCHAR|S_IFBLK|S_IFIFO|S_IFNAM|S_IFSOCK)")])
+ "Other File (S_IFCHAR|S_IFBLK|S_IFIFO|S_IFSOCK)")])
])
])]).
text('black',48,244,2,0,1,54,30,379,12,3,0,0,0,0,2,54,30,0,0,"",0,0,0,0,256,'',[
diff --git a/doc/xfsdump.html b/doc/xfsdump.html
index 9d06129a5e1d..958bc8055bef 100644
--- a/doc/xfsdump.html
+++ b/doc/xfsdump.html
@@ -100,7 +100,6 @@ or stdout. The dump includes all the filesystem objects of:
<li>character special files (S_IFCHR)
<li>block special files (S_IFBLK)
<li>named pipes (S_FIFO)
-<li>XENIX named pipes (S_IFNAM)
</ul>
It does not dump files from <i>/var/xfsdump</i> which is where the
xfsdump inventory is located.
diff --git a/dump/content.c b/dump/content.c
index 7637fe89609e..75b79220daf6 100644
--- a/dump/content.c
+++ b/dump/content.c
@@ -3883,9 +3883,6 @@ dump_file(void *arg1,
case S_IFCHR:
case S_IFBLK:
case S_IFIFO:
-#ifdef S_IFNAM
- case S_IFNAM:
-#endif
case S_IFLNK:
case S_IFSOCK:
/* only need a filehdr_t; no data
diff --git a/dump/inomap.c b/dump/inomap.c
index 85f76df606a9..85d61c353cf0 100644
--- a/dump/inomap.c
+++ b/dump/inomap.c
@@ -1723,9 +1723,6 @@ estimate_dump_space(struct xfs_bstat *statp)
case S_IFIFO:
case S_IFCHR:
case S_IFDIR:
-#ifdef S_IFNAM
- case S_IFNAM:
-#endif
case S_IFBLK:
case S_IFSOCK:
case S_IFLNK:
diff --git a/restore/content.c b/restore/content.c
index 6b22965bd894..97f821322960 100644
--- a/restore/content.c
+++ b/restore/content.c
@@ -7313,9 +7313,6 @@ restore_file_cb(void *cp, bool_t linkpr, char *path1, char *path2)
case S_IFBLK:
case S_IFCHR:
case S_IFIFO:
-#ifdef S_IFNAM
- case S_IFNAM:
-#endif
case S_IFSOCK:
ok = restore_spec(fhdrp, rvp, path1);
return ok;
@@ -7797,11 +7794,6 @@ restore_spec(filehdr_t *fhdrp, rv_t *rvp, char *path)
case S_IFIFO:
printstr = _("named pipe");
break;
-#ifdef S_IFNAM
- case S_IFNAM:
- printstr = _("XENIX named pipe");
- break;
-#endif
case S_IFSOCK:
printstr = _("UNIX domain socket");
break;
--
2.28.0
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH 2/2] xfsdump: remove obsolete code for handling xenix named pipes
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
0 siblings, 0 replies; 5+ messages in thread
From: Darrick J. Wong @ 2020-10-06 22:09 UTC (permalink / raw)
To: Anthony Iliopoulos; +Cc: Eric Sandeen, linux-xfs
On Wed, Oct 07, 2020 at 12:07:04AM +0200, Anthony Iliopoulos wrote:
> We can safely drop support for XENIX named pipes (S_IFNAM) at this
> point, since this was never implemented in Linux.
>
> Signed-off-by: Anthony Iliopoulos <ailiop@suse.com>
<giggle> :)
Reviewed-by: Darrick J. Wong <darrick.wong@oracle.com>
--D
> ---
> doc/files.obj | 2 +-
> doc/xfsdump.html | 1 -
> dump/content.c | 3 ---
> dump/inomap.c | 3 ---
> restore/content.c | 8 --------
> 5 files changed, 1 insertion(+), 16 deletions(-)
>
> diff --git a/doc/files.obj b/doc/files.obj
> index 4f4653ac56fc..098620e356da 100644
> --- a/doc/files.obj
> +++ b/doc/files.obj
> @@ -295,7 +295,7 @@ minilines(486,15,0,0,0,0,0,[
> mini_line(486,12,3,0,0,0,[
> str_block(0,486,12,3,0,-4,0,0,0,[
> str_seg('black','Courier',0,80640,486,12,3,0,-4,0,0,0,0,0,
> - "Other File (S_IFCHAR|S_IFBLK|S_IFIFO|S_IFNAM|S_IFSOCK)")])
> + "Other File (S_IFCHAR|S_IFBLK|S_IFIFO|S_IFSOCK)")])
> ])
> ])]).
> text('black',48,244,2,0,1,54,30,379,12,3,0,0,0,0,2,54,30,0,0,"",0,0,0,0,256,'',[
> diff --git a/doc/xfsdump.html b/doc/xfsdump.html
> index 9d06129a5e1d..958bc8055bef 100644
> --- a/doc/xfsdump.html
> +++ b/doc/xfsdump.html
> @@ -100,7 +100,6 @@ or stdout. The dump includes all the filesystem objects of:
> <li>character special files (S_IFCHR)
> <li>block special files (S_IFBLK)
> <li>named pipes (S_FIFO)
> -<li>XENIX named pipes (S_IFNAM)
> </ul>
> It does not dump files from <i>/var/xfsdump</i> which is where the
> xfsdump inventory is located.
> diff --git a/dump/content.c b/dump/content.c
> index 7637fe89609e..75b79220daf6 100644
> --- a/dump/content.c
> +++ b/dump/content.c
> @@ -3883,9 +3883,6 @@ dump_file(void *arg1,
> case S_IFCHR:
> case S_IFBLK:
> case S_IFIFO:
> -#ifdef S_IFNAM
> - case S_IFNAM:
> -#endif
> case S_IFLNK:
> case S_IFSOCK:
> /* only need a filehdr_t; no data
> diff --git a/dump/inomap.c b/dump/inomap.c
> index 85f76df606a9..85d61c353cf0 100644
> --- a/dump/inomap.c
> +++ b/dump/inomap.c
> @@ -1723,9 +1723,6 @@ estimate_dump_space(struct xfs_bstat *statp)
> case S_IFIFO:
> case S_IFCHR:
> case S_IFDIR:
> -#ifdef S_IFNAM
> - case S_IFNAM:
> -#endif
> case S_IFBLK:
> case S_IFSOCK:
> case S_IFLNK:
> diff --git a/restore/content.c b/restore/content.c
> index 6b22965bd894..97f821322960 100644
> --- a/restore/content.c
> +++ b/restore/content.c
> @@ -7313,9 +7313,6 @@ restore_file_cb(void *cp, bool_t linkpr, char *path1, char *path2)
> case S_IFBLK:
> case S_IFCHR:
> case S_IFIFO:
> -#ifdef S_IFNAM
> - case S_IFNAM:
> -#endif
> case S_IFSOCK:
> ok = restore_spec(fhdrp, rvp, path1);
> return ok;
> @@ -7797,11 +7794,6 @@ restore_spec(filehdr_t *fhdrp, rv_t *rvp, char *path)
> case S_IFIFO:
> printstr = _("named pipe");
> break;
> -#ifdef S_IFNAM
> - case S_IFNAM:
> - printstr = _("XENIX named pipe");
> - break;
> -#endif
> case S_IFSOCK:
> printstr = _("UNIX domain socket");
> break;
> --
> 2.28.0
>
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH 1/2] xfsdump: remove obsolete code for handling mountpoint inodes
2020-10-06 22:07 ` [PATCH 1/2] xfsdump: remove obsolete code for handling mountpoint inodes Anthony Iliopoulos
@ 2020-10-06 22:09 ` Darrick J. Wong
0 siblings, 0 replies; 5+ messages in thread
From: Darrick J. Wong @ 2020-10-06 22:09 UTC (permalink / raw)
To: Anthony Iliopoulos; +Cc: Eric Sandeen, linux-xfs
On Wed, Oct 07, 2020 at 12:07:03AM +0200, Anthony Iliopoulos wrote:
> The S_IFMNT file type was never supported in Linux, remove the last
> vestiges.
>
> Signed-off-by: Anthony Iliopoulos <ailiop@suse.com>
Looks good to me,
Reviewed-by: Darrick J. Wong <darrick.wong@oracle.com>
--D
> ---
> 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
>
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2020-10-06 22:10 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-10-06 22:07 [PATCH 0/2] xfsdump: obsolete file type handling cleanups Anthony Iliopoulos
2020-10-06 22:07 ` [PATCH 1/2] xfsdump: remove obsolete code for handling mountpoint inodes Anthony Iliopoulos
2020-10-06 22:09 ` 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
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).