Re: [PATCH] sata_nv,adma: fix error when rmmod sata_nv

Kuan Luo wrote:
> hi,
> The below error happens when i rmmod sata_nv in adma mode on ck804
> chipset with 2.6.24 kernel.
> I traced the code and found that the driver attempts to write device mem
> that has been unmapped.
> 
> Only simply removing the code" writew(0, mmio + NV_ADMA_CTL);" in the
> nv_adma_port_stop function or remove .port_stop field in nv_adma_ops,
> rmmod is ok.
> 
> static void nv_adma_port_stop(struct ata_port *ap)
> {
> 	struct nv_adma_port_priv *pp = ap->private_data;
> 	void __iomem *mmio = pp->ctl_block;
> 
> 	VPRINTK("ENTER\n");
> -	writew(0, mmio + NV_ADMA_CTL);
> }
> 
> Or
>  Place pcim_iomap_regions before ata_pci_prepare_native_host in
> nv_init_one function.
> This can guarantee that the code "writew(0, mmio + NV_ADMA_CTL) " write
> device mem before the device mem is unmapped.

Which kernel version are you using?  The following commit should have
fixed the problem.  Please give a shot at 2.6.24-rc5.  Thanks.

commit 32ebbc0c0d5d18c0135b55d1eb0029f48c54aff0
Author: Tejun Heo <[email protected]>
Date:   Thu Nov 8 13:09:00 2007 +0900

libata: port and host should be stopped before hardware resources are
released

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

---

• Follow-Ups:
  • RE: [PATCH] sata_nv,adma: fix error when rmmod sata_nv
    • From: "Kuan Luo" <[email protected]>

• References:
  • Re: [PATCH] sata_nv,ahci: add the ahci legacy mode support to sata_nv
    • From: Jeff Garzik <[email protected]>
  • Re: [PATCH] sata_nv,ahci: add the ahci legacy mode support to sata_nv
    • From: "peer chen" <[email protected]>
  • Re: [PATCH] sata_nv,ahci: add the ahci legacy mode support to sata_nv
    • From: Jeff Garzik <[email protected]>
  • Re: [PATCH] sata_nv,ahci: add the ahci legacy mode support to sata_nv
    • From: "peer chen" <[email protected]>
  • Re: [PATCH] sata_nv,ahci: add the ahci legacy mode support to sata_nv
    • From: Jeff Garzik <[email protected]>
  • Re: [PATCH] sata_nv,ahci: add the ahci legacy mode support to sata_nv
    • From: "peer chen" <[email protected]>
  • Re: [PATCH] sata_nv,ahci: add the ahci legacy mode support to sata_nv
    • From: Jeff Garzik <[email protected]>
  • Re: [PATCH] sata_nv,ahci: add the ahci legacy mode support to sata_nv
    • From: Jeff Garzik <[email protected]>
  • Re: [PATCH] sata_nv,ahci: add the ahci legacy mode support to sata_nv
    • From: Robert Hancock <[email protected]>
  • RE: [PATCH] sata_nv,adma: fix error when rmmod sata_nv
    • From: "Kuan Luo" <[email protected]>

• Prev by Date: Re: [RFT] Port 0x80 I/O speed
• Next by Date: Re: [PATCH] Add support for the S-35390A RTC chip.
• Previous by thread: RE: [PATCH] sata_nv,adma: fix error when rmmod sata_nv
• Next by thread: RE: [PATCH] sata_nv,adma: fix error when rmmod sata_nv
• Index(es):
  • Date
  • Thread

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Stuff]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]     [Linux Resources]