This is the mail archive of the ecos-maintainers@sources.redhat.com mailing list for the eCos project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: FWD: JFFS2 : Node info mutex fix suggestion.


On Tue, 2004-02-17 at 08:09, Gary Thomas wrote:
> On Tue, 2004-02-17 at 07:58, David Woodhouse wrote:
> > On Tue, 2004-02-17 at 07:11 -0700, Gary Thomas wrote:
> > > On Mon, 2004-02-16 at 10:07, Andrew Lunn wrote:
> > > > Hi Folks
> > > > 
> > > > Does anybody know how locking should work in the jffs2 package? 
> > > > 
> > > > To me this patch looks the wrong solution. 
> > > 
> > > I agree.  I did already pass this to David, with no reply :-(
> > 
> > Hmmm. I think the answer is to remove the one in jffs2_do_new_inode()
> > instead.
> > 
> > That function should rely on the OS-specific code to pass it a struct
> > jffs2_inode_info which is initialised correctly, and that includes
> > having a locked semaphore in it.
> > 
> > I've committed the appropriate change to my CVS tree.
> 
> Thanks, I'll merge this to the eCos repository.  There are also some 
> changes in our tree (CDL stuff mostly) that need to be pushed your
> way.  I'll take care of that as well.

Done (diffs attached).

David, what magic do you use to run "syncmail" when you make CVS
changes?
-- 
Gary Thomas <gary@mlbassoc.com>
MLB Associates

Attachment: diffs
Description: Text document


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]