From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:41297) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1YNIsJ-0002wm-7O for qemu-devel@nongnu.org; Mon, 16 Feb 2015 05:26:00 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1YNIsH-0005rl-W5 for qemu-devel@nongnu.org; Mon, 16 Feb 2015 05:25:59 -0500 Message-ID: <54E1C44F.9000501@redhat.com> Date: Mon, 16 Feb 2015 12:19:59 +0200 From: Marcel Apfelbaum MIME-Version: 1.0 References: <1424080457-13752-1-git-send-email-marcel@redhat.com> <1424080457-13752-5-git-send-email-marcel@redhat.com> <20150216110706.5a840b02@nial.brq.redhat.com> In-Reply-To: <20150216110706.5a840b02@nial.brq.redhat.com> Content-Type: text/plain; charset=windows-1252; format=flowed Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH RFC V2 04/17] hw/acpi: add _CRS method for extra root busses List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Igor Mammedov Cc: seabios@seabios.org, kraxel@redhat.com, mst@redhat.com, quintela@redhat.com, qemu-devel@nongnu.org, agraf@suse.de, alex.williamson@redhat.com, kevin@koconnor.net, qemu-ppc@nongnu.org, hare@suse.de, amit.shah@redhat.com, pbonzini@redhat.com, leon.alrae@imgtec.com, aurelien@aurel32.net, rth@twiddle.net On 02/16/2015 12:07 PM, Igor Mammedov wrote: > On Mon, 16 Feb 2015 11:54:04 +0200 > Marcel Apfelbaum wrote: > >> Save the IO/mem/bus numbers ranges assigned to the extra root busses >> to be removed from the root bus 0 range. Hi Igor, Thanks for the review. > Is it possible to make BIOS program extra buses and root bus 0 > in sane way so their resources won't intersect? It is possible and actually the first version was coded like that, however the version was rejected by the BIOS review because the resources allocation would not be optimal. This is the best way to allocate the resources so no space will be wasted. > That way QEMU will be able to just read resources from root buses > and generate AML in the same manner for every root bus instead of > collect->exclude hack and special casing root bus 0. This implementation supports actually the possibility to read resources also for bus 0, I chose this approach to have Bus 0 (and all busses behind it) the only owner of the unused/hot-pluggable memory ranges. The extra roots are hot-pluggable because they have a pci-2-pci bridge behind. Thanks, Marcel > >> >> Signed-off-by: Marcel Apfelbaum >> --- >> hw/i386/acpi-build.c | 146 +++++++++++++++++++++++++++++++++++++++++++++++++++ >> 1 file changed, 146 insertions(+) >> >> diff --git a/hw/i386/acpi-build.c b/hw/i386/acpi-build.c >> index ee1a50a..0822a20 100644 >> --- a/hw/i386/acpi-build.c >> +++ b/hw/i386/acpi-build.c >> @@ -719,6 +719,145 @@ static AcpiAml build_prt(void) >> return method; >> } >> >> +typedef struct PciRangeEntry { >> + QLIST_ENTRY(PciRangeEntry) entry; >> + int64_t base; >> + int64_t limit; >> +} PciRangeEntry; >> + >> +typedef QLIST_HEAD(PciRangeQ, PciRangeEntry) PciRangeQ; >> + >> +static void pci_range_insert(PciRangeQ *list, int64_t base, int64_t limit) { >> + PciRangeEntry *entry, *next, *e; >> + >> + if (!base) { >> + return; >> + } >> + >> + if (limit - base + 1 < 0x1000) >> + limit = base + 0x1000 - 1; >> + >> + e = g_malloc(sizeof(*entry)); >> + e->base = base; >> + e->limit = limit; >> + >> + if (QLIST_EMPTY(list)) { >> + QLIST_INSERT_HEAD(list, e, entry); >> + } else { >> + QLIST_FOREACH_SAFE(entry, list, entry, next) { >> + if (base < entry->base) { >> + QLIST_INSERT_BEFORE(entry, e, entry); >> + break; >> + } else if (!next) { >> + QLIST_INSERT_AFTER(entry, e, entry); >> + break; >> + } >> + } >> + } >> +} >> + >> +static void pci_range_list_free(PciRangeQ *list) >> +{ >> + PciRangeEntry *entry, *next; >> + >> + QLIST_FOREACH_SAFE(entry, list, entry, next) { >> + QLIST_REMOVE(entry, entry); >> + g_free(entry); >> + } >> +} >> + >> +static AcpiAml build_crs(PcPciInfo *pci, PciInfo *bus_info, >> + PciRangeQ *io_ranges, PciRangeQ *mem_ranges) >> +{ >> + PciDeviceInfoList *dev_list; >> + PciMemoryRange range; >> + AcpiAml crs; >> + uint8_t max_bus; >> + >> + crs = acpi_resource_template(); >> + max_bus = bus_info->bus; >> + >> + for (dev_list = bus_info->devices; dev_list; dev_list = dev_list->next) { >> + PciMemoryRegionList *region; >> + >> + for (region = dev_list->value->regions; region; region = region->next) { >> + range.base = region->value->address; >> + range.limit = region->value->address + region->value->size - 1; >> + >> + if (!strcmp(region->value->type, "io")) { >> + aml_append(&crs, >> + acpi_word_io(acpi_min_fixed, acpi_max_fixed, >> + acpi_pos_decode, acpi_entire_range, >> + 0x0000, >> + range.base, >> + range.limit, >> + 0x0000, >> + range.limit - range.base + 1)); >> + pci_range_insert(io_ranges, range.base, range.limit); >> + } else { /* "memory" */ >> + aml_append(&crs, >> + acpi_dword_memory(acpi_pos_decode, acpi_min_fixed, >> + acpi_max_fixed, acpi_non_cacheable, acpi_ReadWrite, >> + 0, >> + range.base, >> + range.limit, >> + 0, >> + range.limit - range.base + 1)); >> + pci_range_insert(mem_ranges, range.base, range.limit); >> + } >> + } >> + >> + if (dev_list->value->has_pci_bridge) { >> + PciBridgeInfo *bridge_info = dev_list->value->pci_bridge; >> + >> + if (bridge_info->bus.subordinate > max_bus) { >> + max_bus = bridge_info->bus.subordinate; >> + } >> + >> + range = *bridge_info->bus.io_range; >> + aml_append(&crs, >> + acpi_word_io(acpi_min_fixed, acpi_max_fixed, >> + acpi_pos_decode, acpi_entire_range, >> + 0x0000, >> + range.base, >> + range.limit, >> + 0x0000, >> + range.limit - range.base + 1)); >> + pci_range_insert(io_ranges, range.base, range.limit); >> + >> + range = *bridge_info->bus.memory_range; >> + aml_append(&crs, >> + acpi_dword_memory(acpi_pos_decode, acpi_min_fixed, >> + acpi_max_fixed, acpi_non_cacheable, acpi_ReadWrite, >> + 0, >> + range.base, >> + range.limit, >> + 0, >> + range.limit - range.base + 1)); >> + pci_range_insert(mem_ranges, range.base, range.limit); >> + >> + range = *bridge_info->bus.prefetchable_range; >> + aml_append(&crs, >> + acpi_dword_memory(acpi_pos_decode, acpi_min_fixed, >> + acpi_max_fixed, acpi_non_cacheable, acpi_ReadWrite, >> + 0, >> + range.base, >> + range.limit, >> + 0, >> + range.limit - range.base + 1)); >> + pci_range_insert(mem_ranges, range.base, range.limit); >> + } >> + } >> + >> + aml_append(&crs, >> + acpi_word_bus_number(acpi_min_fixed, acpi_max_fixed, >> + acpi_pos_decode, >> + 0x0000, bus_info->bus, max_bus, >> + 0x0000, max_bus - bus_info->bus + 1)); >> + >> + return crs; >> +} >> + >> static void >> build_ssdt(AcpiAml *table_aml, GArray *linker, >> AcpiCpuInfo *cpu, AcpiPmInfo *pm, AcpiMiscInfo *misc, >> @@ -729,6 +868,8 @@ build_ssdt(AcpiAml *table_aml, GArray *linker, >> unsigned acpi_cpus = guest_info->apic_id_limit; >> AcpiAml pkg, scope, dev, method, crs, field, ifctx, ssdt; >> int i; >> + PciRangeQ io_ranges = QLIST_HEAD_INITIALIZER(io_ranges); >> + PciRangeQ mem_ranges = QLIST_HEAD_INITIALIZER(mem_ranges); >> >> /* The current AML generator can cover the APIC ID range [0..255], >> * inclusive, for VCPU hotplug. */ >> @@ -764,9 +905,14 @@ build_ssdt(AcpiAml *table_aml, GArray *linker, >> aml_append(&dev, >> acpi_name_decl("_BBN", acpi_int((uint8_t)bus_info->bus))); >> aml_append(&dev, build_prt()); >> + crs = build_crs(pci, bus_info, &io_ranges, &mem_ranges); >> + aml_append(&dev, acpi_name_decl("_CRS", crs)); >> aml_append(&scope, dev); >> aml_append(&ssdt, scope); >> } >> + >> + pci_range_list_free(&io_ranges); >> + pci_range_list_free(&mem_ranges); >> qapi_free_PciInfoList(info_list); >> } >> >