public inbox for linux-ia64@vger.kernel.org
 help / color / mirror / Atom feed
* [PATCH 05/15] ia64: remove use of asm/segment.h
       [not found] <Pine.LNX.4.61.0508241139100.23956@nylon.am.freescale.net>
@ 2005-08-24 16:53 ` Kumar Gala
  2005-08-24 20:09   ` Bjorn Helgaas
                     ` (2 more replies)
  0 siblings, 3 replies; 6+ messages in thread
From: Kumar Gala @ 2005-08-24 16:53 UTC (permalink / raw)
  To: linux-kernel; +Cc: Andrew Morton, tony.luck, linux-ia64

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 */

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

* Re: [PATCH 05/15] ia64: remove use of asm/segment.h
  2005-08-24 16:53 ` [PATCH 05/15] ia64: remove use of asm/segment.h Kumar Gala
@ 2005-08-24 20:09   ` Bjorn Helgaas
  2005-08-24 22:49     ` Ralf Baechle
  2005-08-24 20:19   ` Luck, Tony
  2005-08-24 21:42   ` Luck, Tony
  2 siblings, 1 reply; 6+ messages in thread
From: Bjorn Helgaas @ 2005-08-24 20:09 UTC (permalink / raw)
  To: Kumar Gala; +Cc: linux-kernel, Andrew Morton, tony.luck, linux-ia64

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.

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

* RE: [PATCH 05/15] ia64: remove use of asm/segment.h
  2005-08-24 16:53 ` [PATCH 05/15] ia64: remove use of asm/segment.h Kumar Gala
  2005-08-24 20:09   ` Bjorn Helgaas
@ 2005-08-24 20:19   ` Luck, Tony
  2005-08-24 21:32     ` Kumar Gala
  2005-08-24 21:42   ` Luck, Tony
  2 siblings, 1 reply; 6+ messages in thread
From: Luck, Tony @ 2005-08-24 20:19 UTC (permalink / raw)
  To: Bjorn Helgaas, Kumar Gala; +Cc: linux-kernel, Andrew Morton, linux-ia64

>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

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

* Re: [PATCH 05/15] ia64: remove use of asm/segment.h
  2005-08-24 20:19   ` Luck, Tony
@ 2005-08-24 21:32     ` Kumar Gala
  0 siblings, 0 replies; 6+ messages in thread
From: Kumar Gala @ 2005-08-24 21:32 UTC (permalink / raw)
  To: Luck, Tony
  Cc: Bjorn Helgaas, Gala Kumar K.-galak, linux-kernel, Andrew Morton,
	linux-ia64


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



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

* RE: [PATCH 05/15] ia64: remove use of asm/segment.h
  2005-08-24 16:53 ` [PATCH 05/15] ia64: remove use of asm/segment.h Kumar Gala
  2005-08-24 20:09   ` Bjorn Helgaas
  2005-08-24 20:19   ` Luck, Tony
@ 2005-08-24 21:42   ` Luck, Tony
  2 siblings, 0 replies; 6+ messages in thread
From: Luck, Tony @ 2005-08-24 21:42 UTC (permalink / raw)
  To: Kumar Gala
  Cc: Bjorn Helgaas, Gala Kumar K.-galak, linux-kernel, Andrew Morton,
	linux-ia64

>> 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

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

* Re: [PATCH 05/15] ia64: remove use of asm/segment.h
  2005-08-24 20:09   ` Bjorn Helgaas
@ 2005-08-24 22:49     ` Ralf Baechle
  0 siblings, 0 replies; 6+ messages in thread
From: Ralf Baechle @ 2005-08-24 22:49 UTC (permalink / raw)
  To: Bjorn Helgaas
  Cc: Kumar Gala, linux-kernel, Andrew Morton, tony.luck, linux-ia64

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

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

end of thread, other threads:[~2005-08-24 22:49 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
     [not found] <Pine.LNX.4.61.0508241139100.23956@nylon.am.freescale.net>
2005-08-24 16:53 ` [PATCH 05/15] ia64: remove use of asm/segment.h Kumar Gala
2005-08-24 20:09   ` Bjorn Helgaas
2005-08-24 22:49     ` Ralf Baechle
2005-08-24 20:19   ` Luck, Tony
2005-08-24 21:32     ` Kumar Gala
2005-08-24 21:42   ` Luck, Tony

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox