All of lore.kernel.org
 help / color / mirror / Atom feed
From: Andreas Olsowski <andreas.olsowski@leuphana.de>
To: Ian Campbell <Ian.Campbell@citrix.com>,
	"xen-devel@lists.xensource.com" <xen-devel@lists.xensource.com>
Subject: Re: hanging tapdisk2 processes and improper udev rules
Date: Fri, 22 Jul 2011 13:36:22 +0200	[thread overview]
Message-ID: <4E2960B6.5000902@leuphana.de> (raw)
In-Reply-To: <1311326922.12772.14.camel@zakaz.uk.xensource.com>


[-- Attachment #1.1: Type: text/plain, Size: 2621 bytes --]


On 07/22/2011 11:28 AM, Ian Campbell wrote:

> This is because udev and forward/backward compatibility are strangers
> passing in the night. I presume if you make the recommended change to
> SYMLINK+= instead of NAME= in your udev script this goes away?
You assume correctly.

> I posted a patch to fix this "libxl: attempt to cleanup tapdisk
> processes on disk backend destroy" a couple of times, most recently at
> http://marc.info/?l=xen-devel&m=131066210526755 but it hasn't been
> applied yet. Can you try it?

I tried it:

make -j7 tools:
...
libxl_device.c: In function ‘libxl__device_destroy’:
libxl_device.c:253: error: incompatible type for argument 1 of 
‘libxl__device_destroy_tapdisk’
libxl_internal.h:321: note: expected ‘struct libxl__gc *’ but argument 
is of type ‘libxl__gc’
libxl_device.c:274: error: incompatible type for argument 1 of 
‘libxl__device_destroy_tapdisk’
libxl_internal.h:321: note: expected ‘struct libxl__gc *’ but argument 
is of type ‘libxl__gc’

My expertise with C is barely existant, but i took a look at 
tools/libxl/libxl_device.c

and changed your
libxl__device_destroy_tapdisk(gc, be_path);
into
libxl__device_destroy_tapdisk(&gc, be_path);

as i have seen some &gc on other lines of code.

And it compiled.

I then created a guest, shut it down.
First it kept beeing in a -ps--- state, i wanted to take a look at the 
runing processes with "ps auxww" but the ps process hung itself.
I could no longer run "ps" successfully after this point.
syslog showed:
ul 22 13:00:07 xenturio1 xl: tap-err:tap_ctl_read_message: failure 
reading message
Jul 22 13:00:07 xenturio1 xl: tap-err:tap_ctl_send_and_receive: failed 
to receive 'unknown' message

Either my hack to get your code to compile was no good or your patch has 
some unforseen side effects.



I have now rebooted the server.


As i went on to check if multipath had any effect on it i added
devnode "^td" to the blacklist.

Now when i xl create a vm it only boots up to a certain point and then 
does nothing.
If that certain point were to be the login prompt everything would be 
fine, but it isnt:
http://pastebin.com/Lmie6KwY

This is how it should look like:

http://pastebin.com/CsgYypbk

I will try to backtrace my steps and see what i did do to break my system.

In the meantime i have other systems i can test stuff on.



-with best regards


-- 
Andreas Olsowski
Leuphana Universität Lüneburg
Rechen- und Medienzentrum
Scharnhorststraße 1, C7.015
21335 Lüneburg

Tel: ++49 4131 677 1309


[-- Attachment #1.2: S/MIME Cryptographic Signature --]
[-- Type: application/pkcs7-signature, Size: 6595 bytes --]

[-- Attachment #2: Type: text/plain, Size: 138 bytes --]

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xensource.com
http://lists.xensource.com/xen-devel

  reply	other threads:[~2011-07-22 11:36 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-07-22  9:18 hanging tapdisk2 processes and improper udev rules Andreas Olsowski
2011-07-22  9:28 ` Ian Campbell
2011-07-22 11:36   ` Andreas Olsowski [this message]
2011-07-22 14:07     ` Ian Campbell
2011-07-22 14:28       ` Andreas Olsowski
2011-07-22 14:32         ` Ian Campbell
2011-07-25  8:55           ` Andreas Olsowski
2011-08-11 13:32           ` Andreas Olsowski
2011-07-22  9:31 ` Daniel Stodden
2011-07-22  9:32   ` Daniel Stodden
2011-07-22  9:34   ` Sébastien RICCIO
2011-07-22  9:50     ` Daniel Stodden
2011-07-22 10:01       ` Sébastien RICCIO
2011-07-22 18:55         ` hanging tapdisk2 processes and multipathing Daniel Stodden

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=4E2960B6.5000902@leuphana.de \
    --to=andreas.olsowski@leuphana.de \
    --cc=Ian.Campbell@citrix.com \
    --cc=xen-devel@lists.xensource.com \
    /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.