Re: [PATCH v1 2/3] scsi: ufs: Optimize host lock on transfer requests send/compl paths
by kernel test robot
Hi Can,
Thank you for the patch! Perhaps something to improve:
[auto build test WARNING on mkp-scsi/for-next]
[also build test WARNING on next-20210524]
[cannot apply to scsi/for-next v5.13-rc3]
[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]
url: https://github.com/0day-ci/linux/commits/Can-Guo/Optimize-host-lock-on-TR...
base: https://git.kernel.org/pub/scm/linux/kernel/git/mkp/scsi.git for-next
config: arm64-randconfig-r011-20210524 (attached as .config)
compiler: clang version 13.0.0 (https://github.com/llvm/llvm-project 93d1e5822ed64abd777eb94ea9899e96c4c39fbe)
reproduce (this is a W=1 build):
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# install arm64 cross compiling tool for clang build
# apt-get install binutils-aarch64-linux-gnu
# https://github.com/0day-ci/linux/commit/efe94162bf7973be4ed6496871b9bc9ea...
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review Can-Guo/Optimize-host-lock-on-TR-send-compl-paths-and-utilize-UTRLCNR/20210524-163847
git checkout efe94162bf7973be4ed6496871b9bc9ea54e2819
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=arm64
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp(a)intel.com>
All warnings (new ones prefixed by >>):
>> drivers/scsi/ufs/ufshcd.c:2959:6: warning: variable 'lrbp' is used uninitialized whenever 'if' condition is true [-Wsometimes-uninitialized]
if (unlikely(test_bit(tag, &hba->outstanding_reqs))) {
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/compiler.h:78:22: note: expanded from macro 'unlikely'
# define unlikely(x) __builtin_expect(!!(x), 0)
^~~~~~~~~~~~~~~~~~~~~~~~~~
drivers/scsi/ufs/ufshcd.c:2981:32: note: uninitialized use occurs here
(struct utp_upiu_req *)lrbp->ucd_rsp_ptr);
^~~~
drivers/scsi/ufs/ufshcd.c:2959:2: note: remove the 'if' if its condition is always false
if (unlikely(test_bit(tag, &hba->outstanding_reqs))) {
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
drivers/scsi/ufs/ufshcd.c:2939:25: note: initialize the variable 'lrbp' to silence this warning
struct ufshcd_lrb *lrbp;
^
= NULL
1 warning generated.
vim +2959 drivers/scsi/ufs/ufshcd.c
2924
2925 /**
2926 * ufshcd_exec_dev_cmd - API for sending device management requests
2927 * @hba: UFS hba
2928 * @cmd_type: specifies the type (NOP, Query...)
2929 * @timeout: time in seconds
2930 *
2931 * NOTE: Since there is only one available tag for device management commands,
2932 * it is expected you hold the hba->dev_cmd.lock mutex.
2933 */
2934 static int ufshcd_exec_dev_cmd(struct ufs_hba *hba,
2935 enum dev_cmd_type cmd_type, int timeout)
2936 {
2937 struct request_queue *q = hba->cmd_queue;
2938 struct request *req;
2939 struct ufshcd_lrb *lrbp;
2940 int err;
2941 int tag;
2942 struct completion wait;
2943
2944 down_read(&hba->clk_scaling_lock);
2945
2946 /*
2947 * Get free slot, sleep if slots are unavailable.
2948 * Even though we use wait_event() which sleeps indefinitely,
2949 * the maximum wait time is bounded by SCSI request timeout.
2950 */
2951 req = blk_get_request(q, REQ_OP_DRV_OUT, 0);
2952 if (IS_ERR(req)) {
2953 err = PTR_ERR(req);
2954 goto out_unlock;
2955 }
2956 tag = req->tag;
2957 WARN_ON_ONCE(!ufshcd_valid_tag(hba, tag));
2958
> 2959 if (unlikely(test_bit(tag, &hba->outstanding_reqs))) {
2960 err = -EBUSY;
2961 goto out;
2962 }
2963
2964 init_completion(&wait);
2965 lrbp = &hba->lrb[tag];
2966 WARN_ON(lrbp->cmd);
2967 err = ufshcd_compose_dev_cmd(hba, lrbp, cmd_type, tag);
2968 if (unlikely(err))
2969 goto out_put_tag;
2970
2971 hba->dev_cmd.complete = &wait;
2972
2973 ufshcd_add_query_upiu_trace(hba, UFS_QUERY_SEND, lrbp->ucd_req_ptr);
2974 /* Make sure descriptors are ready before ringing the doorbell */
2975 wmb();
2976
2977 ufshcd_send_command(hba, tag);
2978 err = ufshcd_wait_for_dev_cmd(hba, lrbp, timeout);
2979 out:
2980 ufshcd_add_query_upiu_trace(hba, err ? UFS_QUERY_ERR : UFS_QUERY_COMP,
2981 (struct utp_upiu_req *)lrbp->ucd_rsp_ptr);
2982
2983 out_put_tag:
2984 blk_put_request(req);
2985 out_unlock:
2986 up_read(&hba->clk_scaling_lock);
2987 return err;
2988 }
2989
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
6 days, 15 hours
[mtd:nand/next 6/7] drivers/mtd/nand/raw/qcom_nandc.c:765:7: error: 'nandc' undeclared
by kernel test robot
tree: https://git.kernel.org/pub/scm/linux/kernel/git/mtd/linux.git nand/next
head: 612a421079a7fa3042fd713eb62b455aac1a0e88
commit: 4ea7f88f173804f344988f580fb41ba96f7666da [6/7] mtd: rawnand: qcom: avoid writing to obsolete register
config: arc-randconfig-p001-20210510 (attached as .config)
compiler: arceb-elf-gcc (GCC) 9.3.0
reproduce (this is a W=1 build):
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# https://git.kernel.org/pub/scm/linux/kernel/git/mtd/linux.git/commit/?id=...
git remote add mtd https://git.kernel.org/pub/scm/linux/kernel/git/mtd/linux.git
git fetch --no-tags mtd nand/next
git checkout 4ea7f88f173804f344988f580fb41ba96f7666da
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross W=1 ARCH=arc
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp(a)intel.com>
All errors (new ones prefixed by >>):
drivers/mtd/nand/raw/qcom_nandc.c: In function 'update_rw_regs':
>> drivers/mtd/nand/raw/qcom_nandc.c:765:7: error: 'nandc' undeclared (first use in this function)
765 | if (!nandc->props->qpic_v2)
| ^~~~~
drivers/mtd/nand/raw/qcom_nandc.c:765:7: note: each undeclared identifier is reported only once for each function it appears in
vim +/nandc +765 drivers/mtd/nand/raw/qcom_nandc.c
724
725 /*
726 * update_rw_regs: set up read/write register values, these will be
727 * written to the NAND controller registers via DMA
728 *
729 * @num_cw: number of steps for the read/write operation
730 * @read: read or write operation
731 * @cw : which code word
732 */
733 static void update_rw_regs(struct qcom_nand_host *host, int num_cw, bool read, int cw)
734 {
735 struct nand_chip *chip = &host->chip;
736 u32 cmd, cfg0, cfg1, ecc_bch_cfg;
737
738 if (read) {
739 if (host->use_ecc)
740 cmd = OP_PAGE_READ_WITH_ECC | PAGE_ACC | LAST_PAGE;
741 else
742 cmd = OP_PAGE_READ | PAGE_ACC | LAST_PAGE;
743 } else {
744 cmd = OP_PROGRAM_PAGE | PAGE_ACC | LAST_PAGE;
745 }
746
747 if (host->use_ecc) {
748 cfg0 = (host->cfg0 & ~(7U << CW_PER_PAGE)) |
749 (num_cw - 1) << CW_PER_PAGE;
750
751 cfg1 = host->cfg1;
752 ecc_bch_cfg = host->ecc_bch_cfg;
753 } else {
754 cfg0 = (host->cfg0_raw & ~(7U << CW_PER_PAGE)) |
755 (num_cw - 1) << CW_PER_PAGE;
756
757 cfg1 = host->cfg1_raw;
758 ecc_bch_cfg = 1 << ECC_CFG_ECC_DISABLE;
759 }
760
761 nandc_set_reg(chip, NAND_FLASH_CMD, cmd);
762 nandc_set_reg(chip, NAND_DEV0_CFG0, cfg0);
763 nandc_set_reg(chip, NAND_DEV0_CFG1, cfg1);
764 nandc_set_reg(chip, NAND_DEV0_ECC_CFG, ecc_bch_cfg);
> 765 if (!nandc->props->qpic_v2)
766 nandc_set_reg(chip, NAND_EBI2_ECC_BUF_CFG, host->ecc_buf_cfg);
767 nandc_set_reg(chip, NAND_FLASH_STATUS, host->clrflashstatus);
768 nandc_set_reg(chip, NAND_READ_STATUS, host->clrreadstatus);
769 nandc_set_reg(chip, NAND_EXEC_CMD, 1);
770
771 if (read)
772 nandc_set_read_loc(chip, cw, 0, 0, host->use_ecc ?
773 host->cw_data : host->cw_size, 1);
774 }
775
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
1 week, 1 day
mm/migrate.c:1406:5: warning: stack frame size of 2096 bytes in function 'migrate_pages'
by kernel test robot
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head: b90e90f40b4ff23c753126008bf4713a42353af6
commit: dd4ae78a21fc05d91d841e499dddd057ad64a4df mm: migrate: simplify the logic for handling permanent failure
date: 6 months ago
config: powerpc-randconfig-r034-20210530 (attached as .config)
compiler: clang version 13.0.0 (https://github.com/llvm/llvm-project bc6799f2f79f0ae87e9f1ebf9d25ba799fbd25a9)
reproduce (this is a W=1 build):
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# install powerpc cross compiling tool for clang build
# apt-get install binutils-powerpc-linux-gnu
# https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit...
git remote add linus https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
git fetch --no-tags linus master
git checkout dd4ae78a21fc05d91d841e499dddd057ad64a4df
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=powerpc
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp(a)intel.com>
All warnings (new ones prefixed by >>):
In file included from mm/migrate.c:16:
In file included from include/linux/migrate.h:5:
In file included from include/linux/mm.h:9:
In file included from include/linux/mmdebug.h:5:
In file included from include/linux/bug.h:5:
In file included from arch/powerpc/include/asm/bug.h:109:
In file included from include/asm-generic/bug.h:20:
In file included from include/linux/kernel.h:12:
In file included from include/linux/bitops.h:29:
In file included from arch/powerpc/include/asm/bitops.h:62:
arch/powerpc/include/asm/barrier.h:49:9: warning: '__lwsync' macro redefined [-Wmacro-redefined]
#define __lwsync() __asm__ __volatile__ (stringify_in_c(LWSYNC) : : :"memory")
^
<built-in>:309:9: note: previous definition is here
#define __lwsync __builtin_ppc_lwsync
^
>> mm/migrate.c:1406:5: warning: stack frame size of 2096 bytes in function 'migrate_pages' [-Wframe-larger-than=]
int migrate_pages(struct list_head *from, new_page_t get_new_page,
^
2 warnings generated.
vim +/migrate_pages +1406 mm/migrate.c
290408d4a25002 Naoya Horiguchi 2010-09-08 1384
e24f0b8f76cc3d Christoph Lameter 2006-06-23 1385 /*
c73e5c9c59a0f7 Srivatsa S. Bhat 2013-04-29 1386 * migrate_pages - migrate the pages specified in a list, to the free pages
c73e5c9c59a0f7 Srivatsa S. Bhat 2013-04-29 1387 * supplied as the target for the page migration
e24f0b8f76cc3d Christoph Lameter 2006-06-23 1388 *
c73e5c9c59a0f7 Srivatsa S. Bhat 2013-04-29 1389 * @from: The list of pages to be migrated.
c73e5c9c59a0f7 Srivatsa S. Bhat 2013-04-29 1390 * @get_new_page: The function used to allocate free pages to be used
c73e5c9c59a0f7 Srivatsa S. Bhat 2013-04-29 1391 * as the target of the page migration.
68711a746345c4 David Rientjes 2014-06-04 1392 * @put_new_page: The function used to free target pages if migration
68711a746345c4 David Rientjes 2014-06-04 1393 * fails, or NULL if no special handling is necessary.
c73e5c9c59a0f7 Srivatsa S. Bhat 2013-04-29 1394 * @private: Private data to be passed on to get_new_page()
c73e5c9c59a0f7 Srivatsa S. Bhat 2013-04-29 1395 * @mode: The migration mode that specifies the constraints for
c73e5c9c59a0f7 Srivatsa S. Bhat 2013-04-29 1396 * page migration, if any.
c73e5c9c59a0f7 Srivatsa S. Bhat 2013-04-29 1397 * @reason: The reason for page migration.
e24f0b8f76cc3d Christoph Lameter 2006-06-23 1398 *
c73e5c9c59a0f7 Srivatsa S. Bhat 2013-04-29 1399 * The function returns after 10 attempts or if no pages are movable any more
c73e5c9c59a0f7 Srivatsa S. Bhat 2013-04-29 1400 * because the list has become empty or no retryable pages exist any more.
dd4ae78a21fc05 Yang Shi 2020-12-14 1401 * It is caller's responsibility to call putback_movable_pages() to return pages
dd4ae78a21fc05 Yang Shi 2020-12-14 1402 * to the LRU or free list only if ret != 0.
e24f0b8f76cc3d Christoph Lameter 2006-06-23 1403 *
c73e5c9c59a0f7 Srivatsa S. Bhat 2013-04-29 1404 * Returns the number of pages that were not migrated, or an error code.
e24f0b8f76cc3d Christoph Lameter 2006-06-23 1405 */
9c620e2bc5aa42 Hugh Dickins 2013-02-22 @1406 int migrate_pages(struct list_head *from, new_page_t get_new_page,
68711a746345c4 David Rientjes 2014-06-04 1407 free_page_t put_new_page, unsigned long private,
68711a746345c4 David Rientjes 2014-06-04 1408 enum migrate_mode mode, int reason)
e24f0b8f76cc3d Christoph Lameter 2006-06-23 1409 {
e24f0b8f76cc3d Christoph Lameter 2006-06-23 1410 int retry = 1;
1a5bae25e3cf95 Anshuman Khandual 2020-08-11 1411 int thp_retry = 1;
e24f0b8f76cc3d Christoph Lameter 2006-06-23 1412 int nr_failed = 0;
5647bc293ab15f Mel Gorman 2012-10-19 1413 int nr_succeeded = 0;
1a5bae25e3cf95 Anshuman Khandual 2020-08-11 1414 int nr_thp_succeeded = 0;
1a5bae25e3cf95 Anshuman Khandual 2020-08-11 1415 int nr_thp_failed = 0;
1a5bae25e3cf95 Anshuman Khandual 2020-08-11 1416 int nr_thp_split = 0;
e24f0b8f76cc3d Christoph Lameter 2006-06-23 1417 int pass = 0;
1a5bae25e3cf95 Anshuman Khandual 2020-08-11 1418 bool is_thp = false;
e24f0b8f76cc3d Christoph Lameter 2006-06-23 1419 struct page *page;
e24f0b8f76cc3d Christoph Lameter 2006-06-23 1420 struct page *page2;
e24f0b8f76cc3d Christoph Lameter 2006-06-23 1421 int swapwrite = current->flags & PF_SWAPWRITE;
1a5bae25e3cf95 Anshuman Khandual 2020-08-11 1422 int rc, nr_subpages;
dd4ae78a21fc05 Yang Shi 2020-12-14 1423 LIST_HEAD(ret_pages);
e24f0b8f76cc3d Christoph Lameter 2006-06-23 1424
e24f0b8f76cc3d Christoph Lameter 2006-06-23 1425 if (!swapwrite)
e24f0b8f76cc3d Christoph Lameter 2006-06-23 1426 current->flags |= PF_SWAPWRITE;
e24f0b8f76cc3d Christoph Lameter 2006-06-23 1427
1a5bae25e3cf95 Anshuman Khandual 2020-08-11 1428 for (pass = 0; pass < 10 && (retry || thp_retry); pass++) {
e24f0b8f76cc3d Christoph Lameter 2006-06-23 1429 retry = 0;
1a5bae25e3cf95 Anshuman Khandual 2020-08-11 1430 thp_retry = 0;
e24f0b8f76cc3d Christoph Lameter 2006-06-23 1431
e24f0b8f76cc3d Christoph Lameter 2006-06-23 1432 list_for_each_entry_safe(page, page2, from, lru) {
94723aafb9e764 Michal Hocko 2018-04-10 1433 retry:
1a5bae25e3cf95 Anshuman Khandual 2020-08-11 1434 /*
1a5bae25e3cf95 Anshuman Khandual 2020-08-11 1435 * THP statistics is based on the source huge page.
1a5bae25e3cf95 Anshuman Khandual 2020-08-11 1436 * Capture required information that might get lost
1a5bae25e3cf95 Anshuman Khandual 2020-08-11 1437 * during migration.
1a5bae25e3cf95 Anshuman Khandual 2020-08-11 1438 */
6c5c7b9f335272 Zi Yan 2020-09-25 1439 is_thp = PageTransHuge(page) && !PageHuge(page);
6c357848b44b40 Matthew Wilcox (Oracle 2020-08-14 1440) nr_subpages = thp_nr_pages(page);
e24f0b8f76cc3d Christoph Lameter 2006-06-23 1441 cond_resched();
e24f0b8f76cc3d Christoph Lameter 2006-06-23 1442
31caf665e666b5 Naoya Horiguchi 2013-09-11 1443 if (PageHuge(page))
31caf665e666b5 Naoya Horiguchi 2013-09-11 1444 rc = unmap_and_move_huge_page(get_new_page,
68711a746345c4 David Rientjes 2014-06-04 1445 put_new_page, private, page,
dd4ae78a21fc05 Yang Shi 2020-12-14 1446 pass > 2, mode, reason,
dd4ae78a21fc05 Yang Shi 2020-12-14 1447 &ret_pages);
31caf665e666b5 Naoya Horiguchi 2013-09-11 1448 else
68711a746345c4 David Rientjes 2014-06-04 1449 rc = unmap_and_move(get_new_page, put_new_page,
add05cecef803f Naoya Horiguchi 2015-06-24 1450 private, page, pass > 2, mode,
dd4ae78a21fc05 Yang Shi 2020-12-14 1451 reason, &ret_pages);
dd4ae78a21fc05 Yang Shi 2020-12-14 1452 /*
dd4ae78a21fc05 Yang Shi 2020-12-14 1453 * The rules are:
dd4ae78a21fc05 Yang Shi 2020-12-14 1454 * Success: non hugetlb page will be freed, hugetlb
dd4ae78a21fc05 Yang Shi 2020-12-14 1455 * page will be put back
dd4ae78a21fc05 Yang Shi 2020-12-14 1456 * -EAGAIN: stay on the from list
dd4ae78a21fc05 Yang Shi 2020-12-14 1457 * -ENOMEM: stay on the from list
dd4ae78a21fc05 Yang Shi 2020-12-14 1458 * Other errno: put on ret_pages list then splice to
dd4ae78a21fc05 Yang Shi 2020-12-14 1459 * from list
dd4ae78a21fc05 Yang Shi 2020-12-14 1460 */
e24f0b8f76cc3d Christoph Lameter 2006-06-23 1461 switch(rc) {
95a402c3847cc1 Christoph Lameter 2006-06-23 1462 case -ENOMEM:
94723aafb9e764 Michal Hocko 2018-04-10 1463 /*
94723aafb9e764 Michal Hocko 2018-04-10 1464 * THP migration might be unsupported or the
94723aafb9e764 Michal Hocko 2018-04-10 1465 * allocation could've failed so we should
94723aafb9e764 Michal Hocko 2018-04-10 1466 * retry on the same page with the THP split
94723aafb9e764 Michal Hocko 2018-04-10 1467 * to base pages.
94723aafb9e764 Michal Hocko 2018-04-10 1468 *
94723aafb9e764 Michal Hocko 2018-04-10 1469 * Head page is retried immediately and tail
94723aafb9e764 Michal Hocko 2018-04-10 1470 * pages are added to the tail of the list so
94723aafb9e764 Michal Hocko 2018-04-10 1471 * we encounter them after the rest of the list
94723aafb9e764 Michal Hocko 2018-04-10 1472 * is processed.
94723aafb9e764 Michal Hocko 2018-04-10 1473 */
6c5c7b9f335272 Zi Yan 2020-09-25 1474 if (is_thp) {
94723aafb9e764 Michal Hocko 2018-04-10 1475 lock_page(page);
94723aafb9e764 Michal Hocko 2018-04-10 1476 rc = split_huge_page_to_list(page, from);
94723aafb9e764 Michal Hocko 2018-04-10 1477 unlock_page(page);
94723aafb9e764 Michal Hocko 2018-04-10 1478 if (!rc) {
94723aafb9e764 Michal Hocko 2018-04-10 1479 list_safe_reset_next(page, page2, lru);
1a5bae25e3cf95 Anshuman Khandual 2020-08-11 1480 nr_thp_split++;
94723aafb9e764 Michal Hocko 2018-04-10 1481 goto retry;
94723aafb9e764 Michal Hocko 2018-04-10 1482 }
6c5c7b9f335272 Zi Yan 2020-09-25 1483
1a5bae25e3cf95 Anshuman Khandual 2020-08-11 1484 nr_thp_failed++;
1a5bae25e3cf95 Anshuman Khandual 2020-08-11 1485 nr_failed += nr_subpages;
1a5bae25e3cf95 Anshuman Khandual 2020-08-11 1486 goto out;
1a5bae25e3cf95 Anshuman Khandual 2020-08-11 1487 }
dfef2ef4027b13 David Rientjes 2016-05-20 1488 nr_failed++;
95a402c3847cc1 Christoph Lameter 2006-06-23 1489 goto out;
e24f0b8f76cc3d Christoph Lameter 2006-06-23 1490 case -EAGAIN:
1a5bae25e3cf95 Anshuman Khandual 2020-08-11 1491 if (is_thp) {
1a5bae25e3cf95 Anshuman Khandual 2020-08-11 1492 thp_retry++;
1a5bae25e3cf95 Anshuman Khandual 2020-08-11 1493 break;
1a5bae25e3cf95 Anshuman Khandual 2020-08-11 1494 }
b20a35035f983f Christoph Lameter 2006-03-22 1495 retry++;
e24f0b8f76cc3d Christoph Lameter 2006-06-23 1496 break;
78bd52097d0420 Rafael Aquini 2012-12-11 1497 case MIGRATEPAGE_SUCCESS:
1a5bae25e3cf95 Anshuman Khandual 2020-08-11 1498 if (is_thp) {
1a5bae25e3cf95 Anshuman Khandual 2020-08-11 1499 nr_thp_succeeded++;
1a5bae25e3cf95 Anshuman Khandual 2020-08-11 1500 nr_succeeded += nr_subpages;
1a5bae25e3cf95 Anshuman Khandual 2020-08-11 1501 break;
1a5bae25e3cf95 Anshuman Khandual 2020-08-11 1502 }
5647bc293ab15f Mel Gorman 2012-10-19 1503 nr_succeeded++;
e24f0b8f76cc3d Christoph Lameter 2006-06-23 1504 break;
e24f0b8f76cc3d Christoph Lameter 2006-06-23 1505 default:
354a3363363724 Naoya Horiguchi 2014-01-21 1506 /*
354a3363363724 Naoya Horiguchi 2014-01-21 1507 * Permanent failure (-EBUSY, -ENOSYS, etc.):
354a3363363724 Naoya Horiguchi 2014-01-21 1508 * unlike -EAGAIN case, the failed page is
354a3363363724 Naoya Horiguchi 2014-01-21 1509 * removed from migration page list and not
354a3363363724 Naoya Horiguchi 2014-01-21 1510 * retried in the next outer loop.
354a3363363724 Naoya Horiguchi 2014-01-21 1511 */
1a5bae25e3cf95 Anshuman Khandual 2020-08-11 1512 if (is_thp) {
1a5bae25e3cf95 Anshuman Khandual 2020-08-11 1513 nr_thp_failed++;
1a5bae25e3cf95 Anshuman Khandual 2020-08-11 1514 nr_failed += nr_subpages;
1a5bae25e3cf95 Anshuman Khandual 2020-08-11 1515 break;
1a5bae25e3cf95 Anshuman Khandual 2020-08-11 1516 }
b20a35035f983f Christoph Lameter 2006-03-22 1517 nr_failed++;
e24f0b8f76cc3d Christoph Lameter 2006-06-23 1518 break;
2d1db3b1170db4 Christoph Lameter 2006-06-23 1519 }
b20a35035f983f Christoph Lameter 2006-03-22 1520 }
b20a35035f983f Christoph Lameter 2006-03-22 1521 }
1a5bae25e3cf95 Anshuman Khandual 2020-08-11 1522 nr_failed += retry + thp_retry;
1a5bae25e3cf95 Anshuman Khandual 2020-08-11 1523 nr_thp_failed += thp_retry;
f2f81fb2b72b83 Vlastimil Babka 2015-11-05 1524 rc = nr_failed;
95a402c3847cc1 Christoph Lameter 2006-06-23 1525 out:
dd4ae78a21fc05 Yang Shi 2020-12-14 1526 /*
dd4ae78a21fc05 Yang Shi 2020-12-14 1527 * Put the permanent failure page back to migration list, they
dd4ae78a21fc05 Yang Shi 2020-12-14 1528 * will be put back to the right list by the caller.
dd4ae78a21fc05 Yang Shi 2020-12-14 1529 */
dd4ae78a21fc05 Yang Shi 2020-12-14 1530 list_splice(&ret_pages, from);
dd4ae78a21fc05 Yang Shi 2020-12-14 1531
5647bc293ab15f Mel Gorman 2012-10-19 1532 count_vm_events(PGMIGRATE_SUCCESS, nr_succeeded);
5647bc293ab15f Mel Gorman 2012-10-19 1533 count_vm_events(PGMIGRATE_FAIL, nr_failed);
1a5bae25e3cf95 Anshuman Khandual 2020-08-11 1534 count_vm_events(THP_MIGRATION_SUCCESS, nr_thp_succeeded);
1a5bae25e3cf95 Anshuman Khandual 2020-08-11 1535 count_vm_events(THP_MIGRATION_FAIL, nr_thp_failed);
1a5bae25e3cf95 Anshuman Khandual 2020-08-11 1536 count_vm_events(THP_MIGRATION_SPLIT, nr_thp_split);
1a5bae25e3cf95 Anshuman Khandual 2020-08-11 1537 trace_mm_migrate_pages(nr_succeeded, nr_failed, nr_thp_succeeded,
1a5bae25e3cf95 Anshuman Khandual 2020-08-11 1538 nr_thp_failed, nr_thp_split, mode, reason);
7b2a2d4a18fffa Mel Gorman 2012-10-19 1539
b20a35035f983f Christoph Lameter 2006-03-22 1540 if (!swapwrite)
b20a35035f983f Christoph Lameter 2006-03-22 1541 current->flags &= ~PF_SWAPWRITE;
b20a35035f983f Christoph Lameter 2006-03-22 1542
95a402c3847cc1 Christoph Lameter 2006-06-23 1543 return rc;
b20a35035f983f Christoph Lameter 2006-03-22 1544 }
b20a35035f983f Christoph Lameter 2006-03-22 1545
:::::: The code at line 1406 was first introduced by commit
:::::: 9c620e2bc5aa4256c102ada34e6c76204ed5898b mm: remove offlining arg to migrate_pages
:::::: TO: Hugh Dickins <hughd(a)google.com>
:::::: CC: Linus Torvalds <torvalds(a)linux-foundation.org>
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
1 week, 6 days
[input:next 2732/2735] drivers/input/touchscreen/resistive-adc-touch.c:84 grts_cb() error: uninitialized symbol 'z2'.
by Dan Carpenter
tree: https://git.kernel.org/pub/scm/linux/kernel/git/dtor/input.git next
head: 6cf3b3abbf0b3b778138c0f8936aa7820af62cfc
commit: fb082cd59afa7fbd3a610e9835050675040b1b99 [2732/2735] Input: resistive-adc-touch - add support for z1 and z2 channels
config: x86_64-randconfig-m001-20210526 (attached as .config)
compiler: gcc-9 (Debian 9.3.0-22) 9.3.0
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp(a)intel.com>
Reported-by: Dan Carpenter <dan.carpenter(a)oracle.com>
New smatch warnings:
drivers/input/touchscreen/resistive-adc-touch.c:84 grts_cb() error: uninitialized symbol 'z2'.
drivers/input/touchscreen/resistive-adc-touch.c:88 grts_cb() error: uninitialized symbol 'x'.
drivers/input/touchscreen/resistive-adc-touch.c:101 grts_cb() error: uninitialized symbol 'y'.
Old smatch warnings:
drivers/input/touchscreen/resistive-adc-touch.c:101 grts_cb() error: uninitialized symbol 'x'.
drivers/input/touchscreen/resistive-adc-touch.c:109 grts_cb() error: uninitialized symbol 'y'.
vim +/z2 +84 drivers/input/touchscreen/resistive-adc-touch.c
aa132ffb6b0a18 Eugen Hristev 2018-05-22 56 static int grts_cb(const void *data, void *private)
aa132ffb6b0a18 Eugen Hristev 2018-05-22 57 {
aa132ffb6b0a18 Eugen Hristev 2018-05-22 58 const u16 *touch_info = data;
aa132ffb6b0a18 Eugen Hristev 2018-05-22 59 struct grts_state *st = private;
fb082cd59afa7f Oleksij Rempel 2021-05-25 60 unsigned int x, y, press = 0, z1 = 0, z2;
fb082cd59afa7f Oleksij Rempel 2021-05-25 61 unsigned int Rt, i;
aa132ffb6b0a18 Eugen Hristev 2018-05-22 62
fb082cd59afa7f Oleksij Rempel 2021-05-25 63 for (i = 0; i < ARRAY_SIZE(st->ch) && st->ch[i] != GRTS_CH_NONE; i++) {
fb082cd59afa7f Oleksij Rempel 2021-05-25 64 switch (st->ch[i]) {
fb082cd59afa7f Oleksij Rempel 2021-05-25 65 case GRTS_CH_X:
fb082cd59afa7f Oleksij Rempel 2021-05-25 66 x = touch_info[i];
fb082cd59afa7f Oleksij Rempel 2021-05-25 67 break;
fb082cd59afa7f Oleksij Rempel 2021-05-25 68 case GRTS_CH_Y:
fb082cd59afa7f Oleksij Rempel 2021-05-25 69 y = touch_info[i];
fb082cd59afa7f Oleksij Rempel 2021-05-25 70 break;
fb082cd59afa7f Oleksij Rempel 2021-05-25 71 case GRTS_CH_PRESSURE:
fb082cd59afa7f Oleksij Rempel 2021-05-25 72 press = touch_info[i];
fb082cd59afa7f Oleksij Rempel 2021-05-25 73 break;
fb082cd59afa7f Oleksij Rempel 2021-05-25 74 case GRTS_CH_Z1:
fb082cd59afa7f Oleksij Rempel 2021-05-25 75 z1 = touch_info[i];
fb082cd59afa7f Oleksij Rempel 2021-05-25 76 break;
fb082cd59afa7f Oleksij Rempel 2021-05-25 77 case GRTS_CH_Z2:
fb082cd59afa7f Oleksij Rempel 2021-05-25 78 z2 = touch_info[i];
fb082cd59afa7f Oleksij Rempel 2021-05-25 79 break;
fb082cd59afa7f Oleksij Rempel 2021-05-25 80 }
fb082cd59afa7f Oleksij Rempel 2021-05-25 81 }
fb082cd59afa7f Oleksij Rempel 2021-05-25 82
fb082cd59afa7f Oleksij Rempel 2021-05-25 83 if (z1) {
fb082cd59afa7f Oleksij Rempel 2021-05-25 @84 Rt = z2;
It's likely this is a false positive, but we can see why the static
checker would complain.
fb082cd59afa7f Oleksij Rempel 2021-05-25 85 Rt -= z1;
fb082cd59afa7f Oleksij Rempel 2021-05-25 86 Rt *= st->x_plate_ohms;
fb082cd59afa7f Oleksij Rempel 2021-05-25 87 Rt = DIV_ROUND_CLOSEST(Rt, 16);
fb082cd59afa7f Oleksij Rempel 2021-05-25 @88 Rt *= x;
fb082cd59afa7f Oleksij Rempel 2021-05-25 89 Rt /= z1;
fb082cd59afa7f Oleksij Rempel 2021-05-25 90 Rt = DIV_ROUND_CLOSEST(Rt, 256);
fb082cd59afa7f Oleksij Rempel 2021-05-25 91 /*
fb082cd59afa7f Oleksij Rempel 2021-05-25 92 * On increased pressure the resistance (Rt) is decreasing
fb082cd59afa7f Oleksij Rempel 2021-05-25 93 * so, convert values to make it looks as real pressure.
fb082cd59afa7f Oleksij Rempel 2021-05-25 94 */
fb082cd59afa7f Oleksij Rempel 2021-05-25 95 if (Rt < GRTS_DEFAULT_PRESSURE_MAX)
fb082cd59afa7f Oleksij Rempel 2021-05-25 96 press = GRTS_DEFAULT_PRESSURE_MAX - Rt;
fb082cd59afa7f Oleksij Rempel 2021-05-25 97 else
fb082cd59afa7f Oleksij Rempel 2021-05-25 98 press = 0;
fb082cd59afa7f Oleksij Rempel 2021-05-25 99 }
aa132ffb6b0a18 Eugen Hristev 2018-05-22 100
aa132ffb6b0a18 Eugen Hristev 2018-05-22 @101 if ((!x && !y) || (st->pressure && (press < st->pressure_min))) {
aa132ffb6b0a18 Eugen Hristev 2018-05-22 102 /* report end of touch */
aa132ffb6b0a18 Eugen Hristev 2018-05-22 103 input_report_key(st->input, BTN_TOUCH, 0);
aa132ffb6b0a18 Eugen Hristev 2018-05-22 104 input_sync(st->input);
aa132ffb6b0a18 Eugen Hristev 2018-05-22 105 return 0;
aa132ffb6b0a18 Eugen Hristev 2018-05-22 106 }
aa132ffb6b0a18 Eugen Hristev 2018-05-22 107
aa132ffb6b0a18 Eugen Hristev 2018-05-22 108 /* report proper touch to subsystem*/
aa132ffb6b0a18 Eugen Hristev 2018-05-22 109 touchscreen_report_pos(st->input, &st->prop, x, y, false);
aa132ffb6b0a18 Eugen Hristev 2018-05-22 110 if (st->pressure)
aa132ffb6b0a18 Eugen Hristev 2018-05-22 111 input_report_abs(st->input, ABS_PRESSURE, press);
aa132ffb6b0a18 Eugen Hristev 2018-05-22 112 input_report_key(st->input, BTN_TOUCH, 1);
aa132ffb6b0a18 Eugen Hristev 2018-05-22 113 input_sync(st->input);
aa132ffb6b0a18 Eugen Hristev 2018-05-22 114
aa132ffb6b0a18 Eugen Hristev 2018-05-22 115 return 0;
aa132ffb6b0a18 Eugen Hristev 2018-05-22 116 }
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
1 week, 6 days
[linux-next:master 5596/5886] arch/x86/include/asm/fixmap.h:103:48: error: use of undeclared identifier 'NR_CPUS'
by kernel test robot
tree: https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git master
head: 3e029760e6f8ce90c122c267a039ae73b3f1f5a4
commit: 366e9aec4a22d77b869f1b7641a74fdd96f02b4c [5596/5886] mm/early_ioremap: add prototype for early_memremap_pgprot_adjust
config: x86_64-randconfig-a006-20210530 (attached as .config)
compiler: clang version 13.0.0 (https://github.com/llvm/llvm-project bc6799f2f79f0ae87e9f1ebf9d25ba799fbd25a9)
reproduce (this is a W=1 build):
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# install x86_64 cross compiling tool for clang build
# apt-get install binutils-x86-64-linux-gnu
# https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git/commi...
git remote add linux-next https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git
git fetch --no-tags linux-next master
git checkout 366e9aec4a22d77b869f1b7641a74fdd96f02b4c
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=x86_64
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp(a)intel.com>
All errors (new ones prefixed by >>):
In file included from drivers/gpu/drm/udl/udl_modeset.c:12:
In file included from include/linux/dma-buf.h:16:
In file included from include/linux/dma-buf-map.h:9:
In file included from include/linux/io.h:13:
In file included from arch/x86/include/asm/io.h:44:
In file included from ./arch/x86/include/generated/asm/early_ioremap.h:1:
In file included from include/asm-generic/early_ioremap.h:6:
>> arch/x86/include/asm/fixmap.h:103:48: error: use of undeclared identifier 'NR_CPUS'
FIX_KMAP_END = FIX_KMAP_BEGIN + (KM_MAX_IDX * NR_CPUS) - 1,
^
1 error generated.
vim +/NR_CPUS +103 arch/x86/include/asm/fixmap.h
c577b098f9bf46 arch/x86/include/asm/fixmap.h Gustavo Padovan 2009-02-15 61
c577b098f9bf46 arch/x86/include/asm/fixmap.h Gustavo Padovan 2009-02-15 62 /*
c577b098f9bf46 arch/x86/include/asm/fixmap.h Gustavo Padovan 2009-02-15 63 * Here we define all the compile-time 'special' virtual
c577b098f9bf46 arch/x86/include/asm/fixmap.h Gustavo Padovan 2009-02-15 64 * addresses. The point is to have a constant address at
c577b098f9bf46 arch/x86/include/asm/fixmap.h Gustavo Padovan 2009-02-15 65 * compile time, but to set the physical address only
c577b098f9bf46 arch/x86/include/asm/fixmap.h Gustavo Padovan 2009-02-15 66 * in the boot process.
c577b098f9bf46 arch/x86/include/asm/fixmap.h Gustavo Padovan 2009-02-15 67 * for x86_32: We allocate these special addresses
c577b098f9bf46 arch/x86/include/asm/fixmap.h Gustavo Padovan 2009-02-15 68 * from the end of virtual memory (0xfffff000) backwards.
c577b098f9bf46 arch/x86/include/asm/fixmap.h Gustavo Padovan 2009-02-15 69 * Also this lets us do fail-safe vmalloc(), we
c577b098f9bf46 arch/x86/include/asm/fixmap.h Gustavo Padovan 2009-02-15 70 * can guarantee that these special addresses and
c577b098f9bf46 arch/x86/include/asm/fixmap.h Gustavo Padovan 2009-02-15 71 * vmalloc()-ed addresses never overlap.
c577b098f9bf46 arch/x86/include/asm/fixmap.h Gustavo Padovan 2009-02-15 72 *
c577b098f9bf46 arch/x86/include/asm/fixmap.h Gustavo Padovan 2009-02-15 73 * These 'compile-time allocated' memory buffers are
c577b098f9bf46 arch/x86/include/asm/fixmap.h Gustavo Padovan 2009-02-15 74 * fixed-size 4k pages (or larger if used with an increment
c577b098f9bf46 arch/x86/include/asm/fixmap.h Gustavo Padovan 2009-02-15 75 * higher than 1). Use set_fixmap(idx,phys) to associate
c577b098f9bf46 arch/x86/include/asm/fixmap.h Gustavo Padovan 2009-02-15 76 * physical memory with fixmap indices.
c577b098f9bf46 arch/x86/include/asm/fixmap.h Gustavo Padovan 2009-02-15 77 *
c577b098f9bf46 arch/x86/include/asm/fixmap.h Gustavo Padovan 2009-02-15 78 * TLB entries of such buffers will not be flushed across
c577b098f9bf46 arch/x86/include/asm/fixmap.h Gustavo Padovan 2009-02-15 79 * task switches.
c577b098f9bf46 arch/x86/include/asm/fixmap.h Gustavo Padovan 2009-02-15 80 */
c577b098f9bf46 arch/x86/include/asm/fixmap.h Gustavo Padovan 2009-02-15 81 enum fixed_addresses {
96a388de5dc53a include/asm-x86/fixmap.h Thomas Gleixner 2007-10-11 82 #ifdef CONFIG_X86_32
c577b098f9bf46 arch/x86/include/asm/fixmap.h Gustavo Padovan 2009-02-15 83 FIX_HOLE,
96a388de5dc53a include/asm-x86/fixmap.h Thomas Gleixner 2007-10-11 84 #else
1ad83c858c7d4e arch/x86/include/asm/fixmap.h Andy Lutomirski 2014-10-29 85 #ifdef CONFIG_X86_VSYSCALL_EMULATION
f40c330091c7aa arch/x86/include/asm/fixmap.h Andy Lutomirski 2014-05-05 86 VSYSCALL_PAGE = (FIXADDR_TOP - VSYSCALL_ADDR) >> PAGE_SHIFT,
1ad83c858c7d4e arch/x86/include/asm/fixmap.h Andy Lutomirski 2014-10-29 87 #endif
c577b098f9bf46 arch/x86/include/asm/fixmap.h Gustavo Padovan 2009-02-15 88 #endif
c577b098f9bf46 arch/x86/include/asm/fixmap.h Gustavo Padovan 2009-02-15 89 FIX_DBGP_BASE,
c577b098f9bf46 arch/x86/include/asm/fixmap.h Gustavo Padovan 2009-02-15 90 FIX_EARLYCON_MEM_BASE,
ff30a0543e9a6c arch/x86/include/asm/fixmap.h Jan Beulich 2010-03-15 91 #ifdef CONFIG_PROVIDE_OHCI1394_DMA_INIT
ff30a0543e9a6c arch/x86/include/asm/fixmap.h Jan Beulich 2010-03-15 92 FIX_OHCI1394_BASE,
ff30a0543e9a6c arch/x86/include/asm/fixmap.h Jan Beulich 2010-03-15 93 #endif
c577b098f9bf46 arch/x86/include/asm/fixmap.h Gustavo Padovan 2009-02-15 94 #ifdef CONFIG_X86_LOCAL_APIC
c577b098f9bf46 arch/x86/include/asm/fixmap.h Gustavo Padovan 2009-02-15 95 FIX_APIC_BASE, /* local (CPU) APIC) -- required for SMP or not */
c577b098f9bf46 arch/x86/include/asm/fixmap.h Gustavo Padovan 2009-02-15 96 #endif
c577b098f9bf46 arch/x86/include/asm/fixmap.h Gustavo Padovan 2009-02-15 97 #ifdef CONFIG_X86_IO_APIC
c577b098f9bf46 arch/x86/include/asm/fixmap.h Gustavo Padovan 2009-02-15 98 FIX_IO_APIC_BASE_0,
c577b098f9bf46 arch/x86/include/asm/fixmap.h Gustavo Padovan 2009-02-15 99 FIX_IO_APIC_BASE_END = FIX_IO_APIC_BASE_0 + MAX_IO_APICS - 1,
c577b098f9bf46 arch/x86/include/asm/fixmap.h Gustavo Padovan 2009-02-15 100 #endif
14df3267029158 arch/x86/include/asm/fixmap.h Thomas Gleixner 2020-11-18 101 #ifdef CONFIG_KMAP_LOCAL
c577b098f9bf46 arch/x86/include/asm/fixmap.h Gustavo Padovan 2009-02-15 102 FIX_KMAP_BEGIN, /* reserved pte's for temporary kernel mappings */
157e118b55113d arch/x86/include/asm/fixmap.h Thomas Gleixner 2020-11-03 @103 FIX_KMAP_END = FIX_KMAP_BEGIN + (KM_MAX_IDX * NR_CPUS) - 1,
c577b098f9bf46 arch/x86/include/asm/fixmap.h Gustavo Padovan 2009-02-15 104 #ifdef CONFIG_PCI_MMCONFIG
c577b098f9bf46 arch/x86/include/asm/fixmap.h Gustavo Padovan 2009-02-15 105 FIX_PCIE_MCFG,
96a388de5dc53a include/asm-x86/fixmap.h Thomas Gleixner 2007-10-11 106 #endif
c577b098f9bf46 arch/x86/include/asm/fixmap.h Gustavo Padovan 2009-02-15 107 #endif
ecac71816a1829 arch/x86/include/asm/fixmap.h Juergen Gross 2020-08-15 108 #ifdef CONFIG_PARAVIRT_XXL
c577b098f9bf46 arch/x86/include/asm/fixmap.h Gustavo Padovan 2009-02-15 109 FIX_PARAVIRT_BOOTMAP,
c577b098f9bf46 arch/x86/include/asm/fixmap.h Gustavo Padovan 2009-02-15 110 #endif
69218e47994da6 arch/x86/include/asm/fixmap.h Thomas Garnier 2017-03-14 111
:::::: The code at line 103 was first introduced by commit
:::::: 157e118b55113d1e6c7f8ddfcec0a1dbf3a69511 x86/mm/highmem: Use generic kmap atomic implementation
:::::: TO: Thomas Gleixner <tglx(a)linutronix.de>
:::::: CC: Thomas Gleixner <tglx(a)linutronix.de>
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
2 weeks
drivers/gpu/drm/msm/disp/dpu1/dpu_encoder_phys_cmd.c:688:31: warning: variable 'cmd_enc' set but not used
by kernel test robot
Hi AngeloGioacchino,
FYI, the error/warning still remains.
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head: c2131f7e73c9e9365613e323d65c7b9e5b910f56
commit: fe286893ed34b12684659d3efb907d47fe18559b drm/msm/dpu: Remove unused call in wait_for_commit_done
date: 4 months ago
config: arm-randconfig-r012-20210531 (attached as .config)
compiler: arm-linux-gnueabi-gcc (GCC) 9.3.0
reproduce (this is a W=1 build):
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit...
git remote add linus https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
git fetch --no-tags linus master
git checkout fe286893ed34b12684659d3efb907d47fe18559b
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross ARCH=arm
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp(a)intel.com>
All warnings (new ones prefixed by >>):
drivers/gpu/drm/msm/disp/dpu1/dpu_encoder_phys_cmd.c: In function 'dpu_encoder_phys_cmd_wait_for_commit_done':
>> drivers/gpu/drm/msm/disp/dpu1/dpu_encoder_phys_cmd.c:688:31: warning: variable 'cmd_enc' set but not used [-Wunused-but-set-variable]
688 | struct dpu_encoder_phys_cmd *cmd_enc;
| ^~~~~~~
vim +/cmd_enc +688 drivers/gpu/drm/msm/disp/dpu1/dpu_encoder_phys_cmd.c
25fdd5933e4c0f Jeykumar Sankaran 2018-06-27 684
25fdd5933e4c0f Jeykumar Sankaran 2018-06-27 685 static int dpu_encoder_phys_cmd_wait_for_commit_done(
25fdd5933e4c0f Jeykumar Sankaran 2018-06-27 686 struct dpu_encoder_phys *phys_enc)
25fdd5933e4c0f Jeykumar Sankaran 2018-06-27 687 {
25fdd5933e4c0f Jeykumar Sankaran 2018-06-27 @688 struct dpu_encoder_phys_cmd *cmd_enc;
25fdd5933e4c0f Jeykumar Sankaran 2018-06-27 689
25fdd5933e4c0f Jeykumar Sankaran 2018-06-27 690 cmd_enc = to_dpu_encoder_phys_cmd(phys_enc);
25fdd5933e4c0f Jeykumar Sankaran 2018-06-27 691
25fdd5933e4c0f Jeykumar Sankaran 2018-06-27 692 /* only required for master controller */
fe286893ed34b1 AngeloGioacchino Del Regno 2021-01-12 693 if (!dpu_encoder_phys_cmd_is_master(phys_enc))
fe286893ed34b1 AngeloGioacchino Del Regno 2021-01-12 694 return 0;
25fdd5933e4c0f Jeykumar Sankaran 2018-06-27 695
fe286893ed34b1 AngeloGioacchino Del Regno 2021-01-12 696 return _dpu_encoder_phys_cmd_wait_for_ctl_start(phys_enc);
25fdd5933e4c0f Jeykumar Sankaran 2018-06-27 697 }
25fdd5933e4c0f Jeykumar Sankaran 2018-06-27 698
:::::: The code at line 688 was first introduced by commit
:::::: 25fdd5933e4c0f5fe2ea5cd59994f8ac5fbe90ef drm/msm: Add SDM845 DPU support
:::::: TO: Jeykumar Sankaran <jsanka(a)codeaurora.org>
:::::: CC: Sean Paul <seanpaul(a)chromium.org>
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
2 weeks
Re: [PATCH next v1 1/2] dump_stack: move cpu lock to printk.c
by kernel test robot
Hi John,
I love your patch! Perhaps something to improve:
[auto build test WARNING on next-20210528]
url: https://github.com/0day-ci/linux/commits/John-Ogness/introduce-printk-cpu...
base: 3e029760e6f8ce90c122c267a039ae73b3f1f5a4
config: x86_64-randconfig-a014-20210531 (attached as .config)
compiler: clang version 13.0.0 (https://github.com/llvm/llvm-project bc6799f2f79f0ae87e9f1ebf9d25ba799fbd25a9)
reproduce (this is a W=1 build):
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# install x86_64 cross compiling tool for clang build
# apt-get install binutils-x86-64-linux-gnu
# https://github.com/0day-ci/linux/commit/528abf3d0ab80471d02f4f16cbe97b2ea...
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review John-Ogness/introduce-printk-cpu-lock/20210601-013122
git checkout 528abf3d0ab80471d02f4f16cbe97b2eaa918a11
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=x86_64
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp(a)intel.com>
All warnings (new ones prefixed by >>):
lib/dump_stack.c:97: warning: Function parameter or member 'log_lvl' not described in 'dump_stack_lvl'
>> lib/dump_stack.c:97: warning: expecting prototype for dump_stack(). Prototype was for dump_stack_lvl() instead
vim +97 lib/dump_stack.c
^1da177e4c3f415 Linus Torvalds 2005-04-16 90
196779b9b4ce192 Tejun Heo 2013-04-30 91 /**
196779b9b4ce192 Tejun Heo 2013-04-30 92 * dump_stack - dump the current task information and its stack trace
196779b9b4ce192 Tejun Heo 2013-04-30 93 *
196779b9b4ce192 Tejun Heo 2013-04-30 94 * Architectures can override this implementation by implementing its own.
196779b9b4ce192 Tejun Heo 2013-04-30 95 */
d542aa86de8ad0c Alexander Potapenko 2021-05-26 96 asmlinkage __visible void dump_stack_lvl(const char *log_lvl)
^1da177e4c3f415 Linus Torvalds 2005-04-16 @97 {
528abf3d0ab8047 John Ogness 2021-05-31 98 unsigned int cpu_store;
d7ce36924344ace Eric Dumazet 2016-02-05 99 unsigned long flags;
b58d977432c80ee Alex Thorlton 2013-07-03 100
528abf3d0ab8047 John Ogness 2021-05-31 101 printk_cpu_lock(&cpu_store, &flags);
d542aa86de8ad0c Alexander Potapenko 2021-05-26 102 __dump_stack(log_lvl);
528abf3d0ab8047 John Ogness 2021-05-31 103 printk_cpu_unlock(cpu_store, flags);
b58d977432c80ee Alex Thorlton 2013-07-03 104 }
d542aa86de8ad0c Alexander Potapenko 2021-05-26 105 EXPORT_SYMBOL(dump_stack_lvl);
d542aa86de8ad0c Alexander Potapenko 2021-05-26 106
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
2 weeks