[PATCH 00/15] Remove asm/segment.h from low hanging architectures

[PATCH 00/15] Remove asm/segment.h from low hanging architectures

[Kumar Gala]

The following set of patches removes the use and existence of 
asm/segment.h from the architecture ports that it was fairly trivial to do 
so.  I need to work with the arch maintainers on the following 
architectures since they use asm/segment.h heavily:

m32r
um
frv
h8300
i386
m68knommu
m68k
v850
x86_64

If any arch maintainers want to go a head kill asm/segment.h in that last 
I would be very appreciative.

Thanks

- kumar

[PATCH 01/15] alpha: remove use of asm/segment.h

[Kumar Gala]

Removed Alpha architecture specific users of asm/segment.h and
asm-alpha/segment.h itself

Signed-off-by: Kumar Gala <kumar.gala@freescale.com>

---
commit 19150fe1698293dc2c4f4f48c6b05d83595544e6
tree 9c45366378a1c34d755da01efc162bd7d80d48ab
parent acda3853951d4b9dbe97dffbefd6d2a4fd9d3df0
author Kumar K. Gala <kumar.gala@freescale.com> Wed, 24 Aug 2005 10:33:40 -0500
committer Kumar K. Gala <kumar.gala@freescale.com> Wed, 24 Aug 2005 10:33:40 -0500

 arch/alpha/kernel/smc37c669.c |    1 -
 arch/alpha/kernel/smc37c93x.c |    1 -
 include/asm-alpha/segment.h   |    6 ------
 3 files changed, 0 insertions(+), 8 deletions(-)

diff --git a/arch/alpha/kernel/smc37c669.c b/arch/alpha/kernel/smc37c669.c
--- a/arch/alpha/kernel/smc37c669.c
+++ b/arch/alpha/kernel/smc37c669.c
@@ -11,7 +11,6 @@
 
 #include <asm/hwrpb.h>
 #include <asm/io.h>
-#include <asm/segment.h>
 
 #if 0
 # define DBG_DEVS(args)         printk args
diff --git a/arch/alpha/kernel/smc37c93x.c b/arch/alpha/kernel/smc37c93x.c
--- a/arch/alpha/kernel/smc37c93x.c
+++ b/arch/alpha/kernel/smc37c93x.c
@@ -12,7 +12,6 @@
 
 #include <asm/hwrpb.h>
 #include <asm/io.h>
-#include <asm/segment.h>
 
 #define SMC_DEBUG 0
 
diff --git a/include/asm-alpha/segment.h b/include/asm-alpha/segment.h
deleted file mode 100644
--- a/include/asm-alpha/segment.h
+++ /dev/null
@@ -1,6 +0,0 @@
-#ifndef __ALPHA_SEGMENT_H
-#define __ALPHA_SEGMENT_H
-
-/* Only here because we have some old header files that expect it.. */
-
-#endif

[PATCH 02/15] arm26: remove use of asm/segment.h

[Kumar Gala]

Removed ARM26 architecture specific users of asm/segment.h and
asm-alpha/segment.h itself.

Signed-off-by: Kumar Gala <kumar.gala@freescale.com>

---
commit 1d8c8bf789b6925529d84f3ad787cf973763793f
tree 8b009fca0a2f15ac71e72f09d16330990609c4bc
parent 19150fe1698293dc2c4f4f48c6b05d83595544e6
author Kumar K. Gala <kumar.gala@freescale.com> Wed, 24 Aug 2005 10:40:11 -0500
committer Kumar K. Gala <kumar.gala@freescale.com> Wed, 24 Aug 2005 10:40:11 -0500

 arch/arm26/mm/init.c        |    1 -
 include/asm-arm26/segment.h |   11 -----------
 2 files changed, 0 insertions(+), 12 deletions(-)

diff --git a/arch/arm26/mm/init.c b/arch/arm26/mm/init.c
--- a/arch/arm26/mm/init.c
+++ b/arch/arm26/mm/init.c
@@ -24,7 +24,6 @@
 #include <linux/bootmem.h>
 #include <linux/blkdev.h>
 
-#include <asm/segment.h>
 #include <asm/mach-types.h>
 #include <asm/dma.h>
 #include <asm/hardware.h>
diff --git a/include/asm-arm26/segment.h b/include/asm-arm26/segment.h
deleted file mode 100644
--- a/include/asm-arm26/segment.h
+++ /dev/null
@@ -1,11 +0,0 @@
-#ifndef __ASM_ARM_SEGMENT_H
-#define __ASM_ARM_SEGMENT_H
-
-#define __KERNEL_CS   0x0
-#define __KERNEL_DS   0x0
-
-#define __USER_CS     0x1
-#define __USER_DS     0x1
-
-#endif /* __ASM_ARM_SEGMENT_H */
-

[PATCH 03/15] arm: remove use of asm/segment.h

[Kumar Gala]

Removed asm-arm/segment.h as its not used by anything.

Signed-off-by: Kumar Gala <kumar.gala@freescale.com>

---
commit 13e9876eff93029eb24affd8842db94035c370f5
tree acb6c189dd89aef958961eedba5641449865c5aa
parent 1d8c8bf789b6925529d84f3ad787cf973763793f
author Kumar K. Gala <kumar.gala@freescale.com> Wed, 24 Aug 2005 10:42:30 -0500
committer Kumar K. Gala <kumar.gala@freescale.com> Wed, 24 Aug 2005 10:42:30 -0500

 include/asm-arm/segment.h |   11 -----------
 1 files changed, 0 insertions(+), 11 deletions(-)

