[PATCH] Add MODULE_DEVICE_TABLE() so ioatdma module is autoloaded

[PATCH] Add MODULE_DEVICE_TABLE() so ioatdma module is autoloaded

[Roland Dreier]

The ioatdma module is missing aliases for the PCI devices it supports,
so it is not autoloaded on boot.  Add a MODULE_DEVICE_TABLE() to get
these aliases.

Signed-off-by: Roland Dreier <rolandd@cisco.com>
---
Not sure if not autoloading was intentional or if it's just an
oversight, but this patch seems useful to me, so please apply if it
makes sense to you.

 drivers/dma/ioat.c |    2 ++
 1 files changed, 2 insertions(+), 0 deletions(-)

diff --git a/drivers/dma/ioat.c b/drivers/dma/ioat.c
index 2225bb6..a1d8b8d 100644
--- a/drivers/dma/ioat.c
+++ b/drivers/dma/ioat.c
@@ -60,6 +60,8 @@ static struct pci_device_id ioat_pci_tbl[] = {
 	{ 0, }
 };
 
+MODULE_DEVICE_TABLE(pci, ioat_pci_tbl);
+
 struct ioat_device {
 	struct pci_dev		*pdev;
 	void __iomem		*iobase;

Re: [PATCH] Add MODULE_DEVICE_TABLE() so ioatdma module is autoloaded

[Dan Williams]

On Wed, Jul 1, 2009 at 11:05 AM, Roland Dreier<rdreier@cisco.com> wrote:
> The ioatdma module is missing aliases for the PCI devices it supports,
> so it is not autoloaded on boot.  Add a MODULE_DEVICE_TABLE() to get
> these aliases.
>
> Signed-off-by: Roland Dreier <rolandd@cisco.com>
> ---
> Not sure if not autoloading was intentional or if it's just an
> oversight, but this patch seems useful to me, so please apply if it
> makes sense to you.
>

Makes sense.  I believe the only hesitation is that it is a bit
awkward to remove (ifdown all interfaces, stop all raid456 arrays).
However, unless Maciej disagrees I think we should go ahead and
autoload it.

Applied for now to the 'fixes' branch of async_tx.git.

Thanks,
Dan