All of lore.kernel.org
 help / color / mirror / Atom feed
From: Artem Bityutskiy <dedekind1@gmail.com>
To: Stefani Seibold <stefani@seibold.net>
Cc: Artem Bityutskiy <Artem.Bityutskiy@nokia.com>,
	"Enzinger,
	Robert \(EXT-Other - DE/Munich\)" <robert.enzinger.ext@nsn.com>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	"linux-mtd@lists.infradead.org" <linux-mtd@lists.infradead.org>,
	"akpm@linux-foundation.org" <akpm@linux-foundation.org>,
	David Woodhouse <dwmw2@infradead.org>
Subject: Re: [PATCH] Add quick erase format option
Date: Sun, 29 Aug 2010 15:20:24 +0300	[thread overview]
Message-ID: <1283084424.2131.40.camel@brekeke> (raw)
In-Reply-To: <1283081435.2131.24.camel@brekeke>

On Sun, 2010-08-29 at 14:30 +0300, Artem Bityutskiy wrote:
> The reason why this flushing is needed is that we want to make sure
> that
> there are no PEBs which belonged to a volume with the same ID. Indeed,
> imaging you have volume 5 with many used LEBs. Then you remove this
> volume, so that all its LEBs are scheduled for erasure. Then you
> create
> a new volume with the same ID = 5. If we do not flush the queue, we
> may
> end up with rubbish from the older volume 5 in the new one. 

Err, it is important to say that we'll end up with the rubbish if we
have an unclean reboot after the new volume with ID 5 is created, but
before all PEBs belonging to the old volume with ID 5 are erased.
Hopefully, my explanation is not too messy :-)

-- 
Best Regards,
Artem Bityutskiy (Битюцкий Артём)

WARNING: multiple messages have this Message-ID (diff)
From: Artem Bityutskiy <dedekind1@gmail.com>
To: Stefani Seibold <stefani@seibold.net>
Cc: David Woodhouse <dwmw2@infradead.org>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	"akpm@linux-foundation.org" <akpm@linux-foundation.org>,
	Artem Bityutskiy <Artem.Bityutskiy@nokia.com>,
	"linux-mtd@lists.infradead.org" <linux-mtd@lists.infradead.org>,
	"Enzinger,
	Robert (EXT-Other - DE/Munich)"  <robert.enzinger.ext@nsn.com>
Subject: Re: [PATCH] Add quick erase format option
Date: Sun, 29 Aug 2010 15:20:24 +0300	[thread overview]
Message-ID: <1283084424.2131.40.camel@brekeke> (raw)
In-Reply-To: <1283081435.2131.24.camel@brekeke>

On Sun, 2010-08-29 at 14:30 +0300, Artem Bityutskiy wrote:
> The reason why this flushing is needed is that we want to make sure
> that
> there are no PEBs which belonged to a volume with the same ID. Indeed,
> imaging you have volume 5 with many used LEBs. Then you remove this
> volume, so that all its LEBs are scheduled for erasure. Then you
> create
> a new volume with the same ID = 5. If we do not flush the queue, we
> may
> end up with rubbish from the older volume 5 in the new one. 

Err, it is important to say that we'll end up with the rubbish if we
have an unclean reboot after the new volume with ID 5 is created, but
before all PEBs belonging to the old volume with ID 5 are erased.
Hopefully, my explanation is not too messy :-)

-- 
Best Regards,
Artem Bityutskiy (Битюцкий Артём)


  reply	other threads:[~2010-08-29 12:20 UTC|newest]

Thread overview: 26+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-08-09  8:25 [PATCH] Add quick erase format option stefani
2010-08-09  8:37 ` David Woodhouse
2010-08-09  8:37   ` David Woodhouse
2010-08-09  8:52   ` Stefani Seibold
2010-08-09  8:52     ` Stefani Seibold
2010-08-09 11:29     ` Artem Bityutskiy
2010-08-09 11:29       ` Artem Bityutskiy
2010-08-09 13:37       ` Stefani Seibold
2010-08-09 13:37         ` Stefani Seibold
2010-08-09 13:54       ` Stefani Seibold
2010-08-09 13:54         ` Stefani Seibold
2010-08-29 11:30         ` Artem Bityutskiy
2010-08-29 11:30           ` Artem Bityutskiy
2010-08-29 12:20           ` Artem Bityutskiy [this message]
2010-08-29 12:20             ` Artem Bityutskiy
2010-08-31  6:42           ` Stefani Seibold
2010-08-31  6:42             ` Stefani Seibold
2010-09-01  0:47             ` Artem Bityutskiy
2010-09-01  0:47               ` Artem Bityutskiy
2010-09-02  6:53               ` Stefani Seibold
2010-09-02  6:53                 ` Stefani Seibold
2010-09-02 10:58                 ` Artem Bityutskiy
2010-09-02 10:58                   ` Artem Bityutskiy
2010-09-02 11:42                   ` Stefani Seibold
2010-09-02 11:42                     ` Stefani Seibold
2018-06-20  5:38     ` Richard Weinberger

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=1283084424.2131.40.camel@brekeke \
    --to=dedekind1@gmail.com \
    --cc=Artem.Bityutskiy@nokia.com \
    --cc=akpm@linux-foundation.org \
    --cc=dwmw2@infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mtd@lists.infradead.org \
    --cc=robert.enzinger.ext@nsn.com \
    --cc=stefani@seibold.net \
    /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.