Looks good. Thanks Chris
On 1/12/2022 11:11 pm, Sebastian Huber wrote: > Close #4769. > --- > c-user/config/event-record.rst | 43 +++++++++++++++++++++- > user/tracing/eventrecording.rst | 65 +++------------------------------ > 2 files changed, 47 insertions(+), 61 deletions(-) > > diff --git a/c-user/config/event-record.rst b/c-user/config/event-record.rst > index 31a4fa9..f1e7969 100644 > --- a/c-user/config/event-record.rst > +++ b/c-user/config/event-record.rst > @@ -1,6 +1,6 @@ > .. SPDX-License-Identifier: CC-BY-SA-4.0 > > -.. Copyright (C) 2019, 2021 embedded brains GmbH > (http://www.embedded-brains.de) > +.. Copyright (C) 2019, 2022 embedded brains GmbH > (http://www.embedded-brains.de) > > .. This file is part of the RTEMS quality process and was automatically > .. generated. If you find something that needs to be fixed or > @@ -150,6 +150,47 @@ in a fatal error extension (see :ref:`Terminate`). > The zlib compression needs about 512KiB of RAM. This extension can be used > to produce crash dumps. > > +.. Generated from spec:/acfg/if/record-interrupts-enabled > + > +.. raw:: latex > + > + \clearpage > + > +.. index:: CONFIGURE_RECORD_INTERRUPTS_ENABLED > + > +.. _CONFIGURE_RECORD_INTERRUPTS_ENABLED: > + > +CONFIGURE_RECORD_INTERRUPTS_ENABLED > +----------------------------------- > + > +.. rubric:: CONSTANT: > + > +``CONFIGURE_RECORD_INTERRUPTS_ENABLED`` > + > +.. rubric:: OPTION TYPE: > + > +This configuration option is a boolean feature define. > + > +.. rubric:: DEFAULT CONFIGURATION: > + > +If this configuration option is undefined, then the described feature is not > +enabled. > + > +.. rubric:: DESCRIPTION: > + > +In case > + > +* this configuration option is defined > + > +* and :ref:`CONFIGURE_RECORD_PER_PROCESSOR_ITEMS` is properly defined, > + > +then the interrupt event recording is enabled. > + > +.. rubric:: NOTES: > + > +The interrupt event recording generates interrupt entry and exit events when > +interrupt entries are dispatched. > + > .. Generated from spec:/acfg/if/record-per-processor-items > > .. raw:: latex > diff --git a/user/tracing/eventrecording.rst b/user/tracing/eventrecording.rst > index 7130658..b5561cc 100644 > --- a/user/tracing/eventrecording.rst > +++ b/user/tracing/eventrecording.rst > @@ -48,8 +48,11 @@ The application enables the event recording support via > the configuration > option :c:macro:`CONFIGURE_RECORD_PER_PROCESSOR_ITEMS`. The configuration > option :c:macro:`CONFIGURE_RECORD_EXTENSIONS_ENABLED` enables the generation > of > thread create, start, restart, delete, switch, begin, exitted and terminate > -events. Dumps of the event records in a fatal error handler can be enabled > by > -the mutually exclusive :c:macro:`CONFIGURE_RECORD_FATAL_DUMP_BASE64` and > +events. The configuration option > +:c:macro:`CONFIGURE_RECORD_INTERRUPTS_ENABLED` enables the generation of > +interrupt entry and exit events. Dumps of the event records in a fatal error > +handler can be enabled by the mutually exclusive > +:c:macro:`CONFIGURE_RECORD_FATAL_DUMP_BASE64` and > :c:macro:`CONFIGURE_RECORD_FATAL_DUMP_BASE64_ZLIB` configuration options. > > Custom events can be recorded for example with the > @@ -98,64 +101,6 @@ instrumented functions: > ); > } > > -To generate interrupt handler entry/exit events, the following patch can be > -used: > - > -.. code-block:: diff > - > - diff --git a/bsps/arm/shared/clock/clock-armv7m.c > b/bsps/arm/shared/clock/clock-armv7m.c > - index 255de1ca42..0d37c63ac6 100644 > - --- a/bsps/arm/shared/clock/clock-armv7m.c > - +++ b/bsps/arm/shared/clock/clock-armv7m.c > - @@ -29,6 +29,7 @@ > - #include <bsp/clock-armv7m.h> > - > - #include <rtems.h> > - +#include <rtems/record.h> > - #include <rtems/sysinit.h> > - > - #ifdef ARM_MULTILIB_ARCH_V7M > - @@ -45,9 +46,11 @@ static uint32_t _ARMV7M_TC_get_timecount(struct > timecounter *base) > - > - void _ARMV7M_Clock_handler(void) > - { > - + rtems_record_produce(RTEMS_RECORD_INTERRUPT_ENTRY, > ARMV7M_VECTOR_SYSTICK); > - _ARMV7M_Interrupt_service_enter(); > - Clock_isr(NULL); > - _ARMV7M_Interrupt_service_leave(); > - + rtems_record_produce(RTEMS_RECORD_INTERRUPT_EXIT, > ARMV7M_VECTOR_SYSTICK); > - } > - > - static void _ARMV7M_Clock_handler_install(void) > - diff --git a/bsps/include/bsp/irq-generic.h > b/bsps/include/bsp/irq-generic.h > - index 31835d07ba..2ab2f78b65 100644 > - --- a/bsps/include/bsp/irq-generic.h > - +++ b/bsps/include/bsp/irq-generic.h > - @@ -30,6 +30,7 @@ > - #include <stdbool.h> > - > - #include <rtems/irq-extension.h> > - +#include <rtems/record.h> > - #include <rtems/score/assert.h> > - > - #ifdef RTEMS_SMP > - @@ -258,6 +259,7 @@ void > bsp_interrupt_vector_disable(rtems_vector_number vector); > - */ > - static inline void bsp_interrupt_handler_dispatch(rtems_vector_number > vector) > - { > - + rtems_record_produce(RTEMS_RECORD_INTERRUPT_ENTRY, vector); > - if (bsp_interrupt_is_valid_vector(vector)) { > - const bsp_interrupt_handler_entry *e = > - &bsp_interrupt_handler_table > [bsp_interrupt_handler_index(vector)]; > - @@ -276,6 +278,7 @@ static inline void > bsp_interrupt_handler_dispatch(rtems_vector_number vector) > - } else { > - bsp_interrupt_handler_default(vector); > - } > - + rtems_record_produce(RTEMS_RECORD_INTERRUPT_EXIT, vector); > - } > - > - /** > - > Transfer of Event Records to the Host Computer > ---------------------------------------------- > _______________________________________________ devel mailing list devel@rtems.org http://lists.rtems.org/mailman/listinfo/devel
