public inbox for binutils@sourceware.org
 help / color / mirror / Atom feed
From: "H. J. Lu" <hjl@lucon.org>
To: Andi Kleen <ak@suse.de>
Cc: discuss@x86-64.org, "Baker,
	Brian (ISS - Houston)" <Brian.B@hp.com>,
		torvalds@osdl.org, akpm@osdl.org, 	"Altobelli,
	David" <david.altobelli@hp.com>,
		binutils@sources.redhat.com
Subject: Re: [discuss] RE: [PATCH] [1/2] x86_64: Pass -32 to the assembler when compiling  the 32bit vsyscall pages
Date: Mon, 24 Apr 2006 10:13:00 -0000	[thread overview]
Message-ID: <20060421233244.GA16692@lucon.org> (raw)
In-Reply-To: <200604220025.53501.ak@suse.de>

On Sat, Apr 22, 2006 at 12:25:52AM +0200, Andi Kleen wrote:
> 
> > > Index: linux/arch/x86_64/ia32/Makefile
> > > ===================================================================
> > > --- linux.orig/arch/x86_64/ia32/Makefile
> > > +++ linux/arch/x86_64/ia32/Makefile
> > > @@ -27,5 +27,5 @@ $(obj)/vsyscall-sysenter.so $(obj)/vsysc
> > >  $(obj)/vsyscall-%.so: $(src)/vsyscall.lds $(obj)/vsyscall-%.o FORCE
> > >  	$(call if_changed,syscall)
> > >
> > > -AFLAGS_vsyscall-sysenter.o = -m32
> > > -AFLAGS_vsyscall-syscall.o = -m32
> > > +AFLAGS_vsyscall-sysenter.o = -m32 -Wa,-32
> > > +AFLAGS_vsyscall-syscall.o = -m32 -Wa,-32
> >
> > I can't reproduce it
> 
> What did you try exactly? 

I compared the outputs of vsyscall-sysenter.o with -m32 and -m32
-Wa,-32, using gcc 3.4. They are identical.

> 
> You have to recompile the file and either run the pthreads
> cancellation test case with the right glibc version, or examine
> the unwind tables in the .so by hand. Or just check if the warning
> appears or not.

There is no need to do that. If you believe the problem is due to
binutils 2.16.91.0.5, it is a binutils bug. Please open a bug report
at

http://www.sourceware.org/bugzilla/

and assign it to me. I need the preprocessed .s file, the expected .o
file and the expected .so file with procedures to reproduce the
difference. Since they are assembly codes, I will expect -m32 and
-m32 -Wa,-32 should generate the same output.

> 
> > with binutils 2.16.91.0.5. Unless you have a bad 
> > gcc driver, -m32 will pass --32 to assembler, which is the same as -32.
> 
> It definitely fixed the problem here and the warning went away.
> gcc used was 4.1
> 
> But you're right it could have been the gcc - in the case where it worked
> I was using gcc 4.0.
> 

Since they are assembly codes, it sounds like a gcc driver issue to me.


H.J.

  reply	other threads:[~2006-04-21 23:32 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <6095103C4F0ACD4FAE3522E0F27ADF42022CEE33@cceexc13.americas.cpqcorp.net>
2006-04-21 23:22 ` H. J. Lu
2006-04-23 20:24   ` Andi Kleen
2006-04-24 10:13     ` H. J. Lu [this message]
2006-04-24 10:33       ` Andi Kleen

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20060421233244.GA16692@lucon.org \
    --to=hjl@lucon.org \
    --cc=Brian.B@hp.com \
    --cc=ak@suse.de \
    --cc=akpm@osdl.org \
    --cc=binutils@sources.redhat.com \
    --cc=david.altobelli@hp.com \
    --cc=discuss@x86-64.org \
    --cc=torvalds@osdl.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).