On 2/2/21 9:55 PM, Joerg Roedel wrote:
On Tue, Feb 02, 2021 at 12:40:57PM +0800, Lu Baolu wrote:
+ list_for_each_entry_rcu(satcu, &dmar_satc_units, list) {
+ satc = container_of(satcu->hdr, struct acpi_dmar_satc, header);
+ if (satc->segment == pci_domain_nr(dev->bus) &&
satcu->atc_required) {
You can safe a level of indentation and make this look nicer if you do:
if (satc->segment != pci_domain_nr(dev->bus) ||
!satcu->atc_required)
continue;
Yes. Thanks!
Best regards,
baolu
+ for_each_dev_scope(satcu->devices, satcu->devices_cnt,
i, tmp)
+ if (to_pci_dev(tmp) == dev)
+ goto out;
+ }
+ }
+ ret = 0;
+out:
+ rcu_read_unlock();
+ return ret;
+}