On 12/09/2024 3:20 pm, Jan Beulich wrote:
> On 12.09.2024 14:51, Andrew Cooper wrote:
>> @@ -596,6 +594,8 @@ int __init iommu_setup(void)
>> }
>> else
>> {
>> + register_keyhandler('o', &iommu_dump_page_tables, "dump iommu page
>> tables", 0);
>> +
>> if ( iommu_quarantine_init() )
>> panic("Could not set up quarantine\n");
> It's a little odd to have this immediately ahead of something that can
> panic(), but not a big deal of course. The line wants wrapping though,
> to stay within 80 cols.
I'm happy to put it somewhere else if you can suggest somewhere better.
I was just looking for somewhere which was clearly behind an
iommu_enabled check, as it was in iommu_hwdom_init().
~Andrew
