[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH] tools/xl: Add stubdomain_cmdline option to xl.cfg


  • To: Julien Grall <julien@xxxxxxx>, Scott Davis <scottwd@xxxxxxxxx>, <xen-devel@xxxxxxxxxxxxxxxxxxxx>
  • From: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
  • Date: Wed, 21 Jul 2021 10:06:10 +0100
  • Arc-authentication-results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=citrix.com; dmarc=pass action=none header.from=citrix.com; dkim=pass header.d=citrix.com; arc=none
  • Arc-message-signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=uh5PJfiRc7Ja7Loaxr4P52vXB2d0IaOEL6WCp0fZM2M=; b=gv8RpwbaqxmCvcXEhWfIGUJKxrifeaVtpi+YjFsPkKxJbzb0PNAUCKSQMWIXOl10pqtX7ARHCNkeISEvNTGmAAIma2FhopkVDQZZybVTSo3PGgoLylTPj64jVkrXxxnMd3giA3hdxwTFfyhTvcjNB85UmKCT7ZKzA9etYZQ3Q88msl/XoGyvmzJuzR4mJYiXa0yOLDJPKjf1o3WLMaTKOdGHoliDlUQVfsYpTlbkv84O3mZXP8wkBGKWcpXOCUAgjKftXJD2hVq4ww6WnbhTyfE9pBbvqZ0PzHJW1hwQCz0lJ9g1kyTr4oZyLRlIElkDEYTeGgk5jqTHRfpZCcl81g==
  • Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=hSHYDqQoMh/2jtVxRLAhcP9KSgsy1HqTNWt9w8xjI2I8JGtKKQvpXsBLvJqm+PaUeHUmbEyuiSHWiagvSmj+OAQvDnV9DKBWDWorhRquoBKaan9mGMhbt/8IfkpSbL5O5vvrW4RaU4IkvK2DaUae6Uu8td42uviNcoJBIDOjkB+lvl3AMZe7RTlzj/oax+cS/+LCuIRmRQ4S6ZFtSrjsSEz2WFTmwC0Qf/f/JhFqdEVeC+4QfD/A1Ie09piMvoalYoPI4cRR7/zq6UzZ3Rn32x2R6cczgH20BIksnmMccTs/RySRxaFki506Omkx1AS4JpPFg581i90w9r4MvecTsQ==
  • Authentication-results: esa1.hc3370-68.iphmx.com; dkim=pass (signature verified) header.i=@citrix.onmicrosoft.com
  • Cc: Scott Davis <scott.davis@xxxxxxxxxx>, Ian Jackson <iwj@xxxxxxxxxxxxxx>, Wei Liu <wl@xxxxxxx>, George Dunlap <george.dunlap@xxxxxxxxxx>, Nick Rosbrook <rosbrookn@xxxxxxxxxxxx>, Anthony PERARD <anthony.perard@xxxxxxxxxx>, "Juergen Gross" <jgross@xxxxxxxx>
  • Delivery-date: Wed, 21 Jul 2021 09:06:36 +0000
  • Ironport-hdrordr: A9a23:Zw1Vda6jVvpTx/l0dQPXwD7XdLJyesId70hD6qkQc3FomwKj9/ xG/c5rsyMc7Qx6ZJhOo7+90cW7L080sKQFg7X5Xo3SOzUO2lHYT72KhLGKq1Hd8m/Fh4tgPM 9bGJSWY+eAaWSS4/ya3OG5eexQv+Vu8sqT9JnjJ6EGd3AaV0lihT0JejpyCidNNXB77QJSLu vg2iJAzQDQAUg/X4CAKVQuefPMnNHPnIKOW297O/Z2gDP+9g9B8dTBYmKl4is=
  • Ironport-sdr: aPPgSyaDGzD/917tQDdE7c0CQ0jR69lG2uxNfYs8OImL04LtK74wq0cQ4bdE0b7K97oTlUJ8rM cJwdun5w6uTIGsXBdRc9N/prnO2vyjnvcct7s4u9HWrIsCnkVk1jMf6Y2NVdil+zZFe8Oc5L7Y YDsR8mPzpjHqX642GJaVvYi4mkS0ytQKmbmu/0vlUABZ2w/lFxlDXuZ/jEEOBp+/Uf4gbsLnGh OlO9KXJVei1kWJk5y77Ixhm12phw/+Nf+s3ttuujALysQQz605GpPJXFp4LjBJGVGpdwIQNgGM OumFP3zO0AAJRNxN3I4xM24P
  • List-id: Xen developer discussion <xen-devel.lists.xenproject.org>

On 21/07/2021 09:21, Julien Grall wrote:
> Hi Scott,
>
> On 20/07/2021 18:56, Scott Davis wrote:
>> This adds an option to the xl domain configuration file syntax for
>> specifying
>> a kernel command line for device-model stubdomains. It is intended
>> for use with
>> Linux-based stubdomains.
>
> May I ask why embedding the command line in the kernel would not be a
> solution? Do you expect it to change from stubdom to stubdom?

Why should users of stubdoms be forced to embed command line options? 
Especially when its not the normal way of working?

They shouldn't, and this alone is enough justification for the change.

~Andrew



 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.