All of lore.kernel.org
 help / color / mirror / Atom feed
* [Xenomai] RTDM rework
@ 2014-02-01 18:23 Gilles Chanteperdrix
  2014-02-01 18:27 ` [Xenomai] [PATCH 1/3] cobalt/rtdm: base protocol devices on xnid Gilles Chanteperdrix
  2014-02-02  0:21 ` [Xenomai] RTDM rework Gilles Chanteperdrix
  0 siblings, 2 replies; 7+ messages in thread
From: Gilles Chanteperdrix @ 2014-02-01 18:23 UTC (permalink / raw)
  To: Jan Kiszka, Philippe Gerum; +Cc: Xenomai


Hi Jan, Philippe,

I have started working on rebasing RTDM on the common fd implementation
(xnfd). Before I start changing the drivers, the current state of the
changes follow as patches. These patches are based on the commits in the
"for-forge" branch in xenomai-gch repository. Please do not pull from
this branch just yet, it is probably going to be rebased.

The thing which I know we will discuss, even before you read the patches:
- struct xnfd appears instead of struct rtdm_dev_context in callbacks,
we already discussed that, you prefer rtdm typedefs, the thing is, in
forge, the habit to typedef structs seems gone, and typedef struct xnfd
rtdm_dev_contex_t would be confusing since struct rtdm_dev_context still
exists, albeit is no longer exported to drivers;
- the xnfd structure is defined in cobalt, and not cobalt/posix, this
was made so that both rtdm and posix can use it, but maybe it makes more
sense to move it to posix (after all, file descriptors are a posix
thing), and get rtdm to use it anyway;
- doc is probably out of sync, this will be arranged in later commits
- maybe select should be moved to fd.c now, posix/select.c contains 3
functions, and it now supports xnfds exclusively;
- context_flags are gone, since the two flags it used are no longer
used, since driver have a private structure, drivers can move their
flags to their private structure if they want;
- handling -EAGAIN in the close callback is gone, the close callback now
returns void, it is possible to delay file descriptor destruction by
using rtdm_lock/rtdm_unlock, so the -EAGAIN return value was IMO redundant.

Regards.

-- 
                                                                Gilles.


^ permalink raw reply	[flat|nested] 7+ messages in thread

end of thread, other threads:[~2014-02-28 11:27 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-02-01 18:23 [Xenomai] RTDM rework Gilles Chanteperdrix
2014-02-01 18:27 ` [Xenomai] [PATCH 1/3] cobalt/rtdm: base protocol devices on xnid Gilles Chanteperdrix
2014-02-01 18:27   ` [Xenomai] [PATCH 2/3] cobalt/rtdm: base named devices on nucleus registry Gilles Chanteperdrix
2014-02-01 18:27   ` [Xenomai] [PATCH 3/3] cobalt/rtdm: base on common fd implementation Gilles Chanteperdrix
2014-02-02  0:21 ` [Xenomai] RTDM rework Gilles Chanteperdrix
2014-02-04 20:49   ` Gilles Chanteperdrix
     [not found]     ` <52FE14E1.8040303@siemens.com>
     [not found]       ` <52FE18CE.4020702@xenomai.org>
     [not found]         ` <52FE1B85.3070402@siemens.com>
     [not found]           ` <52FE2364.6060608@xenomai.org>
     [not found]             ` <52FE3BCD.3000608@siemens.com>
     [not found]               ` <52FE443D.9060806@xenomai.org>
     [not found]                 ` <52FE4D84.90006@siemens.com>
2014-02-28 11:27                   ` Philippe Gerum

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.