All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] ioctl_fat.2: correctly reference volume ID instead of volume label
@ 2017-02-08  1:13 Brian Masney
       [not found] ` <20170208011336.6578-1-masneyb-1iNe0GrtECGEi8DpZVb4nw@public.gmane.org>
  0 siblings, 1 reply; 2+ messages in thread
From: Brian Masney @ 2017-02-08  1:13 UTC (permalink / raw)
  To: mtk.manpages-Re5JQEeQqe8AvxtiuMwx3w
  Cc: linux-man-u79uwXL29TY76Z2rM5mHXA, masneyb-1iNe0GrtECGEi8DpZVb4nw

The description of FAT_IOCTL_GET_VOLUME_ID references volume name, when
it really should be volume ID since the volume label is different in the
FAT filesystem. This patch renames the incorrect volume name references
to volume id.

This man page supplies an example display_fat_volume_id.c program that
can be used to read the volume ID (serial number) from a filesystem.
Here is an additional test showing that the volume ID and volume name
are two different entities:

$ dd if=/dev/zero of=fat_volume bs=1M count=1
$ mkfs.fat -v -n MASNEYB fat_volume
[snip]
Volume ID is da8cecf2, volume label MASNEYB    .
$ sudo mount -o loop fat_volume /mnt
$ ./display_fat_volume_id /mnt/
Volume ID da8c-ecf2

Signed-off-by: Brian Masney <masneyb-1iNe0GrtECGEi8DpZVb4nw@public.gmane.org>
---
 man2/ioctl_fat.2 | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

diff --git a/man2/ioctl_fat.2 b/man2/ioctl_fat.2
index 1417b61..aa6e9f8 100644
--- a/man2/ioctl_fat.2
+++ b/man2/ioctl_fat.2
@@ -88,9 +88,9 @@ It is reset by an archiving system.
 The zero value
 .B ATTR_NONE
 can be used to indicate that no attribute bit is set.
-.SS Reading the volume label
-FAT filesystems are identified by a volume label.
-The volume label can be read with
+.SS Reading the volume ID
+FAT filesystems are identified by a volume ID.
+The volume ID can be read with
 .BR FAT_IOCTL_GET_VOLUME_ID .
 .PP
 The
@@ -106,7 +106,7 @@ flag.
 The
 .I id
 argument is a pointer to the field that will be filled with the volume ID.
-Typically the volume label is displayed to the user as a group of two
+Typically the volume ID is displayed to the user as a group of two
 16-bit fields:
 .PP
 .in +4n
@@ -347,10 +347,10 @@ main(int argc, char *argv[])
 }
 .fi
 .in
-.SS Reading the volume label
+.SS Reading the volume ID
 The following program demonstrates the use of
 .BR ioctl (2)
-to display the volume label of a FAT filesystem.
+to display the volume ID of a FAT filesystem.
 .PP
 The following output was recorded when applying the program for
 directory
-- 
2.9.3

--
To unsubscribe from this list: send the line "unsubscribe linux-man" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH] ioctl_fat.2: correctly reference volume ID instead of volume label
       [not found] ` <20170208011336.6578-1-masneyb-1iNe0GrtECGEi8DpZVb4nw@public.gmane.org>
@ 2017-02-08  8:30   ` Michael Kerrisk (man-pages)
  0 siblings, 0 replies; 2+ messages in thread
From: Michael Kerrisk (man-pages) @ 2017-02-08  8:30 UTC (permalink / raw)
  To: Brian Masney
  Cc: mtk.manpages-Re5JQEeQqe8AvxtiuMwx3w, linux-man-u79uwXL29TY76Z2rM5mHXA

Hello Brian,

On 02/08/2017 02:13 AM, Brian Masney wrote:
> The description of FAT_IOCTL_GET_VOLUME_ID references volume name, when
> it really should be volume ID since the volume label is different in the
> FAT filesystem. This patch renames the incorrect volume name references
> to volume id.
> 
> This man page supplies an example display_fat_volume_id.c program that
> can be used to read the volume ID (serial number) from a filesystem.
> Here is an additional test showing that the volume ID and volume name
> are two different entities:
> 
> $ dd if=/dev/zero of=fat_volume bs=1M count=1
> $ mkfs.fat -v -n MASNEYB fat_volume
> [snip]
> Volume ID is da8cecf2, volume label MASNEYB    .
> $ sudo mount -o loop fat_volume /mnt
> $ ./display_fat_volume_id /mnt/
> Volume ID da8c-ecf2

Thanks for the patch, and the omformative commit message! Applied.

Cheers,

Michael


> Signed-off-by: Brian Masney <masneyb-1iNe0GrtECGEi8DpZVb4nw@public.gmane.org>
> ---
>  man2/ioctl_fat.2 | 12 ++++++------
>  1 file changed, 6 insertions(+), 6 deletions(-)
> 
> diff --git a/man2/ioctl_fat.2 b/man2/ioctl_fat.2
> index 1417b61..aa6e9f8 100644
> --- a/man2/ioctl_fat.2
> +++ b/man2/ioctl_fat.2
> @@ -88,9 +88,9 @@ It is reset by an archiving system.
>  The zero value
>  .B ATTR_NONE
>  can be used to indicate that no attribute bit is set.
> -.SS Reading the volume label
> -FAT filesystems are identified by a volume label.
> -The volume label can be read with
> +.SS Reading the volume ID
> +FAT filesystems are identified by a volume ID.
> +The volume ID can be read with
>  .BR FAT_IOCTL_GET_VOLUME_ID .
>  .PP
>  The
> @@ -106,7 +106,7 @@ flag.
>  The
>  .I id
>  argument is a pointer to the field that will be filled with the volume ID.
> -Typically the volume label is displayed to the user as a group of two
> +Typically the volume ID is displayed to the user as a group of two
>  16-bit fields:
>  .PP
>  .in +4n
> @@ -347,10 +347,10 @@ main(int argc, char *argv[])
>  }
>  .fi
>  .in
> -.SS Reading the volume label
> +.SS Reading the volume ID
>  The following program demonstrates the use of
>  .BR ioctl (2)
> -to display the volume label of a FAT filesystem.
> +to display the volume ID of a FAT filesystem.
>  .PP
>  The following output was recorded when applying the program for
>  directory
> 


-- 
Michael Kerrisk
Linux man-pages maintainer; http://www.kernel.org/doc/man-pages/
Linux/UNIX System Programming Training: http://man7.org/training/
--
To unsubscribe from this list: send the line "unsubscribe linux-man" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2017-02-08  8:30 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-02-08  1:13 [PATCH] ioctl_fat.2: correctly reference volume ID instead of volume label Brian Masney
     [not found] ` <20170208011336.6578-1-masneyb-1iNe0GrtECGEi8DpZVb4nw@public.gmane.org>
2017-02-08  8:30   ` Michael Kerrisk (man-pages)

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.