From: "Haavard Skinnemoen" <hskinnemoen@gmail.com>
To: "H. Peter Anvin" <hpa@zytor.com>
Cc: "Haavard Skinnemoen" <hskinnemoen@atmel.com>,
akpm@osdl.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH 2/7] AVR32: Fix invalid constraints for stcond
Date: Tue, 11 Jul 2006 21:26:02 +0200 [thread overview]
Message-ID: <1defaf580607111226r2f0eac2n51fb513c3e88b1cc@mail.gmail.com> (raw)
In-Reply-To: <44B3E8CD.8070409@zytor.com>
On 7/11/06, H. Peter Anvin <hpa@zytor.com> wrote:
> Haavard Skinnemoen wrote:
> > Because gcc doesn't seem to like arch-dependent constraints in inline
> > asm, we ended up using "m" as constraint for the stcond instruction.
>
> Bunkum. That would be a bug in the AVR32 gcc, but gcc handles
> arch-specific constraints in inline assembly all the time. The kernel
> wouldn't compile for a large number of architectures otherwise.
Thanks, then we at least know it's a gcc bug. It's sometimes hard to
tell what features are actually supposed to work, especially the stuff
that's only documented in the internals manual...
Anyway, "o" is apparently more correct than "m" since with "o" it
manages to compile all the filesystems without complaining, and the
output looks correct. On the other hand, ocfs2 probably isn't
something you want to use on your AVR32 board...
Håvard
prev parent reply other threads:[~2006-07-11 19:26 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2006-07-11 12:43 [PATCH 0/7] AVR32 update Haavard Skinnemoen
2006-07-11 12:43 ` [PATCH 1/7] AVR32: CONFIG_DEBUG_BUGVERBOSE and CONFIG_FRAME_POINTER Haavard Skinnemoen
2006-07-11 12:43 ` [PATCH 2/7] AVR32: Fix invalid constraints for stcond Haavard Skinnemoen
2006-07-11 12:43 ` [PATCH 3/7] AVR32: Add support for irq-flags state tracing Haavard Skinnemoen
2006-07-11 12:43 ` [PATCH 4/7] AVR32: Turn off support for DISCONTIGMEM and SPARSEMEM Haavard Skinnemoen
2006-07-11 12:43 ` [PATCH 5/7] AVR32: Always enable CONFIG_EMBEDDED Haavard Skinnemoen
2006-07-11 12:43 ` [PATCH 6/7] AVR32: Export the find_*_bit() functions Haavard Skinnemoen
2006-07-11 12:43 ` [PATCH 7/7] AVR32: Add defconfig for AT32STK1002 Haavard Skinnemoen
2006-07-11 18:07 ` [PATCH 2/7] AVR32: Fix invalid constraints for stcond H. Peter Anvin
2006-07-11 19:26 ` Haavard Skinnemoen [this message]
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=1defaf580607111226r2f0eac2n51fb513c3e88b1cc@mail.gmail.com \
--to=hskinnemoen@gmail.com \
--cc=akpm@osdl.org \
--cc=hpa@zytor.com \
--cc=hskinnemoen@atmel.com \
--cc=linux-kernel@vger.kernel.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
all inboxes | Powered by JetHome®