This is the mail archive of the
gcc@gcc.gnu.org
mailing list for the GCC project.
RE: Bootstrap issues in libsanitizer
- From: "Iyer, Balaji V" <balaji dot v dot iyer at intel dot com>
- To: Richard Biener <richard dot guenther at gmail dot com>, "Gopalasubramanian, Ganesh" <Ganesh dot Gopalasubramanian at amd dot com>
- Cc: "gcc at gcc dot gnu dot org" <gcc at gcc dot gnu dot org>, "konstantin dot s dot serebryany at gmail dot com" <konstantin dot s dot serebryany at gmail dot com>, Jonathan Wakely <jwakely dot gcc at gmail dot com>
- Date: Wed, 6 Nov 2013 14:34:32 +0000
- Subject: RE: Bootstrap issues in libsanitizer
- Authentication-results: sourceware.org; auth=none
- References: <BF230D13CA30DD48930C31D4099330003A49B0E1 at FMSMSX101 dot amr dot corp dot intel dot com> <CAH6eHdScLvmgh_ArORKqmtJo5XzbGCEW-Ln03uYG7KvVF_B01g at mail dot gmail dot com> <CAH6eHdRbzuYnF0mK3d1vOZboV7RW3Fzf=VGo4wUFAANjx70gsg at mail dot gmail dot com> <CAH6eHdRke4np4zdLfGRv56sVWzJRJ1LcOWnj6=i-OhnUEF5ZOQ at mail dot gmail dot com> <CAFiYyc1_1_4pAQxYrmGrTddu+XV+Gmv=0rq3kxMYCJr=HNBccw at mail dot gmail dot com> <EB4625145972F94C9680D8CADD6516155C2435AE at SATLEXDAG02 dot amd dot com> <CAFiYyc0_-yOj-=b3nsVGjHhnC6=1ohrAvZRiurZRcSFQ3bb2xw at mail dot gmail dot com>
> -----Original Message-----
> From: Richard Biener [mailto:richard.guenther@gmail.com]
> Sent: Wednesday, November 6, 2013 5:55 AM
> To: Gopalasubramanian, Ganesh
> Cc: Iyer, Balaji V; gcc@gcc.gnu.org; konstantin.s.serebryany@gmail.com;
> Jonathan Wakely
> Subject: Re: Bootstrap issues in libsanitizer
>
> On Wed, Nov 6, 2013 at 11:50 AM, Gopalasubramanian, Ganesh
> <Ganesh.Gopalasubramanian@amd.com> wrote:
> >> Are you including linux/vt.h yourself? If you get it via a glibc header then
> it's a SUSE issue, yes. Can you specify the SUSE version you are looking at?
> Even for 12.1 I see 'newev' here though SLE11 seems to have 'new'.
> >
> > I get the error with
> >
> > SUSE Linux Enterprise Server 11 (x86_64) VERSION = 11 PATCHLEVEL = 1
Hi Ganesh, That's exactly the version I have. One quick workaround I have done to get around this issue (yes I know it is pretty wierd one, but still works) is the following
------------------------------------------------------------------------------------------------------------
#ifndef __cplusplus
#define new_SOMETHING_ELSE new
#endif
struct vt_event {
unsigned int event;
#define VT_EVENT_SWITCH 0x0001 /* Console switch */
#define VT_EVENT_BLANK 0x0002 /* Screen blank */
#define VT_EVENT_UNBLANK 0x0004 /* Screen unblank */
#define VT_EVENT_RESIZE 0x0008 /* Resize display */
#define VT_MAX_EVENT 0x000F
unsigned int old; /* Old console */
unsigned int new_SOMETHING_ELSE; /* New console (if changing) */
unsigned int pad[4]; /* Padding for expansion */
};
--------------------------------------------------------------------------------------------------------------
>
> I've filed a bugreport internally. And yes, sys/vt.h just includes linux/vt.h
> here.
>
Thanks Richard!
-Balaji V. Iyer.
> Richard.
>
> > Regards
> > Ganesh
> >
> > -----Original Message-----
> > From: gcc-owner@gcc.gnu.org [mailto:gcc-owner@gcc.gnu.org] On Behalf
> > Of Richard Biener
> > Sent: Wednesday, November 06, 2013 4:16 PM
> > To: Jonathan Wakely
> > Cc: Iyer, Balaji V; gcc@gcc.gnu.org; konstantin.s.serebryany@gmail.com
> > Subject: Re: Bootstrap issues in libsanitizer
> >
> > On Tue, Nov 5, 2013 at 4:41 PM, Jonathan Wakely
> <jwakely.gcc@gmail.com> wrote:
> >> On 5 November 2013 15:38, Jonathan Wakely wrote:
> >>> On 5 November 2013 15:32, Jonathan Wakely wrote:
> >>>> On 5 November 2013 15:27, Iyer, Balaji V wrote:
> >>>>> In file included from /usr/include/sys/vt.h:1:0,
> >>>>> from ../../../../trunk-
> gcc/libsanitizer/sanitizer_common/sanitizer_platform_limits_posix.cc:49:
> >>>>> /usr/include/linux/vt.h:74:15: error: expected unqualified-id before
> ânewâ
> >>>>> unsigned int new; /* New console (if changing) */
> >>>>> ^
> >>>>
> >>>> 'new' is a keyword in C++, so your linux/vt.h header is not usable in C++
> files.
> >>>>
> >>>> This is not a problem in libsanitizer. Have you recently updated a
> >>>> kernel-headers package on your machine?
> >>>
> >>> With kernel 3.11.6 I see that field is called newev, so it looks as
> >>> though newer kernels fix the problem.
> >>
> >> http://lkml.indiana.edu/hypermail/linux/kernel/0911.2/01720.html
> >> implies the "new" field never made it into an actual release, so it
> >> might be a problem for SuSE kernels only.
> >
> > Are you including linux/vt.h yourself? If you get it via a glibc header then
> it's a SUSE issue, yes. Can you specify the SUSE version you are looking at?
> Even for 12.1 I see 'newev' here though SLE11 seems to have 'new'.
> >
> > Richard.
> >
> >