From mboxrd@z Thu Jan 1 00:00:00 1970 Message-ID: <47977DE1.8060207@domain.hid> Date: Wed, 23 Jan 2008 18:48:17 +0100 From: Philippe Gerum MIME-Version: 1.0 References: <2ff1a98a0801020231k19be7d89k1a6f04b7d497cc34@domain.hid> <18315.63245.160672.547658@domain.hid> <18326.21480.397736.413726@domain.hid> In-Reply-To: <18326.21480.397736.413726@domain.hid> Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Sender: Philippe Gerum Subject: Re: [Xenomai-core] High latencies on ARM. Reply-To: rpm@xenomai.org List-Id: "Xenomai life and development \(bug reports, patches, discussions\)" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Gilles Chanteperdrix Cc: xenomai-core Gilles Chanteperdrix wrote: > Gilles Chanteperdrix wrote: > > Please find attached a patch implementing these ideas. This adds some > > clutter, which I would be happy to reduce. Better ideas are welcome. > > > > Ok. New version of the patch, this time split in two parts, should > hopefully make it more readable. > Ack. I'd suggest the following: - let's have a rate limiter when walking the zombie queue in __xnpod_finalize_zombies. We hold the superlock here, and what the patch also introduces is the potential for flushing more than a single TCB at a time, which might not always be a cheap operation, depending on which cra^H^Hode runs on behalf of the deletion hooks for instance. We may take for granted that no sane code would continuously create more threads than we would be able to finalize in a given time frame anyway. - We could move most of the code depending on XNARCH_WANT_UNLOCKED_CTXSW to conditional inlines in pod.h. This would reduce the visual pollution a lot. -- Philippe.