Re: [PATCH 01/21] sparc: copy exported sparc64 specific header files to asm-sparc

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



From: Sam Ravnborg <sam@xxxxxxxxxxxx>
Date: Fri,  6 Jun 2008 23:00:13 +0200

> To get rid of ALTARCH in Kbuild copy remaining
> exported header files that are sparc64 specific to asm-sparc.
> 
> Copy was done using the following simple script:
> 
> set -e
> SPARC64="apb.h bbc.h display7seg.h envctrl.h psrcompat.h pstate.h uctx.h utrap.h watchdog.h"
> for FILE in ${SPARC64}; do
> 	if [ -f asm-sparc/$FILE ]; then
> 		echo $FILE exist in asm-sparc
> 	fi
> 	cat asm-sparc64/$FILE > asm-sparc/$FILE
> 	printf "#include <asm-sparc/$FILE>\n" > asm-sparc64/$FILE
> done
> 
> Signed-off-by: Sam Ravnborg <sam@xxxxxxxxxxxx>

Thanks for doing this work Sam.

Looking over this particular patch made me notice that
asm/apb.h and asm/bbc.h should not be exported to userspace
at all.  And with some searching I verified that no userland
source bases include these files.

They are simply definitions of chip registers and do not
describe a userland interface of any kind, so this seems
about right.

psrcompat.h and pstate.h fall roughly into the same category,
but unfortunately things like strace include these files
(even though they really don't use the defines) so we have
to keep exporting them in order to not break userspace.

So what I'm going to do is apply the patch below first to
sparc-next-2.6, then apply this patch with the apb.h and bbc.h parts
removed.

Thanks!

sparc64 Kbuild: apb.h and bbc.h should not be exported to userspace

Signed-off-by: David S. Miller <davem@xxxxxxxxxxxxx>
---
 include/asm-sparc64/Kbuild |    2 --
 1 files changed, 0 insertions(+), 2 deletions(-)

diff --git a/include/asm-sparc64/Kbuild b/include/asm-sparc64/Kbuild
index dce1cf9..7e88c12 100644
--- a/include/asm-sparc64/Kbuild
+++ b/include/asm-sparc64/Kbuild
@@ -4,9 +4,7 @@ ALTARCH := sparc
 ARCHDEF := defined __sparc__ && defined __arch64__
 ALTARCHDEF := defined __sparc__ && !defined __arch64__
 
-header-y += apb.h
 header-y += asi.h
-header-y += bbc.h
 header-y += bpp.h
 header-y += display7seg.h
 header-y += envctrl.h
-- 
1.5.5.1.308.g1fbb5

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

[Index of Archives]     [Kernel Development]     [DCCP]     [Linux ARM Development]     [Linux]     [Photo]     [Yosemite Help]     [Linux ARM Kernel]     [Linux SCSI]     [Linux x86_64]     [Linux Hams]

  Powered by Linux