On 16.03.2025 11:08, Andrii Sultanov wrote: > Following a similar change to amd_iommu struct, change the > find_iommu_for_device function to take pci_sbdf_t as a single parameter. > This avoids splitting a single argument into two registers in the majority > of cases. > > Bloat-o-meter reports (on top of the first patch in the series): > add/remove: 0/0 grow/shrink: 11/13 up/down: 90/-154 (-64) > Function old new delta > amd_iommu_get_supported_ivhd_type 54 86 +32 > parse_ivrs_table 3955 3966 +11 > amd_iommu_assign_device 271 282 +11 > __mon_lengths 2928 2936 +8 > update_intremap_entry_from_msi_msg 859 864 +5 > iov_supports_xt 270 275 +5 > amd_setup_hpet_msi 232 237 +5 > amd_iommu_domain_destroy 43 48 +5 > find_iommu_for_device 242 246 +4 > amd_iommu_ioapic_update_ire 572 575 +3 > allocate_domain_resources 82 83 +1 > amd_iommu_read_ioapic_from_ire 347 344 -3 > reassign_device 843 838 -5 > amd_iommu_remove_device 352 347 -5 > amd_iommu_get_reserved_device_memory 524 519 -5 > amd_iommu_flush_iotlb 359 354 -5 > amd_iommu_add_device 844 839 -5 > amd_iommu_setup_domain_device 1478 1472 -6 > build_info 752 744 -8 > amd_iommu_detect_one_acpi 886 876 -10 > register_range_for_device 297 281 -16 > amd_iommu_msi_msg_update_ire 472 448 -24 > parse_ivmd_block 1339 1312 -27 > _hvm_dpci_msi_eoi 168 133 -35 > > Signed-off-by: Andrii Sultanov <sultanovand...@gmail.com> > > --- >> This avoids splitting a single argument into two registers > And adds a few conversions into a pci_sbdf_t... these are addressed further > in the series. - not part of the commit message in case said commit isn't > taken.
What is "said commit" here? Everything done in a patch should be in the commit message. If parts are later dropped, the commit message would need refining. With the commit message amended (assuming I guessed correctly that this is about a few secondary changes done here): Acked-by: Jan Beulich <jbeul...@suse.com> Jan
