public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Arjan van de Ven <arjanv@redhat.com>
To: VDA <VDA@port.imtp.ilyichevsk.odessa.ua>
Cc: linux-kernel@vger.kernel.org
Subject: Re: Duron kernel crash (i686 works)
Date: Wed, 12 Sep 2001 12:23:03 +0100	[thread overview]
Message-ID: <3B9F4597.B10E428E@redhat.com> (raw)
In-Reply-To: <E15goos-0002le-00@the-village.bc.nu> <9184118686.20010912095919@port.imtp.ilyichevsk.odessa.ua> <3B9F3E4B.AB5E1D12@scali.no> <1715812347.20010912140853@port.imtp.ilyichevsk.odessa.ua>

VDA wrote:

> SP> Well, not necessarily. It might be that data just hasn't "arrived" yet because
> SP> of the movntq instruction.

this is wrong; the CPU _internal_ view of the data is always consistent,
regardless of movntq vs movq.
It's only the EXTERNAL view that is slightly different. "sfence" takes
care of syncing that.

 
> So why it is oopses then?
> Also, we don't want this data to arrive late or whatever.
> fast_copy_page must copy page (make it so that memcpy()==0).
> If it does not, it is too much "optimized".

It does; but if you read it back from memory and is corrupted, your
chipset corrupted it.
 
> SP> One thing that also puzzels me is that my is the fast_copy_page() routine laid
> SP> out like this :

[snip]

A better way to do it is to bencmark several routines at
> startup time and pick the best one. It is done now
> for RAID xor'ing routine.

I benchmarked several versions, see the testprogram at
http://www.fenrus.demon.nl/athlon.c

The interleaved one is faster on athlons because it seems to help AMD's
register aliasing logic
to operate better....

Anyway, since this code works for like 99% of the machines, and only 1%
seems to be affected, it really really really looks like a hardware bug.
This is also more or less proven by the reports that certain
biosversions "break" working setups by doing things to the via chipset
that make it break....

Greetings,
   Arjan van de Ven

  reply	other threads:[~2001-09-12 11:23 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2001-09-11  1:11 Duron kernel crash (i686 works) Roberto Jung Drebes
2001-09-11  3:30 ` Roberto Jung Drebes
2001-09-11 14:47 ` Alan Cox
2001-09-12  6:59   ` VDA
2001-09-12 10:51     ` Steffen Persvold
2001-09-12 11:08       ` VDA
2001-09-12 11:23         ` Arjan van de Ven [this message]
2001-09-12 12:48         ` Alan Cox
2001-09-12 13:48           ` VDA
2001-09-12 18:09             ` Mike Fedyk
     [not found] <154763769.20010911115644@port.imtp.ilyichevsk.odessa.ua>
2001-09-11 19:38 ` Roberto Jung Drebes
2001-09-12  6:56   ` Liakakis Kostas
2001-09-12  8:21     ` Morten Helgesen

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=3B9F4597.B10E428E@redhat.com \
    --to=arjanv@redhat.com \
    --cc=VDA@port.imtp.ilyichevsk.odessa.ua \
    --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