Dmitry Adamushko wrote: >> >> >> >> This will set xnintr_t::name to "" if name is NULL - intentionally? >> > >> > xnintr_t::name becomes a reference to RT_INTR::name. The later one >> > exists as long as the former anyway so there shoudn't be a broken >> > reference. >> >> Yes, this specifically addresses Mathias' concern about passed names >> that are temporary variables. >> >> It just leaves xnintr_t::name with an emptry string instead of "unknown" >> in some cases. Either xnintr_init needs to take care of this, or we >> should catch that case already here. I think empty strings in >> /proc/xenomai/irq are bad in case of shared IRQ lists. >> > > Yes, this is what I've mentioned as a possible exception in my previous > mail. > > This patch should address it (although, it makes a code flow a bit > obscure but not too much :) >
I'm fine with it. Mind bundling all the snippets to one complete patch so that people don't have to collect the pieces? Thanks, Jan
signature.asc
Description: OpenPGP digital signature
_______________________________________________ Xenomai-help mailing list [email protected] https://mail.gna.org/listinfo/xenomai-help