diff --git a/include/asm-arm/segment.h b/include/asm-arm/segment.h
deleted file mode 100644
--- a/include/asm-arm/segment.h
+++ /dev/null
@@ -1,11 +0,0 @@
-#ifndef __ASM_ARM_SEGMENT_H
-#define __ASM_ARM_SEGMENT_H
-
-#define __KERNEL_CS   0x0
-#define __KERNEL_DS   0x0
-
-#define __USER_CS     0x1
-#define __USER_DS     0x1
-
-#endif /* __ASM_ARM_SEGMENT_H */
-

[PATCH 04/15] cris: remove use of asm/segment.h

[Kumar Gala]

Removed cris architecture specific users of asm/segment.h and
asm-cris/segment.h itself

Signed-off-by: Kumar Gala <kumar.gala@freescale.com>

---
commit 352a43ca6e4cb29ca7ee1742a00f3b6d98465a0d
tree b3816f312ed0b9206e344ceb62bb4f82a5c469e6
parent 13e9876eff93029eb24affd8842db94035c370f5
author Kumar K. Gala <kumar.gala@freescale.com> Wed, 24 Aug 2005 10:47:26 -0500
committer Kumar K. Gala <kumar.gala@freescale.com> Wed, 24 Aug 2005 10:47:26 -0500

 arch/cris/arch-v10/kernel/fasttimer.c |    1 -
 arch/cris/kernel/sys_cris.c           |    1 -
 include/asm-cris/processor.h          |    4 ++++
 include/asm-cris/segment.h            |    8 --------
 include/asm-cris/thread_info.h        |    1 -
 5 files changed, 4 insertions(+), 11 deletions(-)

diff --git a/arch/cris/arch-v10/kernel/fasttimer.c b/arch/cris/arch-v10/kernel/fasttimer.c
--- a/arch/cris/arch-v10/kernel/fasttimer.c
+++ b/arch/cris/arch-v10/kernel/fasttimer.c
@@ -105,7 +105,6 @@
 #include <linux/time.h>
 #include <linux/delay.h>
 
-#include <asm/segment.h>
 #include <asm/io.h>
 #include <asm/irq.h>
 #include <asm/delay.h>
diff --git a/arch/cris/kernel/sys_cris.c b/arch/cris/kernel/sys_cris.c
--- a/arch/cris/kernel/sys_cris.c
+++ b/arch/cris/kernel/sys_cris.c
@@ -24,7 +24,6 @@
 
 #include <asm/uaccess.h>
 #include <asm/ipc.h>
-#include <asm/segment.h>
 
 /*
  * sys_pipe() is the normal C calling standard for creating
diff --git a/include/asm-cris/processor.h b/include/asm-cris/processor.h
--- a/include/asm-cris/processor.h
+++ b/include/asm-cris/processor.h
@@ -21,6 +21,10 @@
  */
 #define TASK_UNMAPPED_BASE      (PAGE_ALIGN(TASK_SIZE / 3))
 
+typedef struct {
+	unsigned long seg;
+} mm_segment_t;
+
 /* THREAD_SIZE is the size of the task_struct/kernel_stack combo.
  * normally, the stack is found by doing something like p + THREAD_SIZE
  * in CRIS, a page is 8192 bytes, which seems like a sane size
diff --git a/include/asm-cris/segment.h b/include/asm-cris/segment.h
deleted file mode 100644
--- a/include/asm-cris/segment.h
+++ /dev/null
@@ -1,8 +0,0 @@
-#ifndef _ASM_SEGMENT_H
-#define _ASM_SEGMENT_H
-
-typedef struct {
-  unsigned long seg;
-} mm_segment_t;
-
-#endif
diff --git a/include/asm-cris/thread_info.h b/include/asm-cris/thread_info.h
--- a/include/asm-cris/thread_info.h
+++ b/include/asm-cris/thread_info.h
@@ -15,7 +15,6 @@
 #include <asm/types.h>
 #include <asm/processor.h>
 #include <asm/arch/thread_info.h>
-#include <asm/segment.h>
 #endif
 
 

[PATCH 05/15] ia64: remove use of asm/segment.h

[Kumar Gala]

Removed IA64 architecture specific users of asm/segment.h and
asm-ia64/segment.h itself

Signed-off-by: Kumar Gala <kumar.gala@freescale.com>

---
commit 53cbc8f4b0d47965e2d673bcc9dc5e6a8388350b
tree d05c25c406d023dce7c797ede8119b8ab68767e5
parent 352a43ca6e4cb29ca7ee1742a00f3b6d98465a0d
author Kumar K. Gala <kumar.gala@freescale.com> Wed, 24 Aug 2005 10:54:23 -0500
committer Kumar K. Gala <kumar.gala@freescale.com> Wed, 24 Aug 2005 10:54:23 -0500

 arch/ia64/ia32/ia32_signal.c        |    1 -
 arch/ia64/pci/pci.c                 |    1 -
 arch/ia64/sn/kernel/sn2/sn_hwperf.c |    1 -
 include/asm-ia64/segment.h          |    6 ------
 4 files changed, 0 insertions(+), 9 deletions(-)

diff --git a/arch/ia64/ia32/ia32_signal.c b/arch/ia64/ia32/ia32_signal.c
--- a/arch/ia64/ia32/ia32_signal.c
+++ b/arch/ia64/ia32/ia32_signal.c
@@ -29,7 +29,6 @@
 #include <asm/uaccess.h>
 #include <asm/rse.h>
 #include <asm/sigcontext.h>
-#include <asm/segment.h>
 
 #include "ia32priv.h"
 
diff --git a/arch/ia64/pci/pci.c b/arch/ia64/pci/pci.c
--- a/arch/ia64/pci/pci.c
+++ b/arch/ia64/pci/pci.c
@@ -24,7 +24,6 @@
 
 #include <asm/machvec.h>
 #include <asm/page.h>
-#include <asm/segment.h>
 #include <asm/system.h>
 #include <asm/io.h>
 #include <asm/sal.h>
diff --git a/arch/ia64/sn/kernel/sn2/sn_hwperf.c b/arch/ia64/sn/kernel/sn2/sn_hwperf.c
--- a/arch/ia64/sn/kernel/sn2/sn_hwperf.c
+++ b/arch/ia64/sn/kernel/sn2/sn_hwperf.c
@@ -36,7 +36,6 @@
 #include <asm/topology.h>
 #include <asm/smp.h>
 #include <asm/semaphore.h>
-#include <asm/segment.h>
 #include <asm/uaccess.h>
 #include <asm/sal.h>
 #include <asm/sn/io.h>
diff --git a/include/asm-ia64/segment.h b/include/asm-ia64/segment.h
deleted file mode 100644
--- a/include/asm-ia64/segment.h
+++ /dev/null
@@ -1,6 +0,0 @@
-#ifndef _ASM_IA64_SEGMENT_H
-#define _ASM_IA64_SEGMENT_H
-
-/* Only here because we have some old header files that expect it.. */
-
-#endif /* _ASM_IA64_SEGMENT_H */

