Re: 2.6.13.1 locks machine after some time, 2.6.12.5 work fine
From: Adam Kropelin (akropel1_at_rochester.rr.com)
Date: 09/14/05
- Previous message: Dmitrij Bogush: "snd-via82xx-modem do not work from 2.6.12 kernel version"
- In reply to: David S. Miller: "Re: 2.6.13.1 locks machine after some time, 2.6.12.5 work fine"
- Next in thread: Adam Kropelin: "[PATCH] ibmphp: Use dword accessors for PCI_ROM_ADDRESS"
- Reply: Adam Kropelin: "[PATCH] ibmphp: Use dword accessors for PCI_ROM_ADDRESS"
- Reply: Adam Kropelin: "[PATCH] pciehp: Use dword accessors for PCI_ROM_ADDRESS"
- Reply: Adam Kropelin: "[PATCH] shpchp: Use dword accessors for PCI_ROM_ADDRESS"
- Reply: Adam Kropelin: "[PATCH] qla2xxx: Use dword accessors for PCI_ROM_ADDRESS"
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]
Date: Tue, 13 Sep 2005 18:25:50 -0400 To: "David S. Miller" <davem@davemloft.net>
On Tue, Sep 13, 2005 at 01:22:13PM -0700, David S. Miller wrote:
> From: Linus Torvalds <torvalds@osdl.org>
> Date: Tue, 13 Sep 2005 08:55:31 -0700 (PDT)
>
> > > /* Reset expansion ROM address decode enable */
> > > pci_read_config_word(ha->pdev, PCI_ROM_ADDRESS, &w);
> > > w &= ~PCI_ROM_ADDRESS_ENABLE;
> > > pci_write_config_word(ha->pdev, PCI_ROM_ADDRESS, w);
> ...
> > So the above probably works fine, especially since it's just disabling the
> > ROM (ie we don't end up caring at all about the upper bits even if they
> > did get the wrong value). But it's definitely bad practice, and there are
> > probably cards (for which that driver is irrelevant, of course ;) where
> > doing something like the above might not work at all.
>
> I think for consistency the above driver case should still be fixed,
> however. This way when people try to audit the tree for
> PCI_ROM_ADDRESS config space accesses, they won't come across this
> same instance again and again.
Agreed. I'll follow up with patches for the relevant maintainers.
--Adam
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
- Previous message: Dmitrij Bogush: "snd-via82xx-modem do not work from 2.6.12 kernel version"
- In reply to: David S. Miller: "Re: 2.6.13.1 locks machine after some time, 2.6.12.5 work fine"
- Next in thread: Adam Kropelin: "[PATCH] ibmphp: Use dword accessors for PCI_ROM_ADDRESS"
- Reply: Adam Kropelin: "[PATCH] ibmphp: Use dword accessors for PCI_ROM_ADDRESS"
- Reply: Adam Kropelin: "[PATCH] pciehp: Use dword accessors for PCI_ROM_ADDRESS"
- Reply: Adam Kropelin: "[PATCH] shpchp: Use dword accessors for PCI_ROM_ADDRESS"
- Reply: Adam Kropelin: "[PATCH] qla2xxx: Use dword accessors for PCI_ROM_ADDRESS"
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]
Relevant Pages
|
|
