linuxppc-dev.lists.ozlabs.org archive mirror
 help / color / mirror / Atom feed
From: Benjamin Herrenschmidt <benh@kernel.crashing.org>
To: Manoj Sharma <manoj.shar@gmail.com>
Cc: linuxppc-dev@ozlabs.org
Subject: Re: Hang with isync
Date: Thu, 21 Sep 2006 07:35:11 +1000	[thread overview]
Message-ID: <1158788111.6002.310.camel@localhost.localdomain> (raw)
In-Reply-To: <d6dada100609191816q75d1455cobeb1e0bd21dab8a8@mail.gmail.com>

On Tue, 2006-09-19 at 18:16 -0700, Manoj Sharma wrote:
>         Hi,
>         
>         We use linux kernel 2.4.20 on ppc405 and the system hangs once
>         in a while when isync gets called in this function: 
>         
>         _GLOBAL(_nmask_and_or_msr)
>             mfmsr   r0      /* Get current msr */
>             andc    r0,r0,r3    /* And off the bits set in r3 (first
>         parm) */ 
>             or  r0,r0,r4    /* Or on the bits in r4 (second parm) */
>             sync            /* Some chip revs have problems here... */
>             isync
>             mtmsr   r0      /* Update machine state */
>             isync
>             blr         /* Done */ 
>         
>          2.5 onwards, I find that "sync; isync" has been replaced by a
>         macro SYNC (defined only for 601). I don't find it in any
>         changelog and reason for the change.
>         
>         Can someone give some information on this change? 

Regardless of the change... on 2.4, _nmask_and_or_msr() was used for a
number of things. We would need to know where it was called from with
what values as arguments to have an idea of what's going wrong. It's
probably not dying on the isync, but rather on the following mtmsr due
to a problem with the values passed in....

Ben.

  reply	other threads:[~2006-09-20 21:35 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-09-19  1:28 Hang with isync Manoj Sharma
2006-09-20  1:16 ` Manoj Sharma
2006-09-20 21:35   ` Benjamin Herrenschmidt [this message]
2006-09-20 22:31     ` Manoj Sharma
2006-09-20 22:38       ` Benjamin Herrenschmidt
2006-09-20 23:04         ` Linas Vepstas
2006-09-21  0:59           ` Manoj Sharma
2006-09-21  4:24             ` Liu Dave-r63238
2006-09-21  6:17               ` Manoj Sharma
2006-09-21  6:27                 ` Liu Dave-r63238
2006-09-21  7:11                   ` Manoj Sharma

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=1158788111.6002.310.camel@localhost.localdomain \
    --to=benh@kernel.crashing.org \
    --cc=linuxppc-dev@ozlabs.org \
    --cc=manoj.shar@gmail.com \
    /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 NNTP newsgroup(s).