Re: [PATCH 05/15] ia64: remove use of asm/segment.h

[Bjorn Helgaas]

On Wednesday 24 August 2005 10:53 am, Kumar Gala wrote:
> Removed IA64 architecture specific users of asm/segment.h and
> asm-ia64/segment.h itself

I posted a similar patch a month ago, but I only removed the
arch/ia64 includes of asm/segment.h.

There are still a few drivers that include asm/segment.h, so
I don't think we should remove asm/segment.h itself just yet.

Re: [PATCH 05/15] ia64: remove use of asm/segment.h

[Ralf Baechle]

On Wed, Aug 24, 2005 at 02:09:55PM -0600, Bjorn Helgaas wrote:

> On Wednesday 24 August 2005 10:53 am, Kumar Gala wrote:
> > Removed IA64 architecture specific users of asm/segment.h and
> > asm-ia64/segment.h itself
> 
> I posted a similar patch a month ago, but I only removed the
> arch/ia64 includes of asm/segment.h.
> 
> There are still a few drivers that include asm/segment.h, so
> I don't think we should remove asm/segment.h itself just yet.

<asm/segment.h> was replaced by <asm/uaccess.h> in 2.1.4 and we still have
references ...

  Ralf

[PATCH 06/15] mips: remove use of asm/segment.h

[Kumar Gala]

Removed MIPS architecture specific users of asm/segment.h and
asm-mips/segment.h itself

Signed-off-by: Kumar Gala <kumar.gala@freescale.com>

---
commit 8f20e153d5d5c3efd95835e814fae7b3ccbfcd08
tree 17e9ae88b3fe1762302179a4ea08e61360805a29
parent 503a812c1f9cef08e6f96b2b2cf1f32bbfef2bc6
author Kumar K. Gala <kumar.gala@freescale.com> Wed, 24 Aug 2005 10:59:09 -0500
committer Kumar K. Gala <kumar.gala@freescale.com> Wed, 24 Aug 2005 10:59:09 -0500

 arch/mips/au1000/db1x00/mirage_ts.c |    1 -
 include/asm-mips/segment.h          |    6 ------
 2 files changed, 0 insertions(+), 7 deletions(-)

diff --git a/arch/mips/au1000/db1x00/mirage_ts.c b/arch/mips/au1000/db1x00/mirage_ts.c
--- a/arch/mips/au1000/db1x00/mirage_ts.c
+++ b/arch/mips/au1000/db1x00/mirage_ts.c
@@ -44,7 +44,6 @@
 #include <linux/smp_lock.h>
 #include <linux/wait.h>
 
-#include <asm/segment.h>
 #include <asm/irq.h>
 #include <asm/uaccess.h>
 #include <asm/delay.h>
diff --git a/include/asm-mips/segment.h b/include/asm-mips/segment.h
deleted file mode 100644
--- a/include/asm-mips/segment.h
+++ /dev/null
@@ -1,6 +0,0 @@
-#ifndef _ASM_SEGMENT_H
-#define _ASM_SEGMENT_H
-
-/* Only here because we have some old header files that expect it.. */
-
-#endif /* _ASM_SEGMENT_H */

Re: [PATCH 06/15] mips: remove use of asm/segment.h

[Ralf Baechle]

On Wed, Aug 24, 2005 at 11:54:27AM -0500, Kumar Gala wrote:

> Removed MIPS architecture specific users of asm/segment.h and
> asm-mips/segment.h itself

Thanks, applied.

  Ralf

[PATCH 07/15] parisc: remove use of asm/segment.h

[Kumar Gala]

Removed asm-parisc/segment.h as its not used by anything.

