All of lore.kernel.org
 help / color / mirror / Atom feed
From: kernel test robot <lkp@intel.com>
To: Madhavan Srinivasan <maddy@linux.ibm.com>,
	mpe@ellerman.id.au, npiggin@gmail.com,
	christophe.leroy@csgroup.eu
Cc: oe-kbuild-all@lists.linux.dev, linuxppc-dev@lists.ozlabs.org,
	Madhavan Srinivasan <maddy@linux.ibm.com>
Subject: Re: [PATCH v2 2/4] powerpc/perf/hv-24x7: Avoid loading hv-24x7 during dump kernel
Date: Sun, 2 Mar 2025 20:20:39 +0800	[thread overview]
Message-ID: <202503021908.Ed1gO0Gi-lkp@intel.com> (raw)
In-Reply-To: <20250301182310.6832-2-maddy@linux.ibm.com>

Hi Madhavan,

kernel test robot noticed the following build errors:

[auto build test ERROR on powerpc/next]
[also build test ERROR on powerpc/fixes linus/master v6.14-rc4 next-20250228]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch#_base_tree_information]

url:    https://github.com/intel-lab-lkp/linux/commits/Madhavan-Srinivasan/powerpc-perf-hv-24x7-Avoid-loading-hv-24x7-during-dump-kernel/20250302-022531
base:   https://git.kernel.org/pub/scm/linux/kernel/git/powerpc/linux.git next
patch link:    https://lore.kernel.org/r/20250301182310.6832-2-maddy%40linux.ibm.com
patch subject: [PATCH v2 2/4] powerpc/perf/hv-24x7: Avoid loading hv-24x7 during dump kernel
config: powerpc64-randconfig-001-20250302 (https://download.01.org/0day-ci/archive/20250302/202503021908.Ed1gO0Gi-lkp@intel.com/config)
compiler: powerpc64-linux-gcc (GCC) 14.2.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20250302/202503021908.Ed1gO0Gi-lkp@intel.com/reproduce)

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 <lkp@intel.com>
| Closes: https://lore.kernel.org/oe-kbuild-all/202503021908.Ed1gO0Gi-lkp@intel.com/

All errors (new ones prefixed by >>):

   arch/powerpc/perf/hv-24x7.c: In function 'hv_24x7_init':
>> arch/powerpc/perf/hv-24x7.c:1701:13: error: implicit declaration of function 'is_kdump_kernel' [-Wimplicit-function-declaration]
    1701 |         if (is_kdump_kernel() || is_fadump_active())
         |             ^~~~~~~~~~~~~~~


vim +/is_kdump_kernel +1701 arch/powerpc/perf/hv-24x7.c

  1693	
  1694	static int hv_24x7_init(void)
  1695	{
  1696		int r;
  1697		unsigned long hret;
  1698		unsigned int pvr = mfspr(SPRN_PVR);
  1699		struct hv_perf_caps caps;
  1700	
> 1701		if (is_kdump_kernel() || is_fadump_active())
  1702			return 0;
  1703	
  1704		if (!firmware_has_feature(FW_FEATURE_LPAR)) {
  1705			pr_debug("not a virtualized system, not enabling\n");
  1706			return -ENODEV;
  1707		}
  1708	
  1709		/* POWER8 only supports v1, while POWER9 only supports v2. */
  1710		if (PVR_VER(pvr) == PVR_POWER8 || PVR_VER(pvr) == PVR_POWER8E ||
  1711		    PVR_VER(pvr) == PVR_POWER8NVL)
  1712			interface_version = 1;
  1713		else {
  1714			interface_version = 2;
  1715	
  1716			/* SMT8 in POWER9 needs to aggregate result elements. */
  1717			if (threads_per_core == 8)
  1718				aggregate_result_elements = true;
  1719		}
  1720	
  1721		hret = hv_perf_caps_get(&caps);
  1722		if (hret) {
  1723			pr_debug("could not obtain capabilities, not enabling, rc=%ld\n",
  1724					hret);
  1725			return -ENODEV;
  1726		}
  1727	
  1728		hv_page_cache = kmem_cache_create("hv-page-4096", 4096, 4096, 0, NULL);
  1729		if (!hv_page_cache)
  1730			return -ENOMEM;
  1731	
  1732		/* sampling not supported */
  1733		h_24x7_pmu.capabilities |= PERF_PMU_CAP_NO_INTERRUPT;
  1734	
  1735		r = create_events_from_catalog(&event_group.attrs,
  1736					   &event_desc_group.attrs,
  1737					   &event_long_desc_group.attrs);
  1738	
  1739		if (r)
  1740			return r;
  1741	
  1742		/* init cpuhotplug */
  1743		r = hv_24x7_cpu_hotplug_init();
  1744		if (r)
  1745			return r;
  1746	
  1747		r = perf_pmu_register(&h_24x7_pmu, h_24x7_pmu.name, -1);
  1748		if (r)
  1749			return r;
  1750	
  1751		read_24x7_sys_info();
  1752	
  1753		return 0;
  1754	}
  1755	

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

  reply	other threads:[~2025-03-02 12:21 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-03-01 18:23 [PATCH v2 1/4] powerpc/perf/core-book3s: Avoid loading platform pmu driver during dump kernel Madhavan Srinivasan
2025-03-01 18:23 ` [PATCH v2 2/4] powerpc/perf/hv-24x7: Avoid loading hv-24x7 " Madhavan Srinivasan
2025-03-02 12:20   ` kernel test robot [this message]
2025-03-01 18:23 ` [PATCH v2 3/4] powerpc/perf/hv-gpci: Avoid loading hv-gpci pmu " Madhavan Srinivasan
2025-03-01 18:23 ` [PATCH v2 4/4] powerpc/perf/vpa-pmu: Avoid loading vpa-pmu driver " Madhavan Srinivasan
2025-03-02 10:13 ` [PATCH v2 1/4] powerpc/perf/core-book3s: Avoid loading platform pmu " kernel test robot
2025-03-05 15:29 ` kernel test robot

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=202503021908.Ed1gO0Gi-lkp@intel.com \
    --to=lkp@intel.com \
    --cc=christophe.leroy@csgroup.eu \
    --cc=linuxppc-dev@lists.ozlabs.org \
    --cc=maddy@linux.ibm.com \
    --cc=mpe@ellerman.id.au \
    --cc=npiggin@gmail.com \
    --cc=oe-kbuild-all@lists.linux.dev \
    /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.