All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jens Axboe <axboe@kernel.dk>
To: dedekind1@gmail.com
Cc: Christoph Hellwig <hch@lst.de>, linux-fsdevel@vger.kernel.org
Subject: Re: [PATCH, RFC] simplify writeback thread creation
Date: Fri, 09 Jul 2010 10:16:40 +0200	[thread overview]
Message-ID: <4C36DAE8.3070302@kernel.dk> (raw)
In-Reply-To: <1278661979.9953.8.camel@localhost>

On 2010-07-09 09:52, Artem Bityutskiy wrote:
> On Thu, 2010-07-08 at 20:48 +0200, Christoph Hellwig wrote:
>> On Thu, Jul 08, 2010 at 09:43:22PM +0300, Artem Bityutskiy wrote:
>>> Hmm, was thinking about this while driving home - the forker approach
>>> has a good resilience property - if it cannot fork - it'll do the stuff
>>> itself. I have a feeling that if something like this to be implemented
>>> with the approach I suggested, we'll end up with similar level of
>>> complexity that we wanted to get rid of...
>>
>> Yes, the lazy starting is what adds the complexity.  I think starting
>> it once we have any filesystem mounted on the bdi and stop it once all
>> filesystems are gone is a lot simpler and more elegant.
> 
> But what about cases like 'dd if=/dev/zero of=/dev/sda4'? They also
> involve dirty data write-back.

You would have to do it at device open time, if the thread does
not already exist.

Not sure this is all worth it, I think the complexity of the
lazy create/exit is a bit exaggerated. 

-- 
Jens Axboe


  reply	other threads:[~2010-07-09  8:16 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-07-07 22:52 [PATCH, RFC] simplify writeback thread creation Christoph Hellwig
2010-07-08  7:08 ` Artem Bityutskiy
2010-07-08 12:20 ` Jens Axboe
2010-07-08 14:21   ` Artem Bityutskiy
2010-07-08 14:59     ` Jens Axboe
2010-07-08 15:23       ` Artem Bityutskiy
2010-07-08 17:23         ` Jens Axboe
2010-07-08 18:43           ` Artem Bityutskiy
2010-07-08 18:48             ` Christoph Hellwig
2010-07-09  7:52               ` Artem Bityutskiy
2010-07-09  8:16                 ` Jens Axboe [this message]
2010-07-09 11:06                   ` Theodore Tso
2010-07-09 14:51                 ` Christoph Hellwig
2010-07-09 15:49                   ` Artem Bityutskiy
2010-07-08 13:22 ` Artem Bityutskiy

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=4C36DAE8.3070302@kernel.dk \
    --to=axboe@kernel.dk \
    --cc=dedekind1@gmail.com \
    --cc=hch@lst.de \
    --cc=linux-fsdevel@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 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.