Signed-off-by: Kumar Gala <kumar.gala@freescale.com>

---
commit e2a2b88b3c3d60b07be7908579bb772e6bd4cd01
tree f87e38b7cfa643825a937c2d4a65a042ebfe263d
parent 18c435f8b99db307732a02a07bf2099ad64174db
author Kumar K. Gala <kumar.gala@freescale.com> Wed, 24 Aug 2005 11:32:02 -0500
committer Kumar K. Gala <kumar.gala@freescale.com> Wed, 24 Aug 2005 11:32:02 -0500

 include/asm-parisc/segment.h |    6 ------
 1 files changed, 0 insertions(+), 6 deletions(-)

diff --git a/include/asm-parisc/segment.h b/include/asm-parisc/segment.h
deleted file mode 100644
--- a/include/asm-parisc/segment.h
+++ /dev/null
@@ -1,6 +0,0 @@
-#ifndef __PARISC_SEGMENT_H
-#define __PARISC_SEGMENT_H
-
-/* Only here because we have some old header files that expect it.. */
-
-#endif

Re: [parisc-linux] [PATCH 07/15] parisc: remove use of asm/segment.h

[Matthew Wilcox]

On Wed, Aug 24, 2005 at 11:55:30AM -0500, Kumar Gala wrote:
> Removed asm-parisc/segment.h as its not used by anything.

Did you already remove all the uses outside the parisc-specific bits of
the tree, eg ISDN, media/video/, sound/oss/, etc?

If so, ACK, otherwise, NAK.

-- 
"Next the statesmen will invent cheap lies, putting the blame upon 
the nation that is attacked, and every man will be glad of those
conscience-soothing falsities, and will diligently study them, and refuse
to examine any refutations of them; and thus he will by and by convince 
himself that the war is just, and will thank God for the better sleep 
he enjoys after this process of grotesque self-deception." -- Mark Twain

Re: [parisc-linux] [PATCH 07/15] parisc: remove use of asm/segment.h

[Kumar Gala]

On Aug 24, 2005, at 12:14 PM, Matthew Wilcox wrote:

> On Wed, Aug 24, 2005 at 11:55:30AM -0500, Kumar Gala wrote:
>
>> Removed asm-parisc/segment.h as its not used by anything.
>>
>
> Did you already remove all the uses outside the parisc-specific  
> bits of
> the tree, eg ISDN, media/video/, sound/oss/, etc?
>
> If so, ACK, otherwise, NAK.

I did remove them.  I should have included a reference to the patch  
that did this:

http://www.ussg.iu.edu/hypermail/linux/kernel/0508.3/0099.html

- kumar

[PATCH 08/15] ppc32: remove use of asm/segment.h

[Kumar Gala]

Removed ppc32 architecture specific users of asm/segment.h and
asm-ppc/segment.h itself

Signed-off-by: Kumar Gala <kumar.gala@freescale.com>

---
commit 503a812c1f9cef08e6f96b2b2cf1f32bbfef2bc6
tree b61a93eb582e90fafda6ff9d064b2ab25e7d3ede
parent 21a31cb366a764793dc532b525b95bfc3e1723a2
author Kumar K. Gala <kumar.gala@freescale.com> Wed, 24 Aug 2005 10:58:02 -0500
committer Kumar K. Gala <kumar.gala@freescale.com> Wed, 24 Aug 2005 10:58:02 -0500

 arch/ppc/kernel/temp.c         |    1 -
 arch/ppc/kernel/time.c         |    1 -
 arch/ppc/platforms/chrp_time.c |    1 -
 arch/ppc/syslib/prep_nvram.c   |    1 -
 include/asm-ppc/segment.h      |    1 -
 5 files changed, 0 insertions(+), 5 deletions(-)

diff --git a/arch/ppc/kernel/temp.c b/arch/ppc/kernel/temp.c
--- a/arch/ppc/kernel/temp.c
+++ b/arch/ppc/kernel/temp.c
@@ -21,7 +21,6 @@
 #include <linux/interrupt.h>
 #include <linux/init.h>
 
-#include <asm/segment.h>
 #include <asm/io.h>
 #include <asm/reg.h>
 #include <asm/nvram.h>
diff --git a/arch/ppc/kernel/time.c b/arch/ppc/kernel/time.c
--- a/arch/ppc/kernel/time.c
+++ b/arch/ppc/kernel/time.c
@@ -58,7 +58,6 @@
 #include <linux/init.h>
 #include <linux/profile.h>
 
-#include <asm/segment.h>
 #include <asm/io.h>
 #include <asm/nvram.h>
 #include <asm/cache.h>
diff --git a/arch/ppc/platforms/chrp_time.c b/arch/ppc/platforms/chrp_time.c
--- a/arch/ppc/platforms/chrp_time.c
+++ b/arch/ppc/platforms/chrp_time.c
@@ -22,7 +22,6 @@
 #include <linux/init.h>
 #include <linux/bcd.h>
 
-#include <asm/segment.h>
 #include <asm/io.h>
 #include <asm/nvram.h>
 #include <asm/prom.h>
diff --git a/arch/ppc/syslib/prep_nvram.c b/arch/ppc/syslib/prep_nvram.c
--- a/arch/ppc/syslib/prep_nvram.c
+++ b/arch/ppc/syslib/prep_nvram.c
@@ -15,7 +15,6 @@
 #include <linux/ioport.h>
 
 #include <asm/sections.h>
