lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Date:	Tue, 09 Oct 2012 12:35:39 -0600
From:	Alex Williamson <alex.williamson@...hat.com>
To:	Florian Dazinger <florian@...inger.net>
Cc:	Joerg.Roedel@....com, iommu@...ts.linux-foundation.org,
	linux-kernel@...r.kernel.org
Subject: Re: [RFC PATCH 0/5] amd_iommu: Refactor IOMMU group and support
 virtual aliases

On Tue, 2012-10-09 at 20:27 +0200, Florian Dazinger wrote:
> Am Mon, 08 Oct 2012 22:49:28 -0600
> schrieb Alex Williamson <alex.williamson@...hat.com>:
> 
> > This series is meant to refactor IOMMU group support in amd_iommu
> > to properly support virtual aliases.  If multiple devices alias to
> > the same virtual alias, they should be grouped together.  This code
> > also verifies whether the alias should be the root of the group vs
> > devices above the alias.
> > 
> > This seems to do the right thing on my system, but that's not saying
> > a lot since it doesn't do anything interesting with aliases.  I'd
> > appreciate if Joerg and Florian could test this on their systems.
> > Thanks,
> > 
> > Alex
> > 
> > ---
> > 
> > Alex Williamson (5):
> >       amd_iommu: Properly account for virtual aliases in IOMMU groups
> >       amd_iommu: Split IOMMU group allocation and attach
> >       amd_iommu: Split upstream bus device lookup
> >       amd_iommu: Split IOMMU Group topology walk
> >       amd_iommu: Split IOMMU group initialization
> > 
> > 
> >  drivers/iommu/amd_iommu.c       |  184 ++++++++++++++++++++++++++++++---------
> >  drivers/iommu/amd_iommu_types.h |    1 
> >  2 files changed, 142 insertions(+), 43 deletions(-)
> > 
> 
> patch #5 does not apply cleanly on top of 3.6, I had to edit
> amd_iommu_types.h manually (easy enough), apart from that, it is
> working fine!

Thanks Florian.  Would you mind also reporting
'find /sys/kernel/iommu_groups' for both stock 3.6 and with this series
applied?  I think we want to find that devices 07:00.0 and 08:04.0 are
grouped together and I'd like to verify whether that actually works.
Thanks,

Alex

