<div dir="auto"><div><br><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Aug 9, 2019, 8:45 AM Sebastian Huber <<a href="mailto:sebastian.huber@embedded-brains.de">sebastian.huber@embedded-brains.de</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
----- Joel Sherrill <<a href="mailto:joel@rtems.org" target="_blank" rel="noreferrer">joel@rtems.org</a>> schrieb:<br>
> Coming in late but there isn't there a helper method to turn object names<br>
> into strings from either 32 bit values or strings? Shouldn't that be used?<br>
<br>
This program runs on the host.<br></blockquote></div></div><div dir="auto"><br></div><div dir="auto">Couldn't the method be copied? </div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
> <br>
> On Fri, Aug 9, 2019, 8:17 AM Sebastian Huber <<br>
> <a href="mailto:sebastian.huber@embedded-brains.de" target="_blank" rel="noreferrer">sebastian.huber@embedded-brains.de</a>> wrote:<br>
> <br>
> > On 09/08/2019 15:06, Ravindra Kumar Meena wrote:<br>
> > > rtems-main.c cleanup:<br>
> > ><br>
> > <a href="https://github.com/rmeena840/rtems-tools/commit/61d5dc45e43f0998ad9305d565926090215a5bdc" rel="noreferrer noreferrer" target="_blank">https://github.com/rmeena840/rtems-tools/commit/61d5dc45e43f0998ad9305d565926090215a5bdc</a><br>
> > ><br>
> > > Switch_event per CPU added:<br>
> > ><br>
> > <a href="https://github.com/rmeena840/rtems-tools/commit/8ffe8bd2adea27772a9ab0e578539dd99fa0174b" rel="noreferrer noreferrer" target="_blank">https://github.com/rmeena840/rtems-tools/commit/8ffe8bd2adea27772a9ab0e578539dd99fa0174b</a><br>
> > ><br>
> > > Have a look.<br>
> ><br>
> > Ok, good. Please always check if the babeltrace and Trace Compass give<br>
> > the right results after the changes.<br>
> ><br>
> > If you use cctx->switch_event[ item->cpu ] and similar a couple of times<br>
> > in a function, then please assign it to a local pointer and use it, e.g.<br>
> ><br>
> > switch_event *se = &cctx->switch_event[ item->cpu ];<br>
> ><br>
> > Please check all structure names and make the similar to the names used<br>
> > in the metadata, e.g. swich_event -> event_sched_switch;<br>
> ><br>
> > Please check the style of the if again, it should be:<br>
> ><br>
> > if ( condition ) {<br>
> >   ...<br>
> > } else {<br>
> >   ...<br>
> > }<br>
> ><br>
> > Check the white space and position of the { }.<br>
> ><br>
> > When you have a function like get_api_of_id() you return the api, so the<br>
> > variable should be named "api":<br>
> ><br>
> >    size_t api_id = get_api_of_id( cctx->thread_id_name[ item->cpu<br>
> > ].thread_id );<br>
> ><br>
> >    size_t api = get_api_of_id( cctx->thread_id_name[ item->cpu<br>
> > ].thread_id );<br>
> ><br>
> > Declare all variables at the top of the function, e.g.<br>
> ><br>
> > size_t api;<br>
> > ...<br>
> > api = get_api_of_id( cctx->thread_id_name[ item->cpu ].thread_id );<br>
> ><br>
> > --<br>
> > Sebastian Huber, embedded brains GmbH<br>
> ><br>
> > Address : Dornierstr. 4, D-82178 Puchheim, Germany<br>
> > Phone   : +49 89 189 47 41-16<br>
> > Fax     : +49 89 189 47 41-09<br>
> > E-Mail  : <a href="mailto:sebastian.huber@embedded-brains.de" target="_blank" rel="noreferrer">sebastian.huber@embedded-brains.de</a><br>
> > PGP     : Public key available on request.<br>
> ><br>
> > Diese Nachricht ist keine geschäftliche Mitteilung im Sinne des EHUG.<br>
> > _______________________________________________<br>
> > devel mailing list<br>
> > <a href="mailto:devel@rtems.org" target="_blank" rel="noreferrer">devel@rtems.org</a><br>
> > <a href="http://lists.rtems.org/mailman/listinfo/devel" rel="noreferrer noreferrer" target="_blank">http://lists.rtems.org/mailman/listinfo/devel</a><br>
<br>
-- <br>
Sebastian Huber, embedded brains GmbH<br>
<br>
Address : Dornierstr. 4, D-82178 Puchheim, Germany<br>
Phone   : +49 89 189 47 41-16<br>
Fax     : +49 89 189 47 41-09<br>
E-Mail  : <a href="mailto:sebastian.huber@embedded-brains.de" target="_blank" rel="noreferrer">sebastian.huber@embedded-brains.de</a><br>
PGP     : Public key available on request.<br>
<br>
Diese Nachricht ist keine geschäftliche Mitteilung im Sinne des EHUG. <br>
</blockquote></div></div></div>