Messages in this thread | | | From | "Metzger, Markus T" <> | Date | Thu, 15 Jan 2009 08:06:31 +0000 | Subject | RE: [patch 3/5] x86: headers cleanup - ptrace-abi.h |
| |
>-----Original Message----- >From: H. Peter Anvin [mailto:hpa@zytor.com] >Sent: Wednesday, January 14, 2009 11:22 PM >To: Cyrill Gorcunov; Metzger, Markus T >Cc: mingo@elte.hu; linux-kernel@vger.kernel.org; harvey.harrison@gmail.com; sam@ravnborg.org; >jaswinderrajput@gmail.com; Cyrill Gorcunov >Subject: Re: [patch 3/5] x86: headers cleanup - ptrace-abi.h > >Okay, this one really looks wrong to me. > >Markus, could you confirm if this was ever intended to be exported to >userspace? I suspect The Right Thing is simply to remove the #ifdef >CONFIG_X86_TRACE_BTS here...
This is intended for userspace. I added the #ifdef so the entire feature could be turned off. This might not be necessary for declarations and macros, though.
Do you want me to remove the #ifdef CONFIG around the declarations in ptrace headers?
regards, markus.
> > -hpa > > >Cyrill Gorcunov wrote: >> Impact: cleanup >> >> 'make headers_check' warn us about leaking of kernel private >> (mostly compile time vars) data to userspace in headers. Fix it. >> >> ptrace_bts_config struct is wrapped by __KERNEL__ -- >> not sure if it was ever proposed for userland. On the >> other hand branch tracing is really anxious for userland >> developers. >> >> Signed-off-by: Cyrill Gorcunov <gorcunov@openvz.org> >> --- >> arch/x86/include/asm/ptrace-abi.h | 4 +++- >> 1 file changed, 3 insertions(+), 1 deletion(-) >> >> Index: linux-2.6.git/arch/x86/include/asm/ptrace-abi.h >> =================================================================== >> --- linux-2.6.git.orig/arch/x86/include/asm/ptrace-abi.h >> +++ linux-2.6.git/arch/x86/include/asm/ptrace-abi.h >> @@ -50,7 +50,7 @@ >> #define RSP 152 >> #define SS 160 >> #define ARGOFFSET R11 >> -#endif /* __ASSEMBLY__ */ >> +#endif /* __ASSEMBLY__ || __FRAME_OFFSETS */ >> >> /* top of stack page */ >> #define FRAME_SIZE 168 >> @@ -80,6 +80,7 @@ >> >> #define PTRACE_SINGLEBLOCK 33 /* resume execution until next branch */ >> >> +#ifdef __KERNEL__ >> #ifdef CONFIG_X86_PTRACE_BTS >> >> #ifndef __ASSEMBLY__ >> @@ -141,5 +142,6 @@ struct ptrace_bts_config { >> Returns number of BTS records drained. >> */ >> #endif /* CONFIG_X86_PTRACE_BTS */ >> +#endif /* __KERNEL__ */ >> >> #endif /* _ASM_X86_PTRACE_ABI_H */ >> --------------------------------------------------------------------- Intel GmbH Dornacher Strasse 1 85622 Feldkirchen/Muenchen Germany Sitz der Gesellschaft: Feldkirchen bei Muenchen Geschaeftsfuehrer: Douglas Lusk, Peter Gleissner, Hannes Schwaderer Registergericht: Muenchen HRB 47456 Ust.-IdNr. VAT Registration No.: DE129385895 Citibank Frankfurt (BLZ 502 109 00) 600119052
This e-mail and any attachments may contain confidential material for the sole use of the intended recipient(s). Any review or distribution by others is strictly prohibited. If you are not the intended recipient, please contact the sender and delete all copies.
| |