All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Cédric Le Goater" <clg@redhat.com>
To: Aditya Gupta <adityag@linux.ibm.com>
Cc: qemu-devel@nongnu.org, qemu-ppc@nongnu.org,
	"Harsh Prateek Bora" <harshpb@linux.ibm.com>,
	"Philippe Mathieu-Daudé" <philmd@oss.qualcomm.com>,
	"Shivang Upadhyay" <shivangu@linux.ibm.com>,
	"Daniel P. Berrangé" <berrange@redhat.com>,
	"Sourabh Jain" <sourabhjain@linux.ibm.com>,
	"Hari Bathini" <hbathini@linux.ibm.com>,
	"Nicholas Piggin" <npiggin@gmail.com>,
	"Miles Glenn" <milesg@linux.ibm.com>,
	"Fabiano Rosas" <farosas@suse.de>,
	"Laurent Vivier" <lvivier@redhat.com>,
	"Paolo Bonzini" <pbonzini@redhat.com>,
	devel@lists.libvirt.org,
	"Misbah Anjum N" <misanjum@linux.ibm.com>,
	"Anushree Mathur" <anushree.mathur@linux.ibm.com>,
	"Pierrick Bouvier" <pierrick.bouvier@oss.qualcomm.com>,
	kvm@vger.kernel.org, "Gautam Menghani" <gautam@linux.ibm.com>,
	"Chinmay Rath" <rathc@linux.ibm.com>
Subject: Re: [PATCH v3 1/8] tests/functional: Add remote interrupts test for PowerNV
Date: Tue, 30 Jun 2026 13:13:30 +0200	[thread overview]
Message-ID: <dcffa5bb-c092-422c-8685-6f247ffed004@redhat.com> (raw)
In-Reply-To: <b489fbd0-a1f8-429a-90f0-9ed950cfda1a@linux.ibm.com>

On 6/30/26 13:04, Aditya Gupta wrote:
> 
> On 29/06/26 20:13, Cédric Le Goater wrote:
>> On 6/29/26 13:39, Aditya Gupta wrote:
>>> On 26/06/29 01:20PM, Cédric Le Goater wrote:
>>>> On 6/29/26 13:04, Aditya Gupta wrote:
>>>>> In the past there have been hard to recreate issues where XIVE changes
>>>>> cause qemu crashes due to multi-socket interrupts such as in [1].
>>>>>
>>>>> Add a functional test explicitly to test whether remote interrupts work.
>>>>>
>>>>> The test can also work as additional boot test for multi-socket boot,
>>>>> initrd boot test, as well as a check for e1000e to be working in powernv,
>>>>
>>>> you should consider moving to a more modern nic model: igb, which
>>>> has sr-iov support too.
>>>
>>> yes, can do. with igb i see multiple irqs, so test will have to take sum
>>> of all rx irqs, and compare:
>>>
>>>     2026-06-29 17:00:46,288: cat /proc/interrupts | grep eth0
>>>     2026-06-29 17:00:46,391: 135:          0          0 0 PNV-PCI-MSI 2416443392 Edge      eth0
>>>     2026-06-29 17:00:46,393: 136:          0          6 0 PNV-PCI-MSI 2416443393 Edge      eth0-rx-0
>>>     2026-06-29 17:00:46,394: 137:          0         14 0 PNV-PCI-MSI 2416443394 Edge      eth0-rx-1
>>>     2026-06-29 17:00:46,395: 138:          0          4 0 PNV-PCI-MSI 2416443395 Edge      eth0-rx-2
>>>     2026-06-29 17:00:46,397: 139:          0          4 0 PNV-PCI-MSI 2416443396 Edge      eth0-rx-3
>>>     2026-06-29 17:00:46,398: 140:          0         16 0 PNV-PCI-MSI 2416443397 Edge      eth0-tx-0
>>>     2026-06-29 17:00:46,399: 141:          0          6 0 PNV-PCI-MSI 2416443398 Edge      eth0-tx-1
>>>     2026-06-29 17:00:46,399: 142:          0          6 0 PNV-PCI-MSI 2416443399 Edge      eth0-tx-2
>>>     2026-06-29 17:00:46,400: 143:          0          4 0 PNV-PCI-MSI 2416443400 Edge      eth0-tx-3
>>>
>>> I feel e1000e suffices for this test, should i stay with e1000e or use
>>> igb here ?
>>
>> or use a single cpu per socket config. Which seems better for the test.
> 
> 
> The test uses single cpu per socket, since i have `-smp 4,sockets=4,threads=1`. Was this what you meant ?

