All of lore.kernel.org
 help / color / mirror / Atom feed
From: kernel test robot <oliver.sang@intel.com>
To: Herve Codina <herve.codina@bootlin.com>
Cc: <oe-lkp@lists.linux.dev>, <lkp@intel.com>,
	<linux-pci@vger.kernel.org>,
	Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
	"Rafael J. Wysocki" <rafael@kernel.org>,
	Rob Herring <robh@kernel.org>,
	Saravana Kannan <saravanak@google.com>,
	Bjorn Helgaas <bhelgaas@google.com>,
	Lizhi Hou <lizhi.hou@amd.com>, <linux-kernel@vger.kernel.org>,
	<devicetree@vger.kernel.org>,
	Allan Nielsen <allan.nielsen@microchip.com>,
	Horatiu Vultur <horatiu.vultur@microchip.com>,
	Steen Hegelund <steen.hegelund@microchip.com>,
	Thomas Petazzoni <thomas.petazzoni@bootlin.com>,
	Herve Codina <herve.codina@bootlin.com>, <oliver.sang@intel.com>
Subject: Re: [PATCH v5 5/5] PCI: of: Create device-tree PCI host bridge node
Date: Tue, 24 Dec 2024 14:20:51 +0800	[thread overview]
Message-ID: <202412241455.22d84a7f-lkp@intel.com> (raw)
In-Reply-To: <20241209130339.81354-6-herve.codina@bootlin.com>



Hello,

kernel test robot noticed "WARNING:at_drivers/of/base.c:#of_n_addr_cells" on:

commit: 7e0c4ed7342d44f246bc8d905421f8010de74662 ("[PATCH v5 5/5] PCI: of: Create device-tree PCI host bridge node")
url: https://github.com/intel-lab-lkp/linux/commits/Herve-Codina/driver-core-Introduce-device_-add-remove-_of_node/20241209-211305
base: https://git.kernel.org/cgit/linux/kernel/git/pci/pci.git next
patch link: https://lore.kernel.org/all/20241209130339.81354-6-herve.codina@bootlin.com/
patch subject: [PATCH v5 5/5] PCI: of: Create device-tree PCI host bridge node

in testcase: boot

config: i386-randconfig-001-20241220
compiler: clang-19
test machine: qemu-system-i386 -enable-kvm -cpu SandyBridge -smp 2 -m 4G

(please refer to attached dmesg/kmsg for entire log/backtrace)


+-------------------------------------------------------------------------+------------+------------+
|                                                                         | 320b0bf1d5 | 7e0c4ed734 |
+-------------------------------------------------------------------------+------------+------------+
| WARNING:at_drivers/of/base.c:#of_n_addr_cells                           | 0          | 18         |
| EIP:of_n_addr_cells                                                     | 0          | 18         |
+-------------------------------------------------------------------------+------------+------------+


If you fix the issue in a separate patch/commit (i.e. not just a new version of
the same patch/commit), kindly add following tags
| Reported-by: kernel test robot <oliver.sang@intel.com>
| Closes: https://lore.kernel.org/oe-lkp/202412241455.22d84a7f-lkp@intel.com


