Re: [PATCH v2 2/2] Fix "irq [-a|-s]" options on Linux 6.5-rc1 and later

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 2023/07/12 18:54, lijiang wrote:
> Thank you for the update, Kazu.
> On Wed, Jul 12, 2023 at 4:55 PM <crash-utility-request@xxxxxxxxxx> wrote:
> 
>> Date: Wed, 12 Jul 2023 08:55:29 +0000
>> From: HAGIO KAZUHITO(?????)  <k-hagio-ab@xxxxxxx>
>> To: "crash-utility@xxxxxxxxxx" <crash-utility@xxxxxxxxxx>
>> Cc: "lijiang@xxxxxxxxxx" <lijiang@xxxxxxxxxx>
>> Subject:  [PATCH v2 2/2] Fix "irq [-a|-s]" options on
>>          Linux 6.5-rc1 and later
>> Message-ID: <1689152127-28057-1-git-send-email-k-hagio-ab@xxxxxxx>
>> Content-Type: text/plain; charset="iso-2022-jp"
>>
>> From: Kazuhito Hagio <k-hagio-ab@xxxxxxx>
>>
>> Kernel commit 721255b982 ("genirq: Use a maple tree for interrupt
>> descriptor management"), which is contained in Linux 6.5-rc1 and later
>> kernels, replaced irq_desc_tree with a maple tree sparse_irqs.
>>
>> Without the patch, "irq [-a|-s]" options fail with an error, e.g. the
>> following on x86_64, on kernels configured with CONFIG_SPARSE_IRQ=y.
>>
>>    crash> irq
>>    irq: x86_64_dump_irq: irq_desc[] or irq_desc_tree do not exist?
>>
>> Signed-off-by: Kazuhito Hagio <k-hagio-ab@xxxxxxx>
>> ---
>> v1 -> v2
>> - Fixed a typo in the commit log: s/SPARSE_IRQS/SPARSE_IRQ/
>> - Reverted the unintendedly removed line in x86_64_show_interrupts()
>> - Added sparse_irqs support to ia64_dump_irq()
>>
>>
> The v2 looks good to me. For the patches: Ack
> [1] [PATCH 1/2] Exclude NULL entries from do_maple_tree() return value
> <https://listman.redhat.com/archives/crash-utility/2023-July/010832.html>
> [2] [PATCH v2 2/2] Fix "irq [-a|-s]" options on Linux 6.5-rc1 and later
> <https://listman.redhat.com/archives/crash-utility/2023-July/010847.html>

Thanks, applied.

Kazu
--
Crash-utility mailing list
Crash-utility@xxxxxxxxxx
https://listman.redhat.com/mailman/listinfo/crash-utility
Contribution Guidelines: https://github.com/crash-utility/crash/wiki




[Index of Archives]     [Fedora Development]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]

 

Powered by Linux