From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:45469) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Z9enq-00074r-Qt for qemu-devel@nongnu.org; Mon, 29 Jun 2015 15:33:15 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1Z9enn-0007su-DV for qemu-devel@nongnu.org; Mon, 29 Jun 2015 15:33:14 -0400 Received: from userp1040.oracle.com ([156.151.31.81]:39434) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Z9egF-0004ii-M4 for qemu-devel@nongnu.org; Mon, 29 Jun 2015 15:25:24 -0400 From: Konrad Rzeszutek Wilk Date: Mon, 29 Jun 2015 15:25:07 -0400 Message-Id: <1435605913-23826-1-git-send-email-konrad.wilk@oracle.com> Subject: [Qemu-devel] [PATCH RFC v1] Cleanups + various fixes due to libxl ABI + more logging on errors. List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: stefano.stabellini@eu.citrix.com, xen-devel@lists.xenproject.org, qemu-devel@nongnu.org As I am in the process of syncing the 'dev.config' and Xen's internal cache of the PCI config space I noticed that some oddities in the code and figured that having them in would be easier for me (so I don't have to handle 20 odd patches). These: [PATCH RFC 1/6] xen/pt: Update comments with proper function name. [PATCH RFC 2/6] xen/pt: Make xen_pt_msi_set_enable static [PATCH RFC 3/6] xen/pt: xen_host_pci_config_read returns -errno, not [PATCH RFC 4/6] xen: Print and use errno where applicable. are quite simple. The two other ones: [PATCH RFC 5/6] xen/pt/msi: Add the register value when printing [PATCH RFC 6/6] xen: Add backtrace for serious issues. are more for adding the proper infrastructure to debug in the field bugs that may come about. hw/xen/xen_pt.c | 9 ++++++--- hw/xen/xen_pt.h | 1 - hw/xen/xen_pt_config_init.c | 6 +++--- hw/xen/xen_pt_msi.c | 2 +- include/hw/xen/xen_common.h | 1 + xen-hvm.c | 26 ++++++++++++++++++++++---- 6 files changed, 33 insertions(+), 12 deletions(-) Konrad Rzeszutek Wilk (6): xen/pt: Update comments with proper function name. xen/pt: Make xen_pt_msi_set_enable static xen/pt: xen_host_pci_config_read returns -errno, not -1 on failure xen: Print and use errno where applicable. xen/pt/msi: Add the register value when printing logging and error messages xen: Add backtrace for serious issues.