All of lore.kernel.org
 help / color / mirror / Atom feed
From: Julien Grall <julien.grall@citrix.com>
To: qemu-devel@nongnu.org
Cc: Stefano.Stabellini@eu.citrix.com, jan.kiszka@siemens.com,
	Julien Grall <julien.grall@citrix.com>,
	kraxel@redhat.com, afaerber@suse.de, avi@redhat.com
Subject: [Qemu-devel] [PATCH V6 0/8] memory: unify ioport registration
Date: Wed, 29 Aug 2012 04:01:33 +0100	[thread overview]
Message-ID: <cover.1346208853.git.julien.grall@citrix.com> (raw)

This is the sixth version of patch series about ioport registration.

Some part of QEMU still use register_ioport* functions to register ioport.
These functions doesn't allow to use Memory Listener on it.

Modifications between V1 and V2:
   - Remove the use of get_system_io. Instead of use isa and pci IO
     address space.
   - Avoid allocation of PortioList. Use the different device
     structure.
   - Still remove register_ioport* (hw/dma.c, hw/apm.c,
     hw/acpi_piix4.c).
   - Use MemoryRegion when we have only a range of ioport.
   - For some functions, add IO address space as
     argument.
   - Add isa_address_space_io function

Modifications between V2 and V3:
   - Remove some register_ioport_* on hw/vt82c686.c.
   - Split smb ioport part in new patch.
   - Still replace MemoryRegion when we have only a range of ioport.
   - Fix read/write ioports prototype to  be compliant with memory callback.

Modifications between V3 and V4:
   - Fix compilation in hw/dma.c
   - Fix address conversion (hw/dma.c, hw/acpi_piix4.c) with MemorySection.
     Indeed the new version use offset from MemorySection start instead of 0.

Modifications between V4 and V5:
   - Rebase on qemu upstream.
   - Forget some ioport_register_* in acpi_piix4.c.
   - Register 0x3b0 - 0x3df range for cirrus instead of ioport by ioport.

Modifications between V5 and V6:
   - Add read function on cirrus ioport (forget on the previous patch).
   - Rework PM memory range handling.
   - Fix PCI_BASE in acpi_piix4.c (wrong conversion during port).
   - Rewrite isa_address_space_io to use ISA bus address space.
   - Fix compilation in vt82c686.c

Julien Grall (8):
  isa: add isa_address_space_io
  hw/acpi_piix4.c: replace register_ioport*
  hw/cirrus_vga.c: replace register_ioport*
  hw/serial.c: replace register_ioport*
  hw/pc.c: replace register_ioport*
  hw/dma.c: replace register_ioport*
  hw/apm.c: replace register_ioport*
  smb: replace_register_ioport*

 hw/acpi_piix4.c   |  170 +++++++++++++++++++++++++++++++++++++++++-----------
 hw/apm.c          |   24 ++++++--
 hw/apm.h          |    5 +-
 hw/cirrus_vga.c   |   50 +++++++++------
 hw/dma.c          |  108 ++++++++++++++++++++++-----------
 hw/isa-bus.c      |    9 +++
 hw/isa.h          |    1 +
 hw/mips_mipssim.c |    3 +-
 hw/pc.c           |   58 +++++++++++++-----
 hw/pc.h           |    2 +-
 hw/pm_smbus.c     |    7 +-
 hw/pm_smbus.h     |    6 +-
 hw/serial.c       |    8 ++-
 hw/vt82c686.c     |   20 +++++-
 14 files changed, 345 insertions(+), 126 deletions(-)

-- 
Julien Grall

             reply	other threads:[~2012-08-29  9:25 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-08-29  3:01 Julien Grall [this message]
2012-08-29  3:01 ` [Qemu-devel] [PATCH V6 1/8] isa: add isa_address_space_io Julien Grall
2012-08-29  3:01 ` [Qemu-devel] [PATCH V6 2/8] hw/acpi_piix4.c: replace register_ioport* Julien Grall
2012-09-01  7:44   ` Jan Kiszka
2012-09-03 12:56     ` Julien Grall
2012-09-03 12:56       ` Jan Kiszka
2012-09-03 13:04         ` Julien Grall
2012-09-03 11:19   ` Avi Kivity
2012-09-03 13:00     ` Julien Grall
2012-09-03 11:25   ` Avi Kivity
2012-08-29  3:01 ` [Qemu-devel] [PATCH V6 3/8] hw/cirrus_vga.c: " Julien Grall
2012-08-29  3:01 ` [Qemu-devel] [PATCH V6 4/8] hw/serial.c: " Julien Grall
2012-08-29  3:01 ` [Qemu-devel] [PATCH V6 5/8] hw/pc.c: " Julien Grall
2012-08-29  3:01 ` [Qemu-devel] [PATCH V6 6/8] hw/dma.c: " Julien Grall
2012-08-29  3:01 ` [Qemu-devel] [PATCH V6 7/8] hw/apm.c: " Julien Grall
2012-08-29  3:01 ` [Qemu-devel] [PATCH V6 8/8] smb: replace_register_ioport* Julien Grall

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=cover.1346208853.git.julien.grall@citrix.com \
    --to=julien.grall@citrix.com \
    --cc=Stefano.Stabellini@eu.citrix.com \
    --cc=afaerber@suse.de \
    --cc=avi@redhat.com \
    --cc=jan.kiszka@siemens.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.