All of lore.kernel.org
 help / color / mirror / Atom feed
From: Alan Jenkins <alan-jenkins@tuffmail.co.uk>
To: linux-hotplug@vger.kernel.org
Subject: Re: Good news for threading!
Date: Tue, 26 May 2009 09:32:01 +0000	[thread overview]
Message-ID: <4A1BB711.6020008@tuffmail.co.uk> (raw)
In-Reply-To: <4A1AA131.7000202@tuffmail.co.uk>

Kay Sievers wrote:
> On Mon, May 25, 2009 at 22:28, Alan Jenkins <alan-jenkins@tuffmail.co.uk> wrote:
>   
>> Kay Sievers wrote:
>>     
>
>   
>>> Do you have numbers for the difference of threaded vs. non-threaded, for:
>>>  time (udevadm trigger; udevadm settle)
>>> when no rules are active?
>>>       
>
>   
>> Interesting question.  I stop HAL first, it tends to slow my tests down and
>> doesn't represent normal bootup.
>>     
>
> Yeah, it's the RUN+="socket: ..." handling, which can block the event
> process, if something listens, but does not read fast enough. With the
> new netlink events from udev to listeners (no RUN+="socket: ..."
> needed), the sending is de-coupled from the listeners state and will
> not slow-down udev. Hopefully HAL will go away soon, we are pretty
> close already.
>   

From my POV it's more that if I don't kill HAL, I think it appears
higher than udev in the profile.  So udev will have to compete for CPU
time, and I can't simply measure the elapsed time.

>> Test machine is eeepc 701; trigger -n shows 382 devices.
>>
>> no threading:    ~0.5s
>> threading:   ~0.25s
>>     
>
> Oh, nice numbers. In a real setup, can we expect more than 0.25s to
> win with a threaded udevd?

I don't know yet.  I need to re-implement the madvise hack so it doesn't
segfault on a booting system.

I'm pretty sure I know what I did wrong.  But right now I'm working on
submitting everything that's ready.

> What can we expect from the udev-exec
> thing? Will it be a bit slower, because it has a bit more work to do?
>   

I ditched the udev-exec daemon on Scotts suggestion.

Regards
Alan

  parent reply	other threads:[~2009-05-26  9:32 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-05-25 13:46 Good news for threading! Alan Jenkins
2009-05-25 19:09 ` Kay Sievers
2009-05-25 20:28 ` Alan Jenkins
2009-05-25 23:52 ` Kay Sievers
2009-05-26  7:03 ` Scott James Remnant
2009-05-26  9:32 ` Alan Jenkins [this message]
2009-05-26  9:36 ` Alan Jenkins

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=4A1BB711.6020008@tuffmail.co.uk \
    --to=alan-jenkins@tuffmail.co.uk \
    --cc=linux-hotplug@vger.kernel.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.