All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Alexander Belyakov" <abelyako@googlemail.com>
To: "David Woodhouse" <dwmw2@infradead.org>
Cc: "Alexander Belyakov" <abelyako@mail.ru>,
	"Jörn Engel" <joern@logfs.org>,
	"linux-mtd@lists.infradead.org" <linux-mtd@lists.infradead.org>
Subject: Re: [PATCH][JFFS2] Fix garbage collector block search
Date: Wed, 23 Apr 2008 15:31:37 +0400	[thread overview]
Message-ID: <fe810cf0804230431m2bf80e83h4fa71bd5887b7587@mail.gmail.com> (raw)
In-Reply-To: <1208882833.9212.629.camel@pmac.infradead.org>

On Tue, Apr 22, 2008 at 8:47 PM, David Woodhouse <dwmw2@infradead.org> wrote:
>
> On Fri, 2008-01-18 at 19:28 +0300, Alexander Belyakov wrote:
>  >
>  > Question. What is success criteria for jffs2_garbage_collect_pass() execution?
>  >
>
>  It depends. There are two callers of jffs2_garbage_collect_pass().
>
>  For jffs2_flush_wbuf_gc() it's not progress, so it should be handled
>  like failure.
>
>  For jffs2_reserve_space(), we can treat it as success... or we can have
>  a special case which calls jffs2_erase_pending_blocks() right away to
>  wait for the block to erase.
>
>  How about this... does it solve the problem for you?
>

I like this special case for jffs2_reserve_space(). It solves the problem.

Thank you,
Alexander

  reply	other threads:[~2008-04-23 11:31 UTC|newest]

Thread overview: 25+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-12-12 15:51 [PATCH][JFFS2] Fix garbage collector block search Alexander Belyakov
2008-01-14 10:23 ` Alexander Belyakov
2008-01-14 13:28 ` Jörn Engel
2008-01-15 12:24   ` Alexander Belyakov
2008-01-15 13:33     ` Jörn Engel
2008-01-18 16:28       ` Alexander Belyakov
2008-02-25 12:50         ` Jörn Engel
2008-04-22 16:47         ` David Woodhouse
2008-04-23 11:31           ` Alexander Belyakov [this message]
2008-04-23 11:32             ` David Woodhouse
2008-04-23 13:08               ` Alexander Belyakov
2008-04-23 14:01                 ` David Woodhouse
2008-04-23 14:30                   ` David Woodhouse
2008-04-23 15:25                     ` Alexander Belyakov
2008-04-23 15:30                       ` David Woodhouse
2008-04-24  7:17                         ` Alexander Belyakov
2008-04-24  7:28                           ` David Woodhouse
2008-04-24  7:48                             ` Alexander Belyakov
2008-04-24  7:51                               ` David Woodhouse
2008-04-24 10:14                                 ` Alexander Belyakov
2008-04-24 10:32                                   ` David Woodhouse
2008-04-24 10:42                                     ` Alexander Belyakov
2008-04-24 11:05                                       ` David Woodhouse
2008-04-24 11:11                                         ` Jörn Engel
2008-04-24 11:21                                         ` Alexander Belyakov

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=fe810cf0804230431m2bf80e83h4fa71bd5887b7587@mail.gmail.com \
    --to=abelyako@googlemail.com \
    --cc=abelyako@mail.ru \
    --cc=dwmw2@infradead.org \
    --cc=joern@logfs.org \
    --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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.