From patchwork Mon Mar 9 14:02:11 2015 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Alexander Graf X-Patchwork-Id: 448020 Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Received: from lists.gnu.org (lists.gnu.org [IPv6:2001:4830:134:3::11]) (using TLSv1 with cipher AES256-SHA (256/256 bits)) (No client certificate requested) by ozlabs.org (Postfix) with ESMTPS id 8479A140146 for ; Tue, 10 Mar 2015 01:02:49 +1100 (AEDT) Received: from localhost ([::1]:43438 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1YUyGc-0008Do-L2 for incoming@patchwork.ozlabs.org; Mon, 09 Mar 2015 10:02:46 -0400 Received: from eggs.gnu.org ([2001:4830:134:3::10]:41618) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1YUyGH-0007uG-MD for qemu-devel@nongnu.org; Mon, 09 Mar 2015 10:02:32 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1YUyGB-00019z-Fx for qemu-devel@nongnu.org; Mon, 09 Mar 2015 10:02:25 -0400 Received: from cantor2.suse.de ([195.135.220.15]:46548 helo=mx2.suse.de) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1YUyGB-00019o-6H; Mon, 09 Mar 2015 10:02:19 -0400 X-Virus-Scanned: by amavisd-new at test-mx.suse.de Received: from relay2.suse.de (charybdis-ext.suse.de [195.135.220.254]) by mx2.suse.de (Postfix) with ESMTP id 40FDBAC7E; Mon, 9 Mar 2015 14:02:18 +0000 (UTC) Message-ID: <54FDA7E3.2070103@suse.de> Date: Mon, 09 Mar 2015 09:02:11 -0500 From: Alexander Graf User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.10; rv:31.0) Gecko/20100101 Thunderbird/31.5.0 MIME-Version: 1.0 To: Peter Maydell References: <1425804297-53727-1-git-send-email-agraf@suse.de> <54FD927A.1030606@suse.de> In-Reply-To: X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x (no timestamps) [generic] X-Received-From: 195.135.220.15 Cc: "qemu-ppc@nongnu.org" , QEMU Developers Subject: Re: [Qemu-devel] [PULL 2.3 00/38] ppc patch queue 2015-03-08 X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.14 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org Sender: qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org On 09.03.15 08:16, Peter Maydell wrote: > On 9 March 2015 at 21:30, Alexander Graf wrote: >> >> >> On 09.03.15 04:13, Peter Maydell wrote: >>> On 8 March 2015 at 17:44, Alexander Graf wrote: >>>> Hi Peter, >>>> >>>> This is my current patch queue for ppc. Please pull. >>>> >>>> Alex >>>> >>>> >>>> The following changes since commit 3539bbb93e944ffde31c61c369ea9eedcc5697a6: >>>> >>>> Merge remote-tracking branch 'remotes/awilliam/tags/vfio-update-20150302.0' into staging (2015-03-04 14:37:31 +0000) >>>> >>>> are available in the git repository at: >>>> >>>> >>>> git://github.com/agraf/qemu.git tags/signed-ppc-for-upstream >>>> >>>> for you to fetch changes up to fa11371474207feeda6ca316e03a9bffd1710276: >>>> >>>> sPAPR: Implement sPAPRPHBClass EEH callbacks (2015-03-08 09:40:20 +0100) >>>> >>>> ---------------------------------------------------------------- >>>> Patch queue for ppc - 2015-03-08 >>>> >>>> This is my current patch queue for 2.3. Highlights include: >>>> >>>> * pseries: 2.3 machine >>>> * pseries: Export RTC via QOM >>>> * pseries: EEH support >>>> * mac: save/restore support >>>> * fix POWER5 hosts >>>> * random bug fixes >>>> >>>> ---------------------------------------------------------------- > > This is giving merge conflicts in hw/display/vga-pci.c now; > they seem more complicated than merely textual, since they're > adding a new object property and the function they're changing > has changed semantics from an initfn to a realizefn. You > probably need to add a new instance init fn to the devices. Hooray :). I've fixed up the patch, rebased and updated the tag. Updated patch is below. Alex From 3c2784fc864d943ff70a2a5c9dea8e9a9b4d9e1d Mon Sep 17 00:00:00 2001 From: David Gibson Date: Tue, 10 Feb 2015 15:36:15 +1100 Subject: [PULL 17/38] vga: Expose framebuffer byteorder as a QOM property The VGA device model now supports having the framebuffer in either endian, and can be switched between these by the guest via a register in the qext region. However, in some cases (e.g. LE OS on the pseries machine) we have existing guest that don't know about the endian switch register, but other parts of the qemu code have better information to set a default endianness than the VGA code does of itself. In order to allow them to set a correct default endianness in these cases, without breaking abstraction walls, this patch exposes the VGA framebuffer endianness via a writable QOM property. Signed-off-by: David Gibson Reviewed-by: Gerd Hoffmann [agraf: use instance_init for property exposure] Signed-off-by: Alexander Graf --- hw/display/vga-pci.c | 29 +++++++++++++++++++++++++++++ 1 file changed, 29 insertions(+) { PCIVGAState *d = DO_UPCAST(PCIVGAState, dev, dev); @@ -265,7 +286,13 @@ static void pci_secondary_vga_realize(PCIDevice *dev, Error **errp) pci_register_bar(&d->dev, 0, PCI_BASE_ADDRESS_MEM_PREFETCH, &s->vram); pci_register_bar(&d->dev, 2, PCI_BASE_ADDRESS_SPACE_MEMORY, &d->mmio); +} +static void pci_secondary_vga_init(Object *obj) +{ + /* Expose framebuffer byteorder via QOM */ + object_property_add_bool(obj, "big-endian-framebuffer", + vga_get_big_endian_fb, vga_set_big_endian_fb, NULL); } static void pci_secondary_vga_reset(DeviceState *dev) @@ -324,6 +351,7 @@ static void secondary_class_init(ObjectClass *klass, void *data) static const TypeInfo vga_info = { .name = "VGA", .parent = TYPE_PCI_DEVICE, + .instance_init = pci_std_vga_init, .instance_size = sizeof(PCIVGAState), .class_init = vga_class_init, }; @@ -331,6 +359,7 @@ static const TypeInfo vga_info = { static const TypeInfo secondary_info = { .name = "secondary-vga", .parent = TYPE_PCI_DEVICE, + .instance_init = pci_secondary_vga_init, .instance_size = sizeof(PCIVGAState), .class_init = secondary_class_init, }; -- 1.8.1.4 diff --git a/hw/display/vga-pci.c b/hw/display/vga-pci.c index d67f748..aabfc23 100644 --- a/hw/display/vga-pci.c +++ b/hw/display/vga-pci.c @@ -181,6 +181,20 @@ static void pci_vga_qext_write(void *ptr, hwaddr addr, } } +static bool vga_get_big_endian_fb(Object *obj, Error **errp) +{ + PCIVGAState *d = DO_UPCAST(PCIVGAState, dev, PCI_DEVICE(obj)); + + return d->vga.big_endian_fb; +} + +static void vga_set_big_endian_fb(Object *obj, bool value, Error **errp) +{ + PCIVGAState *d = DO_UPCAST(PCIVGAState, dev, PCI_DEVICE(obj)); + + d->vga.big_endian_fb = value; +} + static const MemoryRegionOps pci_vga_qext_ops = { .read = pci_vga_qext_read, .write = pci_vga_qext_write, @@ -234,6 +248,13 @@ static void pci_std_vga_realize(PCIDevice *dev, Error **errp) } } +static void pci_std_vga_init(Object *obj) +{ + /* Expose framebuffer byteorder via QOM */ + object_property_add_bool(obj, "big-endian-framebuffer", + vga_get_big_endian_fb, vga_set_big_endian_fb, NULL); +} + static void pci_secondary_vga_realize(PCIDevice *dev, Error **errp)