[    1.417876][    T1] ------------[ cut here ]------------
[    1.418708][    T1] Missing '#address-cells' in
[ 1.419500][ T1] WARNING: CPU: 1 PID: 1 at drivers/of/base.c:107 of_n_addr_cells (drivers/of/base.c:106 drivers/of/base.c:117)
[    1.420197][    T1] Modules linked in:
[    1.420818][    T1] CPU: 1 UID: 0 PID: 1 Comm: swapper/0 Not tainted 6.13.0-rc1-00012-g7e0c4ed7342d #1 4d930b120051bd0d99d76027ef034fe5e41e6f6f
[    1.422678][    T1] Hardware name: QEMU Standard PC (i440FX + PIIX, 1996), BIOS 1.16.2-debian-1.16.2-1 04/01/2014
[ 1.424206][ T1] EIP: of_n_addr_cells (drivers/of/base.c:106 drivers/of/base.c:117)
[ 1.424986][ T1] Code: c0 79 2b 80 3d 6f 0c 00 44 00 74 09 8b 7f 34 85 ff 75 d7 eb 1c c6 05 6f 0c 00 44 01 57 68 d7 08 1e 43 e8 53 12 a3 fe 83 c4 08 <0f> 0b eb de 8b 75 f0 89 f0 83 c4 04 5e 5f 5b 5d 31 c9 31 d2 c3 cc
All code
========
   0:	c0 79 2b 80          	sarb   $0x80,0x2b(%rcx)
   4:	3d 6f 0c 00 44       	cmp    $0x44000c6f,%eax
   9:	00 74 09 8b          	add    %dh,-0x75(%rcx,%rcx,1)
   d:	7f 34                	jg     0x43
   f:	85 ff                	test   %edi,%edi
  11:	75 d7                	jne    0xffffffffffffffea
  13:	eb 1c                	jmp    0x31
  15:	c6 05 6f 0c 00 44 01 	movb   $0x1,0x44000c6f(%rip)        # 0x44000c8b
  1c:	57                   	push   %rdi
  1d:	68 d7 08 1e 43       	push   $0x431e08d7
  22:	e8 53 12 a3 fe       	call   0xfffffffffea3127a
  27:	83 c4 08             	add    $0x8,%esp
  2a:*	0f 0b                	ud2		<-- trapping instruction
  2c:	eb de                	jmp    0xc
  2e:	8b 75 f0             	mov    -0x10(%rbp),%esi
  31:	89 f0                	mov    %esi,%eax
  33:	83 c4 04             	add    $0x4,%esp
  36:	5e                   	pop    %rsi
  37:	5f                   	pop    %rdi
  38:	5b                   	pop    %rbx
  39:	5d                   	pop    %rbp
  3a:	31 c9                	xor    %ecx,%ecx
  3c:	31 d2                	xor    %edx,%edx
  3e:	c3                   	ret
  3f:	cc                   	int3

Code starting with the faulting instruction
===========================================
   0:	0f 0b                	ud2
   2:	eb de                	jmp    0xffffffffffffffe2
   4:	8b 75 f0             	mov    -0x10(%rbp),%esi
   7:	89 f0                	mov    %esi,%eax
   9:	83 c4 04             	add    $0x4,%esp
   c:	5e                   	pop    %rsi
   d:	5f                   	pop    %rdi
   e:	5b                   	pop    %rbx
   f:	5d                   	pop    %rbp
  10:	31 c9                	xor    %ecx,%ecx
  12:	31 d2                	xor    %edx,%edx
  14:	c3                   	ret
  15:	cc                   	int3
