From: "Hervé Poussineau" <hpoussin@reactos.org>
To: qemu-devel@nongnu.org
Cc: "Paolo Bonzini" <pbonzini@redhat.com>,
"Hervé Poussineau" <hpoussin@reactos.org>
Subject: [Qemu-devel] [PATCH v2 0/8] memory: remove old_portio usage
Date: Sun, 9 Jun 2013 01:43:58 +0800 [thread overview]
Message-ID: <1370713446-9460-1-git-send-email-hpoussin@reactos.org> (raw)
These proposed patches aim at removing the .old_portio member of
MemoryRegionOps structure, and replacing their usage by .read/.write
handlers.
That way, faked I/O address space can be removed from architectures
which don't have it (MIPS, PowerPC...), and commits like
a178274efabcbbc5d44805b51def874e47051325 ("PPC: pseries: Remove hack
for PIO window") can be reapplied.
The first 4 patches do the cleanup and remove the old_portio handler
in MemoryRegion structure.
The last 4 patches are simplifications, now that Portio handlers
can be called from memory core without limitation.
Changes since v1:
- handling of Portio has been moved to ioport.c (instead of isa-bus.c)
This prevents creating an ISA bus on machines which don't have one
- added last 4 patches to see benefits of removing old_portio
Alexander Graf (1):
PPC: pseries: Remove hack for PIO window
Hervé Poussineau (7):
isa: fix documentation of isa_register_portio_list
memory: handle old_portio accesses in MMIO path
ioport: register memory regions for I/O port lists
memory: remove code dealing with old_portio
ioport: reimplement cpu_in/cpu_out using address_space_rw
ppc: simplify access to PReP I/O region
isa_mmio: simplify access to system I/O region
hw/isa/isa_mmio.c | 42 +++------
hw/ppc/prep.c | 65 ++-----------
hw/ppc/spapr_pci.c | 44 +--------
include/exec/ioport.h | 3 +-
include/exec/memory.h | 4 -
include/hw/isa/isa.h | 2 +-
include/hw/pci-host/spapr.h | 2 +-
ioport.c | 215 ++++++++++++++++++++++---------------------
memory.c | 44 ---------
9 files changed, 137 insertions(+), 284 deletions(-)
--
1.7.10.4
next reply other threads:[~2013-06-08 17:43 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-06-08 17:43 Hervé Poussineau [this message]
2013-06-08 17:43 ` [Qemu-devel] [PATCH v2 1/8] isa: fix documentation of isa_register_portio_list Hervé Poussineau
2013-06-08 17:44 ` [Qemu-devel] [PATCH v2 2/8] memory: handle old_portio accesses in MMIO path Hervé Poussineau
2013-06-08 17:44 ` [Qemu-devel] [PATCH v2 3/8] ioport: register memory regions for I/O port lists Hervé Poussineau
2013-06-08 17:44 ` [Qemu-devel] [PATCH v2 4/8] memory: remove code dealing with old_portio Hervé Poussineau
2013-06-08 17:44 ` [Qemu-devel] [PATCH v2 5/8] ioport: reimplement cpu_in/cpu_out using address_space_rw Hervé Poussineau
2013-06-08 17:44 ` [Qemu-devel] [PATCH v2 6/8] ppc: simplify access to PReP I/O region Hervé Poussineau
2013-06-08 17:44 ` [Qemu-devel] [PATCH v2 7/8] PPC: pseries: Remove hack for PIO window Hervé Poussineau
2013-06-08 17:44 ` [Qemu-devel] [PATCH v2 8/8] isa_mmio: simplify access to system I/O region Hervé Poussineau
2013-06-16 18:20 ` [Qemu-devel] [PATCH v2 0/8] memory: remove old_portio usage Hervé Poussineau
2013-06-17 7:32 ` Paolo Bonzini
2013-06-17 7:43 ` Jan Kiszka
2013-06-17 20:39 ` Hervé Poussineau
2013-06-18 14:38 ` Jan Kiszka
2013-06-18 18:09 ` Hervé Poussineau
2013-06-21 18:11 ` 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=1370713446-9460-1-git-send-email-hpoussin@reactos.org \
--to=hpoussin@reactos.org \
--cc=pbonzini@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.