[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: Fatfs patch for writing support

From: Marco Gerards
Subject: Re: Fatfs patch for writing support
Date: 02 Aug 2003 13:33:21 +0200
User-agent: Gnus/5.09 (Gnus v5.9.0) Emacs/21.2

Marcus Brinkmann <Marcus.Brinkmann@ruhr-uni-bochum.de> writes:

Thanks for you comments Marcus, I've uploaded the new patch and
changelog entry to savannah.

> > 2003-07-07  Marco Gerards  <metgerards@student.han.nl>
> > 
> >     * node-create.c: New file.
> It would be nice to have a comment in that file (like, before the actual
> content) why we need to override this from libdiskfs.  IE, explain the main
> difference in operation.

I've added it after the GPL notice.

> We don't repeat the function name this way, just list all changes in one
> entry.

> >     (diskfs_rewrite_hard): Function rewritten.
> >     
> >     (diskfs_dirempty): Change logic to test if a file was deleted.
> I don't like the empty lines here, with no new "* dir.c".  Either they are
> different changes and require a new "*", or they belong together.  In this
> case, I suggest to just lump everything together.  They are lots of small
> cleanups for one goal, write support.

Hmm... ok...
> >     * fat.c (fat_extend_chain): Move spin_lock to prevent deadlock.
> This one is wrong, you must not access dn->length_of_chain without holding
> the lock (the caller only has the reader lock of alloc_lock).  So just
> unlock the lock before the "return 0" in the if (because that is the bug you
> were seeing, I guess).



reply via email to

[Prev in Thread] Current Thread [Next in Thread]