All of lore.kernel.org
 help / color / mirror / Atom feed
From: Philippe Gerum <rpm@xenomai.org>
To: Gilles Chanteperdrix <gilles.chanteperdrix@xenomai.org>
Cc: Xenomai <xenomai@xenomai.org>
Subject: Re: [Xenomai] rehashing
Date: Mon, 16 Dec 2013 17:12:54 +0100	[thread overview]
Message-ID: <52AF2686.7010907@xenomai.org> (raw)
In-Reply-To: <52AEB440.5090103@xenomai.org>

On 12/16/2013 09:05 AM, Gilles Chanteperdrix wrote:
>
> Hi Philippe,
>
> looking at the registry code, I had an idea: we could increase the
> number of descriptors dynamically when they are exhausted, and increase
> the hash size as well. This would make the configurable number of slots
> a starting point, but not a liimit. The access to the registry by name
> does not really need to be fast, so we could protect it with a mutex

The registry hash is a low contention resource, so using a dedicated 
mutex is likely more efficient than grabbing the big nklock in most 
cases anyway.

> (that would mean that all services accessing the registry by name would
> need to run in secondary mode, but I am not sure we care, and if we do,
> we can use an xnsynch instead).
>
> What do you think?
>

Recent Cobalt users of the registry put aside, this is basically about 
deciding whether:

- issuing connect() on a rtipc socket should switch the caller to 
secondary mode, since other in-kernel users disappeared during the Great 
Refactoring (tm). bind() does have such requirement already.

- all present and future callers of xnregistry_remove() should run in 
secondary mode as well. There is only one caller remaining so far, and 
it does (i.e. thread cleanup code).

-- 
Philippe.


  reply	other threads:[~2013-12-16 16:12 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-12-16  8:05 [Xenomai] rehashing Gilles Chanteperdrix
2013-12-16 16:12 ` Philippe Gerum [this message]
2013-12-16 16:13   ` Gilles Chanteperdrix
2013-12-16 16:35     ` Philippe Gerum
2013-12-16 16:38       ` Gilles Chanteperdrix
2013-12-16 16:48         ` Philippe Gerum
2013-12-29 13:45           ` Gilles Chanteperdrix
2013-12-29 18:04             ` Philippe Gerum
2013-12-29 19:49               ` Gilles Chanteperdrix
2014-01-19 22:08               ` Gilles Chanteperdrix

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=52AF2686.7010907@xenomai.org \
    --to=rpm@xenomai.org \
    --cc=gilles.chanteperdrix@xenomai.org \
    --cc=xenomai@xenomai.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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.