| Message ID | 1535199590-68475-5-git-send-email-peng.hao2@zte.com.cn (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | introduce coalesced pio support | expand |
> On 25 Aug 2018, at 15:19, Peng Hao <peng.hao2@zte.com.cn> wrote: > > diff --git a/hw/pci-host/piix.c b/hw/pci-host/piix.c > index 0e60834..da73743 100644 > --- a/hw/pci-host/piix.c > +++ b/hw/pci-host/piix.c > @@ -327,6 +327,10 @@ static void i440fx_pcihost_realize(DeviceState *dev, Error **errp) > > sysbus_add_io(sbd, 0xcfc, &s->data_mem); > sysbus_init_ioports(sbd, 0xcfc, 4); > + > + /* register i440fx 0xcf8 port as coalesced pio */ > + memory_region_set_flush_coalesced(&s->data_mem); > + memory_region_add_coalescing(&s->conf_mem, 0, 4); > } > Is there a reason to not register this port as coalesced PIO also for Q35? In q35_host_realize()? If not, I would do that as an extra patch as part of this series. -Liran
diff --git a/hw/pci-host/piix.c b/hw/pci-host/piix.c index 0e60834..da73743 100644 --- a/hw/pci-host/piix.c +++ b/hw/pci-host/piix.c @@ -327,6 +327,10 @@ static void i440fx_pcihost_realize(DeviceState *dev, Error **errp) sysbus_add_io(sbd, 0xcfc, &s->data_mem); sysbus_init_ioports(sbd, 0xcfc, 4); + + /* register i440fx 0xcf8 port as coalesced pio */ + memory_region_set_flush_coalesced(&s->data_mem); + memory_region_add_coalescing(&s->conf_mem, 0, 4); } static void i440fx_realize(PCIDevice *dev, Error **errp)
Signed-off-by: Peng Hao <peng.hao2@zte.com.cn> --- hw/pci-host/piix.c | 4 ++++ 1 file changed, 4 insertions(+)