From 47e1f6e06b188bbf8f4da587f2fca15dbe304af4 Mon Sep 17 00:00:00 2001 From: Sebastian Huber Date: Thu, 1 Dec 2022 13:04:08 +0100 Subject: Document CONFIGURE_RECORD_INTERRUPTS_ENABLED 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 - - #include - +#include - #include - - #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 - - #include - +#include - #include - - #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 ---------------------------------------------- -- cgit v1.2.3