| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Message-ID: <86d23e69-e6e5-476b-9582-28352852ea94@163.com>
Date: Fri, 28 Feb 2025 23:17:01 +0800
From: Hans Zhang <18255117159@....com>
To: Thomas Gleixner <tglx@...utronix.de>,
Manivannan Sadhasivam <manivannan.sadhasivam@...aro.org>
Cc: kw@...ux.com, kwilczynski@...nel.org, bhelgaas@...gle.com,
cassel@...nel.org, linux-pci@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] genirq/msi: Add the address and data that show MSI/MSIX
On 2025/2/28 19:26, Thomas Gleixner wrote:
> On Fri, Feb 28 2025 at 17:04, Hans Zhang wrote:
>> Is the following patch OK?
>
> No.
>
>> static void
>> irq_debug_show_chip(struct seq_file *m, struct irq_data *data, int ind)
>> {
>> @@ -178,6 +199,7 @@ static int irq_debug_show(struct seq_file *m, void *p)
>> seq_printf(m, "node: %d\n", irq_data_get_node(data));
>> irq_debug_show_masks(m, desc);
>> irq_debug_show_data(m, data, 0);
>> + irq_debug_show_msi_msix(m, data, 0);
>> raw_spin_unlock_irq(&desc->lock);
>> return 0;
>> }
>
> This is just violating the layering and I told you what to do:
>
> "implement a debug_show() callback in the MSI core code and assign
> it to domain ops::debug_show() on domain creation, if it does not
> provide its own callback."
>
> If you don't understand what I tell you, then please ask instead of
> going off and hacking up something completely different.
>
> Here is another hint:
>
> Look at msi_domain_ops_default and at msi_domain_update_dom_ops()
>
> If you still have questions, feel free to ask.
>
Hi Thomas(tglx),
I'm very sorry that I didn't understand what you meant at the beginning.
Thank you very much for your patient guidance.
Now, how about this patch? If you agree, I will resubmit the next version.
Best regards
Hans
patch:
diff --git a/kernel/irq/msi.c b/kernel/irq/msi.c
index 396a067a8a56..98771a0b7d70 100644
--- a/kernel/irq/msi.c
+++ b/kernel/irq/msi.c
@@ -756,12 +756,33 @@ static int msi_domain_translate(struct irq_domain
*domain, struct irq_fwspec *fw
return info->ops->msi_translate(domain, fwspec, hwirq, type);
}
+static void msi_domain_debug_show(struct seq_file *m, struct irq_domain *d,
+ struct irq_data *irqd, int ind)
+{
+ struct msi_desc *desc;
+ bool is_msix;
+
+ desc = irq_get_msi_desc(irqd->irq);
+ if (!desc)
+ return;
+
+ is_msix = desc->pci.msi_attrib.is_msix;
+ seq_printf(m, "%*s%s:", ind, "", is_msix ? "msix" : "msi");
+ seq_printf(m, "\n%*saddress_hi: 0x%08x", ind + 1, "",
+ desc->msg.address_hi);
+ seq_printf(m, "\n%*saddress_lo: 0x%08x", ind + 1, "",
+ desc->msg.address_lo);
+ seq_printf(m, "\n%*smsg_data: 0x%08x\n", ind + 1, "",
+ desc->msg.data);
+}
+
static const struct irq_domain_ops msi_domain_ops = {
.alloc = msi_domain_alloc,
.free = msi_domain_free,
.activate = msi_domain_activate,
.deactivate = msi_domain_deactivate,
.translate = msi_domain_translate,
+ .debug_show = msi_domain_debug_show,
};
static irq_hw_number_t msi_domain_ops_get_hwirq(struct msi_domain_info
*info,
e.g.
root@...t:/sys/kernel/debug/irq/irqs# cat /proc/interrupts | grep ITS
85: 0 0 0 0 0 0
0 0 0 0 0 0
ITS-MSI 75497472 Edge PCIe PME, aerdrv
86: 0 30 0 0 0 0
0 0 0 0 0 0
ITS-MSI 76021760 Edge nvme0q0
87: 287 0 0 0 0 0
0 0 0 0 0 0
ITS-MSI 76021761 Edge nvme0q1
88: 0 265 0 0 0 0
0 0 0 0 0 0
ITS-MSI 76021762 Edge nvme0q2
89: 0 0 177 0 0 0
0 0 0 0 0 0
ITS-MSI 76021763 Edge nvme0q3
90: 0 0 0 76 0 0
0 0 0 0 0 0
ITS-MSI 76021764 Edge nvme0q4
91: 0 0 0 0 161 0
0 0 0 0 0 0
ITS-MSI 76021765 Edge nvme0q5
92: 0 0 0 0 0 991
0 0 0 0 0 0
ITS-MSI 76021766 Edge nvme0q6
93: 0 0 0 0 0 0
194 0 0 0 0 0
ITS-MSI 76021767 Edge nvme0q7
94: 0 0 0 0 0 0
0 94 0 0 0 0
ITS-MSI 76021768 Edge nvme0q8
95: 0 0 0 0 0 0
0 0 148 0 0 0
ITS-MSI 76021769 Edge nvme0q9
96: 0 0 0 0 0 0
0 0 0 261 0 0
ITS-MSI 76021770 Edge nvme0q10
97: 0 0 0 0 0 0
0 0 0 0 127 0
ITS-MSI 76021771 Edge nvme0q11
98: 0 0 0 0 0 0
0 0 0 0 0 317
ITS-MSI 76021772 Edge nvme0q12
root@...t:/sys/kernel/debug/irq/irqs#
root@...t:/sys/kernel/debug/irq/irqs# cat 87
handler: handle_fasteoi_irq
device: 0000:91:00.0
status: 0x00000000
istate: 0x00004000
ddepth: 0
wdepth: 0
dstate: 0x31600200
IRQD_ACTIVATED
IRQD_IRQ_STARTED
IRQD_SINGLE_TARGET
IRQD_AFFINITY_MANAGED
IRQD_AFFINITY_ON_ACTIVATE
IRQD_HANDLE_ENFORCE_IRQCTX
node: 0
affinity: 0
effectiv: 0
domain: :soc@0:interrupt-controller@...01000:its@...50000-3
hwirq: 0x4880001
chip: ITS-MSI
flags: 0x20
IRQCHIP_ONESHOT_SAFE
msix:
address_hi: 0x00000000
address_lo: 0x0e060040
msg_data: 0x00000001
parent:
domain: :soc@0:interrupt-controller@...01000:its@...50000-5
hwirq: 0x2002
chip: ITS
flags: 0x0
parent:
domain: :soc@0:interrupt-controller@...01000-1
hwirq: 0x2002
chip: GICv3
flags: 0x15
IRQCHIP_SET_TYPE_MASKED
IRQCHIP_MASK_ON_SUSPEND
IRQCHIP_SKIP_SET_WAKE
Powered by blists - more mailing lists