On 2016-05-12 18:59, Gilles Chanteperdrix wrote: > On Thu, May 12, 2016 at 06:50:03PM +0200, Jan Kiszka wrote: >> On 2016-05-12 18:31, Gilles Chanteperdrix wrote: >>> On Thu, May 12, 2016 at 06:06:16PM +0200, Jan Kiszka wrote: >>>> Gilles, >>>> >>>> regarding commit bec5d0dd42 (rtdm: make syscalls conforming rather than >>>> current) - I remember a discussion on that topic, but I do not find its >>>> traces any more. Do you have a pointer >>>> >>>> In any case, I'm confronted with a use case for the old (Xenomai 2), >>>> lazy switching behaviour: lightweight, performance sensitive IOCTL >>>> services that can (and should) be called without any switching from both >>>> domains. >>> >>> Why not using a plain linux driver? ioctl_nrt callbacks are >>> redundant with plain linux drivers. >> >> Because that enforces the calling layer to either call the same service >> via a plain Linux device if the calling thread is currently relaxed or >> go for the RT device if the caller is in primary. Doable, but I would >> really like to avoid this pain for the users. >> >>> >>>> >>>> What were the arguments in favour of migrating threads to real-time first? >>>> >>>> I currently see the real need only for IOCTLs, but the question is then >>>> if we shouldn't go back to "__xn_exec_current" in all RTDM cases to >>>> avoid unwanted migration costs (which are significantly higher than >>>> syscall restarts). >>> >>> I do not find commit bec5d0dd42 in xenomai-2.6 git tree, and I do >> >> Xenomai 2 is still following the lazy scheme - we reverted that commit >> later on in 7df0c1d96b. Xenomai 3 changed it again with the commit above. >> >>> not remember merging this. However I find commit >>> 13bfdd477ab880499d2e8f3b82c49ef4d2cccff0 from 2010 which seems to >>> explain the reason pretty clear. >>> >>> At the time of the discussion we had concluded that it was the way >>> to go. With __xn_exec_current you may enter the ioctl_rt callback >>> from secondary domain, which is counter-intuitive, error-prone, and >>> forces you to cripple driver code for checks for the current domain. >> >> Nope, normal drivers are not affected as they just implement those >> services in the respective mode they want to support there and have a >> simple -ENOSYS for the rest (explicitly in IOCTLs or implicitly by >> leaving out the implementation of the counterpart handler). > > Yes, I got mixed up trying to remember. I think the crux of the > problem is that if a thread running in primary mode gets > (temporarily) switched to secondary mode by gdb, the ioctl_nrt > handler gets invoked, which is almost certainly the wrong thing to > do. You want the thread to migrate to primary mode to execute > ioctl_rt, which __xn_exec_conforming achieves. Otherwise running an > application in gdb causes the application to behave differently.
Well, if that service is not handling nrt, the only difference under gdb will be the extra switch. But gdb does these kind of things to applications all time. Jan -- Siemens AG, Corporate Technology, CT RDA ITP SES-DE Corporate Competence Center Embedded Linux _______________________________________________ Xenomai mailing list Xenomai@xenomai.org https://xenomai.org/mailman/listinfo/xenomai