-#include <asm/segment.h>
 #include <asm/io.h>
 #include <asm/machdep.h>
 #include <asm/prep_nvram.h>
diff --git a/include/asm-ppc/segment.h b/include/asm-ppc/segment.h
deleted file mode 100644
--- a/include/asm-ppc/segment.h
+++ /dev/null
@@ -1 +0,0 @@
-#include <asm/uaccess.h>

[PATCH 09/15] ppc64: remove use of asm/segment.h

[Kumar Gala]

Removed PPC64 architecture specific users of asm/segment.h and
asm-ppc64/segment.h itself

Signed-off-by: Kumar Gala <kumar.gala@freescale.com>

---
commit 21a31cb366a764793dc532b525b95bfc3e1723a2
tree 2c40ec80e781b61fef42e5914d8f336dc72f2200
parent 53cbc8f4b0d47965e2d673bcc9dc5e6a8388350b
author Kumar K. Gala <kumar.gala@freescale.com> Wed, 24 Aug 2005 10:55:55 -0500
committer Kumar K. Gala <kumar.gala@freescale.com> Wed, 24 Aug 2005 10:55:55 -0500

 arch/ppc64/kernel/time.c    |    1 -
 include/asm-ppc64/segment.h |    6 ------
 2 files changed, 0 insertions(+), 7 deletions(-)

diff --git a/arch/ppc64/kernel/time.c b/arch/ppc64/kernel/time.c
--- a/arch/ppc64/kernel/time.c
+++ b/arch/ppc64/kernel/time.c
@@ -51,7 +51,6 @@
 #include <linux/cpu.h>
 #include <linux/security.h>
 
-#include <asm/segment.h>
 #include <asm/io.h>
 #include <asm/processor.h>
 #include <asm/nvram.h>
diff --git a/include/asm-ppc64/segment.h b/include/asm-ppc64/segment.h
deleted file mode 100644
--- a/include/asm-ppc64/segment.h
+++ /dev/null
@@ -1,6 +0,0 @@
-#ifndef __PPC64_SEGMENT_H
-#define __PPC64_SEGMENT_H
-
-/* Only here because we have some old header files that expect it.. */
-
-#endif /* __PPC64_SEGMENT_H */

[PATCH 10/15] s390: remove use of asm/segment.h

[Kumar Gala]

Removed S390 architecture specific users of asm/segment.h and
asm-s390/segment.h itself

Signed-off-by: Kumar Gala <kumar.gala@freescale.com>

---
commit 6ee9ded730a875d63e42add1c3094eca7d5a6cdf
tree 5d340a409fd593e5f5d7e71467a038fd90def51b
parent 8f20e153d5d5c3efd95835e814fae7b3ccbfcd08
author Kumar K. Gala <kumar.gala@freescale.com> Wed, 24 Aug 2005 11:01:20 -0500
committer Kumar K. Gala <kumar.gala@freescale.com> Wed, 24 Aug 2005 11:01:20 -0500

 arch/s390/kernel/ptrace.c  |    1 -
 include/asm-s390/segment.h |    4 ----
 2 files changed, 0 insertions(+), 5 deletions(-)

diff --git a/arch/s390/kernel/ptrace.c b/arch/s390/kernel/ptrace.c
--- a/arch/s390/kernel/ptrace.c
+++ b/arch/s390/kernel/ptrace.c
@@ -34,7 +34,6 @@
 #include <linux/audit.h>
 #include <linux/signal.h>
 
-#include <asm/segment.h>
 #include <asm/page.h>
 #include <asm/pgtable.h>
 #include <asm/pgalloc.h>
diff --git a/include/asm-s390/segment.h b/include/asm-s390/segment.h
deleted file mode 100644
--- a/include/asm-s390/segment.h
+++ /dev/null
@@ -1,4 +0,0 @@
-#ifndef _ASM_SEGMENT_H
-#define _ASM_SEGMENT_H
-
-#endif

[PATCH 11/15] sh64: remove use of asm/segment.h

[Kumar Gala]

Removed asm-sh64/segment.h as its not used by anything.

Signed-off-by: Kumar Gala <kumar.gala@freescale.com>

---
commit 36b349eaba7a399c06219d3553d571261dbf6333
tree 5ac502496f084585407bf5d8fd9040a489525edf
parent ab81566ace65ac748c885d696c11b5ae839fb328
author Kumar K. Gala <kumar.gala@freescale.com> Wed, 24 Aug 2005 11:33:32 -0500
committer Kumar K. Gala <kumar.gala@freescale.com> Wed, 24 Aug 2005 11:33:32 -0500

 include/asm-sh64/segment.h |    6 ------
 1 files changed, 0 insertions(+), 6 deletions(-)

diff --git a/include/asm-sh64/segment.h b/include/asm-sh64/segment.h
deleted file mode 100644
--- a/include/asm-sh64/segment.h
+++ /dev/null
@@ -1,6 +0,0 @@
-#ifndef _ASM_SEGMENT_H
-#define _ASM_SEGMENT_H
-
-/* Only here because we have some old header files that expect it.. */
-
-#endif /* _ASM_SEGMENT_H */

[PATCH 12/15] sh: remove use of asm/segment.h

[Kumar Gala]

Removed asm-sh/segment.h as its not used by anything.

Signed-off-by: Kumar Gala <kumar.gala@freescale.com>

