<div dir="auto"><div><br><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Feb 19, 2021, 5:32 PM Chris Johns <<a href="mailto:chrisj@rtems.org" target="_blank" rel="noreferrer">chrisj@rtems.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 20/2/21 7:56 am, Joel Sherrill wrote:<br>
> On Fri, Feb 19, 2021 at 2:51 PM Gedare Bloom <<a href="mailto:gedare@rtems.org" rel="noreferrer noreferrer" target="_blank">gedare@rtems.org</a><br>
> <mailto:<a href="mailto:gedare@rtems.org" rel="noreferrer noreferrer" target="_blank">gedare@rtems.org</a>>> wrote:<br>
> <br>
>     I think the suggestion is to provide a catch-all rather than try to add new<br>
>     faults for every possible condition. This mkdir is a pretty esoteric fault<br>
>     that is unlikely to happen in properly developed code.<br>
> <br>
> Then why shouldn't this just be a debug _Assert and value not check deliberately?<br>
<br>
Will the call ever fail in production? Could a user configure RTEMS in a manner<br>
that generates the failure?<br>
<br>
> Isn't an assert something that should not happen in a properly designed BSP. In<br>
> this case, it would be the sysinit magic that would be utterly broken.<br>
<br>
I would not step out as far as utterly broken but yes I see your point. There<br>
are other places where we have taken this approach.<br>
<br>
If the lack of making a directory in GRLIB is handled by errors in the other<br>
dependent calls then why not add some documentation to the BSP.<br></blockquote></div></div><div dir="auto"><br></div><div dir="auto">Confirmation appreciated but it is making the directory to out a device node. The device node create will fail if there isn't a directory so this will return an error.</div><div dir="auto"><br></div><div dir="auto"><a href="https://git.rtems.org/rtems/tree/bsps/shared/grlib/pci/gr_rasta_io.c#n577">https://git.rtems.org/rtems/tree/bsps/shared/grlib/pci/gr_rasta_io.c#n577</a><br></div><div dir="auto"><br></div><div dir="auto">Which means an assert is ok </div><div dir="auto"><br></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>
Chris<br>
</blockquote></div></div></div>