Re: [PATCH V6 1/8] pvpanic: move pvpanic to misc as common driver

From: kbuild test robot
Date: Fri Nov 02 2018 - 02:42:26 EST

Next message: Jon Masters: "Re: [PATCH v2 00/11] arch/x86: AMD QoS support"
Previous message: Daniel Vacek: "Re: [PATCH] tsc: make calibration refinement more robust"
In reply to: Peng Hao: "[PATCH V6 6/8] misc/pvpanic: remove unnecessary header file"
Next in thread: kbuild test robot: "Re: [PATCH V6 1/8] pvpanic: move pvpanic to misc as common driver"
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

Hi Peng,

Thank you for the patch! Yet something to improve:

[auto build test ERROR on char-misc/char-misc-testing]
[also build test ERROR on v4.19 next-20181102]
[if your patch is applied to the wrong git tree, please drop us a note to help improve the system]

url: https://github.com/0day-ci/linux/commits/Peng-Hao/pvpanic-move-pvpanic-to-misc-as-common-driver/20181102-124907
config: openrisc-allyesconfig (attached as .config)
compiler: or1k-linux-gcc (GCC) 6.0.0 20160327 (experimental)
reproduce:
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# save the attached .config to linux build tree
make.cross ARCH=openrisc

All errors (new ones prefixed by >>):