---
commit ab81566ace65ac748c885d696c11b5ae839fb328
tree e36ec9a52dc26db8bc20e1c57a9dce5e2b58fd39
parent e2a2b88b3c3d60b07be7908579bb772e6bd4cd01
author Kumar K. Gala <kumar.gala@freescale.com> Wed, 24 Aug 2005 11:32:48 -0500
committer Kumar K. Gala <kumar.gala@freescale.com> Wed, 24 Aug 2005 11:32:48 -0500

 include/asm-sh/segment.h |    6 ------
 1 files changed, 0 insertions(+), 6 deletions(-)

diff --git a/include/asm-sh/segment.h b/include/asm-sh/segment.h
deleted file mode 100644
--- a/include/asm-sh/segment.h
+++ /dev/null
@@ -1,6 +0,0 @@
-#ifndef __ASM_SH_SEGMENT_H
-#define __ASM_SH_SEGMENT_H
-
-/* Only here because we have some old header files that expect it.. */
-
-#endif /* __ASM_SH_SEGMENT_H */

[PATCH 13/15] sparc64: remove use of asm/segment.h

[Kumar Gala]

Removed sparc64 architecture specific users of asm/segment.h and
asm-sparc64/segment.h itself

Signed-off-by: Kumar Gala <kumar.gala@freescale.com>

---
commit 4c42a86cb24cd46d20ccb883b196e29bef6df18f
tree 83888c48157ecf0783f19cd73634d6446ac29482
parent 6ee9ded730a875d63e42add1c3094eca7d5a6cdf
author Kumar K. Gala <kumar.gala@freescale.com> Wed, 24 Aug 2005 11:04:10 -0500
committer Kumar K. Gala <kumar.gala@freescale.com> Wed, 24 Aug 2005 11:04:10 -0500

 b/arch/sparc64/kernel/setup.c     |    1 -
 b/include/asm-sparc64/processor.h |    1 -
 include/asm-sparc64/segment.h     |    6 ------
 3 files changed, 8 deletions(-)

diff --git a/arch/sparc64/kernel/setup.c b/arch/sparc64/kernel/setup.c
--- a/arch/sparc64/kernel/setup.c
+++ b/arch/sparc64/kernel/setup.c
@@ -33,7 +33,6 @@
 #include <linux/cpu.h>
 #include <linux/initrd.h>
 
-#include <asm/segment.h>
 #include <asm/system.h>
 #include <asm/io.h>
 #include <asm/processor.h>
diff --git a/include/asm-sparc64/processor.h b/include/asm-sparc64/processor.h
--- a/include/asm-sparc64/processor.h
+++ b/include/asm-sparc64/processor.h
@@ -18,7 +18,6 @@
 #include <asm/a.out.h>
 #include <asm/pstate.h>
 #include <asm/ptrace.h>
-#include <asm/segment.h>
 #include <asm/page.h>
 
 /* The sparc has no problems with write protection */
diff --git a/include/asm-sparc64/segment.h b/include/asm-sparc64/segment.h
deleted file mode 100644
--- a/include/asm-sparc64/segment.h
+++ /dev/null
@@ -1,6 +0,0 @@
-#ifndef __SPARC64_SEGMENT_H
-#define __SPARC64_SEGMENT_H
-
-/* Only here because we have some old header files that expect it.. */
-
-#endif

[PATCH 14/15] sparc: remove use of asm/segment.h

[Kumar Gala]

Removed sparc architecture specific users of asm/segment.h and
asm-sparc/segment.h itself

Signed-off-by: Kumar Gala <kumar.gala@freescale.com>

---
commit bd988bbd37cb610800f19ac6891a519c81c96d95
tree dff7982106a16af59a65c158631f76d9fa118a1e
parent 4c42a86cb24cd46d20ccb883b196e29bef6df18f
author Kumar K. Gala <kumar.gala@freescale.com> Wed, 24 Aug 2005 11:14:02 -0500
committer Kumar K. Gala <kumar.gala@freescale.com> Wed, 24 Aug 2005 11:14:02 -0500

 arch/sparc/kernel/setup.c     |    1 -
 arch/sparc/kernel/tick14.c    |    1 -
 arch/sparc/kernel/time.c      |    1 -
 arch/sparc/mm/fault.c         |    1 -
 arch/sparc/mm/init.c          |    1 -
 include/asm-sparc/processor.h |    1 -
 include/asm-sparc/segment.h   |    6 ------
 include/asm-sparc/system.h    |    1 -
 8 files changed, 0 insertions(+), 13 deletions(-)

diff --git a/arch/sparc/kernel/setup.c b/arch/sparc/kernel/setup.c
--- a/arch/sparc/kernel/setup.c
+++ b/arch/sparc/kernel/setup.c
@@ -32,7 +32,6 @@
 #include <linux/spinlock.h>
 #include <linux/root_dev.h>
 
-#include <asm/segment.h>
 #include <asm/system.h>
 #include <asm/io.h>
 #include <asm/processor.h>
diff --git a/arch/sparc/kernel/tick14.c b/arch/sparc/kernel/tick14.c
--- a/arch/sparc/kernel/tick14.c
+++ b/arch/sparc/kernel/tick14.c
@@ -19,7 +19,6 @@
 #include <linux/interrupt.h>
 
 #include <asm/oplib.h>
-#include <asm/segment.h>
 #include <asm/timer.h>
 #include <asm/mostek.h>
 #include <asm/system.h>
