public inbox for ecos-discuss@sourceware.org
 help / color / mirror / Atom feed
* AW: [ECOS] [PATCH] fix possible jffs2 deadlock (sync with mtd cvs)
@ 2004-08-02 13:09 Neundorf, Alexander
  2004-08-02 14:08 ` Andrew Lunn
  0 siblings, 1 reply; 3+ messages in thread
From: Neundorf, Alexander @ 2004-08-02 13:09 UTC (permalink / raw)
  To: Andrew Lunn; +Cc: ecos-discuss


...
> > Ok, I'll have a look.
> > What about the patch ?
> 
> It was decided by the jffs2 maintainer to reject the patch. Its the
> compiler that is broken, not JFFS2. Fixing the compiler is the proper
> solution.
> 
>         Andrew

No, not that patch.
I mean the patch which is attached to my previous mail, which fixes the semaphore-unlocking problem (and which is taken from mtd cvs HEAD). This is unrelated to the arm-gcc problem.

Bye
Alex
 

--
Before posting, please read the FAQ: http://ecos.sourceware.org/fom/ecos
and search the list archive: http://ecos.sourceware.org/ml/ecos-discuss

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

* Re: [ECOS] [PATCH] fix possible jffs2 deadlock (sync with mtd cvs)
  2004-08-02 13:09 AW: [ECOS] [PATCH] fix possible jffs2 deadlock (sync with mtd cvs) Neundorf, Alexander
@ 2004-08-02 14:08 ` Andrew Lunn
  0 siblings, 0 replies; 3+ messages in thread
From: Andrew Lunn @ 2004-08-02 14:08 UTC (permalink / raw)
  To: Neundorf, Alexander; +Cc: ecos-discuss

On Mon, Aug 02, 2004 at 03:09:51PM +0200, Neundorf, Alexander wrote:
> 
> ...
> > > Ok, I'll have a look.
> > > What about the patch ?
> > 
> > It was decided by the jffs2 maintainer to reject the patch. Its the
> > compiler that is broken, not JFFS2. Fixing the compiler is the proper
> > solution.
> > 
> >         Andrew
> 

> No, not that patch.  I mean the patch which is attached to my
> previous mail, which fixes the semaphore-unlocking problem (and
> which is taken from mtd cvs HEAD). This is unrelated to the arm-gcc
> problem.

Sorry, my mistake. 

I want to resync the eCos jffs2 code with dwnw2 repository which will
automatically pull in your patch. But its not as simple as that
becasue the last time this was done an eCos change was overwritten so
that needs putting back and then committing to the dwnw2 repository,
plus i have some other local changes in my tree for the reworked flash
API which should not be committed....

This will happen, but not today.

        Andrew

-- 
Before posting, please read the FAQ: http://ecos.sourceware.org/fom/ecos
and search the list archive: http://ecos.sourceware.org/ml/ecos-discuss

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

* AW: [ECOS] [PATCH] fix possible jffs2 deadlock (sync with mtd cvs)
@ 2004-08-02 12:55 Neundorf, Alexander
  0 siblings, 0 replies; 3+ messages in thread
From: Neundorf, Alexander @ 2004-08-02 12:55 UTC (permalink / raw)
  To: Andrew Lunn; +Cc: ecos-discuss


...
> > Another question:
> > 
> > The file jffs2/..../fs-ecos.c contains a check for gcc 
> 3.2.x and ARM_ARCH_4 -> #error This compiler is known to be broken
> > 
> > Does this also apply for ARM_ARCH_5 ?
> > It compiles here with -mcpu=xscale and gcc 3.2.1 ...
> 
> I don't know if the strongARM code is also broken. I suggest you take
> a look at the referenced discussion and look at the assembly language
> your compiler is generating. If it is also broken let me know and i
> will extend the #error for ARM_ARCH_5.
> 

Ok, I'll have a look.
What about the patch ?

Bye
Alex

--
Before posting, please read the FAQ: http://ecos.sourceware.org/fom/ecos
and search the list archive: http://ecos.sourceware.org/ml/ecos-discuss

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

end of thread, other threads:[~2004-08-02 14:08 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2004-08-02 13:09 AW: [ECOS] [PATCH] fix possible jffs2 deadlock (sync with mtd cvs) Neundorf, Alexander
2004-08-02 14:08 ` Andrew Lunn
  -- strict thread matches above, loose matches on Subject: below --
2004-08-02 12:55 AW: " Neundorf, Alexander

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).