All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jan Kiszka <jan.kiszka@siemens.com>
To: Alexander Graf <agraf@suse.de>
Cc: "Avi Kivity" <avi@redhat.com>,
	"Andreas Färber" <andreas.faerber@web.de>,
	"Gerd Hoffmann" <kraxel@redhat.com>,
	"Anthony Liguori" <aliguori@us.ibm.com>,
	qemu-devel <qemu-devel@nongnu.org>
Subject: Re: [Qemu-devel] [PATCH v3 5/6] vga: Use linear mapping + dirty logging in chain 4 memory access mode
Date: Tue, 13 Sep 2011 11:00:31 +0200	[thread overview]
Message-ID: <4E6F1BAF.2000105@siemens.com> (raw)
In-Reply-To: <6BA6355D-D77A-40F4-A8C4-61901A926E71@suse.de>

On 2011-09-13 10:40, Alexander Graf wrote:
> Btw, it still tries to execute invalid code even with your patch. #if 0'ing out the memory region updates at least get the guest booting for me. Btw, to get it working you also need a patch for the interrupt controller (another breakage thanks to memory api).
> 
> diff --git a/hw/heathrow_pic.c b/hw/heathrow_pic.c
> index 51996ab..16f48d1 100644
> --- a/hw/heathrow_pic.c
> +++ b/hw/heathrow_pic.c
> @@ -126,7 +126,7 @@ static uint64_t pic_read(void *opaque, target_phys_addr_t addr,
>  static const MemoryRegionOps heathrow_pic_ops = {
>      .read = pic_read,
>      .write = pic_write,
> -    .endianness = DEVICE_NATIVE_ENDIAN,
> +    .endianness = DEVICE_LITTLE_ENDIAN,
>  };
>  
>  static void heathrow_pic_set_irq(void *opaque, int num, int level)
> 

With out without this fix, with or without active chain-4 optimization,
I just get an empty yellow screen when firing up qemu-system-ppc (also
when using the Debian ISO). Do I need to specify a specific machine type?

Jan

-- 
Siemens AG, Corporate Technology, CT T DE IT 1
Corporate Competence Center Embedded Linux

  reply	other threads:[~2011-09-13  9:00 UTC|newest]

Thread overview: 68+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-08-22 17:12 [Qemu-devel] [PATCH v3 0/6] vga: dirty log cleanup, more linear mapping Jan Kiszka
2011-08-22 17:12 ` [Qemu-devel] [PATCH v3 1/6] vmware-vga: Register reset service Jan Kiszka
2011-08-22 17:12 ` [Qemu-devel] [PATCH v3 2/6] vmware-vga: Disable verbose mode Jan Kiszka
2011-08-22 17:12 ` [Qemu-devel] [PATCH v3 3/6] vmware-vga: Remove dead DIRECT_VRAM mode Jan Kiszka
2011-08-22 17:12 ` [Qemu-devel] [PATCH v3 4/6] vmware-vga: Eliminate vga_dirty_log_restart Jan Kiszka
2011-08-22 17:12 ` [Qemu-devel] [PATCH v3 5/6] vga: Use linear mapping + dirty logging in chain 4 memory access mode Jan Kiszka
2011-08-25  7:19   ` Avi Kivity
2011-08-25  9:07     ` Jan Kiszka
2011-08-25  9:16       ` Avi Kivity
2011-08-25  9:21         ` Jan Kiszka
2011-08-25 10:45           ` Avi Kivity
2011-08-25 10:51             ` Jan Kiszka
2011-08-25 11:19               ` Avi Kivity
2011-08-25 11:28                 ` Jan Kiszka
2011-09-12 15:20   ` Alexander Graf
2011-09-12 15:33     ` Jan Kiszka
2011-09-12 15:45       ` Andreas Färber
2011-09-12 15:49         ` Jan Kiszka
2011-09-12 15:57           ` Jan Kiszka
2011-09-13  7:39             ` Alexander Graf
2011-09-13  8:14               ` Jan Kiszka
2011-09-13  8:17                 ` Alexander Graf
2011-09-13  8:19                 ` Andreas Färber
2011-09-13  8:40                 ` Alexander Graf
2011-09-13  9:00                   ` Jan Kiszka [this message]
2011-09-13  9:42                     ` Alexander Graf
2011-09-13 11:34                       ` Jan Kiszka
2011-09-13 19:39                         ` Blue Swirl
2011-09-14  7:11                           ` Avi Kivity
2011-09-14  7:42                             ` Alexander Graf
2011-09-14  8:02                               ` Jan Kiszka
2011-09-14  8:17                               ` Avi Kivity
2011-09-14  8:20                                 ` Jan Kiszka
2011-09-14  8:22                                   ` Avi Kivity
2011-09-14  8:24                                     ` Jan Kiszka
2011-09-14  8:27                                       ` Alexander Graf
2011-09-14  8:33                                         ` Jan Kiszka
2011-09-14  8:35                                         ` Avi Kivity
2011-09-14 20:06                                           ` Blue Swirl
2011-09-14 20:14                                             ` Alexander Graf
2011-09-14 20:16                                               ` Avi Kivity
2011-09-14 20:35                                                 ` Alexander Graf
2011-09-14 20:42                                                   ` Richard Henderson
2011-09-14 21:27                                                     ` Andreas Färber
2011-09-14 21:41                                                     ` Alexander Graf
2011-09-15  1:24                                                       ` Benjamin Herrenschmidt
2011-09-15  7:32                                                         ` Avi Kivity
2011-09-15  7:35                                                         ` Alexander Graf
2011-09-15 10:01                                                           ` Benjamin Herrenschmidt
2011-09-15 11:31                                                             ` Avi Kivity
2011-09-17 21:40                                                               ` Blue Swirl
2011-09-18 11:28                                                                 ` Avi Kivity
2011-09-19  9:15                                                                 ` Alexander Graf
2011-09-19  9:22                                                                   ` Avi Kivity
2011-09-19  9:36                                                                     ` Alexander Graf
2011-09-19  9:48                                                                       ` Avi Kivity
2011-09-14 20:15                                             ` Avi Kivity
2011-09-14 20:25                                               ` Blue Swirl
2011-09-15  7:30                                                 ` Avi Kivity
2011-09-13  9:43                     ` Andreas Färber
2011-09-12 20:21     ` Blue Swirl
2011-09-13  6:54       ` Alexander Graf
2011-09-13  7:51         ` Avi Kivity
2011-09-13  7:54           ` Alexander Graf
2011-09-13  8:10             ` Avi Kivity
2011-08-22 17:12 ` [Qemu-devel] [PATCH v3 6/6] vga: Drop some unused fields Jan Kiszka
2011-08-22 20:18 ` [Qemu-devel] [PATCH v3 0/6] vga: dirty log cleanup, more linear mapping Anthony Liguori
2011-08-22 20:36   ` Jan Kiszka

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4E6F1BAF.2000105@siemens.com \
    --to=jan.kiszka@siemens.com \
    --cc=agraf@suse.de \
    --cc=aliguori@us.ibm.com \
    --cc=andreas.faerber@web.de \
    --cc=avi@redhat.com \
    --cc=kraxel@redhat.com \
    --cc=qemu-devel@nongnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.