public inbox for linux-sh@vger.kernel.org
 help / color / mirror / Atom feed
From: Kristoffer Ericson <kristoffer.ericson@gmail.com>
To: linux-sh@vger.kernel.org
Subject: Re: [PATCH 04/04] sh: prefetch early exception data on sh3/sh4/sh4a
Date: Mon, 16 Mar 2009 12:00:47 +0000	[thread overview]
Message-ID: <20090316130047.984f938b.kristoffer.ericson@gmail.com> (raw)
In-Reply-To: <20090223071931.12300.72669.sendpatchset@rx1.opensource.se>

[-- Attachment #1: Type: text/plain, Size: 2416 bytes --]

On Mon, 2 Mar 2009 11:30:13 +0900
Paul Mundt <lethal@linux-sh.org> wrote:

> On Fri, Feb 27, 2009 at 04:53:00PM +0900, Magnus Damm wrote:
> > On Fri, Feb 27, 2009 at 4:22 PM, Paul Mundt <lethal@linux-sh.org> wrote:
> > > On Mon, Feb 23, 2009 at 04:19:31PM +0900, Magnus Damm wrote:
> > >> From: Magnus Damm <damm@igel.co.jp>
> > >>
> > >> Prefetch early exception data. There is unused space in our
> > >> exception handler cache line anyway, so this is almost free.
> > >>
> > > Hmm, this won't work, 'pref' doesn't exist on SH-3. The binutils opcode
> > > table defines it as arch_sh2a_nofpu_or_sh4_nommu_nofpu_up, which excludes
> > > SH-3. mova is ok, though.
> > >
> > > This does show how often SH-3 gets tested though, as asm/processor_32.h
> > > happily defines prefetch() for SH-3 when it shouldn't. I'll tidy this up
> > > and drop in a PREF() macro.
> > 
> > Right, I do recall building for hp6xx at some point. Thanks for fixing it up.
> > 
> The fact that that didn't blow up on the opcode is likely because the
> toolchain you were using had no idea about SH-3 ISA tuning, and so fell
> back on an anything-goes tuning policy. If you are building for exotic
> combinations, always do a single run with V=1 to make sure that the
> CFLAGS are sticking and doing what you think they are doing.
> 
> There are far more people building SH-3 targetted kernels with an SH-4+
> specific toolchain than there are actual SH-3 users, the latter of which
> tend to only provide feedback a few times a year. This unfortunately
> means that these sorts of bugs can be long lived and tedious to bisect at
> a later point in time, but there is not much that can be done about that.
> --

I usually build a couple of kernels per month for the hp6xx (sh3) so the biggest
stuff kernel wise usually gets caught there. I however use an old sh3 toolchain so
gcc stuff often runs me by. Lack of time to fix the hd64461 mfd driver is probably the main
reason why hp6xx isnt more in sync currently.
If theres anything specific you want to test just shout. I keep an close eye
on the mailinglists even if I dont say much :)



> To unsubscribe from this list: send the line "unsubscribe linux-sh" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html


-- 
Kristoffer Ericson <kristoffer.ericson@gmail.com>

[-- Attachment #2: Type: application/pgp-signature, Size: 197 bytes --]

  parent reply	other threads:[~2009-03-16 12:00 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-02-23  7:19 [PATCH 04/04] sh: prefetch early exception data on sh3/sh4/sh4a Magnus Damm
2009-02-27  7:22 ` Paul Mundt
2009-02-27  7:53 ` Magnus Damm
2009-03-02  2:30 ` Paul Mundt
2009-03-02  2:47 ` Paul Mundt
2009-03-02  2:48 ` Magnus Damm
2009-03-02  2:55 ` Magnus Damm
2009-03-16 12:00 ` Kristoffer Ericson [this message]
2009-03-16 12:50 ` Paul Mundt

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=20090316130047.984f938b.kristoffer.ericson@gmail.com \
    --to=kristoffer.ericson@gmail.com \
    --cc=linux-sh@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