From mboxrd@z Thu Jan 1 00:00:00 1970 From: Konrad Rzeszutek Wilk Subject: Re: Second regression due to libxl: Remove linux udev rules (2ba368d13893402b2f1fb3c283ddcc714659dd9b) Date: Tue, 22 Sep 2015 10:15:32 -0400 Message-ID: <20150922141532.GA4454@l.oracle.com> References: <20150728194741.GA13430@l.oracle.com> <55B9E614.4040504@citrix.com> <55C07468.4040909@citrix.com> <20150807145440.GK29527@l.oracle.com> <55C4C7B1.8020604@citrix.com> <20150812140912.GA17002@l.oracle.com> <55D2E397.3020405@citrix.com> Mime-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable Return-path: Received: from mail6.bemta5.messagelabs.com ([195.245.231.135]) by lists.xen.org with esmtp (Exim 4.72) (envelope-from ) id 1ZeOMC-0002gD-KQ for xen-devel@lists.xenproject.org; Tue, 22 Sep 2015 14:15:44 +0000 Content-Disposition: inline In-Reply-To: <55D2E397.3020405@citrix.com> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: Roger Pau =?iso-8859-1?Q?Monn=E9?= Cc: george.dunlap@eu.citrix.com, xen-devel@lists.xenproject.org, wei.liu2@citrix.com, Ian Campbell List-Id: xen-devel@lists.xenproject.org On Tue, Aug 18, 2015 at 09:49:43AM +0200, Roger Pau Monn=E9 wrote: > El 12/08/15 a les 16.09, Konrad Rzeszutek Wilk ha escrit: > > On Fri, Aug 07, 2015 at 04:58:57PM +0200, Roger Pau Monn=E9 wrote: > >> El 07/08/15 a les 16.54, Konrad Rzeszutek Wilk ha escrit: > >>> Ok. I hadn't run your patch yet. Do you want me to run the latest sta= ging > >>> instead once more with my test-case? > >> > >> Yes please, 40s in my test case seemed to be fine. > > = > > I get inconsistent values. Sometimes it works, sometimes it does not. > > = > > I will try 120 on my test-bed :-( > = > Thanks, this is not unexpected. IMHO the best way to solve this would be > to make the hotplug timeout settable by the user by adding a new option > to xl.conf: > = > hotplug_timeout=3D40 > = > Allowing the user to set it to -1 in order to remove the timeout if desir= ed. > = > This however seems too intrusive IMHO at this point in the release, so I > would argue for just increasing the default timeout to a suitable value. > Konrad do you have any suggestions? I've been running with staging for months now and I don't see this problem. Going forward I would think that making this work as you explained with the 'hotplug_timeout=3D-1' would be nice. But then I seem to be the only one doing it - and I don't see the normal user re-using the same file or phy to launch tons of guests in parallel. In my mind this is very much a feature for a developer and as such I would say less important as a developer I can always just modify the source code with a higher timeout. > = > Roger. > =