[    1.427777][    T1] EAX: 00000000 EBX: 44d5992c ECX: 00000000 EDX: 00000000
[    1.428821][    T1] ESI: 00000001 EDI: ee7fdeb8 EBP: 44d5993c ESP: 44d5992c
[    1.429891][    T1] DS: 007b ES: 007b FS: 00d8 GS: 0000 SS: 0068 EFLAGS: 00010202
[    1.430985][    T1] CR0: 80050033 CR2: 00000000 CR3: 042ec000 CR4: 000406d0
[    1.432049][    T1] DR0: 00000000 DR1: 00000000 DR2: 00000000 DR3: 00000000
[    1.433214][    T1] DR6: fffe0ff0 DR7: 00000400
[    1.433946][    T1] Call Trace:
[ 1.434502][ T1] ? show_regs (arch/x86/kernel/dumpstack.c:478)
[ 1.435199][ T1] ? of_n_addr_cells (drivers/of/base.c:106 drivers/of/base.c:117)
[ 1.435949][ T1] ? __warn (kernel/panic.c:242)
[ 1.436615][ T1] ? of_n_addr_cells (drivers/of/base.c:106 drivers/of/base.c:117)
[ 1.437327][ T1] ? of_n_addr_cells (drivers/of/base.c:106 drivers/of/base.c:117)
[ 1.437993][ T1] ? report_bug (lib/bug.c:199)
[ 1.438659][ T1] ? exc_overflow (arch/x86/kernel/traps.c:301)
[ 1.439808][ T1] ? handle_bug (arch/x86/kernel/traps.c:?)
[ 1.440043][ T1] ? exc_invalid_op (arch/x86/kernel/traps.c:309)
[ 1.440791][ T1] ? handle_exception (arch/x86/entry/entry_32.S:1048)
[ 1.441635][ T1] ? exc_overflow (arch/x86/kernel/traps.c:301)
[ 1.442355][ T1] ? of_n_addr_cells (drivers/of/base.c:106 drivers/of/base.c:117)
[ 1.443145][ T1] ? exc_overflow (arch/x86/kernel/traps.c:301)
[ 1.443829][ T1] ? of_n_addr_cells (drivers/of/base.c:106 drivers/of/base.c:117)
[ 1.444564][ T1] ? of_pci_add_host_bridge_properties (drivers/pci/of_property.c:? drivers/pci/of_property.c:493)
[ 1.445519][ T1] ? of_changeset_create_node (drivers/of/dynamic.c:917 include/linux/of.h:1614 drivers/of/dynamic.c:512)
[ 1.446365][ T1] ? of_pci_make_host_bridge_node (drivers/pci/of.c:786)
[ 1.447291][ T1] ? pci_register_host_bridge (drivers/pci/probe.c:1056)
[ 1.448154][ T1] ? __raw_spin_lock_init (include/linux/lockdep.h:135 include/linux/lockdep.h:142 kernel/locking/spinlock_debug.c:25)
[ 1.448953][ T1] ? __init_waitqueue_head (kernel/sched/wait.c:11)
[ 1.449806][ T1] ? pci_create_root_bus (drivers/pci/probe.c:3093)
[ 1.450039][ T1] ? acpi_pci_root_create (drivers/acpi/pci_root.c:1025)
[ 1.450869][ T1] ? pci_acpi_scan_root (arch/x86/pci/acpi.c:574)
[ 1.451682][ T1] ? acpi_pci_root_add (drivers/acpi/pci_root.c:728)
[ 1.452495][ T1] ? acpi_bus_get_status (drivers/acpi/bus.c:82)
[ 1.453269][ T1] ? acpi_bus_attach (drivers/acpi/scan.c:2261 drivers/acpi/scan.c:2309)
[ 1.454031][ T1] ? klist_next (lib/klist.c:384)
[ 1.454743][ T1] ? acpi_dev_for_one_check (drivers/acpi/bus.c:1145)
[ 1.455569][ T1] ? acpi_dev_for_each_child (drivers/acpi/bus.c:1139)
[ 1.456416][ T1] ? device_for_each_child (drivers/base/core.c:3993)
[ 1.457227][ T1] ? acpi_dev_for_each_child (drivers/acpi/bus.c:1157)
[ 1.458056][ T1] ? acpi_scan_clear_dep_fn (drivers/acpi/scan.c:2274)
[ 1.459806][ T1] ? acpi_bus_attach (drivers/acpi/scan.c:2331)
[ 1.459806][ T1] ? klist_next (lib/klist.c:384)
[ 1.460029][ T1] ? acpi_dev_for_one_check (drivers/acpi/bus.c:1145)
[ 1.460869][ T1] ? acpi_dev_for_each_child (drivers/acpi/bus.c:1139)
[ 1.461729][ T1] ? device_for_each_child (drivers/base/core.c:3993)
[ 1.462550][ T1] ? acpi_dev_for_each_child (drivers/acpi/bus.c:1157)
[ 1.463350][ T1] ? acpi_scan_clear_dep_fn (drivers/acpi/scan.c:2274)
[ 1.464160][ T1] ? acpi_bus_attach (drivers/acpi/scan.c:2331)
[ 1.464912][ T1] ? acpi_bus_scan (drivers/acpi/scan.c:2541)
[ 1.465639][ T1] ? acpi_scan_init (drivers/acpi/scan.c:2747)
[ 1.466388][ T1] ? __pci_mmcfg_init (include/linux/list.h:373 arch/x86/pci/mmconfig-shared.c:691)
[ 1.467151][ T1] ? acpi_init (drivers/acpi/bus.c:1468)
[ 1.467841][ T1] ? __initstub__kmod_acpi__390_1478_acpi_init4 (drivers/acpi/bus.c:1478)
[ 1.469805][ T1] ? do_one_initcall (init/main.c:1266)
[ 1.469805][ T1] ? __lock_acquire (kernel/locking/lockdep.c:4670)
[ 1.470057][ T1] ? kvm_sched_clock_read (arch/x86/kernel/kvmclock.c:91)
[ 1.470854][ T1] ? sched_clock_noinstr (arch/x86/kernel/tsc.c:268)
[ 1.471606][ T1] ? local_clock_noinstr (kernel/sched/clock.c:269 kernel/sched/clock.c:306)
[ 1.472397][ T1] ? local_clock (arch/x86/include/asm/preempt.h:84 kernel/sched/clock.c:316)
[ 1.473076][ T1] ? kvm_sched_clock_read (arch/x86/kernel/kvmclock.c:91)
[ 1.473884][ T1] ? sched_clock_noinstr (arch/x86/kernel/tsc.c:268)
[ 1.474646][ T1] ? local_clock_noinstr (kernel/sched/clock.c:269 kernel/sched/clock.c:306)
[ 1.475469][ T1] ? local_clock (arch/x86/include/asm/preempt.h:84 kernel/sched/clock.c:316)
[ 1.476194][ T1] ? ktime_get (include/linux/seqlock.h:226)
[ 1.476927][ T1] ? ktime_get (kernel/time/timekeeping.c:226 kernel/time/timekeeping.c:335 kernel/time/timekeeping.c:812)
[ 1.477633][ T1] ? sched_balance_trigger (kernel/sched/fair.c:12886)
[ 1.479808][ T1] ? clockevents_program_event (kernel/time/clockevents.c:336)
[ 1.479808][ T1] ? profile_tick (include/linux/profile.h:50)
[ 1.480043][ T1] ? tick_periodic (kernel/time/tick-common.c:103)
[ 1.480781][ T1] ? __lock_acquire (kernel/locking/lockdep.c:4670)
[ 1.481550][ T1] ? sched_clock_noinstr (arch/x86/kernel/tsc.c:268)
[ 1.482352][ T1] ? tick_handle_periodic (kernel/time/tick-common.c:132)
[ 1.483169][ T1] ? trace_hardirqs_on (kernel/trace/trace_preemptirq.c:49)
[ 1.483939][ T1] ? irqentry_exit (kernel/entry/common.c:?)
[ 1.484656][ T1] ? sysvec_call_function_single (arch/x86/kernel/apic/apic.c:1049)
[ 1.485505][ T1] ? sysvec_apic_timer_interrupt (arch/x86/kernel/apic/apic.c:1049)
[ 1.486375][ T1] ? handle_exception (arch/x86/entry/entry_32.S:1048)
[ 1.487143][ T1] ? xa_extract (lib/xarray.c:2157)
[ 1.487863][ T1] ? next_arg (lib/cmdline.c:273)
[ 1.489879][ T1] ? parse_args (kernel/params.c:153)
[ 1.489879][ T1] ? acpi_sleep_init (drivers/acpi/bus.c:1478)
[ 1.489879][ T1] ? do_initcall_level (init/main.c:1327)
[ 1.490609][ T1] ? rest_init (init/main.c:1458)
[ 1.491323][ T1] ? do_initcalls (init/main.c:1341)
[ 1.492037][ T1] ? rest_init (init/main.c:1458)
[ 1.492745][ T1] ? do_basic_setup (init/main.c:1364)
[ 1.493510][ T1] ? kernel_init_freeable (init/main.c:1579)
[ 1.494299][ T1] ? kernel_init (init/main.c:1468)
[ 1.494991][ T1] ? ret_from_fork (arch/x86/kernel/process.c:153)
[ 1.495713][ T1] ? ret_from_fork_asm (arch/x86/entry/entry_32.S:737)
[ 1.496483][ T1] ? entry_INT80_32 (arch/x86/entry/entry_32.S:945)
[    1.497262][    T1] irq event stamp: 46823
[ 1.499897][ T1] hardirqs last enabled at (46835): __console_unlock (arch/x86/include/asm/irqflags.h:26 arch/x86/include/asm/irqflags.h:87 arch/x86/include/asm/irqflags.h:147 kernel/printk/printk.c:344 kernel/printk/printk.c:2869)
[ 1.499897][ T1] hardirqs last disabled at (46844): __console_unlock (kernel/printk/printk.c:342)
[ 1.500213][ T1] softirqs last enabled at (46858): __do_softirq (kernel/softirq.c:589)
[ 1.501440][ T1] softirqs last disabled at (46853): __do_softirq (kernel/softirq.c:589)
[    1.502694][    T1] ---[ end trace 0000000000000000 ]---
[    1.503814][    T1] pci 0000:00:00.0: [8086:1237] type 00 class 0x060000 conventional PCI endpoint
[    1.507268][    T1] pci 0000:00:01.0: [8086:7000] type 00 class 0x060100 conventional PCI endpoint
[    1.510876][    T1] pci 0000:00:01.1: [8086:7010] type 00 class 0x010180 conventional PCI endpoint
[    1.514831][    T1] pci 0000:00:01.1: BAR 4 [io  0xc140-0xc14f]
[    1.516807][    T1] pci 0000:00:01.1: BAR 0 [io  0x01f0-0x01f7]: legacy IDE quirk
[    1.519807][    T1] pci 0000:00:01.1: BAR 1 [io  0x03f6]: legacy IDE quirk