drivers/misc/pvpanic.c:36:36: error: array type has incomplete element type 'struct acpi_device_id'
static const struct acpi_device_id pvpanic_device_ids[] = {
^~~~~~~~~~~~~~~~~~
drivers/misc/pvpanic.c:46:15: error: variable 'pvpanic_driver' has initializer but incomplete type
static struct acpi_driver pvpanic_driver = {
^~~~~~~~~~~
>> drivers/misc/pvpanic.c:47:2: error: unknown field 'name' specified in initializer
.name = "pvpanic",
^
drivers/misc/pvpanic.c:47:11: warning: excess elements in struct initializer
.name = "pvpanic",
^~~~~~~~~
drivers/misc/pvpanic.c:47:11: note: (near initialization for 'pvpanic_driver')
>> drivers/misc/pvpanic.c:48:2: error: unknown field 'class' specified in initializer
.class = "QEMU",
^
drivers/misc/pvpanic.c:48:11: warning: excess elements in struct initializer
.class = "QEMU",
^~~~~~
drivers/misc/pvpanic.c:48:11: note: (near initialization for 'pvpanic_driver')
>> drivers/misc/pvpanic.c:49:2: error: unknown field 'ids' specified in initializer
.ids = pvpanic_device_ids,
^
drivers/misc/pvpanic.c:49:10: warning: excess elements in struct initializer
.ids = pvpanic_device_ids,
^~~~~~~~~~~~~~~~~~
drivers/misc/pvpanic.c:49:10: note: (near initialization for 'pvpanic_driver')
>> drivers/misc/pvpanic.c:50:2: error: unknown field 'ops' specified in initializer
.ops = {
^
drivers/misc/pvpanic.c:50:10: error: extra brace group at end of initializer
.ops = {
^
drivers/misc/pvpanic.c:50:10: note: (near initialization for 'pvpanic_driver')
drivers/misc/pvpanic.c:50:10: warning: excess elements in struct initializer
drivers/misc/pvpanic.c:50:10: note: (near initialization for 'pvpanic_driver')
>> drivers/misc/pvpanic.c:54:2: error: unknown field 'owner' specified in initializer
.owner = THIS_MODULE,
^
In file included from include/linux/linkage.h:7:0,
from include/linux/kernel.h:7,
from drivers/misc/pvpanic.c:23:
include/linux/export.h:18:21: warning: excess elements in struct initializer
#define THIS_MODULE ((struct module *)0)
^
drivers/misc/pvpanic.c:54:11: note: in expansion of macro 'THIS_MODULE'
.owner = THIS_MODULE,
^~~~~~~~~~~
include/linux/export.h:18:21: note: (near initialization for 'pvpanic_driver')
#define THIS_MODULE ((struct module *)0)
^
drivers/misc/pvpanic.c:54:11: note: in expansion of macro 'THIS_MODULE'
.owner = THIS_MODULE,
^~~~~~~~~~~
drivers/misc/pvpanic.c: In function 'pvpanic_send_event':
drivers/misc/pvpanic.c:60:2: error: implicit declaration of function 'outb' [-Werror=implicit-function-declaration]
outb(event, port);
^~~~
drivers/misc/pvpanic.c: In function 'pvpanic_add':
>> drivers/misc/pvpanic.c:97:8: error: implicit declaration of function 'acpi_bus_get_status' [-Werror=implicit-function-declaration]
ret = acpi_bus_get_status(device);
^~~~~~~~~~~~~~~~~~~
drivers/misc/pvpanic.c:101:13: error: dereferencing pointer to incomplete type 'struct acpi_device'
if (!device->status.enabled || !device->status.functional)
^~
drivers/misc/pvpanic.c: At top level:
drivers/misc/pvpanic.c:124:1: warning: data definition has no type or storage class
module_acpi_driver(pvpanic_driver);
^~~~~~~~~~~~~~~~~~
drivers/misc/pvpanic.c:124:1: error: type defaults to 'int' in declaration of 'module_acpi_driver' [-Werror=implicit-int]
drivers/misc/pvpanic.c:124:1: warning: parameter names (without types) in function declaration
drivers/misc/pvpanic.c:46:27: error: storage size of 'pvpanic_driver' isn't known
static struct acpi_driver pvpanic_driver = {
^~~~~~~~~~~~~~
drivers/misc/pvpanic.c:46:27: warning: 'pvpanic_driver' defined but not used [-Wunused-variable]
drivers/misc/pvpanic.c:36:36: warning: 'pvpanic_device_ids' defined but not used [-Wunused-variable]
static const struct acpi_device_id pvpanic_device_ids[] = {
^~~~~~~~~~~~~~~~~~
cc1: some warnings being treated as errors

vim +/name +47 drivers/misc/pvpanic.c

8b10acd74 drivers/platform/x86/pvpanic.c Hu Tao 2013-05-08 35
8b10acd74 drivers/platform/x86/pvpanic.c Hu Tao 2013-05-08 @36 static const struct acpi_device_id pvpanic_device_ids[] = {
8b10acd74 drivers/platform/x86/pvpanic.c Hu Tao 2013-05-08 37 { "QEMU0001", 0 },
8b10acd74 drivers/platform/x86/pvpanic.c Hu Tao 2013-05-08 38 { "", 0 },
8b10acd74 drivers/platform/x86/pvpanic.c Hu Tao 2013-05-08 39 };
8b10acd74 drivers/platform/x86/pvpanic.c Hu Tao 2013-05-08 40 MODULE_DEVICE_TABLE(acpi, pvpanic_device_ids);
8b10acd74 drivers/platform/x86/pvpanic.c Hu Tao 2013-05-08 41
8b10acd74 drivers/platform/x86/pvpanic.c Hu Tao 2013-05-08 42 #define PVPANIC_PANICKED (1 << 0)
8b10acd74 drivers/platform/x86/pvpanic.c Hu Tao 2013-05-08 43
8b10acd74 drivers/platform/x86/pvpanic.c Hu Tao 2013-05-08 44 static u16 port;
8b10acd74 drivers/platform/x86/pvpanic.c Hu Tao 2013-05-08 45
8b10acd74 drivers/platform/x86/pvpanic.c Hu Tao 2013-05-08 46 static struct acpi_driver pvpanic_driver = {
8b10acd74 drivers/platform/x86/pvpanic.c Hu Tao 2013-05-08 @47 .name = "pvpanic",
8b10acd74 drivers/platform/x86/pvpanic.c Hu Tao 2013-05-08 @48 .class = "QEMU",
8b10acd74 drivers/platform/x86/pvpanic.c Hu Tao 2013-05-08 @49 .ids = pvpanic_device_ids,
8b10acd74 drivers/platform/x86/pvpanic.c Hu Tao 2013-05-08 @50 .ops = {
8b10acd74 drivers/platform/x86/pvpanic.c Hu Tao 2013-05-08 51 .add = pvpanic_add,
8b10acd74 drivers/platform/x86/pvpanic.c Hu Tao 2013-05-08 52 .remove = pvpanic_remove,
8b10acd74 drivers/platform/x86/pvpanic.c Hu Tao 2013-05-08 53 },
8b10acd74 drivers/platform/x86/pvpanic.c Hu Tao 2013-05-08 @54 .owner = THIS_MODULE,
8b10acd74 drivers/platform/x86/pvpanic.c Hu Tao 2013-05-08 55 };
8b10acd74 drivers/platform/x86/pvpanic.c Hu Tao 2013-05-08 56
8b10acd74 drivers/platform/x86/pvpanic.c Hu Tao 2013-05-08 57 static void
8b10acd74 drivers/platform/x86/pvpanic.c Hu Tao 2013-05-08 58 pvpanic_send_event(unsigned int event)
8b10acd74 drivers/platform/x86/pvpanic.c Hu Tao 2013-05-08 59 {
8b10acd74 drivers/platform/x86/pvpanic.c Hu Tao 2013-05-08 60 outb(event, port);
8b10acd74 drivers/platform/x86/pvpanic.c Hu Tao 2013-05-08 61 }
8b10acd74 drivers/platform/x86/pvpanic.c Hu Tao 2013-05-08 62
8b10acd74 drivers/platform/x86/pvpanic.c Hu Tao 2013-05-08 63 static int
8b10acd74 drivers/platform/x86/pvpanic.c Hu Tao 2013-05-08 64 pvpanic_panic_notify(struct notifier_block *nb, unsigned long code,
8b10acd74 drivers/platform/x86/pvpanic.c Hu Tao 2013-05-08 65 void *unused)
8b10acd74 drivers/platform/x86/pvpanic.c Hu Tao 2013-05-08 66 {
8b10acd74 drivers/platform/x86/pvpanic.c Hu Tao 2013-05-08 67 pvpanic_send_event(PVPANIC_PANICKED);
8b10acd74 drivers/platform/x86/pvpanic.c Hu Tao 2013-05-08 68 return NOTIFY_DONE;
8b10acd74 drivers/platform/x86/pvpanic.c Hu Tao 2013-05-08 69 }
8b10acd74 drivers/platform/x86/pvpanic.c Hu Tao 2013-05-08 70
8b10acd74 drivers/platform/x86/pvpanic.c Hu Tao 2013-05-08 71 static struct notifier_block pvpanic_panic_nb = {
8b10acd74 drivers/platform/x86/pvpanic.c Hu Tao 2013-05-08 72 .notifier_call = pvpanic_panic_notify,
7939831ea drivers/platform/x86/pvpanic.c Takashi Iwai 2014-05-06 73 .priority = 1, /* let this called before broken drm_fb_helper */
8b10acd74 drivers/platform/x86/pvpanic.c Hu Tao 2013-05-08 74 };
8b10acd74 drivers/platform/x86/pvpanic.c Hu Tao 2013-05-08 75
8b10acd74 drivers/platform/x86/pvpanic.c Hu Tao 2013-05-08 76
8b10acd74 drivers/platform/x86/pvpanic.c Hu Tao 2013-05-08 77 static acpi_status
8b10acd74 drivers/platform/x86/pvpanic.c Hu Tao 2013-05-08 78 pvpanic_walk_resources(struct acpi_resource *res, void *context)
8b10acd74 drivers/platform/x86/pvpanic.c Hu Tao 2013-05-08 79 {
8b10acd74 drivers/platform/x86/pvpanic.c Hu Tao 2013-05-08 80 switch (res->type) {
8b10acd74 drivers/platform/x86/pvpanic.c Hu Tao 2013-05-08 81 case ACPI_RESOURCE_TYPE_END_TAG:
8b10acd74 drivers/platform/x86/pvpanic.c Hu Tao 2013-05-08 82 return AE_OK;
8b10acd74 drivers/platform/x86/pvpanic.c Hu Tao 2013-05-08 83
8b10acd74 drivers/platform/x86/pvpanic.c Hu Tao 2013-05-08 84 case ACPI_RESOURCE_TYPE_IO:
8b10acd74 drivers/platform/x86/pvpanic.c Hu Tao 2013-05-08 85 port = res->data.io.minimum;
8b10acd74 drivers/platform/x86/pvpanic.c Hu Tao 2013-05-08 86 return AE_OK;
8b10acd74 drivers/platform/x86/pvpanic.c Hu Tao 2013-05-08 87
8b10acd74 drivers/platform/x86/pvpanic.c Hu Tao 2013-05-08 88 default:
8b10acd74 drivers/platform/x86/pvpanic.c Hu Tao 2013-05-08 89 return AE_ERROR;
8b10acd74 drivers/platform/x86/pvpanic.c Hu Tao 2013-05-08 90 }
8b10acd74 drivers/platform/x86/pvpanic.c Hu Tao 2013-05-08 91 }
8b10acd74 drivers/platform/x86/pvpanic.c Hu Tao 2013-05-08 92
8b10acd74 drivers/platform/x86/pvpanic.c Hu Tao 2013-05-08 93 static int pvpanic_add(struct acpi_device *device)
8b10acd74 drivers/platform/x86/pvpanic.c Hu Tao 2013-05-08 94 {
55cd3f01d drivers/platform/x86/pvpanic.c Radim KrÄmÃÅ 2015-05-29 95 int ret;
8b10acd74 drivers/platform/x86/pvpanic.c Hu Tao 2013-05-08 96
55cd3f01d drivers/platform/x86/pvpanic.c Radim KrÄmÃÅ 2015-05-29 @97 ret = acpi_bus_get_status(device);
55cd3f01d drivers/platform/x86/pvpanic.c Radim KrÄmÃÅ 2015-05-29 98 if (ret < 0)
55cd3f01d drivers/platform/x86/pvpanic.c Radim KrÄmÃÅ 2015-05-29 99 return ret;
8b10acd74 drivers/platform/x86/pvpanic.c Hu Tao 2013-05-08 100
55cd3f01d drivers/platform/x86/pvpanic.c Radim KrÄmÃÅ 2015-05-29 101 if (!device->status.enabled || !device->status.functional)
8b10acd74 drivers/platform/x86/pvpanic.c Hu Tao 2013-05-08 102 return -ENODEV;
8b10acd74 drivers/platform/x86/pvpanic.c Hu Tao 2013-05-08 103
8b10acd74 drivers/platform/x86/pvpanic.c Hu Tao 2013-05-08 104 acpi_walk_resources(device->handle, METHOD_NAME__CRS,
8b10acd74 drivers/platform/x86/pvpanic.c Hu Tao 2013-05-08 105 pvpanic_walk_resources, NULL);
8b10acd74 drivers/platform/x86/pvpanic.c Hu Tao 2013-05-08 106
8b10acd74 drivers/platform/x86/pvpanic.c Hu Tao 2013-05-08 107 if (!port)
8b10acd74 drivers/platform/x86/pvpanic.c Hu Tao 2013-05-08 108 return -ENODEV;
8b10acd74 drivers/platform/x86/pvpanic.c Hu Tao 2013-05-08 109
8b10acd74 drivers/platform/x86/pvpanic.c Hu Tao 2013-05-08 110 atomic_notifier_chain_register(&panic_notifier_list,
8b10acd74 drivers/platform/x86/pvpanic.c Hu Tao 2013-05-08 111 &pvpanic_panic_nb);
8b10acd74 drivers/platform/x86/pvpanic.c Hu Tao 2013-05-08 112
8b10acd74 drivers/platform/x86/pvpanic.c Hu Tao 2013-05-08 113 return 0;
8b10acd74 drivers/platform/x86/pvpanic.c Hu Tao 2013-05-08 114 }
8b10acd74 drivers/platform/x86/pvpanic.c Hu Tao 2013-05-08 115

:::::: The code at line 47 was first introduced by commit
:::::: 8b10acd74cdad9063c7a63468e31759d9ac877d9 pvpanic: pvpanic device driver

:::::: TO: Hu Tao <hutao@xxxxxxxxxxxxxx>
:::::: CC: Matthew Garrett <matthew.garrett@xxxxxxxxxx>

---
0-DAY kernel test infrastructure Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all Intel Corporation

Attachment: .config.gz
Description: application/gzip

Next message: Jon Masters: "Re: [PATCH v2 00/11] arch/x86: AMD QoS support"
Previous message: Daniel Vacek: "Re: [PATCH] tsc: make calibration refinement more robust"
In reply to: Peng Hao: "[PATCH V6 6/8] misc/pvpanic: remove unnecessary header file"
Next in thread: kbuild test robot: "Re: [PATCH V6 1/8] pvpanic: move pvpanic to misc as common driver"
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]