From: "Andreas Färber" <afaerber@suse.de>
To: Anthony Liguori <aliguori@us.ibm.com>,
Wanpeng Li <liwanp@linux.vnet.ibm.com>
Cc: juzhang@redhat.com, mst@redhat.com, jan.kiszka@siemens.com,
Jason Baron <jbaron@redhat.com>,
qemu-devel@nongnu.org, agraf@suse.de, blauwirbel@gmail.com,
yamahata@valinux.co.jp, alex.williamson@redhat.com,
kevin@koconnor.net, avi@redhat.com, mkletzan@redhat.com,
pbonzini@redhat.com, lcapitulino@redhat.com, armbru@redhat.com,
kraxel@redhat.com
Subject: Re: [Qemu-devel] [PATCH v1 01/13] pc/piix_pci: factor out smram/pam logic
Date: Tue, 30 Oct 2012 21:26:35 +0100 [thread overview]
Message-ID: <509037FB.20000@suse.de> (raw)
In-Reply-To: <878vanvkwt.fsf@codemonkey.ws>
Am 30.10.2012 20:07, schrieb Anthony Liguori:
> Jason Baron <jbaron@redhat.com> writes:
>
>> From: Isaku Yamahata <yamahata@valinux.co.jp>
>>
>> Factor out smram/pam logic for later use.
>> Which will be used by q35 too.
>>
>> [jbaron@redhat.com: changes for updated memory API]
>> Signed-off-by: Isaku Yamahata <yamahata@valinux.co.jp>
>> Signed-off-by: Jason Baron <jbaron@redhat.com>
>
> This is really not the right approach to solving this problem.
>
> "pam" is not a device.
>
> Instead, the common bits here are essentially the northbridge logic.
> That's what we should model here.
>
> It's pretty simple actually. Just throw this all into a device where
> the memory regions are owned by the device. But also make the device
> own creation of all RAM (not just the areas controlled by PAM).
>
> Then embed the northbridge into the i440fx and q35.
Anthony/Wanpeng, any update on i440fx? Haven't heard of it since we
finally merged the PCI host bridge stuff for v1.2...
Regards,
Andreas
--
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer; HRB 16746 AG Nürnberg
next prev parent reply other threads:[~2012-10-30 20:26 UTC|newest]
Thread overview: 35+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-10-30 2:11 [Qemu-devel] [PATCH v1 00/13] q35 patches for pci tree Jason Baron
2012-10-30 2:11 ` [Qemu-devel] [PATCH v1 01/13] pc/piix_pci: factor out smram/pam logic Jason Baron
2012-10-30 19:07 ` Anthony Liguori
2012-10-30 20:26 ` Andreas Färber [this message]
2012-10-30 2:11 ` [Qemu-devel] [PATCH v1 02/13] Back out add of i21154 Jason Baron
2012-10-31 9:54 ` Michael S. Tsirkin
2012-10-30 2:11 ` [Qemu-devel] [PATCH v1 03/13] blockdev: Introduce QEMUMachine->default_drive_if Jason Baron
2012-10-30 19:08 ` Anthony Liguori
2012-10-30 2:11 ` [Qemu-devel] [PATCH v1 04/13] blockdev: Introduce IF_AHCI Jason Baron
2012-10-30 2:11 ` [Qemu-devel] [PATCH v1 06/13] pc: Move ioapic_init() from pc_piix.c to pc.c Jason Baron
2012-10-31 10:02 ` Michael S. Tsirkin
2012-10-30 2:11 ` [Qemu-devel] [PATCH v1 05/13] pc, pc_piix: split out pc nic initialization Jason Baron
2012-10-30 19:09 ` Anthony Liguori
2012-10-31 9:57 ` Michael S. Tsirkin
2012-10-30 2:11 ` [Qemu-devel] [PATCH v1 07/13] q35: Introduce q35 pc based chipset emulator Jason Baron
2012-10-30 19:18 ` Anthony Liguori
2012-10-31 10:04 ` Michael S. Tsirkin
2012-10-31 12:53 ` Anthony Liguori
2012-10-30 2:11 ` [Qemu-devel] [PATCH v1 09/13] q35: Add kvmclock support Jason Baron
2012-10-30 2:11 ` [Qemu-devel] [PATCH v1 08/13] q35: Suppress SMM BIOS initialization under KVM Jason Baron
2012-10-30 2:11 ` [Qemu-devel] [PATCH v1 10/13] Add a fallback bios file search, if -L fails Jason Baron
2012-10-30 2:11 ` [Qemu-devel] [PATCH v1 11/13] q35: automatically load the q35 dsdt table Jason Baron
2012-10-30 2:11 ` [Qemu-devel] [PATCH v1 12/13] q35: fill in usb pci slots with -usb Jason Baron
2012-10-30 6:34 ` Gerd Hoffmann
2012-10-30 15:19 ` Jason Baron
2012-10-30 16:19 ` Gerd Hoffmann
2012-10-30 18:00 ` Jason Baron
2012-10-30 2:11 ` [Qemu-devel] [PATCH v1 13/13] Fixup q35/ich9 Licenses Jason Baron
2012-10-31 8:59 ` Michael S. Tsirkin
2012-10-31 9:34 ` Isaku Yamahata
2012-10-31 9:57 ` Michael S. Tsirkin
2012-10-30 19:20 ` [Qemu-devel] [PATCH v1 00/13] q35 patches for pci tree Anthony Liguori
2012-10-31 8:42 ` Michael S. Tsirkin
2012-10-31 12:55 ` Anthony Liguori
2012-10-31 14:42 ` Jason Baron
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=509037FB.20000@suse.de \
--to=afaerber@suse.de \
--cc=agraf@suse.de \
--cc=alex.williamson@redhat.com \
--cc=aliguori@us.ibm.com \
--cc=armbru@redhat.com \
--cc=avi@redhat.com \
--cc=blauwirbel@gmail.com \
--cc=jan.kiszka@siemens.com \
--cc=jbaron@redhat.com \
--cc=juzhang@redhat.com \
--cc=kevin@koconnor.net \
--cc=kraxel@redhat.com \
--cc=lcapitulino@redhat.com \
--cc=liwanp@linux.vnet.ibm.com \
--cc=mkletzan@redhat.com \
--cc=mst@redhat.com \
--cc=pbonzini@redhat.com \
--cc=qemu-devel@nongnu.org \
--cc=yamahata@valinux.co.jp \
/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.