From: Alexander Holler <holler@ahsoftware.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v3] ARM: Avoid compiler optimization for usages of readb, writeb and friends.
Date: Tue, 11 Jan 2011 04:53:47 +0100 [thread overview]
Message-ID: <4D2BD44B.7040900@ahsoftware.de> (raw)
In-Reply-To: <20110110150546.B3CBA150A44@gemini.denx.de>
Am 10.01.2011 16:05, schrieb Wolfgang Denk:
> Dear Alexander Holler,
>
> In message<4D2B1D75.70809@ahsoftware.de> you wrote:
>>
>> Beeing kind of a defensive programmer, I still would prefer to use have
>> that __asm__ for write* too. That would at least prevent us from a
>> possible bug there too.
>
> So why don't you simply test and, assuming it's working, ACK the patch
> I submitted yesterday? We should be on the safe side, then, and don't
> have to care about which mood the current compiler's optimizer might
> be in or what the POM is.
Sorry, I haven't had your last patch (mail) before I've written the mail
you are referencing.
I have updated my mail-system at home (armv5 with 128mb ram) and the
incoming queue, mainly filled through lkml, is still not completly
processed. ~2000 messages (3 days) need some time to go through
spamassassin on such a low-level hardware. ;)
I've seen you've switched from do {} while() to "something else", but I
can't comment on that "something else". Because I've already switched to
4.5.2. I'll have to dig out a system where I have a 4.5.1 to test the
problem occured with the write.
If anybody else already has tested it, I'm fine with it.
Regards,
Alexander
next prev parent reply other threads:[~2011-01-11 3:53 UTC|newest]
Thread overview: 23+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-12-22 11:04 [U-Boot] [PATCH v3] ARM: Avoid compiler optimization for usages of readb, writeb and friends Alexander Holler
2010-12-22 14:50 ` Dirk Behme
2010-12-22 15:07 ` Alexander Holler
2011-01-01 12:04 ` Dirk Behme
2011-01-01 17:52 ` Alexander Holler
2011-01-01 18:25 ` Dirk Behme
2011-01-01 18:47 ` Alexander Holler
2011-01-01 19:21 ` Dirk Behme
2011-01-02 12:43 ` Alexander Holler
2011-01-02 13:29 ` Dirk Behme
2011-01-02 21:00 ` Alexander Holler
2011-01-10 14:53 ` Alexander Holler
2011-01-10 15:05 ` Wolfgang Denk
2011-01-11 3:53 ` Alexander Holler [this message]
2011-01-09 22:25 ` Wolfgang Denk
2011-01-10 16:13 ` Dirk Behme
2011-01-17 21:59 ` Wolfgang Denk
2011-01-09 22:19 ` [U-Boot] [PATCH v4] ARM: Avoid compiler optimization for " Wolfgang Denk
2011-01-12 15:17 ` Thomas Weber
2011-01-12 15:39 ` Alexander Holler
2011-01-12 16:40 ` Wolfgang Denk
2011-01-12 16:49 ` Alexander Holler
2011-01-15 13:13 ` Albert ARIBAUD
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=4D2BD44B.7040900@ahsoftware.de \
--to=holler@ahsoftware.de \
--cc=u-boot@lists.denx.de \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.