From: Ralf Baechle <ralf@linux-mips.org>
To: Thiemo Seufer <ica2_ts@csv.ica.uni-stuttgart.de>
Cc: linux-mips@linux-mips.org
Subject: Re: [PATCH] Improve o32 syscall handling
Date: Mon, 22 Nov 2004 08:13:13 +0100 [thread overview]
Message-ID: <20041122071313.GC25433@linux-mips.org> (raw)
In-Reply-To: <20041122070003.GA902@rembrandt.csv.ica.uni-stuttgart.de>
On Mon, Nov 22, 2004 at 08:00:04AM +0100, Thiemo Seufer wrote:
> > Why bother, the unaligned exception handler should take care of this.
>
> It really does so for unaligned accesses from kernel space?
Yes. In fact it's crucially important for this very case. TCP for example
may result in missalignment. And not everybody is using get_unaligned /
put_unaligned as they were intended. Relying on the unaligned handler
is preferable where we expect pointers to be properly aligned almost
always.
The MIPS ABI mandates at least 8 byte stack alignment and funny things
happen if that assumption is violated. So there is no motivation at all
to care about the performance of missalignment. Aside of me defining this
to be verboten by punishment of signal 9 ;-)
> has 4 bytes and is loaded with lw. Using a macro which abstracts for
> 32/64bit compilation hides this needlessly, and can even lead to the
> erraneous impression the code would be useful for 64bit, too.
I'm more following the religion of using such abstractions everywhere
because code tends to be copied around mindlessly ...
Ralf
next prev parent reply other threads:[~2004-11-22 7:15 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2004-11-21 16:45 [PATCH] Improve o32 syscall handling Thiemo Seufer
2004-11-22 6:18 ` Ralf Baechle
2004-11-22 7:00 ` Thiemo Seufer
2004-11-22 7:13 ` Ralf Baechle [this message]
2004-11-22 9:37 ` Thiemo Seufer
2004-11-27 0:29 ` Thiemo Seufer
2004-11-23 12:47 ` Gleb O. Raiko
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=20041122071313.GC25433@linux-mips.org \
--to=ralf@linux-mips.org \
--cc=ica2_ts@csv.ica.uni-stuttgart.de \
--cc=linux-mips@linux-mips.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