From: kernel test robot <lkp@intel.com>
To: Vasant Hegde <vasant.hegde@amd.com>
Cc: oe-kbuild-all@lists.linux.dev, iommu@lists.linux.dev,
Joerg Roedel <jroedel@suse.de>, Jason Gunthorpe <jgg@nvidia.com>,
Jerry Snitselaar <jsnitsel@redhat.com>
Subject: [joro-iommu:x86/amd 16/19] drivers/iommu/amd/iommu.c:829:21: warning: variable 'entry' set but not used
Date: Sat, 7 Oct 2023 11:20:35 +0800 [thread overview]
Message-ID: <202310071103.BYatoc0b-lkp@intel.com> (raw)
tree: https://git.kernel.org/pub/scm/linux/kernel/git/joro/iommu.git x86/amd
head: 189116d5ad54eee2bab82c5f12e837912fc0b4c4
commit: d55b0d2e075ad32d560cdd35a2016973fda52ed6 [16/19] iommu/amd: Remove PPR support
config: x86_64-randconfig-003-20231007 (https://download.01.org/0day-ci/archive/20231007/202310071103.BYatoc0b-lkp@intel.com/config)
compiler: gcc-12 (Debian 12.2.0-14) 12.2.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20231007/202310071103.BYatoc0b-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/202310071103.BYatoc0b-lkp@intel.com/
All warnings (new ones prefixed by >>):
drivers/iommu/amd/iommu.c: In function 'iommu_poll_ppr_log':
>> drivers/iommu/amd/iommu.c:829:21: warning: variable 'entry' set but not used [-Wunused-but-set-variable]
829 | u64 entry[2];
| ^~~~~
vim +/entry +829 drivers/iommu/amd/iommu.c
90008ee4b811c9 arch/x86/kernel/amd_iommu.c Joerg Roedel 2008-09-09 816
72e1dcc4192288 drivers/iommu/amd_iommu.c Joerg Roedel 2011-11-10 817 static void iommu_poll_ppr_log(struct amd_iommu *iommu)
72e1dcc4192288 drivers/iommu/amd_iommu.c Joerg Roedel 2011-11-10 818 {
72e1dcc4192288 drivers/iommu/amd_iommu.c Joerg Roedel 2011-11-10 819 u32 head, tail;
72e1dcc4192288 drivers/iommu/amd_iommu.c Joerg Roedel 2011-11-10 820
72e1dcc4192288 drivers/iommu/amd_iommu.c Joerg Roedel 2011-11-10 821 if (iommu->ppr_log == NULL)
72e1dcc4192288 drivers/iommu/amd_iommu.c Joerg Roedel 2011-11-10 822 return;
72e1dcc4192288 drivers/iommu/amd_iommu.c Joerg Roedel 2011-11-10 823
72e1dcc4192288 drivers/iommu/amd_iommu.c Joerg Roedel 2011-11-10 824 head = readl(iommu->mmio_base + MMIO_PPR_HEAD_OFFSET);
72e1dcc4192288 drivers/iommu/amd_iommu.c Joerg Roedel 2011-11-10 825 tail = readl(iommu->mmio_base + MMIO_PPR_TAIL_OFFSET);
72e1dcc4192288 drivers/iommu/amd_iommu.c Joerg Roedel 2011-11-10 826
72e1dcc4192288 drivers/iommu/amd_iommu.c Joerg Roedel 2011-11-10 827 while (head != tail) {
eee53537c476c9 drivers/iommu/amd_iommu.c Joerg Roedel 2012-06-01 828 volatile u64 *raw;
eee53537c476c9 drivers/iommu/amd_iommu.c Joerg Roedel 2012-06-01 @829 u64 entry[2];
eee53537c476c9 drivers/iommu/amd_iommu.c Joerg Roedel 2012-06-01 830 int i;
72e1dcc4192288 drivers/iommu/amd_iommu.c Joerg Roedel 2011-11-10 831
eee53537c476c9 drivers/iommu/amd_iommu.c Joerg Roedel 2012-06-01 832 raw = (u64 *)(iommu->ppr_log + head);
72e1dcc4192288 drivers/iommu/amd_iommu.c Joerg Roedel 2011-11-10 833
eee53537c476c9 drivers/iommu/amd_iommu.c Joerg Roedel 2012-06-01 834 /*
eee53537c476c9 drivers/iommu/amd_iommu.c Joerg Roedel 2012-06-01 835 * Hardware bug: Interrupt may arrive before the entry is
eee53537c476c9 drivers/iommu/amd_iommu.c Joerg Roedel 2012-06-01 836 * written to memory. If this happens we need to wait for the
eee53537c476c9 drivers/iommu/amd_iommu.c Joerg Roedel 2012-06-01 837 * entry to arrive.
eee53537c476c9 drivers/iommu/amd_iommu.c Joerg Roedel 2012-06-01 838 */
eee53537c476c9 drivers/iommu/amd_iommu.c Joerg Roedel 2012-06-01 839 for (i = 0; i < LOOP_TIMEOUT; ++i) {
eee53537c476c9 drivers/iommu/amd_iommu.c Joerg Roedel 2012-06-01 840 if (PPR_REQ_TYPE(raw[0]) != 0)
eee53537c476c9 drivers/iommu/amd_iommu.c Joerg Roedel 2012-06-01 841 break;
eee53537c476c9 drivers/iommu/amd_iommu.c Joerg Roedel 2012-06-01 842 udelay(1);
eee53537c476c9 drivers/iommu/amd_iommu.c Joerg Roedel 2012-06-01 843 }
eee53537c476c9 drivers/iommu/amd_iommu.c Joerg Roedel 2012-06-01 844
eee53537c476c9 drivers/iommu/amd_iommu.c Joerg Roedel 2012-06-01 845 /* Avoid memcpy function-call overhead */
eee53537c476c9 drivers/iommu/amd_iommu.c Joerg Roedel 2012-06-01 846 entry[0] = raw[0];
eee53537c476c9 drivers/iommu/amd_iommu.c Joerg Roedel 2012-06-01 847 entry[1] = raw[1];
eee53537c476c9 drivers/iommu/amd_iommu.c Joerg Roedel 2012-06-01 848
eee53537c476c9 drivers/iommu/amd_iommu.c Joerg Roedel 2012-06-01 849 /*
05d227efbd8d3c drivers/iommu/amd/iommu.c Tom Lendacky 2023-01-17 850 * To detect the hardware errata 733 we need to clear the
05d227efbd8d3c drivers/iommu/amd/iommu.c Tom Lendacky 2023-01-17 851 * entry back to zero. This issue does not exist on SNP
05d227efbd8d3c drivers/iommu/amd/iommu.c Tom Lendacky 2023-01-17 852 * enabled system. Also this buffer is not writeable on
05d227efbd8d3c drivers/iommu/amd/iommu.c Tom Lendacky 2023-01-17 853 * SNP enabled system.
eee53537c476c9 drivers/iommu/amd_iommu.c Joerg Roedel 2012-06-01 854 */
05d227efbd8d3c drivers/iommu/amd/iommu.c Tom Lendacky 2023-01-17 855 if (!amd_iommu_snp_en)
eee53537c476c9 drivers/iommu/amd_iommu.c Joerg Roedel 2012-06-01 856 raw[0] = raw[1] = 0UL;
eee53537c476c9 drivers/iommu/amd_iommu.c Joerg Roedel 2012-06-01 857
eee53537c476c9 drivers/iommu/amd_iommu.c Joerg Roedel 2012-06-01 858 /* Update head pointer of hardware ring-buffer */
72e1dcc4192288 drivers/iommu/amd_iommu.c Joerg Roedel 2011-11-10 859 head = (head + PPR_ENTRY_SIZE) % PPR_LOG_SIZE;
72e1dcc4192288 drivers/iommu/amd_iommu.c Joerg Roedel 2011-11-10 860 writel(head, iommu->mmio_base + MMIO_PPR_HEAD_OFFSET);
eee53537c476c9 drivers/iommu/amd_iommu.c Joerg Roedel 2012-06-01 861
d55b0d2e075ad3 drivers/iommu/amd/iommu.c Vasant Hegde 2023-10-06 862 /* TODO: PPR Handler will be added when we add IOPF support */
eee53537c476c9 drivers/iommu/amd_iommu.c Joerg Roedel 2012-06-01 863
eee53537c476c9 drivers/iommu/amd_iommu.c Joerg Roedel 2012-06-01 864 /* Refresh ring-buffer information */
eee53537c476c9 drivers/iommu/amd_iommu.c Joerg Roedel 2012-06-01 865 head = readl(iommu->mmio_base + MMIO_PPR_HEAD_OFFSET);
72e1dcc4192288 drivers/iommu/amd_iommu.c Joerg Roedel 2011-11-10 866 tail = readl(iommu->mmio_base + MMIO_PPR_TAIL_OFFSET);
72e1dcc4192288 drivers/iommu/amd_iommu.c Joerg Roedel 2011-11-10 867 }
72e1dcc4192288 drivers/iommu/amd_iommu.c Joerg Roedel 2011-11-10 868 }
72e1dcc4192288 drivers/iommu/amd_iommu.c Joerg Roedel 2011-11-10 869
:::::: The code at line 829 was first introduced by commit
:::::: eee53537c476c947bf7faa1c916d2f5a0ae8ec93 iommu/amd: Fix deadlock in ppr-handling error path
:::::: TO: Joerg Roedel <joerg.roedel@amd.com>
:::::: CC: Joerg Roedel <joerg.roedel@amd.com>
--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki
next reply other threads:[~2023-10-07 3:20 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-10-07 3:20 kernel test robot [this message]
2023-10-12 5:26 ` [joro-iommu:x86/amd 16/19] drivers/iommu/amd/iommu.c:829:21: warning: variable 'entry' set but not used Vasant Hegde
2023-10-12 13:21 ` Jason Gunthorpe
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=202310071103.BYatoc0b-lkp@intel.com \
--to=lkp@intel.com \
--cc=iommu@lists.linux.dev \
--cc=jgg@nvidia.com \
--cc=jroedel@suse.de \
--cc=jsnitsel@redhat.com \
--cc=oe-kbuild-all@lists.linux.dev \
--cc=vasant.hegde@amd.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.