Re: [Xen-devel] [PATCH v2 2/2] x86/mmcfg/drhd: Move acpi_mmcfg_init() call before calling acpi_parse_dmar()

2018-08-20 Thread Jan Beulich
>>> On 20.08.18 at 11:38, wrote: > On 2018/8/20 15:45, Jan Beulich wrote: > On 20.08.18 at 05:38, wrote: >>> I'm thinking about moving below piece of code earlier too, and I checked >>> pci_mmcfg_check_hostbridge() carefully, it's secure, what do you think >>> about that? >>> >>>

Re: [Xen-devel] [PATCH v2 2/2] x86/mmcfg/drhd: Move acpi_mmcfg_init() call before calling acpi_parse_dmar()

2018-08-20 Thread Zhenzhong Duan
On 2018/8/20 15:45, Jan Beulich wrote: On 20.08.18 at 05:38, wrote: On 2018/8/17 20:28, Jan Beulich wrote: On 17.08.18 at 09:01, wrote: pci_conf_read8() needs pci mmcfg mapping to work on multiple pci segments system such as HPE Superdome-Flex. Move acpi_mmcfg_init() call in

Re: [Xen-devel] [PATCH v2 2/2] x86/mmcfg/drhd: Move acpi_mmcfg_init() call before calling acpi_parse_dmar()

2018-08-20 Thread Jan Beulich
>>> On 20.08.18 at 05:38, wrote: > On 2018/8/17 20:28, Jan Beulich wrote: > On 17.08.18 at 09:01, wrote: >>> pci_conf_read8() needs pci mmcfg mapping to work on multiple pci segments >>> system such as HPE Superdome-Flex. >>> >>> Move acpi_mmcfg_init() call in acpi_boot_init() before calling

Re: [Xen-devel] [PATCH v2 2/2] x86/mmcfg/drhd: Move acpi_mmcfg_init() call before calling acpi_parse_dmar()

2018-08-19 Thread Zhenzhong Duan
On 2018/8/17 20:28, Jan Beulich wrote: On 17.08.18 at 09:01, wrote: pci_conf_read8() needs pci mmcfg mapping to work on multiple pci segments system such as HPE Superdome-Flex. Move acpi_mmcfg_init() call in acpi_boot_init() before calling acpi_parse_dmar() so that when pci_conf_read8() is

Re: [Xen-devel] [PATCH v2 2/2] x86/mmcfg/drhd: Move acpi_mmcfg_init() call before calling acpi_parse_dmar()

2018-08-17 Thread zhenzhong.duan
Ok, that make sense for me. Thanks for your detailed explanation, I will rewrite the patch next week. Sent from mobile 2018年8月17日 20:28于 Jan Beulich 写道: > > >>> On 17.08.18 at 09:01, wrote: > > pci_conf_read8() needs pci mmcfg mapping to work on multiple pci segments > > system such as HPE

Re: [Xen-devel] [PATCH v2 2/2] x86/mmcfg/drhd: Move acpi_mmcfg_init() call before calling acpi_parse_dmar()

2018-08-17 Thread Jan Beulich
>>> On 17.08.18 at 09:01, wrote: > pci_conf_read8() needs pci mmcfg mapping to work on multiple pci segments > system such as HPE Superdome-Flex. > > Move acpi_mmcfg_init() call in acpi_boot_init() before calling > acpi_parse_dmar() so that when pci_conf_read8() is called in >

[Xen-devel] [PATCH v2 2/2] x86/mmcfg/drhd: Move acpi_mmcfg_init() call before calling acpi_parse_dmar()

2018-08-17 Thread Zhenzhong Duan
pci_conf_read8() needs pci mmcfg mapping to work on multiple pci segments system such as HPE Superdome-Flex. Move acpi_mmcfg_init() call in acpi_boot_init() before calling acpi_parse_dmar() so that when pci_conf_read8() is called in acpi_parse_dev_scope(), we already have the mapping set up.