This is the mail archive of the ecos-patches@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: new memory debug feature


I had to make some more changes to make it compile again.

><cyg/kernel/kapi.h> is not needed. In fact i think memalloc is
>independant of the kernel, so using this header file would be a bad
>idea since it might not exist.

I had to add it back in since  memalloc/common/current/include/kapi.h
depends on the definition on e.g. cyg_handle_t.


> Now the problem. I added the missing weak attribute. However, this
> causes problems for me. When the option is enabled and a failure
> happens, it crashes. All the tests in memalloc package fail
> horribly. If i remove the weak attribute they work. 

Can't we just leave out the weak attribute?

cyg_memalloc_alloc_fail() has its own file "debug.c".

I find bugs in GCC as "fascinating" as the next guy :-), but I'm happy
to leave this one be.

> It looks like a problem with the linker or maybe the compiler. The
> image does not contain the cyg_memalloc_alloc_fail function. I've got
> binutils 2.14.90.0 and gcc 3.3.4 (Debian) and im testing on synth. 

I'm running arm-elf-gcc 3.2.1 and I get compilation errors with the weak
attribute defined.

> Could you try this patch on your system and let me know if it
> works/crashes.


My applogies for the late reply, this is not due to lack of interest
in this patch on my part :-)

-- 
Øyvind Harboe
http://www.zylin.com

Attachment: memdebug3.txt
Description: Text document


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