[PATCH] record: Add support for interrupt handlers

Sebastian Huber sebastian.huber at embedded-brains.de
Thu Sep 5 04:42:31 UTC 2019


----- Am 5. Sep 2019 um 0:28 schrieb Chris Johns chrisj at rtems.org:

> On 4/9/19 9:46 pm, Sebastian Huber wrote:
>> On 04/09/2019 08:06, Chris Johns wrote:
>>>> +void LTTNGClient::WriteIRQHandlerExit(PerCPUContext* pcpu, const ClientItem&
>>>> item) {
>>>> +  pcpu->content_size += kEventIRQHandlerExitBits;
>>>> +  pcpu->packet_size += kEventIRQHandlerExitBits;
>>>> +
>>>> +  EventIRQHandlerExit& ih = pcpu->irq_handler_exit;
>>>> +  ih.header.ns = item.ns;
>>>> +  ih.irq = static_cast<int32_t>(item.data);
>>>> +  fwrite(&ih, sizeof(ih), 1, pcpu->event_stream);
>>> https://en.cppreference.com/w/cpp/io  :)
>> 
>> I had a look a this, it seems the error reporting capabilities of <fstream> are
>> quite bad (nothing like strerror(errno)).
> 
> Interesting point to make given the return code from `fwrite` is not checked. :)

Yes, but at least the return status of fopen() is checked.

> Also it should be `std::fwrite`.

Ok, it seems <stdio.h> was included via <iostream>. I will fix this.



More information about the devel mailing list