linux-kernel - Re: [PATCH] Unbreak MSI on ATI devices

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:	Thu, 21 Dec 2006 03:21:37 -0500
From:	Jeff Garzik <jeff@...zik.org>
To:	Petr Vandrovec <petr@...drovec.name>
CC:	linux-kernel@...r.kernel.org, akpm@...l.org,
	Linus Torvalds <torvalds@...l.org>
Subject: Re: [PATCH] Unbreak MSI on ATI devices

Petr Vandrovec wrote:
> After poking around I've found that problem is that at least ATI USB-HCDs
> apply INTX enable even for MSI, despite warning in the PCI specification that
> it should apply only to MSI (actually I have feeling that on these USB devices 
> disabling INTX in MSI mode drives their INTA# line active as when ohci1394 
> module got loaded kernel complained about interrupt being continuously 
> activated for no good reason (TI's 7421 is one of few MSI-incapable devices
> in my box).
> 
> So my question is - what is real reason for disabling INTX when in MSI mode?
> According to PCI spec it should not be needed, and it hurts at least chips
> listed below:

> Do not disable INTX in MSI mode.  It breaks ATI USB HCDs (both OHCI and EHCI).
> 
> Signed-off-by: Petr Vandrovec <petr@...drovec.name>
> 
> diff -uprdN linux/drivers/pci/msi.c linux/drivers/pci/msi.c
> --- linux/drivers/pci/msi.c	2006-12-16 13:34:52.000000000 -0800
> +++ linux/drivers/pci/msi.c	2006-12-20 23:18:10.000000000 -0800
> @@ -256,7 +256,7 @@ static void enable_msi_mode(struct pci_d
>  		dev->msix_enabled = 1;
>  	}
>  
> -	pci_intx(dev, 0);  /* disable intx */
> +	pci_intx(dev, 1);  /* enable intx, on some devices it affects MSI as well */
>  }

I'm just going to CC Linus, and run ;-)

More seriously.  Some other chips choke if you forget to disable INTx, 
before going into MSI mode.

Thus, turning off one irq source before turning on another is the most 
logical course of action.

I suppose we'll have to quirk ATI for being dumb...?

	Jeff


-
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/