From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:40510) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1elyQO-00085h-Mk for qemu-devel@nongnu.org; Wed, 14 Feb 2018 09:52:45 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1elyQL-000087-ST for qemu-devel@nongnu.org; Wed, 14 Feb 2018 09:52:44 -0500 Received: from mx0b-001b2d01.pphosted.com ([148.163.158.5]:52552 helo=mx0a-001b2d01.pphosted.com) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1elyQL-00006e-O8 for qemu-devel@nongnu.org; Wed, 14 Feb 2018 09:52:41 -0500 Received: from pps.filterd (m0098414.ppops.net [127.0.0.1]) by mx0b-001b2d01.pphosted.com (8.16.0.22/8.16.0.22) with SMTP id w1EEn57D003599 for ; Wed, 14 Feb 2018 09:52:39 -0500 Received: from e32.co.us.ibm.com (e32.co.us.ibm.com [32.97.110.150]) by mx0b-001b2d01.pphosted.com with ESMTP id 2g4m8tgstq-1 (version=TLSv1.2 cipher=AES256-SHA bits=256 verify=NOT) for ; Wed, 14 Feb 2018 09:52:38 -0500 Received: from localhost by e32.co.us.ibm.com with IBM ESMTP SMTP Gateway: Authorized Use Only! Violators will be prosecuted for from ; Wed, 14 Feb 2018 07:52:38 -0700 References: <1517864246-11101-1-git-send-email-walling@linux.vnet.ibm.com> <1517864246-11101-6-git-send-email-walling@linux.vnet.ibm.com> <0c40e592-32e7-315e-fbad-1969c90b9b01@redhat.com> <60be1af6-ab00-5425-1cac-edb722dfcc1a@linux.vnet.ibm.com> From: "Collin L. Walling" Date: Wed, 14 Feb 2018 09:51:17 -0500 MIME-Version: 1.0 In-Reply-To: <60be1af6-ab00-5425-1cac-edb722dfcc1a@linux.vnet.ibm.com> Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-US Message-Id: <5d52424e-c6ee-2c0a-a8ee-14f01bc25747@linux.vnet.ibm.com> Content-Transfer-Encoding: quoted-printable Subject: Re: [Qemu-devel] [qemu-s390x] [PATCH v5 05/12] s390-ccw: move auxiliary IPL data to separate location List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Viktor Mihajlovski , Thomas Huth , qemu-s390x@nongnu.org, qemu-devel@nongnu.org Cc: frankja@linux.vnet.ibm.com, cohuck@redhat.com, david@redhat.com, alifm@linux.vnet.ibm.com, borntraeger@de.ibm.com, eblake@redhat.com On 02/06/2018 05:13 AM, Viktor Mihajlovski wrote: > On 06.02.2018 10:23, Thomas Huth wrote: >> On 05.02.2018 21:57, Collin L. Walling wrote: >> [...] >>> diff --git a/pc-bios/s390-ccw/main.c b/pc-bios/s390-ccw/main.c >>> index e857ce4..825a1a3 100644 >>> --- a/pc-bios/s390-ccw/main.c >>> +++ b/pc-bios/s390-ccw/main.c >>> @@ -16,6 +16,7 @@ char stack[PAGE_SIZE * 8] __attribute__((__aligned_= _(PAGE_SIZE))); >>> static SubChannelId blk_schid =3D { .one =3D 1 }; >>> IplParameterBlock iplb __attribute__((__aligned__(PAGE_SIZE))); >>> static char loadparm[8] =3D { 0, 0, 0, 0, 0, 0, 0, 0 }; >>> +QemuIplParameters qipl; >>> =20 >>> /* >>> * Priniciples of Operations (SA22-7832-09) chapter 17 requires tha= t >>> @@ -81,6 +82,7 @@ static void virtio_setup(void) >>> uint16_t dev_no; >>> char ldp[] =3D "LOADPARM=3D[________]\n"; >>> VDev *vdev =3D virtio_get_device(); >>> + QemuIplParameters *early_qipl =3D (QemuIplParameters *)204; >> Could you please introduce a proper #define for that magic 204 value >> (and use it in s390_ipl_prepare_qipl, too)? ... so that it is later >> easier to grep for this. >> > Hm ... I was following the pattern established in > write_subsystem_identification(). Christian, what is your opinion? Any updates on this?=C2=A0 Last change I need to make before the next pos= t. --=20 - Collin L Walling