diff --git a/arch/sparc/kernel/time.c b/arch/sparc/kernel/time.c
--- a/arch/sparc/kernel/time.c
+++ b/arch/sparc/kernel/time.c
@@ -32,7 +32,6 @@
 #include <linux/profile.h>
 
 #include <asm/oplib.h>
-#include <asm/segment.h>
 #include <asm/timer.h>
 #include <asm/mostek.h>
 #include <asm/system.h>
diff --git a/arch/sparc/mm/fault.c b/arch/sparc/mm/fault.c
--- a/arch/sparc/mm/fault.c
+++ b/arch/sparc/mm/fault.c
@@ -23,7 +23,6 @@
 #include <linux/module.h>
 
 #include <asm/system.h>
-#include <asm/segment.h>
 #include <asm/page.h>
 #include <asm/pgtable.h>
 #include <asm/memreg.h>
diff --git a/arch/sparc/mm/init.c b/arch/sparc/mm/init.c
--- a/arch/sparc/mm/init.c
+++ b/arch/sparc/mm/init.c
@@ -25,7 +25,6 @@
 #include <linux/bootmem.h>
 
 #include <asm/system.h>
-#include <asm/segment.h>
 #include <asm/vac-ops.h>
 #include <asm/page.h>
 #include <asm/pgtable.h>
diff --git a/include/asm-sparc/processor.h b/include/asm-sparc/processor.h
--- a/include/asm-sparc/processor.h
+++ b/include/asm-sparc/processor.h
@@ -19,7 +19,6 @@
 #include <asm/ptrace.h>
 #include <asm/head.h>
 #include <asm/signal.h>
-#include <asm/segment.h>
 #include <asm/btfixup.h>
 #include <asm/page.h>
 
diff --git a/include/asm-sparc/segment.h b/include/asm-sparc/segment.h
deleted file mode 100644
--- a/include/asm-sparc/segment.h
+++ /dev/null
@@ -1,6 +0,0 @@
-#ifndef __SPARC_SEGMENT_H
-#define __SPARC_SEGMENT_H
-
-/* Only here because we have some old header files that expect it.. */
-
-#endif
diff --git a/include/asm-sparc/system.h b/include/asm-sparc/system.h
--- a/include/asm-sparc/system.h
+++ b/include/asm-sparc/system.h
@@ -9,7 +9,6 @@
 #include <linux/threads.h>	/* NR_CPUS */
 #include <linux/thread_info.h>
 
-#include <asm/segment.h>
 #include <asm/page.h>
 #include <asm/psr.h>
 #include <asm/ptrace.h>

Re: [PATCH 14/15] sparc: remove use of asm/segment.h

[David S. Miller]

Both sparc64 and sparc patches applied, thanks Kumar.

[PATCH 15/15] xtensa: remove use of asm/segment.h

[Kumar Gala]

Removed asm-xtensa/segment.h as its not used by anything.

Signed-off-by: Kumar Gala <kumar.gala@freescale.com>

---
commit 51e70a5236bf8f2f30f1a86513c18e05e3cb6fee
tree 57c39eaf3761a71b9d78c25893f7696fdc703915
parent 36b349eaba7a399c06219d3553d571261dbf6333
author Kumar K. Gala <kumar.gala@freescale.com> Wed, 24 Aug 2005 11:37:29 -0500
committer Kumar K. Gala <kumar.gala@freescale.com> Wed, 24 Aug 2005 11:37:29 -0500

 include/asm-xtensa/segment.h |   16 ----------------
 1 files changed, 0 insertions(+), 16 deletions(-)

diff --git a/include/asm-xtensa/segment.h b/include/asm-xtensa/segment.h
deleted file mode 100644
--- a/include/asm-xtensa/segment.h
+++ /dev/null
@@ -1,16 +0,0 @@
-/*
- * include/asm-xtensa/segment.h
- *
- * This file is subject to the terms and conditions of the GNU General Public
- * License.  See the file "COPYING" in the main directory of this archive
- * for more details.
- *
- * Copyright (C) 2001 - 2005 Tensilica Inc.
- */
-
-#ifndef _XTENSA_SEGMENT_H
-#define _XTENSA_SEGMENT_H
-
-#include <asm/uaccess.h>
-
-#endif	/* _XTENSA_SEGEMENT_H */

Re: [PATCH 00/15] Remove asm/segment.h from low hanging architectures

[Alan Cox]

On Mer, 2005-08-24 at 11:43 -0500, Kumar Gala wrote:
> The following set of patches removes the use and existence of 
> asm/segment.h from the architecture ports 

You've broken various things by doing this because some driver code
rightly or wrongly uses segment.h. That is fine because they shouldn't
do so. However asm/segment.h isn't supoosed to be removed on
architectures that use segments- like x86, and x86-64. There it is a
real arch private file and shouldn't be disappearing.

It shouldn't be leaking into drivers any more (eg mxser.c is an offender
there)

Re: [PATCH 00/15] Remove asm/segment.h from low hanging architectures

[Kumar Gala]

On Aug 24, 2005, at 4:50 PM, Alan Cox wrote:

> On Mer, 2005-08-24 at 11:43 -0500, Kumar Gala wrote:
>
>> The following set of patches removes the use and existence of
>> asm/segment.h from the architecture ports
>>
>
> You've broken various things by doing this because some driver code
> rightly or wrongly uses segment.h. That is fine because they shouldn't
> do so. However asm/segment.h isn't supoosed to be removed on
> architectures that use segments- like x86, and x86-64. There it is a
> real arch private file and shouldn't be disappearing.
>
> It shouldn't be leaking into drivers any more (eg mxser.c is an  
> offender
> there)

