| 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
| ||
|
Message-Id: <200806111638.50933.rjw@sisk.pl>
Date: Wed, 11 Jun 2008 16:38:50 +0200
From: "Rafael J. Wysocki" <rjw@...k.pl>
To: Patrick <ragamuffin@...acomm.ch>
Cc: Pavel Machek <pavel@...e.cz>, linux-kernel@...r.kernel.org,
Ingo Molnar <mingo@...e.hu>,
pm list <linux-pm@...ts.linux-foundation.org>,
Andi Kleen <andi@...stfloor.org>
Subject: Re: >3G => iommu => suspend problems -- was Re: SB600 AHCI: Hard Disk Corruption
On Wednesday, 11 of June 2008, Patrick wrote:
> Hello
Hi,
> On Mon, 2008-06-09 at 00:36 +0200, Rafael J. Wysocki wrote:
>
> > > > It's on top of the patch that adds the GART sysdev.
>
> > Updated patch follows. It has been tested a little on my new 4 GB test box on
> > which 2.6.26-rc4 failed miserably with severe consequences. More testing
> > welcome, but please be careful.
>
> No risk no fun! :-)
>
> As the suspend-vs-iommu-prevent-suspend-if-we-could-not-resume.patch,
> where the GART sysdev is added, is now included in 2.6.26-rc5, I was
> able to apply this one seamlessly to it (-rc5 tree) and have just
> rebooted *, suspended to disk and ram once wiht all filesystems (>400GB)
> mounted rw ** and everything is _working fine_.
>
> As usual, a kernel log, the applied patch and a test report are
> available at http://zefir.890m.com/kernel-testing/ .
Thanks for the testing!
Well, I was hoping to be able to get this patch into 2.6.26, as I don't really
like the temporary hack, preventing the affected systems from resuming
at all, that we have in there, but it seems to be too late. :-(
Hopefully, we'll get it into 2.6.27.
Thanks,
Rafael
> > ---
> > Add resume handling to GART IOMMU.
> >
> > Signed-off-by: Rafael J. Wysocki <rjw@...k.pl>
> > ---
> > arch/x86/kernel/aperture_64.c | 2 +
> > arch/x86/kernel/pci-gart_64.c | 75 +++++++++++++++++++++++++++++++++---------
> > include/asm-x86/gart.h | 1
> > 3 files changed, 62 insertions(+), 16 deletions(-)
> >
> > Index: linux-2.6/arch/x86/kernel/pci-gart_64.c
> > ===================================================================
> > --- linux-2.6.orig/arch/x86/kernel/pci-gart_64.c
> > +++ linux-2.6/arch/x86/kernel/pci-gart_64.c
> > @@ -549,14 +549,70 @@ static __init unsigned read_aperture(str
> > return aper_base;
> > }
> >
> > +static void enable_gart_translations(void)
> > +{
> > + int i;
> > +
> > + for (i = 0; i < num_k8_northbridges; i++) {
> > + struct pci_dev *dev;
> > + u32 gatt_reg;
> > + u32 ctl;
> > +
> > + dev = k8_northbridges[i];
> > + gatt_reg = __pa(agp_gatt_table) >> 12;
> > + gatt_reg <<= 4;
> > + pci_write_config_dword(dev, 0x98, gatt_reg);
> > + pci_read_config_dword(dev, 0x90, &ctl);
> > +
> > + ctl |= 1;
> > + ctl &= ~((1<<4) | (1<<5));
> > +
> > + pci_write_config_dword(dev, 0x90, ctl);
> > + }
> > +}
> > +
> > +/*
> > + * If fix_up_north_bridges is set, the north bridges have to be fixed up on
> > + * resume in the same way as they are handled in gart_iommu_hole_init().
> > + */
> > +static bool fix_up_north_bridges;
> > +static u32 aperture_order;
> > +static u32 aperture_alloc;
> > +
> > +void set_up_gart_resume(u32 aper_order, u32 aper_alloc)
> > +{
> > + fix_up_north_bridges = true;
> > + aperture_order = aper_order;
> > + aperture_alloc = aper_alloc;
> > +}
> > +
> > static int gart_resume(struct sys_device *dev)
> > {
> > + printk(KERN_INFO "PCI-DMA: Resuming GART IOMMU\n");
> > +
> > + if (fix_up_north_bridges) {
> > + int i;
> > +
> > + for (i = 0; i < num_k8_northbridges; i++) {
> > + struct pci_dev *dev = k8_northbridges[i];
> > +
> > + /*
> > + * Don't enable translations just yet. That is the next
> > + * step. Restore the pre-suspend aperture settings.
> > + */
> > + pci_write_config_dword(dev, 0x90, aperture_order << 1);
> > + pci_write_config_dword(dev, 0x94, aperture_alloc >> 25);
> > + }
> > + }
> > +
> > + enable_gart_translations();
> > +
> > return 0;
> > }
> >
> > static int gart_suspend(struct sys_device *dev, pm_message_t state)
> > {
> > - return -EINVAL;
> > + return 0;
> > }
> >
> > static struct sysdev_class gart_sysdev_class = {
> > @@ -614,27 +670,14 @@ static __init int init_k8_gatt(struct ag
> > memset(gatt, 0, gatt_size);
> > agp_gatt_table = gatt;
> >
> > - for (i = 0; i < num_k8_northbridges; i++) {
> > - u32 gatt_reg;
> > - u32 ctl;
> > -
> > - dev = k8_northbridges[i];
> > - gatt_reg = __pa(gatt) >> 12;
> > - gatt_reg <<= 4;
> > - pci_write_config_dword(dev, 0x98, gatt_reg);
> > - pci_read_config_dword(dev, 0x90, &ctl);
> > -
> > - ctl |= 1;
> > - ctl &= ~((1<<4) | (1<<5));
> > -
> > - pci_write_config_dword(dev, 0x90, ctl);
> > - }
> > + enable_gart_translations();
> >
> > error = sysdev_class_register(&gart_sysdev_class);
> > if (!error)
> > error = sysdev_register(&device_gart);
> > if (error)
> > panic("Could not register gart_sysdev -- would corrupt data on next suspend");
> > +
> > flush_gart();
> >
> > printk(KERN_INFO "PCI-DMA: aperture base @ %x size %u KB\n",
> > Index: linux-2.6/arch/x86/kernel/aperture_64.c
> > ===================================================================
> > --- linux-2.6.orig/arch/x86/kernel/aperture_64.c
> > +++ linux-2.6/arch/x86/kernel/aperture_64.c
> > @@ -413,4 +413,6 @@ void __init gart_iommu_hole_init(void)
> > write_pci_config(0, num, 3, 0x90, aper_order<<1);
> > write_pci_config(0, num, 3, 0x94, aper_alloc>>25);
> > }
> > +
> > + set_up_gart_resume(aper_order, aper_alloc);
> > }
> > Index: linux-2.6/include/asm-x86/gart.h
> > ===================================================================
> > --- linux-2.6.orig/include/asm-x86/gart.h
> > +++ linux-2.6/include/asm-x86/gart.h
> > @@ -11,6 +11,7 @@ extern void gart_iommu_shutdown(void);
> > extern void __init gart_parse_options(char *);
> > extern void early_gart_iommu_check(void);
> > extern void gart_iommu_hole_init(void);
> > +extern void set_up_gart_resume(u32, u32);
> > extern int fallback_aper_order;
> > extern int fallback_aper_force;
> > extern int gart_iommu_aperture;
--
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