This is the mail archive of the ecos-patches@sourceware.org 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: fix thread instrumentation parameters


Hi Pieter-Jan

Pieter-Jan Busschaert wrote:

> While looking through instrumentation data, we found a bug in the data
> passed as arguments for THREAD_WAKE, THREAD_SUSPEND and THREAD_RESUME.
> 
> In revision 1.10 of sched.hxx, SMP support was added, which changed
> current_thread from a simple pointer to an array-of-pointers. A
> convenience function get_current_thread() was added. The below
> instances of instrumentation code were not adapted, which makes them
> always pass the address of the array instead of a real thread pointer.

Yes, this is clearly a bug and the fix is trivial. Thank you for
bringing this to our attention.

John Dallaway


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