From: kernel test robot <lkp@intel.com>
To: Daniel Tsai <danielsftsai@google.com>,
Jingoo Han <jingoohan1@gmail.com>,
Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>
Cc: llvm@lists.linux.dev, oe-kbuild-all@lists.linux.dev,
"Lorenzo Pieralisi" <lpieralisi@kernel.org>,
"Krzysztof Wilczyński" <kw@linux.com>,
"Rob Herring" <robh@kernel.org>,
"Bjorn Helgaas" <helgaas@kernel.org>,
"Andrew Chant" <achant@google.com>,
"Brian Norris" <briannorris@google.com>,
"Sajid Dalvi" <sdalvi@google.com>,
"Mark Cheng" <markcheng@google.com>,
"Ben Cheng" <bccheng@google.com>,
"Thomas Gleixner" <tglx@linutronix.de>,
linux-pci@vger.kernel.org, linux-kernel@vger.kernel.org,
"Tsai Sung-Fu" <danielsftsai@google.com>
Subject: Re: [PATCH] PCI: dwc: Chain the set IRQ affinity request back to the parent
Date: Mon, 3 Mar 2025 17:25:59 +0800 [thread overview]
Message-ID: <202503031726.KLKoMMZk-lkp@intel.com> (raw)
In-Reply-To: <20250303070501.2740392-1-danielsftsai@google.com>
Hi Daniel,
kernel test robot noticed the following build errors:
[auto build test ERROR on pci/next]
[also build test ERROR on pci/for-linus mani-mhi/mhi-next linus/master v6.14-rc5 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/Daniel-Tsai/PCI-dwc-Chain-the-set-IRQ-affinity-request-back-to-the-parent/20250303-150704
base: https://git.kernel.org/pub/scm/linux/kernel/git/pci/pci.git next
patch link: https://lore.kernel.org/r/20250303070501.2740392-1-danielsftsai%40google.com
patch subject: [PATCH] PCI: dwc: Chain the set IRQ affinity request back to the parent
config: x86_64-buildonly-randconfig-006-20250303 (https://download.01.org/0day-ci/archive/20250303/202503031726.KLKoMMZk-lkp@intel.com/config)
compiler: clang version 19.1.7 (https://github.com/llvm/llvm-project cd708029e0b2869e80abe31ddb175f7c35361f90)
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20250303/202503031726.KLKoMMZk-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/202503031726.KLKoMMZk-lkp@intel.com/
All errors (new ones prefixed by >>):
>> drivers/pci/controller/dwc/pcie-designware-host.c:223:45: error: no member named 'affinity' in 'struct irq_common_data'
223 | cpumask_copy(desc_parent->irq_common_data.affinity, mask);
| ~~~~~~~~~~~~~~~~~~~~~~~~~~~~ ^
drivers/pci/controller/dwc/pcie-designware-host.c:507:30: warning: shift count >= width of type [-Wshift-count-overflow]
507 | dma_set_coherent_mask(dev, DMA_BIT_MASK(64));
| ^~~~~~~~~~~~~~~~
include/linux/dma-mapping.h:73:54: note: expanded from macro 'DMA_BIT_MASK'
73 | #define DMA_BIT_MASK(n) (((n) == 64) ? ~0ULL : ((1ULL<<(n))-1))
| ^ ~~~
1 warning and 1 error generated.
vim +223 drivers/pci/controller/dwc/pcie-designware-host.c
178
179 static int dw_pci_msi_set_affinity(struct irq_data *d,
180 const struct cpumask *mask, bool force)
181 {
182 struct dw_pcie_rp *pp = irq_data_get_irq_chip_data(d);
183 struct dw_pcie *pci = to_dw_pcie_from_pp(pp);
184 int ret;
185 int virq_parent;
186 unsigned long hwirq = d->hwirq;
187 unsigned long flags, ctrl;
188 struct irq_desc *desc_parent;
189 const struct cpumask *effective_mask;
190 cpumask_var_t mask_result;
191
192 ctrl = hwirq / MAX_MSI_IRQS_PER_CTRL;
193 if (!alloc_cpumask_var(&mask_result, GFP_ATOMIC))
194 return -ENOMEM;
195
196 /*
197 * Loop through all possible MSI vector to check if the
198 * requested one is compatible with all of them
199 */
200 raw_spin_lock_irqsave(&pp->lock, flags);
201 cpumask_copy(mask_result, mask);
202 ret = dw_pci_check_mask_compatibility(pp, ctrl, hwirq, mask_result);
203 if (ret) {
204 dev_dbg(pci->dev, "Incompatible mask, request %*pbl, irq num %u\n",
205 cpumask_pr_args(mask), d->irq);
206 goto unlock;
207 }
208
209 dev_dbg(pci->dev, "Final mask, request %*pbl, irq num %u\n",
210 cpumask_pr_args(mask_result), d->irq);
211
212 virq_parent = pp->msi_irq[ctrl];
213 desc_parent = irq_to_desc(virq_parent);
214 ret = desc_parent->irq_data.chip->irq_set_affinity(&desc_parent->irq_data,
215 mask_result, force);
216
217 if (ret < 0)
218 goto unlock;
219
220 switch (ret) {
221 case IRQ_SET_MASK_OK:
222 case IRQ_SET_MASK_OK_DONE:
> 223 cpumask_copy(desc_parent->irq_common_data.affinity, mask);
224 fallthrough;
225 case IRQ_SET_MASK_OK_NOCOPY:
226 break;
227 }
228
229 effective_mask = irq_data_get_effective_affinity_mask(&desc_parent->irq_data);
230 dw_pci_update_effective_affinity(pp, ctrl, effective_mask, hwirq);
231
232 unlock:
233 free_cpumask_var(mask_result);
234 raw_spin_unlock_irqrestore(&pp->lock, flags);
235 return ret < 0 ? ret : IRQ_SET_MASK_OK_NOCOPY;
236 }
237
--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki
next prev parent reply other threads:[~2025-03-03 9:26 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-03-03 7:05 [PATCH] PCI: dwc: Chain the set IRQ affinity request back to the parent Daniel Tsai
2025-03-03 9:10 ` Thomas Gleixner
2025-03-04 5:48 ` Tsai Sung-Fu
2025-03-04 9:46 ` Thomas Gleixner
2025-03-05 11:21 ` Tsai Sung-Fu
2025-03-06 7:44 ` Thomas Gleixner
2025-03-07 11:10 ` Tsai Sung-Fu
2025-03-07 19:49 ` Thomas Gleixner
2025-03-11 9:52 ` Tsai Sung-Fu
2025-03-11 14:05 ` Thomas Gleixner
2025-03-25 6:38 ` Tsai Sung-Fu
2025-05-07 11:10 ` Krishna Chaitanya Chundru
2025-03-03 9:25 ` kernel test robot [this message]
2025-03-03 10:13 ` 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=202503031726.KLKoMMZk-lkp@intel.com \
--to=lkp@intel.com \
--cc=achant@google.com \
--cc=bccheng@google.com \
--cc=briannorris@google.com \
--cc=danielsftsai@google.com \
--cc=helgaas@kernel.org \
--cc=jingoohan1@gmail.com \
--cc=kw@linux.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-pci@vger.kernel.org \
--cc=llvm@lists.linux.dev \
--cc=lpieralisi@kernel.org \
--cc=manivannan.sadhasivam@linaro.org \
--cc=markcheng@google.com \
--cc=oe-kbuild-all@lists.linux.dev \
--cc=robh@kernel.org \
--cc=sdalvi@google.com \
--cc=tglx@linutronix.de \
/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.