diff options
| author | Maarten Lankhorst <[email protected]> | 2020-11-24 12:57:07 +0100 |
|---|---|---|
| committer | Maarten Lankhorst <[email protected]> | 2020-11-25 10:48:28 +0100 |
| commit | bf8975837dac156c33a4d15d46602700998cb6dd (patch) | |
| tree | 7bac66a93ef0397f6ca4eb738f56c739b5aa1b7b | |
| parent | a14c6b0eef771c017c3549490fc321f3cd5c0e32 (diff) | |
dma-buf/dma-resv: Respect num_fences when initializing the shared fence list.
We hardcode the maximum number of shared fences to 4, instead of
respecting num_fences. Use a minimum of 4, but more if num_fences
is higher.
This seems to have been an oversight when first implementing the
api.
Fixes: 04a5faa8cbe5 ("reservation: update api and add some helpers")
Cc: <[email protected]> # v3.17+
Reported-by: Niranjana Vishwanathapura <[email protected]>
Signed-off-by: Maarten Lankhorst <[email protected]>
Reviewed-by: Thomas Hellström <[email protected]>
Link: https://patchwork.freedesktop.org/patch/msgid/[email protected]
| -rw-r--r-- | drivers/dma-buf/dma-resv.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/dma-buf/dma-resv.c b/drivers/dma-buf/dma-resv.c index bb5a42b10c29..6ddbeb5dfbf6 100644 --- a/drivers/dma-buf/dma-resv.c +++ b/drivers/dma-buf/dma-resv.c @@ -200,7 +200,7 @@ int dma_resv_reserve_shared(struct dma_resv *obj, unsigned int num_fences) max = max(old->shared_count + num_fences, old->shared_max * 2); } else { - max = 4; + max = max(4ul, roundup_pow_of_two(num_fences)); } new = dma_resv_list_alloc(max); |