public inbox for linux-mtd@lists.infradead.org
 help / color / mirror / Atom feed
From: David Woodhouse <dwmw2@infradead.org>
To: joakim.tjernlund@lumentis.se
Cc: "Dave Ellis" <DGE@sixnetio.com>, linux-mtd@lists.infradead.org
Subject: Re: Disk blocks for long periods
Date: Thu, 08 Aug 2002 09:08:42 +0100	[thread overview]
Message-ID: <28529.1028794122@redhat.com> (raw)
In-Reply-To: <IGEFJKJNHJDCBKALBJLLMEJEFFAA.joakim.tjernlund@lumentis.se>

joakim.tjernlund@lumentis.se said:
>  You can move the wake_up() call up one step(xxxx_write_words()) and
> call it just before  you leave that function. I tried that and it's a
> improvement(note, I using buffer writes, so word writes should see an
> even better improvement), but that will also hold reads back(not that
> I noticed anything) until xxx_write_words has completed.  We can fix
> that by having separate wait queues for read and erase. 

Aren't reads held back anyway for the same reason that erases are -- by the 
time we actually schedule after waking them, the chip is in FL_WRITING 
again?

What happens if you stick a cond_resched() immediately after the wake_up() 
(ok, after the spin_unlock).?

--
dwmw2

  reply	other threads:[~2002-08-08  8:08 UTC|newest]

Thread overview: 28+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2002-08-07 16:42 Disk blocks for long periods Dave Ellis
2002-08-08  7:08 ` Joakim Tjernlund
2002-08-08  8:08   ` David Woodhouse [this message]
2002-08-08  9:15     ` Joakim Tjernlund
2002-08-08  9:18       ` David Woodhouse
  -- strict thread matches above, loose matches on Subject: below --
2002-08-06 20:16 Dave Ellis
2002-08-06 21:06 ` Joakim Tjernlund
2002-08-06 14:53 Dave Ellis
2002-08-06 15:09 ` Joakim Tjernlund
2002-08-07 11:11 ` David Woodhouse
2002-08-05 18:50 Dave Ellis
2002-08-06 10:06 ` Joakim Tjernlund
2002-08-06 12:14   ` David Woodhouse
2002-08-06 13:52     ` Jörn Engel
2002-08-06 13:53       ` David Woodhouse
2002-08-06 16:45     ` Joakim Tjernlund
2002-08-07  9:51       ` David Woodhouse
2002-08-08  7:23         ` Joakim Tjernlund
2002-08-08  8:02           ` David Woodhouse
2002-08-08  8:32             ` Joakim Tjernlund
2002-08-08  8:40               ` David Woodhouse
2002-08-05 10:35 MTD Partition problems David Woodhouse
2002-08-05 13:35 ` Disk blocks for long periods Joakim Tjernlund
2002-08-05 13:44   ` David Woodhouse
2002-08-05 13:59     ` Joakim Tjernlund
2002-08-05 14:12       ` David Woodhouse
2002-08-05 14:32         ` Joakim Tjernlund
2002-08-05 14:42           ` David Woodhouse
2002-08-05 21:45             ` Jasmine Strong

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=28529.1028794122@redhat.com \
    --to=dwmw2@infradead.org \
    --cc=DGE@sixnetio.com \
    --cc=joakim.tjernlund@lumentis.se \
    --cc=linux-mtd@lists.infradead.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