On Monday, December 02, 2013 04:14:32 PM Bjorn Helgaas wrote: > [+cc Lv, Rafael, linux-acpi] > > Is this fallout related to "ACPI: Cleanup <acpi/acpi.h>, > <acpi/acpi_bus.h> and <acpi/acpi_drivers.h> inclusions"? We have a new version of that patch that should avoid at least some of these errors. I'm going to replace the one in my tree with it shortly. > On Mon, Dec 2, 2013 at 7:57 AM, Jim Davis <jim.epost@xxxxxxxxx> wrote: > > Building with the attached random configuration file, > > > > arch/x86/pci/mmconfig-shared.c:373:1: error: unknown type name ‘acpi_status’ > > static acpi_status check_mcfg_resource(struct acpi_resource *res, void *data) > > ^ > > arch/x86/pci/mmconfig-shared.c:373:47: warning: ‘struct acpi_resource’ > > declared inside parameter list [enabled by default] > > static acpi_status check_mcfg_resource(struct acpi_resource *res, void *data) > > ^ > > arch/x86/pci/mmconfig-shared.c:373:47: warning: its scope is only this > > definition or declaration, which is probably not what you want > > [enabled by default] > > arch/x86/pci/mmconfig-shared.c: In function ‘check_mcfg_resource’: > > arch/x86/pci/mmconfig-shared.c:376:33: error: storage size of > > ‘address’ isn’t known > > struct acpi_resource_address64 address; > > ^ > > arch/x86/pci/mmconfig-shared.c:377:2: error: unknown type name ‘acpi_status’ > > acpi_status status; > > ^ > > arch/x86/pci/mmconfig-shared.c:379:9: error: dereferencing pointer to > > incomplete type > > if (res->type == ACPI_RESOURCE_TYPE_FIXED_MEMORY32) { > > ^ > > arch/x86/pci/mmconfig-shared.c:379:19: error: > > ‘ACPI_RESOURCE_TYPE_FIXED_MEMORY32’ undeclared (first use in this > > function) > > if (res->type == ACPI_RESOURCE_TYPE_FIXED_MEMORY32) { > > ^ > > arch/x86/pci/mmconfig-shared.c:379:19: note: each undeclared > > identifier is reported only once for each function it appears in > > arch/x86/pci/mmconfig-shared.c:381:8: error: dereferencing pointer to > > incomplete type > > &res->data.fixed_memory32; > > ^ > > arch/x86/pci/mmconfig-shared.c:383:11: error: ‘AE_OK’ undeclared > > (first use in this function) > > return AE_OK; > > ^ > > arch/x86/pci/mmconfig-shared.c:384:35: error: dereferencing pointer to > > incomplete type > > if ((mcfg_res->start >= fixmem32->address) && > > ^ > > arch/x86/pci/mmconfig-shared.c:385:33: error: dereferencing pointer to > > incomplete type > > (mcfg_res->end < (fixmem32->address + > > ^ > > arch/x86/pci/mmconfig-shared.c:386:19: error: dereferencing pointer to > > incomplete type > > fixmem32->address_length))) { > > ^ > > arch/x86/pci/mmconfig-shared.c:388:11: error: ‘AE_CTRL_TERMINATE’ > > undeclared (first use in this function) > > return AE_CTRL_TERMINATE; > > ^ > > arch/x86/pci/mmconfig-shared.c:391:10: error: dereferencing pointer to > > incomplete type > > if ((res->type != ACPI_RESOURCE_TYPE_ADDRESS32) && > > ^ > > arch/x86/pci/mmconfig-shared.c:391:20: error: > > ‘ACPI_RESOURCE_TYPE_ADDRESS32’ undeclared (first use in this function) > > if ((res->type != ACPI_RESOURCE_TYPE_ADDRESS32) && > > ^ > > arch/x86/pci/mmconfig-shared.c:392:10: error: dereferencing pointer to > > incomplete type > > (res->type != ACPI_RESOURCE_TYPE_ADDRESS64)) > > ^ > > arch/x86/pci/mmconfig-shared.c:392:20: error: > > ‘ACPI_RESOURCE_TYPE_ADDRESS64’ undeclared (first use in this function) > > (res->type != ACPI_RESOURCE_TYPE_ADDRESS64)) > > ^ > > arch/x86/pci/mmconfig-shared.c:395:2: error: implicit declaration of > > function ‘acpi_resource_to_address64’ > > [-Werror=implicit-function-declaration] > > status = acpi_resource_to_address64(res, &address); > > ^ > > arch/x86/pci/mmconfig-shared.c:396:2: error: implicit declaration of > > function ‘ACPI_FAILURE’ [-Werror=implicit-function-declaration] > > if (ACPI_FAILURE(status) || > > ^ > > arch/x86/pci/mmconfig-shared.c:398:31: error: ‘ACPI_MEMORY_RANGE’ > > undeclared (first use in this function) > > (address.resource_type != ACPI_MEMORY_RANGE)) > > ^ > > arch/x86/pci/mmconfig-shared.c:376:33: warning: unused variable > > ‘address’ [-Wunused-variable] > > struct acpi_resource_address64 address; > > ^ > > arch/x86/pci/mmconfig-shared.c: At top level: > > arch/x86/pci/mmconfig-shared.c:409:1: error: unknown type name ‘acpi_status’ > > static acpi_status find_mboard_resource(acpi_handle handle, u32 lvl, > > ^ > > arch/x86/pci/mmconfig-shared.c:409:41: error: unknown type name ‘acpi_handle’ > > static acpi_status find_mboard_resource(acpi_handle handle, u32 lvl, > > ^ > > arch/x86/pci/mmconfig-shared.c: In function ‘is_acpi_reserved’: > > arch/x86/pci/mmconfig-shared.c:431:2: error: implicit declaration of > > function ‘acpi_get_devices’ [-Werror=implicit-function-declaration] > > acpi_get_devices("PNP0C01", find_mboard_resource, &mcfg_res, NULL); > > ^ > > arch/x86/pci/mmconfig-shared.c:431:30: error: ‘find_mboard_resource’ > > undeclared ^ > > jim@krebstar:~$ more errs > > jim@krebstar:~$ cat errs > > arch/x86/pci/mmconfig-shared.c:373:1: error: unknown type name ‘acpi_status’ > > static acpi_status check_mcfg_resource(struct acpi_resource *res, void *data) > > ^ > > arch/x86/pci/mmconfig-shared.c:373:47: warning: ‘struct acpi_resource’ > > declared inside parameter list [enabled by default] > > static acpi_status check_mcfg_resource(struct acpi_resource *res, void *data) > > ^ > > arch/x86/pci/mmconfig-shared.c:373:47: warning: its scope is only this > > definition or declaration, which is probably not what you want > > [enabled by default] > > arch/x86/pci/mmconfig-shared.c: In function ‘check_mcfg_resource’: > > arch/x86/pci/mmconfig-shared.c:376:33: error: storage size of > > ‘address’ isn’t known > > struct acpi_resource_address64 address; > > ^ > > arch/x86/pci/mmconfig-shared.c:377:2: error: unknown type name ‘acpi_status’ > > acpi_status status; > > ^ > > arch/x86/pci/mmconfig-shared.c:379:9: error: dereferencing pointer to > > incomplete type > > if (res->type == ACPI_RESOURCE_TYPE_FIXED_MEMORY32) { > > ^ > > arch/x86/pci/mmconfig-shared.c:379:19: error: > > ‘ACPI_RESOURCE_TYPE_FIXED_MEMORY32’ undeclared (first use in this > > function) > > if (res->type == ACPI_RESOURCE_TYPE_FIXED_MEMORY32) { > > ^ > > arch/x86/pci/mmconfig-shared.c:379:19: note: each undeclared > > identifier is reported only once for each function it appears in > > arch/x86/pci/mmconfig-shared.c:381:8: error: dereferencing pointer to > > incomplete type > > &res->data.fixed_memory32; > > ^ > > arch/x86/pci/mmconfig-shared.c:383:11: error: ‘AE_OK’ undeclared > > (first use in this function) > > return AE_OK; > > ^ > > arch/x86/pci/mmconfig-shared.c:384:35: error: dereferencing pointer to > > incomplete type > > if ((mcfg_res->start >= fixmem32->address) && > > ^ > > arch/x86/pci/mmconfig-shared.c:385:33: error: dereferencing pointer to > > incomplete type > > (mcfg_res->end < (fixmem32->address + > > ^ > > arch/x86/pci/mmconfig-shared.c:386:19: error: dereferencing pointer to > > incomplete type > > fixmem32->address_length))) { > > ^ > > arch/x86/pci/mmconfig-shared.c:388:11: error: ‘AE_CTRL_TERMINATE’ > > undeclared (first use in this function) > > return AE_CTRL_TERMINATE; > > ^ > > arch/x86/pci/mmconfig-shared.c:391:10: error: dereferencing pointer to > > incomplete type > > if ((res->type != ACPI_RESOURCE_TYPE_ADDRESS32) && > > ^ > > arch/x86/pci/mmconfig-shared.c:391:20: error: > > ‘ACPI_RESOURCE_TYPE_ADDRESS32’ undeclared (first use in this function) > > if ((res->type != ACPI_RESOURCE_TYPE_ADDRESS32) && > > ^ > > arch/x86/pci/mmconfig-shared.c:392:10: error: dereferencing pointer to > > incomplete type > > (res->type != ACPI_RESOURCE_TYPE_ADDRESS64)) > > ^ > > arch/x86/pci/mmconfig-shared.c:392:20: error: > > ‘ACPI_RESOURCE_TYPE_ADDRESS64’ undeclared (first use in this function) > > (res->type != ACPI_RESOURCE_TYPE_ADDRESS64)) > > ^ > > arch/x86/pci/mmconfig-shared.c:395:2: error: implicit declaration of > > function ‘acpi_resource_to_address64’ > > [-Werror=implicit-function-declaration] > > status = acpi_resource_to_address64(res, &address); > > ^ > > arch/x86/pci/mmconfig-shared.c:396:2: error: implicit declaration of > > function ‘ACPI_FAILURE’ [-Werror=implicit-function-declaration] > > if (ACPI_FAILURE(status) || > > ^ > > arch/x86/pci/mmconfig-shared.c:398:31: error: ‘ACPI_MEMORY_RANGE’ > > undeclared (first use in this function) > > (address.resource_type != ACPI_MEMORY_RANGE)) > > ^ > > arch/x86/pci/mmconfig-shared.c:376:33: warning: unused variable > > ‘address’ [-Wunused-variable] > > struct acpi_resource_address64 address; > > ^ > > arch/x86/pci/mmconfig-shared.c: At top level: > > arch/x86/pci/mmconfig-shared.c:409:1: error: unknown type name ‘acpi_status’ > > static acpi_status find_mboard_resource(acpi_handle handle, u32 lvl, > > ^ > > arch/x86/pci/mmconfig-shared.c:409:41: error: unknown type name ‘acpi_handle’ > > static acpi_status find_mboard_resource(acpi_handle handle, u32 lvl, > > ^ > > arch/x86/pci/mmconfig-shared.c: In function ‘is_acpi_reserved’: > > arch/x86/pci/mmconfig-shared.c:431:2: error: implicit declaration of > > function ‘acpi_get_devices’ [-Werror=implicit-function-declaration] > > acpi_get_devices("PNP0C01", find_mboard_resource, &mcfg_res, NULL); > > ^ > > arch/x86/pci/mmconfig-shared.c:431:30: error: ‘find_mboard_resource’ > > undeclared (first use in this function) > > acpi_get_devices("PNP0C01", find_mboard_resource, &mcfg_res, NULL); > > ^ > > arch/x86/pci/mmconfig-shared.c: At top level: > > arch/x86/pci/mmconfig-shared.c:544:13: warning: ‘struct > > acpi_mcfg_allocation’ declared inside parameter list [enabled by > > default] > > struct acpi_mcfg_allocation *cfg) > > ^ > > arch/x86/pci/mmconfig-shared.c:544:13: warning: ‘struct > > acpi_table_mcfg’ declared inside parameter list [enabled by default] > > arch/x86/pci/mmconfig-shared.c: In function ‘acpi_mcfg_check_entry’: > > arch/x86/pci/mmconfig-shared.c:548:9: error: dereferencing pointer to > > incomplete type > > if (cfg->address < 0xFFFFFFFF) > > ^ > > arch/x86/pci/mmconfig-shared.c:551:19: error: dereferencing pointer to > > incomplete type > > if (!strncmp(mcfg->header.oem_id, "SGI", 3)) > > ^ > > arch/x86/pci/mmconfig-shared.c:554:10: error: dereferencing pointer to > > incomplete type > > if (mcfg->header.revision >= 1) { > > ^ > > In file included from include/linux/kernel.h:13:0, > > from include/linux/kernfs.h:10, > > from include/linux/sysfs.h:15, > > from include/linux/kobject.h:21, > > from include/linux/pci.h:28, > > from arch/x86/pci/mmconfig-shared.c:13: > > arch/x86/pci/mmconfig-shared.c:561:39: error: dereferencing pointer to > > incomplete type > > "is above 4GB, ignored\n", cfg->pci_segment, > > ^ > > include/linux/printk.h:217:33: note: in definition of macro ‘pr_err’ > > printk(KERN_ERR pr_fmt(fmt), ##__VA_ARGS__) > > ^ > > ^ > > arch/x86/pci/mmconfig-shared.c:600:32: error: dereferencing pointer to > > incomplete type > > cfg->end_bus_number, cfg->address) == NULL) { > > ^ > > arch/x86/pci/mmconfig-shared.c: In function ‘pci_mmcfg_early_init’: > > arch/x86/pci/mmconfig-shared.c:642:25: error: ‘ACPI_SIG_MCFG’ > > undeclared (first use in this function) > > acpi_sfi_table_parse(ACPI_SIG_MCFG, pci_parse_mcfg); > > ^ > > arch/x86/pci/mmconfig-shared.c: In function ‘pci_mmcfg_late_init’: > > arch/x86/pci/mmconfig-shared.c:658:24: error: ‘ACPI_SIG_MCFG’ > > undeclared (first use in this function) > > acpi_sfi_table_parse(ACPI_SIG_MCFG, pci_parse_mcfg); > > ^ > > arch/x86/pci/mmconfig-shared.c: At top level: > > arch/x86/pci/mmconfig-shared.c:373:20: warning: ‘check_mcfg_resource’ > > defined but not used [-Wunused-function] > > static acpi_status check_mcfg_resource(struct acpi_resource *res, void *data) > > ^ > > cc1: some warnings being treated as errors > > make[1]: *** [arch/x86/pci/mmconfig-shared.o] Error 1 -- I speak only for myself. Rafael J. Wysocki, Intel Open Source Technology Center. -- To unsubscribe from this list: send the line "unsubscribe linux-next" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html