yes. or cores=1, as you wish.

Thanks,

C.



  reply	other threads:[~2026-06-30 11:14 UTC|newest]

Thread overview: 21+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-06-29 11:04 [PATCH v3 0/8] Testcase improvements and Power8E/NVL removal Aditya Gupta
2026-06-29 11:04 ` [PATCH v3 1/8] tests/functional: Add remote interrupts test for PowerNV Aditya Gupta
2026-06-29 11:20   ` Cédric Le Goater
2026-06-29 11:39     ` Aditya Gupta
2026-06-29 14:43       ` Cédric Le Goater
2026-06-30 11:04         ` Aditya Gupta
2026-06-30 11:13           ` Cédric Le Goater [this message]
2026-06-30 11:16             ` Aditya Gupta
2026-06-30 12:54               ` Cédric Le Goater
2026-06-30 11:13           ` Aditya Gupta
2026-06-29 11:04 ` [PATCH v3 2/8] tests/qtest/pnv_spi: Test Power11 PNV_SPI Aditya Gupta
2026-06-29 11:04 ` [PATCH v3 3/8] tests/qtest: Add Power11 chip & machine to qtests Aditya Gupta
2026-06-29 11:18   ` Cédric Le Goater
2026-06-29 11:28     ` Aditya Gupta
2026-06-29 11:04 ` [PATCH v3 4/8] tests/functional: Use default powernv machine instead of power10 Aditya Gupta
2026-06-29 11:04 ` [PATCH v3 5/8] ppc/pnv: Replace Power8E with Power11 for 'none' machine test Aditya Gupta
2026-06-29 11:04 ` [PATCH v3 6/8] ppc/pnv: Remove Power8E and Power8NVL pnv chips Aditya Gupta
2026-06-29 11:04 ` [PATCH v3 7/8] ppc/pnv: Remove Power8E and Power8NVL CPUs Aditya Gupta
2026-06-29 11:04 ` [PATCH v3 8/8] MAINTAINERS: Add self as maintainer for PowerNV Aditya Gupta
2026-06-30  4:19 ` [PATCH v3 0/8] Testcase improvements and Power8E/NVL removal Misbah Anjum N
2026-06-30 18:34   ` Aditya Gupta

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=dcffa5bb-c092-422c-8685-6f247ffed004@redhat.com \
    --to=clg@redhat.com \
    --cc=adityag@linux.ibm.com \
    --cc=anushree.mathur@linux.ibm.com \
    --cc=berrange@redhat.com \
    --cc=devel@lists.libvirt.org \
    --cc=farosas@suse.de \
    --cc=gautam@linux.ibm.com \
    --cc=harshpb@linux.ibm.com \
    --cc=hbathini@linux.ibm.com \
    --cc=kvm@vger.kernel.org \
    --cc=lvivier@redhat.com \
    --cc=milesg@linux.ibm.com \
    --cc=misanjum@linux.ibm.com \
    --cc=npiggin@gmail.com \
    --cc=pbonzini@redhat.com \
    --cc=philmd@oss.qualcomm.com \
    --cc=pierrick.bouvier@oss.qualcomm.com \
    --cc=qemu-devel@nongnu.org \
    --cc=qemu-ppc@nongnu.org \
    --cc=rathc@linux.ibm.com \
    --cc=shivangu@linux.ibm.com \
    --cc=sourabhjain@linux.ibm.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.