Ok, so it sounds like my patch to remove drivers from include asm/ 
segment.h is fine, and we will leave it up to the various arch  
maintainers to decide if they want to keep asm/segment.h for arch  
specific usage.

- kumar

Re: [PATCH 00/15] Remove asm/segment.h from low hanging architectures

[Zachary Amsden]

Alan Cox wrote:

>On Mer, 2005-08-24 at 11:43 -0500, Kumar Gala wrote:
>  
>
>>The following set of patches removes the use and existence of 
>>asm/segment.h from the architecture ports 
>>    
>>
>
>You've broken various things by doing this because some driver code
>rightly or wrongly uses segment.h. That is fine because they shouldn't
>do so. However asm/segment.h isn't supoosed to be removed on
>architectures that use segments- like x86, and x86-64. There it is a
>real arch private file and shouldn't be disappearing.
>
>It shouldn't be leaking into drivers any more (eg mxser.c is an offender
>there)
>  
>

Yes, agree totally,  i386 _requires_ asm/segment.h.  It is used in 
low-level trap handling and bootup code from assembly files.  In 
addition, even parts of userspace on i386 depend on asm/segment.h, 
although that is a different beast.

It is a total bug if generic drivers include it, there is simply no 
reason to use segment.h unless you have to use segmentation.  Ideally, 
all includes of segment.h should be confined to architectures that 
require it, and limited to either arch/foo or include/asm-foo.

I'm looking at -rc6-mm1, and I see the following bad guys:

./drivers/char/mxser.c:#include <asm/segment.h>
./drivers/char/speakup/speakup_drvcommon.c:#include <asm/segment.h>     
/* for put_user_byte */
./drivers/isdn/hisax/hisax.h:#include <asm/segment.h>
./drivers/media/video/adv7170.c:#include <asm/segment.h>
./drivers/media/video/adv7175.c:#include <asm/segment.h>
./drivers/media/video/bt819.c:#include <asm/segment.h>
./drivers/media/video/bt856.c:#include <asm/segment.h>
./drivers/media/video/saa7111.c:#include <asm/segment.h>
./drivers/media/video/saa7114.c:#include <asm/segment.h>
./drivers/media/video/saa7185.c:#include <asm/segment.h>
./drivers/serial/68328serial.c:#include <asm/segment.h>
./drivers/serial/crisv10.c:#include <asm/segment.h>
./drivers/serial/icom.c:#include <asm/segment.h>
./drivers/serial/mcfserial.c:#include <asm/segment.h>
./drivers/video/q40fb.c:#include <asm/segment.h>
./include/linux/isdn.h:#include <asm/segment.h>
./sound/oss/os.h:#include <asm/segment.h>

Zach

Re: [PATCH 00/15] Remove asm/segment.h from low hanging architectures

[Christoph Hellwig]

On Thu, Aug 25, 2005 at 02:24:56AM -0700, Zachary Amsden wrote:
> Yes, agree totally,  i386 _requires_ asm/segment.h.  It is used in 
> low-level trap handling and bootup code from assembly files.  In 
> addition,

but keeping the header under that name will just encorage people
to put it back into drivers, after all it compiles on i386..

Let's rename it even for i386.

>even parts of userspace on i386 depend on asm/segment.h, 
> although that is a different beast.

That's a problem of glibc-kernheaders (or insert alternative $name here)

RE: [PATCH 05/15] ia64: remove use of asm/segment.h

[Luck, Tony]

>There are still a few drivers that include asm/segment.h, so
>I don't think we should remove asm/segment.h itself just yet.

Agreed.  The sequence should be to send patches to get rid of
all "#include <asm/segment.h>" references.

Once they have all gone, then a patch can remove the files.

If you are concerned that people would start adding new
references and you don't want to get into a game of whack-a-mole,
then you could add #warning "include of deprecated asm/segment.h",
but that might be overkill.

I'll apply this for ia64 w/o the deletion.

-Tony

Re: [PATCH 05/15] ia64: remove use of asm/segment.h

[Kumar Gala]

On Aug 24, 2005, at 3:19 PM, Luck, Tony wrote:

>> There are still a few drivers that include asm/segment.h, so
>> I don't think we should remove asm/segment.h itself just yet.
>>
>
> Agreed.  The sequence should be to send patches to get rid of
> all "#include <asm/segment.h>" references.
>
> Once they have all gone, then a patch can remove the files.
>
> If you are concerned that people would start adding new
> references and you don't want to get into a game of whack-a-mole,
> then you could add #warning "include of deprecated asm/segment.h",
> but that might be overkill.
>
> I'll apply this for ia64 w/o the deletion.

I've posted a patch before this to remove all non-architecture users  
of asm/segment.h.

http://www.ussg.iu.edu/hypermail/linux/kernel/0508.3/0099.html

- kumar

RE: [PATCH 05/15] ia64: remove use of asm/segment.h

[Luck, Tony]

>> I'll apply this for ia64 w/o the deletion.

This is now in my test tree.  I will send to Linus soon after
2.6.13 is released.

>I've posted a patch before this to remove all non-architecture users  
>of asm/segment.h.
>
>http://www.ussg.iu.edu/hypermail/linux/kernel/0508.3/0099.html

Good.  After that gets I applied I will delete include/asm-ia64/segment.h

-Tony