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

Re: [Xen-devel] [PATCH][XEN] Don't throw away cmdline parameter

  • To: Christoph Egger <Christoph.Egger@xxxxxxx>, <xen-devel@xxxxxxxxxxxxxxxxxxx>
  • From: Keir Fraser <keir.fraser@xxxxxxxxxxxxx>
  • Date: Mon, 20 Oct 2008 14:43:51 +0100
  • Cc:
  • Delivery-date: Mon, 20 Oct 2008 06:44:16 -0700
  • List-id: Xen developer discussion <xen-devel.lists.xensource.com>
  • Thread-index: AckyueKrIXFb0p6tEd2bHQAX8io7RQ==
  • Thread-topic: [Xen-devel] [PATCH][XEN] Don't throw away cmdline parameter

On 20/10/08 14:23, "Christoph Egger" <Christoph.Egger@xxxxxxx> wrote:

> attach patch avoids throwing away the first cmdline parameter.
> It contains the dom0 kernel filename.
> On NetBSD, this is used to set sysctl.booted_kernel
> This sysctl entry is used by the kvm framework
> (http://netbsd.gw.com/cgi-bin/man-cgi?kvm+3+NetBSD-current)
> and savecore (http://netbsd.gw.com/cgi-bin/man-cgi?savecore+8+NetBSD-current).
> The kvm framework is used by almost all system utilities.
> Signed-off-by: Christoph Egger <Christoph.Egger@xxxxxxx>

Doing this would make dom0 start_info format differ from domU start_info
format. If an accurate filename is actually useful then I'd consider making
an extension to start_info format (e.g., an extra SIF_ flag and use that to
indicate an extra string appended to cmd_line[] containing the kernel
filename). Or those who do care can explicitly duplicate the filename into
the cmdline in their domain config file or bootloader config file.

 -- Keir

Xen-devel mailing list



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