The kernel config and materials to reproduce are available at:
https://download.01.org/0day-ci/archive/20241224/202412241455.22d84a7f-lkp@intel.com



-- 
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki


      reply	other threads:[~2024-12-24  6:21 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-12-09 13:03 [PATCH v5 0/5] Add support for the PCI host bridge device-tree node creation Herve Codina
2024-12-09 13:03 ` [PATCH v5 1/5] driver core: Introduce device_{add,remove}_of_node() Herve Codina
2024-12-24  8:29   ` Greg Kroah-Hartman
2024-12-09 13:03 ` [PATCH v5 2/5] PCI: of: Use device_{add,remove}_of_node() to attach of_node to existing device Herve Codina
2024-12-09 13:03 ` [PATCH v5 3/5] PCI: of_property: Add support for NULL pdev in of_pci_set_address() Herve Codina
2024-12-09 13:03 ` [PATCH v5 4/5] PCI: of_property: Constify parameter in of_pci_get_addr_flags() Herve Codina
2024-12-09 13:03 ` [PATCH v5 5/5] PCI: of: Create device-tree PCI host bridge node Herve Codina
2024-12-24  6:20   ` kernel test robot [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=202412241455.22d84a7f-lkp@intel.com \
    --to=oliver.sang@intel.com \
    --cc=allan.nielsen@microchip.com \
    --cc=bhelgaas@google.com \
    --cc=devicetree@vger.kernel.org \
    --cc=gregkh@linuxfoundation.org \
    --cc=herve.codina@bootlin.com \
    --cc=horatiu.vultur@microchip.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-pci@vger.kernel.org \
    --cc=lizhi.hou@amd.com \
    --cc=lkp@intel.com \
    --cc=oe-lkp@lists.linux.dev \
    --cc=rafael@kernel.org \
    --cc=robh@kernel.org \
    --cc=saravanak@google.com \
    --cc=steen.hegelund@microchip.com \
    --cc=thomas.petazzoni@bootlin.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.