public inbox for linux-mtd@lists.infradead.org
 help / color / mirror / Atom feed
From: "Martin Creutziger" <martin.creutziger@barco.com>
To: "linux-mtd" <linux-mtd@lists.infradead.org>
Subject: Re: [PATCH] [JFFS2] Fix free space leaking
Date: Thu, 03 Apr 2008 11:03:04 +0200	[thread overview]
Message-ID: <1207213384.6161.4.camel@KARCLT0275> (raw)
In-Reply-To: <1203593969.15409.45.camel@shinybook.infradead.org>


On Thu, 2008-02-21 at 20:39 +0900, an unknown sender wrote:
> On Thu, 2008-02-21 at 12:11 +0100, J?rn Engel wrote:
> > No idea, but it might help to add dwmw2 to Cc:.
> 
> dwmw2 moderately confused by it. I don't see why it's correct to account
> for the space as 'free' when it isn't really. If it's causing a
> persistent leak, shouldn't we deal with that when the block is erased
> instead?

Hi, 

I am currently testing the patch on 2.6.24.2 (as a result of
http://lists.infradead.org/pipermail/linux-mtd/2008-March/021024.html),
and so far it seems to at least cure the symptoms, as in: The partition
does not "fill up" any more so far. 
But: For later production use I would certainly prefer a patch that also
dwmw2 agrees being a cure to the _cause_, not just the symptoms.
Any suggestions?

Greetings . . . Martin



DISCLAIMER:
Unless indicated otherwise, the information contained in this message is privileged and confidential, and is intended only for the use of the addressee(s) named above and others who have been specifically authorized to receive it. If you are not the intended recipient, you are hereby notified that any dissemination, distribution or copying of this message and/or attachments is strictly prohibited. The company accepts no liability for any damage caused by any virus transmitted by this email. Furthermore, the company does not warrant a proper and complete transmission of this information, nor does it accept liability for any delays. If you have received this message in error, please contact the sender and delete the message. Thank you.

  reply	other threads:[~2008-04-03  9:03 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-11-13 10:23 [PATCH] [JFFS2] Fix free space leaking Damir Shayhutdinov
2008-02-06 10:20 ` Joakim Tjernlund
2008-02-06 18:48   ` Damir Shayhutdinov
2008-02-21 11:11   ` Jörn Engel
2008-02-21 11:39     ` David Woodhouse
2008-04-03  9:03       ` Martin Creutziger [this message]
2008-04-22 23:16         ` David Woodhouse

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=1207213384.6161.4.camel@KARCLT0275 \
    --to=martin.creutziger@barco.com \
    --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