From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43)
	id 1ND0WR-0006J3-Kg
	for qemu-devel@nongnu.org; Tue, 24 Nov 2009 13:53:55 -0500
Received: from exim by lists.gnu.org with spam-scanned (Exim 4.43)
	id 1ND0WM-0006GZ-Ui
	for qemu-devel@nongnu.org; Tue, 24 Nov 2009 13:53:55 -0500
Received: from [199.232.76.173] (port=32994 helo=monty-python.gnu.org)
	by lists.gnu.org with esmtp (Exim 4.43) id 1ND0WM-0006GO-Nt
	for qemu-devel@nongnu.org; Tue, 24 Nov 2009 13:53:50 -0500
Received: from mail-bw0-f212.google.com ([209.85.218.212]:36791)
	by monty-python.gnu.org with esmtp (Exim 4.60)
	(envelope-from <anthony@codemonkey.ws>) id 1ND0WM-0003D1-9m
	for qemu-devel@nongnu.org; Tue, 24 Nov 2009 13:53:50 -0500
Received: by bwz4 with SMTP id 4so8345188bwz.2
	for <qemu-devel@nongnu.org>; Tue, 24 Nov 2009 10:53:49 -0800 (PST)
Message-ID: <4B0C2BB7.5010203@codemonkey.ws>
Date: Tue, 24 Nov 2009 12:53:43 -0600
From: Anthony Liguori <anthony@codemonkey.ws>
MIME-Version: 1.0
Subject: Re: [Qemu-devel] [PATCH 12/13] Add zipl bootloader interpreter
References: <1259083781-14642-1-git-send-email-agraf@suse.de>
	<1259083781-14642-13-git-send-email-agraf@suse.de>
In-Reply-To: <1259083781-14642-13-git-send-email-agraf@suse.de>
Content-Type: text/plain; charset=ISO-8859-1; format=flowed
Content-Transfer-Encoding: 7bit
List-Id: qemu-devel.nongnu.org
List-Unsubscribe: <http://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=unsubscribe>
List-Archive: <http://lists.gnu.org/pipermail/qemu-devel>
List-Post: <mailto:qemu-devel@nongnu.org>
List-Help: <mailto:qemu-devel-request@nongnu.org?subject=help>
List-Subscribe: <http://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=subscribe>
To: Alexander Graf <agraf@suse.de>
Cc: Carsten Otte <cotte@de.ibm.com>, qemu-devel@nongnu.org, Aurelien Jarno <aurelien@aurel32.net>

Alexander Graf wrote:
> The default bootloader on S390 is zipl. Because we don't emulate normal S390
> hardware we need to write our own parser for the bootloader configuration,
> so we can boot off real hard disks.
>
> This patch adds a pretty simple implementation of such an interpreter. It only
> supports 512 bytes sector sizes, always boots the default entry and doesn't
> work with reboots yet. But it's better than nothing.
>   

This is a bit unfortunate.  Wouldn't it be better to write a custom 
version of zipl that ran in the guest?

This is like implementing grub in qemu (or pygrub in Xen).  The level of 
security exposure this introduces is really scary.

Regards,

Anthony Liguori