This is the mail archive of the
ecos-patches@sourceware.org
mailing list for the eCos project.
Re: add cdl_option CYGDBG_KERNEL_INSTRUMENT_MLQ
- From: John Dallaway <john at dallaway dot org dot uk>
- To: Pieter-Jan Busschaert <pieterjan dot busschaert at gmail dot com>
- Cc: ecos-patches at ecos dot sourceware dot org
- Date: Thu, 01 Oct 2009 09:40:41 +0100
- Subject: Re: add cdl_option CYGDBG_KERNEL_INSTRUMENT_MLQ
- References: <950a36fb0909280912sd2eed50yb27f54957b1af37a@mail.gmail.com>
Hi Pieter-Jan
Pieter-Jan Busschaert wrote:
> In kernel\current\src\sched\mlqueue.cxx, several calls are made to
> CYG_INSTRUMENT_MLQ().
> In kernel\current\include\instrmnt.h, this call is either defined as
> nothing or as an actual instrumentation instruction, based on the
> setting of CYGDBG_KERNEL_INSTRUMENT_MLQ.
> However, this setting can not be modified in ecos.ecc because the
> cdl_option is missing in kernel\current\cdl\instrument.cdl
> This patch fixes this:
[ snip ]
This seems perfectly reasonable. Is the resulting MLQ instrumentation
data sensible? If so, I will add the CDL option.
John Dallaway