On Mon, 2014-04-14 at 15:22 -0600, Alex Williamson wrote:
>
> > + if (dev_tmp) {
> > + if (pci_is_pcie(dev_tmp)) {
> > + bridge_bus = dev_tmp->subordinate->number;
> > + bridge_devfn = 0;
> > + } else {
> > + bridge_bus = dev_tmp->bus->number;
> > + bridge_devfn = dev_tmp->devfn;
> > + }
> > + spin_lock_irqsave(&device_domain_lock, flags);
> > + info = dmar_search_domain_by_dev_info(segment, bus,
> > devfn);
>
>
> bus and devfn are uninitialized here, CID 1197747 & 1197746. Thanks,Oops. That should be using bridge_bus and bridge_devfn, shouldn't it? Will fix; thanks. -- David Woodhouse Open Source Technology Centre [email protected] Intel Corporation
smime.p7s
Description: S/MIME cryptographic signature
_______________________________________________ iommu mailing list [email protected] https://lists.linuxfoundation.org/mailman/listinfo/iommu
