Gilles Chanteperdrix wrote:
> Hi,
> 
> here comes a third edition of the patchset adding support for select. 
> 
> The core support now depends on a CONFIG_XENO_OPT_SELECT option, whereas the
> posix support depends on a CONFIG_XENO_OPT_POSIX_SELECT option.
> 
> The support for using rtdm_event_t and rtdm_sem_t objects is now built-in to
> these objects without creating new objects types (only if
> CONFIG_XENO_OPT_SELECT is enabled).
> 
> The rtdm_context_get modification, allowing to check that the caller of a 
> service
> belongs to the same process as the file descriptor it uses, is now made in
> __rtdm_context_get while rtdm_context_get passes a NULL first argument to this
> new service.
> 
> When implementing these modifications, a few questions arise:
> - RTDM support currently depends on the CONFIG_XENO_OPT_SELECT option, should 
> we
> add an RTDM specific option ?

IIRC, the policy is so far that the skin selects the nucleus features it
wants/needs, so we may better add the visible switch to RTDM as well.

> - in order to avoid numerous #ifdefs, and especially in drivers code, the new
> callback added to device operations for binding RTDM file descriptors to
> selector blocks is always there even if CONFIG_XENO_OPT_SELECT is disabled. Is
> it the right thing to do ?

That's ok, otherwise ifdef'ery would crawl into the drivers as well.
Given the conversion you posted for RTnet, we don't add noticeable bloat
this way.

Jan

Attachment: signature.asc
Description: OpenPGP digital signature

_______________________________________________
Xenomai-core mailing list
Xenomai-core@gna.org
https://mail.gna.org/listinfo/xenomai-core

Reply via email to