linux-kernel - Re: [PATCH] dma-debug: hash_bucket_find needs to allow for offsets within an entry (v2)

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:	Wed, 10 Aug 2011 15:31:59 +0200
From:	"Roedel, Joerg" <Joerg.Roedel@....com>
To:	Neil Horman <nhorman@...driver.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 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

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