回复:linux-mtd Digest, Vol 51, Issue 8

[Michael Wang]

Sent by mobile phone.

原邮件
自:linux-mtd-request at lists.infradead.org
日期:06/04/2007 21:32
至:linux-mtd at lists.infradead.org
主题: linux-mtd Digest, Vol 51, Issue 8

Send linux-mtd mailing list submissions to
	linux-mtd at lists.infradead.org

To subscribe or unsubscribe via the World Wide Web, visit
	http://lists.infradead.org/mailman/listinfo/linux-mtd
or, via email, send a message with subject or body 'help' to
	linux-mtd-request at lists.infradead.org

You can reach the person managing the list at
	linux-mtd-owner at lists.infradead.org

When replying, please edit your Subject line so it is more specific
than "Re: Contents of linux-mtd digest..."


Today's Topics:

   1. Re: [PATCH] JFFS2 appears to "freeze" during erase
      (Joakim Tjernlund)
   2. Re: [Patch 05/18] fs/logfs/logfs.h (Jan Engelhardt)
   3. Re: [PATCH] JFFS2 appears to "freeze" during erase
      (Joakim Tjernlund)
   4. Re: [PATCH] powerpc: Create "rom" (MTD) device prpmc2800
      (Sergei Shtylyov)
   5. Re: [PATCH] powerpc: Create "rom" (MTD) device prpmc2800
      (Sergei Shtylyov)
   6. [Yaffs1] mkyaffs exits with "MTD Erase failure"
      (Martin Egholm Nielsen)
   7. Re: [PATCH] powerpc: Create "rom" (MTD) device prpmc2800
      (Sergei Shtylyov)
   8. Re: [Patch 05/18] fs/logfs/logfs.h (J?rn Engel)
   9. Re: [PATCH] powerpc: Create "rom" (MTD) device prpmc2800
      (Sergei Shtylyov)


----------------------------------------------------------------------

Message: 1
Date: Mon, 04 Jun 2007 11:20:37 +0200
From: Joakim Tjernlund <joakim.tjernlund at transmode.se>
Subject: Re: [PATCH] JFFS2 appears to "freeze" during erase
To: David Woodhouse <dwmw2 at infradead.org>
Cc: radbis at googlemail.com,	Linux MTD mailing list
	<linux-mtd at lists.infradead.org>
Message-ID: <1180948837.30670.3.camel at gentoo-jocke.transmode.se>
Content-Type: text/plain

On Fri, 2007-06-01 at 19:04 +0100, David Woodhouse wrote:
> On Fri, 2007-06-01 at 11:51 +0200, Joakim Tjernlund wrote:
> > This is a old patch from Radoslaw Bisewski(see MTD list). The patch
> > fixes the long stalls I am seeing in "GC starves write's in 2.6.20" I
> > sent yesterday. Please apply.
> 
> It seems to make sense; I need to convince myself that every other user
> of the erase_free_sem is actually OK with it. I think
> jffs2_g_c_deletion_dirent() should be OK, at first glance. I'm less
> convinced about jffs2_mark_node_obsolete(), where we currently hold the
> erase_free_sem while we actually mark the node obsolete. Will look into
> it...
> 

looked a litte closer on the new locking in erase.c and wonder
why not jffs2_erase_succeeded() and jffs2_erase_failed() needs
to take erase_free_sem?

 Jocke



------------------------------

Message: 2
Date: Mon, 4 Jun 2007 10:17:47 +0200 (MEST)
From: Jan Engelhardt <jengelh at linux01.gwdg.de>
Subject: Re: [Patch 05/18] fs/logfs/logfs.h
To: Arnd Bergmann <arnd at arndb.de>
Cc: akpm at osdl.org, David Weinehall <tao at acc.umu.se>,	Dongjun Shin
	<djshin90 at gmail.com>, Kyle Moffett <mrmacman_g4 at mac.com>,
	linux-mtd at lists.infradead.org, Pavel Machek <pavel at ucw.cz>,	Sam
	Ravnborg <sam at ravnborg.org>,	Ulisses Furquim <ulissesf at gmail.com>, CaT
	<cat at zip.com.au>,	Evgeniy Polyakov <johnpol at 2ka.mipt.ru>, Roland
	Dreier <rdreier at cisco.com>, J?rn Engel <joern at lazybastard.org>,	Jamie
	Lokier <jamie at shareable.org>, Pekka Enberg <penberg at cs.helsinki.fi>,
	Thomas Gleixner <tglx at linutronix.de>, Bill Davidsen
	<davidsen at tmr.com>,	Albert Cahalan <acahalan at gmail.com>,	John Stoffel
	<john at stoffel.org>, linux-kernel at vger.kernel.org,	Ondrej Zajicek
	<santiago at crfreenet.org>, linux-fsdevel at vger.kernel.org,	David
	Woodhouse <dwmw2 at infradead.org>, Willy Tarreau <w at 1wt.eu>
Message-ID: <Pine.LNX.4.61.0706041017270.20829 at yvahk01.tjqt.qr>
Content-Type: TEXT/PLAIN; charset=UTF-8


>On Sunday 03 June 2007, J?rn Engel wrote:
>> +/**
>> + * struct logfs_device_ops - device access operations
>> + *
>> + * @read:??????????????????????read from the device
>> + * @write:?????????????????????write to the device
>> + * @erase:?????????????????????erase part of the device
>> + */
>> +struct logfs_device_ops {
>> +???????int (*read)(struct super_block *sb, loff_t ofs, size_t len, void *buf);
>> +???????int (*write)(struct super_block *sb, loff_t ofs, size_t len, void *buf);
>> +???????int (*erase)(struct super_block *sb, loff_t ofs, size_t len);
>> +};
>
>I wonder if there is a way to document the prototypes of these function
>pointers with kerneldoc, other than having a typedef for each.
>
>What brought me to this point is that I first assumed they would return
>the number of bytes transferred, like read/write file operations, where
>your functions return zero on success.

read/write functions returning bytes written would return ssize_t,
just as vfs_read and vfs_write do.



	Jan
-- 



------------------------------

Message: 3
Date: Mon, 04 Jun 2007 11:44:30 +0200
From: Joakim Tjernlund <joakim.tjernlund at transmode.se>