[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20110822104621.GA12337@hmsreliant.think-freely.org>
Date: Mon, 22 Aug 2011 06:46:21 -0400
From: Neil Horman <nhorman@...driver.com>
To: "Roedel, Joerg" <Joerg.Roedel@....com>
Cc: "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
Divy LeRay <divy@...lsio.com>,
Stanislaw Gruszka <sgruszka@...hat.com>,
Arnd Bergmann <arnd@...db.de>
Subject: Re: [PATCH] dma-debug: hash_bucket_find needs to allow for offsets
within an entry (v2)
On Wed, Aug 10, 2011 at 03:31:59PM +0200, Roedel, Joerg wrote:
> On Mon, Aug 08, 2011 at 03:13:54PM -0400, Neil Horman wrote:
> > dma-debug: hash_bucket_find needs to allow for offsets within an entry (v2)
> >
> > Summary:
> > Users of the pci_dma_sync_single_* api allow users to sync address ranges within
> > the range of a mapped entry (i.e. you can dma map address X to dma_addr_t A and
> > then pci_dma_sync_single on dma_addr_t A+1. The dma-debug library however
> > assume dma syncs will always occur using the base address of a mapped region,
> > and uses that assumption to find entries in its hash table. Since thats often
> > (but not always the case), the dma debug library can give us false errors about
> > missing entries, which are reported as syncing of memory not allocated by the
> > driver. This was noted in the cxgb3 driver as this error:
>
> Hi Neil,
>
> thanks for your patch. Im out-of-office this week and traveling next
> week. So it may take a couple of days until I can take an in-depth look
> at it. But I get to it as soon as I can :-)
>
> Regards,
>
> Joerg
>
Hey, Jeorg, just wondering if you managed to get a look at this?
Neil
> --
> AMD Operating System Research Center
>
> Advanced Micro Devices GmbH Einsteinring 24 85609 Dornach
> General Managers: Alberto Bozzo, Andrew Bowd
> Registration: Dornach, Landkr. Muenchen; Registerger. Muenchen, HRB Nr. 43632
>
> --
> 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/
>
--
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