mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Andrey Savochkin <saw@saw.sw.com.sg>
To: root@chaos.analogic.com
Cc: Colin Walters <walters@cis.ohio-state.edu>, linux-kernel@vger.kernel.org
Subject: Re: eepro100 (PCI ID 82820) lockups/failure
Date: Tue, 7 Aug 2001 03:24:43 -0700	[thread overview]
Message-ID: <20010807032443.A10193@saw.sw.com.sg> (raw)
In-Reply-To: <873d75janh.church.of.emacs@space-ghost.verbum.org> <Pine.LNX.3.95.1010806144632.8686A-100000@chaos.analogic.com>
In-Reply-To: <Pine.LNX.3.95.1010806144632.8686A-100000@chaos.analogic.com>; from "Richard B. Johnson" on Mon, Aug 06, 2001 at 03:00:07PM

Hi,

On Mon, Aug 06, 2001 at 03:00:07PM -0400, Richard B. Johnson wrote:
[snip]
> This may not be a timing problem, but rather a problem that was
> attempted to be fixed with some timing change.
> 
> Possible problem (and solution). Given:
> 
> 	writel(value, pci_reg);
> 	status = readl(pci_reg);
> 
> The second readl() may (read will) complete before the writel().
> This is because writes to the PCI bus may be posted (queued). The
> first read will force all writes to complete, however the value
> read may be something that was not yet affected by the write.
> 
> 	writel(value, pci_reg);
> 	status = readl(pci_reg);
> 	status = readl(pci_reg);

Thanks for the note, I'll keep it in mind.

However, for this particular case I'm interested about a loop like
	while((a = readb(reg)) && --count >= 0);
I wonder if there are circumstances in which the repeated read's can return
"cached" values or whatever, so that the loop will result in significantly less
number of bus cycles than it's supposed?
My understanding is that there shouldn't be such.

Best regards
		Andrey

  reply	other threads:[~2001-08-07 10:29 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2001-08-04  6:06 Colin Walters
2001-08-06  9:27 ` Andrey Savochkin
2001-08-06 15:48   ` Joseph Cheek
2001-08-07 10:48     ` Andrey Savochkin
2001-08-07 16:35       ` Joseph Cheek
2001-08-06 18:39   ` Colin Walters
2001-08-06 19:00     ` Richard B. Johnson
2001-08-07 10:24       ` Andrey Savochkin [this message]
2001-08-07 12:11         ` Richard B. Johnson
2001-08-06 19:14     ` Alan Cox
2001-08-07 10:46     ` Andrey Savochkin

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=20010807032443.A10193@saw.sw.com.sg \
    --to=saw@saw.sw.com.sg \
    --cc=linux-kernel@vger.kernel.org \
    --cc=root@chaos.analogic.com \
    --cc=walters@cis.ohio-state.edu \
    /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

all inboxes | Powered by JetHome®