From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:54760) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1X0Asb-000497-0Y for qemu-devel@nongnu.org; Thu, 26 Jun 2014 10:42:32 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1X0AsT-0006pp-Ea for qemu-devel@nongnu.org; Thu, 26 Jun 2014 10:42:24 -0400 Received: from cantor2.suse.de ([195.135.220.15]:44789 helo=mx2.suse.de) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1X0AsT-0006pQ-9A for qemu-devel@nongnu.org; Thu, 26 Jun 2014 10:42:17 -0400 Message-ID: <53AC3147.1090209@suse.de> Date: Thu, 26 Jun 2014 16:42:15 +0200 From: Alexander Graf MIME-Version: 1.0 References: <1403793009-54176-1-git-send-email-jfrei@linux.vnet.ibm.com> In-Reply-To: <1403793009-54176-1-git-send-email-jfrei@linux.vnet.ibm.com> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH 00/10] pc-bios/s390-ccw: Add DASD IPL support List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Jens Freimann , Christian Borntraeger , Cornelia Huck Cc: qemu-devel@nongnu.org On 26.06.14 16:29, Jens Freimann wrote: > Conny, Alex, Christian, > > here are some fixes for the s390-ccw bios. It's a mixture of > additional features (DASD IPL support for different formats) > and cleanups. From a quick glimpse it looks quite clean and straight forward, but I'd like to make sure we get rid completely of the static sector size assumption. Also, are we guaranteed that virtio always uses 512 byte block size? Or was that just an internal API thing? Alex