diff options
| author | Sebastian Ott <[email protected]> | 2015-04-16 12:43:25 -0700 |
|---|---|---|
| committer | Linus Torvalds <[email protected]> | 2015-04-17 09:03:54 -0400 |
| commit | a7a2c02a40151811609ad8cd3bf5c4fc516fece5 (patch) | |
| tree | d34cfab8c8e31e1332a5fdba300f464a7cb86d2b | |
| parent | 7c43d9a30c527d9e06e2c55f82b56f28df43caed (diff) | |
lib/dma-debug: fix bucket_find_contain()
bucket_find_contain() will search the bucket list for a dma_debug_entry.
When the entry isn't found it needs to search other buckets too, since
only the start address of a dma range is hashed (which might be in a
different bucket).
A copy of the dma_debug_entry is used to get the previous hash bucket
but when its list is searched the original dma_debug_entry is to be used
not its modified copy.
This fixes false "device driver tries to sync DMA memory it has not allocated"
warnings.
Signed-off-by: Sebastian Ott <[email protected]>
Cc: Florian Fainelli <[email protected]>
Cc: Horia Geanta <[email protected]>
Cc: Jiri Kosina <[email protected]>
Signed-off-by: Andrew Morton <[email protected]>
Signed-off-by: Linus Torvalds <[email protected]>
| -rw-r--r-- | lib/dma-debug.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/lib/dma-debug.c b/lib/dma-debug.c index 9722bd2dbc9b..ae4b65e17e64 100644 --- a/lib/dma-debug.c +++ b/lib/dma-debug.c @@ -361,7 +361,7 @@ static struct dma_debug_entry *bucket_find_contain(struct hash_bucket **bucket, unsigned int range = 0; while (range <= max_range) { - entry = __hash_bucket_find(*bucket, &index, containing_match); + entry = __hash_bucket_find(*bucket, ref, containing_match); if (entry) return entry; |