From: kernel test robot <lkp@intel.com>
To: oe-kbuild@lists.linux.dev
Cc: lkp@intel.com
Subject: drivers/char/hpet.c:483:45: warning: '%d' directive writing between 1 and 9 bytes into a region of size 3
Date: Mon, 18 May 2026 15:12:58 +0800 [thread overview]
Message-ID: <202605181530.3bRAIVeO-lkp@intel.com> (raw)
::::::
:::::: Manual check reason: "bisect to a FBC not belonging to original linux-review patches: branch: linux-review/Nathan-Chancellor/kbuild-Bump-minimum-version-of-LLVM-for-building-the-kernel-to-17-0-1/20260518-071448, commit: efe6a2a0073c5db6aaa2d79dd68bde98dc335112"
:::::: Manual check reason: "only suspicious fbc files changed"
::::::
BCC: lkp@intel.com
CC: oe-kbuild-all@lists.linux.dev
TO: Nathan Chancellor <nathan@kernel.org>
CC: 0day robot <lkp@intel.com>
tree: https://github.com/intel-lab-lkp/linux/commits/Nathan-Chancellor/kbuild-Bump-minimum-version-of-LLVM-for-building-the-kernel-to-17-0-1/20260518-071448
head: 5b11b4afdc532a7c1d022b5f572c17428b613f0f
commit: efe6a2a0073c5db6aaa2d79dd68bde98dc335112 x86/build: Drop unnecessary '-ffreestanding' addition to KBUILD_CFLAGS
date: 8 hours ago
:::::: branch date: 8 hours ago
:::::: commit date: 8 hours ago
config: i386-buildonly-randconfig-003-20260518 (https://download.01.org/0day-ci/archive/20260518/202605181530.3bRAIVeO-lkp@intel.com/config)
compiler: gcc-14 (Debian 14.2.0-19) 14.2.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20260518/202605181530.3bRAIVeO-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/r/202605181530.3bRAIVeO-lkp@intel.com/
All warnings (new ones prefixed by >>):
drivers/char/hpet.c: In function 'hpet_ioctl_ieon':
>> drivers/char/hpet.c:483:45: warning: '%d' directive writing between 1 and 9 bytes into a region of size 3 [-Wformat-overflow=]
483 | sprintf(devp->hd_name, "hpet%d", (int)(devp - hpetp->hp_dev));
| ^~
drivers/char/hpet.c:483:40: note: directive argument in the range [-22369621, 22369621]
483 | sprintf(devp->hd_name, "hpet%d", (int)(devp - hpetp->hp_dev));
| ^~~~~~~~
drivers/char/hpet.c:483:17: note: 'sprintf' output between 6 and 14 bytes into a destination of size 7
483 | sprintf(devp->hd_name, "hpet%d", (int)(devp - hpetp->hp_dev));
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
vim +483 drivers/char/hpet.c
^1da177e4c3f4152 Linus Torvalds 2005-04-16 433
^1da177e4c3f4152 Linus Torvalds 2005-04-16 434 static int hpet_ioctl_ieon(struct hpet_dev *devp)
^1da177e4c3f4152 Linus Torvalds 2005-04-16 435 {
^1da177e4c3f4152 Linus Torvalds 2005-04-16 436 struct hpet_timer __iomem *timer;
^1da177e4c3f4152 Linus Torvalds 2005-04-16 437 struct hpet __iomem *hpet;
^1da177e4c3f4152 Linus Torvalds 2005-04-16 438 struct hpets *hpetp;
^1da177e4c3f4152 Linus Torvalds 2005-04-16 439 int irq;
^1da177e4c3f4152 Linus Torvalds 2005-04-16 440 unsigned long g, v, t, m;
^1da177e4c3f4152 Linus Torvalds 2005-04-16 441 unsigned long flags, isr;
^1da177e4c3f4152 Linus Torvalds 2005-04-16 442
^1da177e4c3f4152 Linus Torvalds 2005-04-16 443 timer = devp->hd_timer;
^1da177e4c3f4152 Linus Torvalds 2005-04-16 444 hpet = devp->hd_hpet;
^1da177e4c3f4152 Linus Torvalds 2005-04-16 445 hpetp = devp->hd_hpets;
^1da177e4c3f4152 Linus Torvalds 2005-04-16 446
9090e6db87e3bdb2 Clemens Ladisch 2005-10-30 447 if (!devp->hd_ireqfreq)
9090e6db87e3bdb2 Clemens Ladisch 2005-10-30 448 return -EIO;
9090e6db87e3bdb2 Clemens Ladisch 2005-10-30 449
^1da177e4c3f4152 Linus Torvalds 2005-04-16 450 spin_lock_irq(&hpet_lock);
^1da177e4c3f4152 Linus Torvalds 2005-04-16 451
^1da177e4c3f4152 Linus Torvalds 2005-04-16 452 if (devp->hd_flags & HPET_IE) {
^1da177e4c3f4152 Linus Torvalds 2005-04-16 453 spin_unlock_irq(&hpet_lock);
^1da177e4c3f4152 Linus Torvalds 2005-04-16 454 return -EBUSY;
^1da177e4c3f4152 Linus Torvalds 2005-04-16 455 }
^1da177e4c3f4152 Linus Torvalds 2005-04-16 456
^1da177e4c3f4152 Linus Torvalds 2005-04-16 457 devp->hd_flags |= HPET_IE;
0d29086177aaa1e7 Clemens Ladisch 2005-10-30 458
0d29086177aaa1e7 Clemens Ladisch 2005-10-30 459 if (readl(&timer->hpet_config) & Tn_INT_TYPE_CNF_MASK)
0d29086177aaa1e7 Clemens Ladisch 2005-10-30 460 devp->hd_flags |= HPET_SHARED_IRQ;
^1da177e4c3f4152 Linus Torvalds 2005-04-16 461 spin_unlock_irq(&hpet_lock);
^1da177e4c3f4152 Linus Torvalds 2005-04-16 462
^1da177e4c3f4152 Linus Torvalds 2005-04-16 463 irq = devp->hd_hdwirq;
^1da177e4c3f4152 Linus Torvalds 2005-04-16 464
^1da177e4c3f4152 Linus Torvalds 2005-04-16 465 if (irq) {
0d29086177aaa1e7 Clemens Ladisch 2005-10-30 466 unsigned long irq_flags;
^1da177e4c3f4152 Linus Torvalds 2005-04-16 467
96e9694df446d115 Clemens Ladisch 2010-10-26 468 if (devp->hd_flags & HPET_SHARED_IRQ) {
96e9694df446d115 Clemens Ladisch 2010-10-26 469 /*
96e9694df446d115 Clemens Ladisch 2010-10-26 470 * To prevent the interrupt handler from seeing an
96e9694df446d115 Clemens Ladisch 2010-10-26 471 * unwanted interrupt status bit, program the timer
96e9694df446d115 Clemens Ladisch 2010-10-26 472 * so that it will not fire in the near future ...
96e9694df446d115 Clemens Ladisch 2010-10-26 473 */
96e9694df446d115 Clemens Ladisch 2010-10-26 474 writel(readl(&timer->hpet_config) & ~Tn_TYPE_CNF_MASK,
96e9694df446d115 Clemens Ladisch 2010-10-26 475 &timer->hpet_config);
96e9694df446d115 Clemens Ladisch 2010-10-26 476 write_counter(read_counter(&hpet->hpet_mc),
96e9694df446d115 Clemens Ladisch 2010-10-26 477 &timer->hpet_compare);
96e9694df446d115 Clemens Ladisch 2010-10-26 478 /* ... and clear any left-over status. */
96e9694df446d115 Clemens Ladisch 2010-10-26 479 isr = 1 << (devp - devp->hd_hpets->hp_dev);
96e9694df446d115 Clemens Ladisch 2010-10-26 480 writel(isr, &hpet->hpet_isr);
96e9694df446d115 Clemens Ladisch 2010-10-26 481 }
96e9694df446d115 Clemens Ladisch 2010-10-26 482
0d29086177aaa1e7 Clemens Ladisch 2005-10-30 @483 sprintf(devp->hd_name, "hpet%d", (int)(devp - hpetp->hp_dev));
158f0bb005fc7fdb Michael Opdenacker 2013-10-13 484 irq_flags = devp->hd_flags & HPET_SHARED_IRQ ? IRQF_SHARED : 0;
0d29086177aaa1e7 Clemens Ladisch 2005-10-30 485 if (request_irq(irq, hpet_interrupt, irq_flags,
0d29086177aaa1e7 Clemens Ladisch 2005-10-30 486 devp->hd_name, (void *)devp)) {
^1da177e4c3f4152 Linus Torvalds 2005-04-16 487 printk(KERN_ERR "hpet: IRQ %d is not free\n", irq);
^1da177e4c3f4152 Linus Torvalds 2005-04-16 488 irq = 0;
^1da177e4c3f4152 Linus Torvalds 2005-04-16 489 }
^1da177e4c3f4152 Linus Torvalds 2005-04-16 490 }
^1da177e4c3f4152 Linus Torvalds 2005-04-16 491
^1da177e4c3f4152 Linus Torvalds 2005-04-16 492 if (irq == 0) {
^1da177e4c3f4152 Linus Torvalds 2005-04-16 493 spin_lock_irq(&hpet_lock);
^1da177e4c3f4152 Linus Torvalds 2005-04-16 494 devp->hd_flags ^= HPET_IE;
^1da177e4c3f4152 Linus Torvalds 2005-04-16 495 spin_unlock_irq(&hpet_lock);
^1da177e4c3f4152 Linus Torvalds 2005-04-16 496 return -EIO;
^1da177e4c3f4152 Linus Torvalds 2005-04-16 497 }
^1da177e4c3f4152 Linus Torvalds 2005-04-16 498
^1da177e4c3f4152 Linus Torvalds 2005-04-16 499 devp->hd_irq = irq;
^1da177e4c3f4152 Linus Torvalds 2005-04-16 500 t = devp->hd_ireqfreq;
^1da177e4c3f4152 Linus Torvalds 2005-04-16 501 v = readq(&timer->hpet_config);
64a76f667d987a55 David Brownell 2008-07-29 502
64a76f667d987a55 David Brownell 2008-07-29 503 /* 64-bit comparators are not yet supported through the ioctls,
64a76f667d987a55 David Brownell 2008-07-29 504 * so force this into 32-bit mode if it supports both modes
64a76f667d987a55 David Brownell 2008-07-29 505 */
64a76f667d987a55 David Brownell 2008-07-29 506 g = v | Tn_32MODE_CNF_MASK | Tn_INT_ENB_CNF_MASK;
^1da177e4c3f4152 Linus Torvalds 2005-04-16 507
^1da177e4c3f4152 Linus Torvalds 2005-04-16 508 if (devp->hd_flags & HPET_PERIODIC) {
^1da177e4c3f4152 Linus Torvalds 2005-04-16 509 g |= Tn_TYPE_CNF_MASK;
ae21cf9248584d9b Nils Carlson 2009-09-23 510 v |= Tn_TYPE_CNF_MASK | Tn_VAL_SET_CNF_MASK;
^1da177e4c3f4152 Linus Torvalds 2005-04-16 511 writeq(v, &timer->hpet_config);
^1da177e4c3f4152 Linus Torvalds 2005-04-16 512 local_irq_save(flags);
64a76f667d987a55 David Brownell 2008-07-29 513
ae21cf9248584d9b Nils Carlson 2009-09-23 514 /*
ae21cf9248584d9b Nils Carlson 2009-09-23 515 * NOTE: First we modify the hidden accumulator
64a76f667d987a55 David Brownell 2008-07-29 516 * register supported by periodic-capable comparators.
64a76f667d987a55 David Brownell 2008-07-29 517 * We never want to modify the (single) counter; that
ae21cf9248584d9b Nils Carlson 2009-09-23 518 * would affect all the comparators. The value written
ae21cf9248584d9b Nils Carlson 2009-09-23 519 * is the counter value when the first interrupt is due.
64a76f667d987a55 David Brownell 2008-07-29 520 */
^1da177e4c3f4152 Linus Torvalds 2005-04-16 521 m = read_counter(&hpet->hpet_mc);
^1da177e4c3f4152 Linus Torvalds 2005-04-16 522 write_counter(t + m + hpetp->hp_delta, &timer->hpet_compare);
ae21cf9248584d9b Nils Carlson 2009-09-23 523 /*
ae21cf9248584d9b Nils Carlson 2009-09-23 524 * Then we modify the comparator, indicating the period
ae21cf9248584d9b Nils Carlson 2009-09-23 525 * for subsequent interrupt.
ae21cf9248584d9b Nils Carlson 2009-09-23 526 */
ae21cf9248584d9b Nils Carlson 2009-09-23 527 write_counter(t, &timer->hpet_compare);
^1da177e4c3f4152 Linus Torvalds 2005-04-16 528 } else {
^1da177e4c3f4152 Linus Torvalds 2005-04-16 529 local_irq_save(flags);
^1da177e4c3f4152 Linus Torvalds 2005-04-16 530 m = read_counter(&hpet->hpet_mc);
^1da177e4c3f4152 Linus Torvalds 2005-04-16 531 write_counter(t + m + hpetp->hp_delta, &timer->hpet_compare);
^1da177e4c3f4152 Linus Torvalds 2005-04-16 532 }
^1da177e4c3f4152 Linus Torvalds 2005-04-16 533
0d29086177aaa1e7 Clemens Ladisch 2005-10-30 534 if (devp->hd_flags & HPET_SHARED_IRQ) {
3d5640d1c7584689 Clemens Ladisch 2005-10-30 535 isr = 1 << (devp - devp->hd_hpets->hp_dev);
0d29086177aaa1e7 Clemens Ladisch 2005-10-30 536 writel(isr, &hpet->hpet_isr);
0d29086177aaa1e7 Clemens Ladisch 2005-10-30 537 }
^1da177e4c3f4152 Linus Torvalds 2005-04-16 538 writeq(g, &timer->hpet_config);
^1da177e4c3f4152 Linus Torvalds 2005-04-16 539 local_irq_restore(flags);
^1da177e4c3f4152 Linus Torvalds 2005-04-16 540
^1da177e4c3f4152 Linus Torvalds 2005-04-16 541 return 0;
^1da177e4c3f4152 Linus Torvalds 2005-04-16 542 }
^1da177e4c3f4152 Linus Torvalds 2005-04-16 543
--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki
reply other threads:[~2026-05-18 7:13 UTC|newest]
Thread overview: [no followups] expand[flat|nested] mbox.gz Atom feed
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=202605181530.3bRAIVeO-lkp@intel.com \
--to=lkp@intel.com \
--cc=oe-kbuild@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.