> dmesg:
> [    1.475665] pci 0000:01:00.0: Boot video device
> [    1.475762] PCI: CLS 64 bytes, default 64
> [    1.478371] AMD-Vi: device: 00:00.2 cap: 0040 seg: 0 flags: 3e info 1300
> [    1.478409] AMD-Vi:        mmio-addr: 00000000feb20000
> [    1.478487] AMD-Vi:   DEV_SELECT_RANGE_START  devid: 00:00.0 flags: 00
> [    1.478521] AMD-Vi:   DEV_RANGE_END           devid: 00:00.2
> [    1.478555] AMD-Vi:   DEV_SELECT                      devid: 00:02.0 flags: 00
> [    1.478588] AMD-Vi:   DEV_SELECT_RANGE_START  devid: 01:00.0 flags: 00
> [    1.478622] AMD-Vi:   DEV_RANGE_END           devid: 01:00.1
> [    1.478655] AMD-Vi:   DEV_SELECT                      devid: 00:04.0 flags: 00
> [    1.478688] AMD-Vi:   DEV_SELECT                      devid: 02:00.0 flags: 00
> [    1.478722] AMD-Vi:   DEV_SELECT                      devid: 00:05.0 flags: 00
> [    1.478755] AMD-Vi:   DEV_SELECT                      devid: 03:00.0 flags: 00
> [    1.478788] AMD-Vi:   DEV_SELECT                      devid: 00:06.0 flags: 00
> [    1.478822] AMD-Vi:   DEV_SELECT                      devid: 04:00.0 flags: 00
> [    1.478855] AMD-Vi:   DEV_SELECT                      devid: 00:07.0 flags: 00
> [    1.478888] AMD-Vi:   DEV_SELECT                      devid: 05:00.0 flags: 00
> [    1.478921] AMD-Vi:   DEV_SELECT                      devid: 00:09.0 flags: 00
> [    1.478955] AMD-Vi:   DEV_SELECT                      devid: 06:00.0 flags: 00
> [    1.478988] AMD-Vi:   DEV_SELECT                      devid: 00:0d.0 flags: 00
> [    1.479021] AMD-Vi:   DEV_SELECT                      devid: 07:00.0 flags: 00
> [    1.479055] AMD-Vi:   DEV_ALIAS_RANGE                 devid: 08:01.0 flags: 00 devid_to: 08:00.0
> [    1.479091] AMD-Vi:   DEV_RANGE_END           devid: 08:1f.7
> [    1.479128] AMD-Vi:   DEV_SELECT                      devid: 00:11.0 flags: 00
> [    1.479162] AMD-Vi:   DEV_SELECT_RANGE_START  devid: 00:12.0 flags: 00
> [    1.479195] AMD-Vi:   DEV_RANGE_END           devid: 00:12.2
> [    1.479229] AMD-Vi:   DEV_SELECT_RANGE_START  devid: 00:13.0 flags: 00
> [    1.479262] AMD-Vi:   DEV_RANGE_END           devid: 00:13.2
> [    1.479295] AMD-Vi:   DEV_SELECT                      devid: 00:14.0 flags: d7
> [    1.479329] AMD-Vi:   DEV_SELECT                      devid: 00:14.3 flags: 00
> [    1.479362] AMD-Vi:   DEV_SELECT                      devid: 00:14.4 flags: 00
> [    1.479396] AMD-Vi:   DEV_ALIAS_RANGE                 devid: 09:00.0 flags: 00 devid_to: 00:14.4
> [    1.479432] AMD-Vi:   DEV_RANGE_END           devid: 09:1f.7
> [    1.479478] AMD-Vi:   DEV_SELECT                      devid: 00:14.5 flags: 00
> [    1.479513] AMD-Vi:   DEV_SELECT_RANGE_START  devid: 00:16.0 flags: 00
> [    1.479547] AMD-Vi:   DEV_RANGE_END           devid: 00:16.2
> [    1.533745] AMD-Vi: Found IOMMU at 0000:00:00.2 cap 0x40
> [    1.533781] 
> [    1.533830] pci 0000:00:00.2: irq 72 for MSI/MSI-X
> [    1.542957] AMD-Vi: Lazy IO/TLB flushing enabled
> 
> lspci:
> 00:00.0 Host bridge: Advanced Micro Devices [AMD] nee ATI RD890 PCI to PCI bridge (external gfx0 port B) (rev 02)
> 00:00.2 IOMMU: Advanced Micro Devices [AMD] nee ATI RD990 I/O Memory Management Unit (IOMMU)
> 00:02.0 PCI bridge: Advanced Micro Devices [AMD] nee ATI RD890 PCI to PCI bridge (PCI express gpp port B)
> 00:04.0 PCI bridge: Advanced Micro Devices [AMD] nee ATI RD890 PCI to PCI bridge (PCI express gpp port D)
> 00:05.0 PCI bridge: Advanced Micro Devices [AMD] nee ATI RD890 PCI to PCI bridge (PCI express gpp port E)
> 00:06.0 PCI bridge: Advanced Micro Devices [AMD] nee ATI RD890 PCI to PCI bridge (PCI express gpp port F)
> 00:07.0 PCI bridge: Advanced Micro Devices [AMD] nee ATI RD890 PCI to PCI bridge (PCI express gpp port G)
> 00:09.0 PCI bridge: Advanced Micro Devices [AMD] nee ATI RD890 PCI to PCI bridge (PCI express gpp port H)
> 00:0d.0 PCI bridge: Advanced Micro Devices [AMD] nee ATI RD890 PCI to PCI bridge (external gfx1 port B)
> 00:11.0 SATA controller: Advanced Micro Devices [AMD] nee ATI SB7x0/SB8x0/SB9x0 SATA Controller [AHCI mode] (rev 40)
> 00:12.0 USB controller: Advanced Micro Devices [AMD] nee ATI SB7x0/SB8x0/SB9x0 USB OHCI0 Controller
> 00:12.2 USB controller: Advanced Micro Devices [AMD] nee ATI SB7x0/SB8x0/SB9x0 USB EHCI Controller
> 00:13.0 USB controller: Advanced Micro Devices [AMD] nee ATI SB7x0/SB8x0/SB9x0 USB OHCI0 Controller
> 00:13.2 USB controller: Advanced Micro Devices [AMD] nee ATI SB7x0/SB8x0/SB9x0 USB EHCI Controller
> 00:14.0 SMBus: Advanced Micro Devices [AMD] nee ATI SBx00 SMBus Controller (rev 42)
> 00:14.3 ISA bridge: Advanced Micro Devices [AMD] nee ATI SB7x0/SB8x0/SB9x0 LPC host controller (rev 40)
> 00:14.4 PCI bridge: Advanced Micro Devices [AMD] nee ATI SBx00 PCI to PCI Bridge (rev 40)
> 00:14.5 USB controller: Advanced Micro Devices [AMD] nee ATI SB7x0/SB8x0/SB9x0 USB OHCI2 Controller
> 00:16.0 USB controller: Advanced Micro Devices [AMD] nee ATI SB7x0/SB8x0/SB9x0 USB OHCI0 Controller
> 00:16.2 USB controller: Advanced Micro Devices [AMD] nee ATI SB7x0/SB8x0/SB9x0 USB EHCI Controller
> 00:18.0 Host bridge: Advanced Micro Devices [AMD] Family 10h Processor HyperTransport Configuration
> 00:18.1 Host bridge: Advanced Micro Devices [AMD] Family 10h Processor Address Map
> 00:18.2 Host bridge: Advanced Micro Devices [AMD] Family 10h Processor DRAM Controller
> 00:18.3 Host bridge: Advanced Micro Devices [AMD] Family 10h Processor Miscellaneous Control
> 00:18.4 Host bridge: Advanced Micro Devices [AMD] Family 10h Processor Link Control
> 01:00.0 VGA compatible controller: Advanced Micro Devices [AMD] nee ATI RV730XT [Radeon HD 4670]
> 01:00.1 Audio device: Advanced Micro Devices [AMD] nee ATI RV710/730 HDMI Audio [Radeon HD 4000 series]
> 02:00.0 SATA controller: ASMedia Technology Inc. ASM1062 Serial ATA Controller (rev 01)
> 03:00.0 Ethernet controller: Intel Corporation 82583V Gigabit Network Connection
> 04:00.0 USB controller: ASMedia Technology Inc. ASM1042 SuperSpeed USB Host Controller
> 05:00.0 USB controller: ASMedia Technology Inc. ASM1042 SuperSpeed USB Host Controller
> 06:00.0 USB controller: ASMedia Technology Inc. ASM1042 SuperSpeed USB Host Controller
> 07:00.0 PCI bridge: PLX Technology, Inc. PEX8112 x1 Lane PCI Express-to-PCI Bridge (rev aa)
> 08:04.0 Multimedia audio controller: C-Media Electronics Inc CMI8788 [Oxygen HD Audio]



--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