From: Christophe Varoqui <christophe.varoqui@gmail.com>
To: Hannes Reinecke <hare@suse.de>
Cc: device-mapper development <dm-devel@redhat.com>
Subject: Re: SUSE multipath-tools patch resync
Date: Wed, 25 May 2011 23:29:50 +0200 [thread overview]
Message-ID: <1306358990.2290.43.camel@lapoo.opensvc.com> (raw)
In-Reply-To: <1306326126.2290.33.camel@lapoo.opensvc.com>
> on a 'multipathd reconfigure' command, the uxclient gets stuck and the
> multipathd daemon strace shows:
>
> $ sudo strace -f -p 17721
> Process 17721 attached with 7 threads - interrupt to quit
> [pid 17757] futex(0x7fdc6a1540a4, FUTEX_WAIT_PRIVATE, 3, NULL
> <unfinished ...>
> [pid 17756] futex(0x11167f0, FUTEX_WAIT_PRIVATE, 2, NULL
> <unfinished ...>
> [pid 17755] ioctl(3, DM_DEV_WAIT <unfinished ...>
> [pid 17724] futex(0x11167f0, FUTEX_WAIT_PRIVATE, 2, NULL
> <unfinished ...>
> [pid 17723] recvmsg(6, <unfinished ...>
> [pid 17722] futex(0x110a1b4, FUTEX_WAIT_PRIVATE, 15, NULL
> <unfinished ...>
> [pid 17721] futex(0x612624, FUTEX_WAIT_PRIVATE, 1, NULL
>
ok, I dug it to 9e7b4d8d6fa8dc9433c1e60d4bd6717aec2f5296
Here you add acquire/release the vector lock inside
multipathd/main.c:reconfigure(), but as seen in the following LCKDBG
trace, the lock is already acquired in
multipathd/main.c:uxsock_trigger()
Hence the lock -> lock = hang.
I commited and pushed a partial revert of
9e7b4d8d6fa8dc9433c1e60d4bd6717aec2f5296
But maybe you'd rather see us stop acquiring the lock from
uxsock_trigger() to acquire more selectively in the functions called
from there ... Please comment.
Regards,
--
Christophe Varoqui
OpenSVC - Tools to scale
http://www.opensvc.com/
next prev parent reply other threads:[~2011-05-25 21:29 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-05-18 15:03 SUSE multipath-tools patch resync Hannes Reinecke
2011-05-20 6:26 ` Christophe Varoqui
2011-05-25 7:51 ` Christophe Varoqui
2011-05-25 9:08 ` Hannes Reinecke
2011-05-25 11:14 ` Christophe Varoqui
2011-05-25 12:10 ` Christophe Varoqui
2011-05-25 12:22 ` Christophe Varoqui
2011-05-25 12:45 ` Hannes Reinecke
2011-05-25 21:29 ` Christophe Varoqui [this message]
2011-05-26 6:28 ` Hannes Reinecke
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=1306358990.2290.43.camel@lapoo.opensvc.com \
--to=christophe.varoqui@gmail.com \
--cc=christophe.varoqui@opensvc.com \
--cc=dm-devel@redhat.com \
--cc=hare@suse.de \
/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.