From mboxrd@z Thu Jan 1 00:00:00 1970 Date: Mon, 7 Jul 2014 18:02:39 +0200 From: Maxime Ripard Message-ID: <20140707160239.GF13423@lukather> References: <1404210421-17081-1-git-send-email-maxime.ripard@free-electrons.com> <53B294A7.5010803@xenomai.org> <20140701141536.GN28647@lukather> <53B30D96.60500@xenomai.org> <20140704092736.GC13487@lukather> <53B7B3BF.3090807@xenomai.org> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <53B7B3BF.3090807@xenomai.org> Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Subject: Re: [Xenomai] [PATCH] AT91: SAMA5D3: Adapt Ipipe for AIC5 List-Id: Discussions about the Xenomai project List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Gilles Chanteperdrix Cc: Thomas Petazzoni , Nicolas Ferre , Boris Brezillon , Alexandre Belloni , xenomai@xenomai.org On Sat, Jul 05, 2014 at 10:13:51AM +0200, Gilles Chanteperdrix wrote: > >>> Ok, so, with the changes you mentionned, I can't make the system > >>> crash anymore (or at least, not as easily as it used to be). > >>> > >>> But: - whenever the program mentionned above calls exit(), it > >>> stalls. However, ctrl+c makes the program exit properly, and > >>> everything seems fine otherwise - whenever we don't link it against > >>> xenomai, it just hangs. I've not figured out why yet > >>> > >>> With CONFIG_XENOMAI and CONFIG_IPIPE disabled, it works fine. > >> > >> My answer was wrong, you probably need to keep the > >> set_backup/clear_backup calls in he ->hold and ->release callbacks, as > >> the linux interrupt may expect the backup areas to be in sync. Did you > >> do this, or did you go for the alternative? > > > > I went for the alternative. I'm sending you a v2 with what I have so > > far, that shows the behaviour I was describing. > > Could you try the following patch? It actually works much better, thanks! The above mentionned issues are gone, there's only one last glitch I guess. The max latency under load is around a few 100's of ms (while idle, it's actually around 200-300us, which seems more reasonable. Maxime -- Maxime Ripard, Free Electrons Embedded Linux, Kernel and Android engineering http://free-electrons.com -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 819 bytes Desc: Digital signature URL: