* [PATCH] Remove a few UMSDOS leftovers
@ 2007-05-24 22:58 Jesper Juhl
2007-05-24 23:07 ` H. Peter Anvin
0 siblings, 1 reply; 5+ messages in thread
From: Jesper Juhl @ 2007-05-24 22:58 UTC (permalink / raw)
To: Linux Kernel Mailing List
Cc: Jacques Gelinas, Matija Nalis, Matija Nalis, trivial, Jesper Juhl
The UMSDOS filesystem was removed back in 2.6.11, but some tiny bits
stuck around. This patch removes the few leftovers.
The only things left behind after this are the entries in the CREDITS file.
Signed-off-by: Jesper Juhl <jesper.juhl@gmail.com>
---
Documentation/ioctl-number.txt | 1 -
arch/arm26/defconfig | 1 -
arch/cris/arch-v10/defconfig | 1 -
arch/um/config.release | 1 -
include/linux/ncp_fs.h | 2 --
5 files changed, 0 insertions(+), 6 deletions(-)
diff --git a/Documentation/ioctl-number.txt b/Documentation/ioctl-number.txt
index 3de7d37..f9f15bf 100644
--- a/Documentation/ioctl-number.txt
+++ b/Documentation/ioctl-number.txt
@@ -67,7 +67,6 @@ Code Seq# Include File Comments
0x00 00-1F linux/wavefront.h conflict!
0x02 all linux/fd.h
0x03 all linux/hdreg.h
-0x04 all linux/umsdos_fs.h
0x06 all linux/lp.h
0x09 all linux/md.h
0x12 all linux/fs.h
diff --git a/arch/arm26/defconfig b/arch/arm26/defconfig
index c4a8970..2b7d44b 100644
--- a/arch/arm26/defconfig
+++ b/arch/arm26/defconfig
@@ -248,7 +248,6 @@ CONFIG_I2C_CHARDEV=y
# CONFIG_JBD_DEBUG is not set
# CONFIG_FAT_FS is not set
# CONFIG_MSDOS_FS is not set
-# CONFIG_UMSDOS_FS is not set
# CONFIG_VFAT_FS is not set
# CONFIG_EFS_FS is not set
# CONFIG_JFFS_FS is not set
diff --git a/arch/cris/arch-v10/defconfig b/arch/cris/arch-v10/defconfig
index 2a3411e..710c20b 100644
--- a/arch/cris/arch-v10/defconfig
+++ b/arch/cris/arch-v10/defconfig
@@ -429,7 +429,6 @@ CONFIG_NET_ETHERNET=y
# CONFIG_BFS_FS is not set
# CONFIG_FAT_FS is not set
# CONFIG_MSDOS_FS is not set
-# CONFIG_UMSDOS_FS is not set
# CONFIG_VFAT_FS is not set
# CONFIG_EFS_FS is not set
# CONFIG_JFFS_FS is not set
diff --git a/arch/um/config.release b/arch/um/config.release
index fc68bcb..aba42f8 100644
--- a/arch/um/config.release
+++ b/arch/um/config.release
@@ -200,7 +200,6 @@ CONFIG_JBD=y
# CONFIG_JBD_DEBUG is not set
CONFIG_FAT_FS=y
CONFIG_MSDOS_FS=y
-CONFIG_UMSDOS_FS=y
CONFIG_VFAT_FS=y
CONFIG_EFS_FS=m
# CONFIG_JFFS_FS is not set
diff --git a/include/linux/ncp_fs.h b/include/linux/ncp_fs.h
index 83e39eb..88766e4 100644
--- a/include/linux/ncp_fs.h
+++ b/include/linux/ncp_fs.h
@@ -148,8 +148,6 @@ struct ncp_nls_ioctl
#include <linux/ncp_fs_i.h>
#include <linux/ncp_fs_sb.h>
-/* undef because public define in umsdos_fs.h (ncp_fs.h isn't public) */
-#undef PRINTK
/* define because it is easy to change PRINTK to {*}PRINTK */
#define PRINTK(format, args...) printk(KERN_DEBUG format , ## args)
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH] Remove a few UMSDOS leftovers
2007-05-24 22:58 [PATCH] Remove a few UMSDOS leftovers Jesper Juhl
@ 2007-05-24 23:07 ` H. Peter Anvin
2007-05-24 23:15 ` Jesper Juhl
0 siblings, 1 reply; 5+ messages in thread
From: H. Peter Anvin @ 2007-05-24 23:07 UTC (permalink / raw)
To: Jesper Juhl
Cc: Linux Kernel Mailing List, Jacques Gelinas, Matija Nalis,
Matija Nalis, trivial
Jesper Juhl wrote:
> The UMSDOS filesystem was removed back in 2.6.11, but some tiny bits
> stuck around. This patch removes the few leftovers.
> The only things left behind after this are the entries in the CREDITS file.
>
> Signed-off-by: Jesper Juhl <jesper.juhl@gmail.com>
> ---
>
> Documentation/ioctl-number.txt | 1 -
> arch/arm26/defconfig | 1 -
> arch/cris/arch-v10/defconfig | 1 -
> arch/um/config.release | 1 -
> include/linux/ncp_fs.h | 2 --
> 5 files changed, 0 insertions(+), 6 deletions(-)
>
> diff --git a/Documentation/ioctl-number.txt b/Documentation/ioctl-number.txt
> index 3de7d37..f9f15bf 100644
> --- a/Documentation/ioctl-number.txt
> +++ b/Documentation/ioctl-number.txt
> @@ -67,7 +67,6 @@ Code Seq# Include File Comments
> 0x00 00-1F linux/wavefront.h conflict!
> 0x02 all linux/fd.h
> 0x03 all linux/hdreg.h
> -0x04 all linux/umsdos_fs.h
> 0x06 all linux/lp.h
> 0x09 all linux/md.h
> 0x12 all linux/fs.h
NAK!
We should at least document which ioctl numbers have been burned, since
they SHOULD NOT be reused.
-hpa
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] Remove a few UMSDOS leftovers
2007-05-24 23:07 ` H. Peter Anvin
@ 2007-05-24 23:15 ` Jesper Juhl
2007-05-24 23:24 ` H. Peter Anvin
0 siblings, 1 reply; 5+ messages in thread
From: Jesper Juhl @ 2007-05-24 23:15 UTC (permalink / raw)
To: H. Peter Anvin
Cc: Linux Kernel Mailing List, Jacques Gelinas, Matija Nalis,
Matija Nalis, trivial
On Friday 25 May 2007 01:07:17 H. Peter Anvin wrote:
> Jesper Juhl wrote:
> > The UMSDOS filesystem was removed back in 2.6.11, but some tiny bits
> > stuck around. This patch removes the few leftovers.
> > The only things left behind after this are the entries in the CREDITS file.
> >
> > Signed-off-by: Jesper Juhl <jesper.juhl@gmail.com>
> > ---
[snip]
> > --- a/Documentation/ioctl-number.txt
> > +++ b/Documentation/ioctl-number.txt
> > @@ -67,7 +67,6 @@ Code Seq# Include File Comments
> > 0x00 00-1F linux/wavefront.h conflict!
> > 0x02 all linux/fd.h
> > 0x03 all linux/hdreg.h
> > -0x04 all linux/umsdos_fs.h
> > 0x06 all linux/lp.h
> > 0x09 all linux/md.h
> > 0x12 all linux/fs.h
>
> NAK!
>
> We should at least document which ioctl numbers have been burned, since
> they SHOULD NOT be reused.
>
Ok, that makes sense. How about this version :
The UMSDOS filesystem was removed back in 2.6.11, but some tiny bits
stuck around. This patch removes the few leftovers.
The only things left behind after this are the entries in the CREDITS file
and the ioctl number in Documentation/ioctl-number.txt as documentation.
Signed-off-by: Jesper Juhl <jesper.juhl@gmail.com>
---
Documentation/ioctl-number.txt | 2 +-
arch/arm26/defconfig | 1 -
arch/cris/arch-v10/defconfig | 1 -
arch/um/config.release | 1 -
include/linux/ncp_fs.h | 2 --
5 files changed, 1 insertions(+), 6 deletions(-)
diff --git a/Documentation/ioctl-number.txt b/Documentation/ioctl-number.txt
index 3de7d37..4413866 100644
--- a/Documentation/ioctl-number.txt
+++ b/Documentation/ioctl-number.txt
@@ -67,7 +67,7 @@ Code Seq# Include File Comments
0x00 00-1F linux/wavefront.h conflict!
0x02 all linux/fd.h
0x03 all linux/hdreg.h
-0x04 all linux/umsdos_fs.h
+0x04 all linux/umsdos_fs.h Dead, but don't reuse this ioctl number.
0x06 all linux/lp.h
0x09 all linux/md.h
0x12 all linux/fs.h
diff --git a/arch/arm26/defconfig b/arch/arm26/defconfig
index c4a8970..2b7d44b 100644
--- a/arch/arm26/defconfig
+++ b/arch/arm26/defconfig
@@ -248,7 +248,6 @@ CONFIG_I2C_CHARDEV=y
# CONFIG_JBD_DEBUG is not set
# CONFIG_FAT_FS is not set
# CONFIG_MSDOS_FS is not set
-# CONFIG_UMSDOS_FS is not set
# CONFIG_VFAT_FS is not set
# CONFIG_EFS_FS is not set
# CONFIG_JFFS_FS is not set
diff --git a/arch/cris/arch-v10/defconfig b/arch/cris/arch-v10/defconfig
index 2a3411e..710c20b 100644
--- a/arch/cris/arch-v10/defconfig
+++ b/arch/cris/arch-v10/defconfig
@@ -429,7 +429,6 @@ CONFIG_NET_ETHERNET=y
# CONFIG_BFS_FS is not set
# CONFIG_FAT_FS is not set
# CONFIG_MSDOS_FS is not set
-# CONFIG_UMSDOS_FS is not set
# CONFIG_VFAT_FS is not set
# CONFIG_EFS_FS is not set
# CONFIG_JFFS_FS is not set
diff --git a/arch/um/config.release b/arch/um/config.release
index fc68bcb..aba42f8 100644
--- a/arch/um/config.release
+++ b/arch/um/config.release
@@ -200,7 +200,6 @@ CONFIG_JBD=y
# CONFIG_JBD_DEBUG is not set
CONFIG_FAT_FS=y
CONFIG_MSDOS_FS=y
-CONFIG_UMSDOS_FS=y
CONFIG_VFAT_FS=y
CONFIG_EFS_FS=m
# CONFIG_JFFS_FS is not set
diff --git a/include/linux/ncp_fs.h b/include/linux/ncp_fs.h
index 83e39eb..88766e4 100644
--- a/include/linux/ncp_fs.h
+++ b/include/linux/ncp_fs.h
@@ -148,8 +148,6 @@ struct ncp_nls_ioctl
#include <linux/ncp_fs_i.h>
#include <linux/ncp_fs_sb.h>
-/* undef because public define in umsdos_fs.h (ncp_fs.h isn't public) */
-#undef PRINTK
/* define because it is easy to change PRINTK to {*}PRINTK */
#define PRINTK(format, args...) printk(KERN_DEBUG format , ## args)
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH] Remove a few UMSDOS leftovers
2007-05-24 23:15 ` Jesper Juhl
@ 2007-05-24 23:24 ` H. Peter Anvin
2007-05-24 23:27 ` Jesper Juhl
0 siblings, 1 reply; 5+ messages in thread
From: H. Peter Anvin @ 2007-05-24 23:24 UTC (permalink / raw)
To: Jesper Juhl
Cc: Linux Kernel Mailing List, Jacques Gelinas, Matija Nalis,
Matija Nalis, trivial
Jesper Juhl wrote:
> Ok, that makes sense. How about this version :
Could you document which numbers were actually used by umsdos, instead
of reserving a full block of 256? Since it's dead, it's not going to
eat any more numbers.
-hpa
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] Remove a few UMSDOS leftovers
2007-05-24 23:24 ` H. Peter Anvin
@ 2007-05-24 23:27 ` Jesper Juhl
0 siblings, 0 replies; 5+ messages in thread
From: Jesper Juhl @ 2007-05-24 23:27 UTC (permalink / raw)
To: H. Peter Anvin
Cc: Linux Kernel Mailing List, Jacques Gelinas, Matija Nalis,
Matija Nalis, trivial
On 25/05/07, H. Peter Anvin <hpa@zytor.com> wrote:
> Jesper Juhl wrote:
> > Ok, that makes sense. How about this version :
>
> Could you document which numbers were actually used by umsdos, instead
> of reserving a full block of 256? Since it's dead, it's not going to
> eat any more numbers.
>
Sure, I'll dig out that information and prepare a new patch tomorrow
(need to catch some sleep right now).
--
Jesper Juhl <jesper.juhl@gmail.com>
Don't top-post http://www.catb.org/~esr/jargon/html/T/top-post.html
Plain text mails only, please http://www.expita.com/nomime.html
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2007-05-24 23:27 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2007-05-24 22:58 [PATCH] Remove a few UMSDOS leftovers Jesper Juhl
2007-05-24 23:07 ` H. Peter Anvin
2007-05-24 23:15 ` Jesper Juhl
2007-05-24 23:24 ` H. Peter Anvin
2007-05-24 23:27 ` Jesper Juhl
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox