public inbox for linux-mtd@lists.infradead.org
 help / color / mirror / Atom feed
* Erase.c in CVS tree, 08-13-2001 version does not compile
@ 2001-08-14 13:55 Frederic Giasson
  2001-08-14 14:03 ` David Woodhouse
  0 siblings, 1 reply; 2+ messages in thread
From: Frederic Giasson @ 2001-08-14 13:55 UTC (permalink / raw)
  To: 'David Woodhouse', Vipin Malik
  Cc: Frederic Giasson, MTD mailing list (E-mail)

Erase.c in CVS tree does not compile.  This line have to be put this line at
the beggining of the D2 block:
        struct jffs2_raw_node_ref *this;

Frédéric Giasson





-----Original Message-----
From: David Woodhouse [mailto:dwmw2@infradead.org]
Sent: Monday, August 13, 2001 8:00 PM
To: Vipin Malik
Cc: Frederic Giasson; MTD mailing list (E-mail)
Subject: Re: JFFS2 is broken 



vipin@embeddedlinuxworks.com said:
> David once mentioned that there are some obvious optimizations that
> can be carried out in the JFFS2 code. Maybe he could list out his
> favorite ones  and maybe someone could volunteer to tackle them one by
> one. 

I committed the cleanup to jffs2_remove_node_refs_from_ino_list() (Christ, 
did I name that function?), which showed up near the top of the profiles 
you did. What's next? ISTR we got strange data from the profiling. 

--
dwmw2

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: Erase.c in CVS tree, 08-13-2001 version does not compile
  2001-08-14 13:55 Erase.c in CVS tree, 08-13-2001 version does not compile Frederic Giasson
@ 2001-08-14 14:03 ` David Woodhouse
  0 siblings, 0 replies; 2+ messages in thread
From: David Woodhouse @ 2001-08-14 14:03 UTC (permalink / raw)
  To: Frederic Giasson; +Cc: Vipin Malik, MTD mailing list (E-mail)

fgiasson@mediatrix.com said:
> Erase.c in CVS tree does not compile.  This line have to be put this
> line at the beggining of the D2 block:
>         struct jffs2_raw_node_ref *this;

Sorry about that. Please commit it.

--
dwmw2

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2001-08-14 13:58 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2001-08-14 13:55 Erase.c in CVS tree, 08-13-2001 version does not compile Frederic Giasson
2001-08-14 14:03 ` David Woodhouse

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox