Re: [Xenomai] [PATCH] Split rtdm_fd_enter up, move the functionality where we store the fd until after the open() call succeeds. Calls where open() fail a fd is left in the tree even after the cleanup

2017-12-05 Thread Philippe Gerum
On 12/04/2017 04:22 AM, Greg Gallagher wrote: > --- > include/cobalt/kernel/rtdm/fd.h | 2 ++ > kernel/cobalt/posix/mqueue.c| 7 ++- > kernel/cobalt/posix/timerfd.c | 4 > kernel/cobalt/rtdm/core.c | 12 +--- > kernel/cobalt/rtdm/fd.c | 24

[Xenomai] [PATCH] Split rtdm_fd_enter up

2017-12-04 Thread Greg Gallagher
--- split rtdm_fd_enter, move the functionality where we store the fd until after the open() call succeeds. Calls where open() fail a fd is left in the tree even after the cleanup code is executed. If this fd number is used again we will fail the call to open until a different fd is used.

Re: [Xenomai] [PATCH] Split rtdm_fd_enter up, move the functionality where we store the fd until after the open() call succeeds. Calls where open() fail a fd is left in the tree even after the cleanup

2017-12-04 Thread Henning Schild
Hi, could you please change that commit to have a short "subject" followed by the description. Your commit comment does not seem to have any newlines causing all to show up in the seubject. Am Sun, 3 Dec 2017 22:22:24 -0500 schrieb Greg Gallagher : > --- >

[Xenomai] [PATCH] Split rtdm_fd_enter up, move the functionality where we store the fd until after the open() call succeeds. Calls where open() fail a fd is left in the tree even after the cleanup cod

2017-12-03 Thread Greg Gallagher
--- include/cobalt/kernel/rtdm/fd.h | 2 ++ kernel/cobalt/posix/mqueue.c| 7 ++- kernel/cobalt/posix/timerfd.c | 4 kernel/cobalt/rtdm/core.c | 12 +--- kernel/cobalt/rtdm/fd.c | 24 5 files changed, 37 insertions(+), 12