From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:50136) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1dn5yh-0007Fl-Aq for qemu-devel@nongnu.org; Wed, 30 Aug 2017 12:36:32 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1dn5ye-0006H2-6a for qemu-devel@nongnu.org; Wed, 30 Aug 2017 12:36:31 -0400 Received: from mx0a-001b2d01.pphosted.com ([148.163.156.1]:40148) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1dn5yd-0006GX-Tw for qemu-devel@nongnu.org; Wed, 30 Aug 2017 12:36:28 -0400 Received: from pps.filterd (m0098394.ppops.net [127.0.0.1]) by mx0a-001b2d01.pphosted.com (8.16.0.21/8.16.0.21) with SMTP id v7UGYaQp143845 for ; Wed, 30 Aug 2017 12:36:25 -0400 Received: from e06smtp12.uk.ibm.com (e06smtp12.uk.ibm.com [195.75.94.108]) by mx0a-001b2d01.pphosted.com with ESMTP id 2cp09w2rrd-1 (version=TLSv1.2 cipher=AES256-SHA bits=256 verify=NOT) for ; Wed, 30 Aug 2017 12:36:25 -0400 Received: from localhost by e06smtp12.uk.ibm.com with IBM ESMTP SMTP Gateway: Authorized Use Only! Violators will be prosecuted for from ; Wed, 30 Aug 2017 17:36:23 +0100 From: Halil Pasic Date: Wed, 30 Aug 2017 18:36:00 +0200 Message-Id: <20170830163609.50260-1-pasic@linux.vnet.ibm.com> Subject: [Qemu-devel] [PATCH 0/9] List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Cornelia Huck Cc: Dong Jia Shi , Pierre Morel , qemu-devel@nongnu.org, Halil Pasic This series has a character of a refactoring, as the initial motivation was improving readability and reducing complexity. Despite of the original intent the tree first patches buxfixes, and according to Dong Jia the patch-set also has a functional value: for ccw pass-through, he is planing to pass-through the instruction completion information (cc or interruption condition) from the kernel, and this patch set can pretty much be seen as a preparation for that. The basic idea is: tell how to handle an unusual conditon where it's identified, instead of mapping it to an errno (more or less arbitrarily), then possibly mapping these errnos around, to finally (mentally) map the errno back to the condition and take appropriate action. At the end of the series we also have 125 lines of code less to maintain, and the executable got a bit smaller too. Halil Pasic (9): s390x/css: fix cc handling for XSCH s390x: fix invalid use of cc 1 for SSCH s390x/css: be more consistent if broken beyond repair s390x: refactor error handling for SSCH and RSCH s390x: refactor error handling for XSCH handler s390x: refactor error handling for CSCH handler s390x: refactor error handling for HSCH handler s390x: refactor error handling for MSCH handler s390x: factor out common ioinst handler logic hw/s390x/css.c | 158 ++++++++++++-------------------------- hw/s390x/s390-ccw.c | 8 +- hw/vfio/ccw.c | 32 ++++++-- include/hw/s390x/css.h | 38 +++++++--- include/hw/s390x/s390-ccw.h | 2 +- target/s390x/ioinst.c | 181 ++++++++++---------------------------------- 6 files changed, 147 insertions(+), 272 deletions(-) -- 2.13.5