Re: [PATCH] Make struct alt_instr acceptable to userland

Re: [PATCH] Make struct alt_instr acceptable to userland

[Andi Kleen]

Bryan O'Sullivan <bos@serpentine.com> writes:

> Andi's new struct alt_instr is visible to klibc.  The attached patch
> changes the types it uses, so that klibc becomes happy.

Thanks. I changed it in my sources.

I already have some updates/cleanups for the mechanism queued, will
submit it with that.

-Andi

[PATCH] Make struct alt_instr acceptable to userland

[Bryan O'Sullivan]

Andi's new struct alt_instr is visible to klibc.  The attached patch
changes the types it uses, so that klibc becomes happy.

	<b

# This is a BitKeeper generated patch for the following project:
# Project Name: Linux kernel tree
# This patch format is intended for GNU patch command version 2.5 or higher.
# This patch includes the following deltas:
#	           ChangeSet	1.1142  -> 1.1143 
#	include/asm-i386/system.h	1.27    -> 1.28   
#
# The following is the BitKeeper ChangeSet Log
# --------------------------------------------
# 03/04/28	bos@serpentine.com	1.1143
# struct alt_instr is visible to userland, so change the types it uses
# --------------------------------------------
#
diff -Nru a/include/asm-i386/system.h b/include/asm-i386/system.h
--- a/include/asm-i386/system.h	Mon Apr 28 15:43:42 2003
+++ b/include/asm-i386/system.h	Mon Apr 28 15:43:42 2003
@@ -278,11 +278,11 @@
 #endif
 
 struct alt_instr { 
-	u8 *instr; 		/* original instruction */
-	u8  cpuid;		/* cpuid bit set for replacement */
-	u8  instrlen;		/* length of original instruction */
-	u8  replacementlen; 	/* length of new instruction, <= instrlen */ 
-	u8  replacement[0];   	/* new instruction */
+	__u8 *instr; 		/* original instruction */
+	__u8  cpuid;		/* cpuid bit set for replacement */
+	__u8  instrlen;		/* length of original instruction */
+	__u8  replacementlen; 	/* length of new instruction, <= instrlen */ 
+	__u8  replacement[0];	/* new instruction */
 }; 
 
 /* 

Re: [PATCH] Make struct alt_instr acceptable to userland

[Andrew Morton]

"Bryan O'Sullivan" <bos@serpentine.com> wrote:
>
> Andi's new struct alt_instr is visible to klibc.  The attached patch
> changes the types it uses, so that klibc becomes happy.

Should it be inside #ifdef __KERNEL__?