diff options
| author | Kyle Spiers <ksspiers@google.com> | 2018-06-01 13:20:16 -0700 |
|---|---|---|
| committer | Vinod Koul <vkoul@kernel.org> | 2018-06-18 20:17:38 +0530 |
| commit | 89a7e2f752859c1a1696adf7b00b6ca9a5da2cda (patch) | |
| tree | 20efdb47f75740523ed334e1f3e816c7b07dc039 /crypto/async_tx/raid6test.c | |
| parent | ce397d215ccd07b8ae3f71db689aedb85d56ab40 (diff) | |
async_pq: Remove VLA usage
In the quest to remove VLAs from the kernel[1], this adjusts the
allocation of coefs and blocks to use the existing maximum values
(with one new define, MAX_DISKS for coefs, and a reuse of the
existing NDISKS for blocks).
[1] https://lkml.org/lkml/2018/3/7/621
Signed-off-by: Kyle Spiers <ksspiers@google.com>
Reviewed-by: Kees Cook <keescook@chromium.org>
Reviewed-by: Dan Williams <dan.j.williams@intel.com>
Signed-off-by: Vinod Koul <vkoul@kernel.org>
Diffstat (limited to 'crypto/async_tx/raid6test.c')
| -rw-r--r-- | crypto/async_tx/raid6test.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/crypto/async_tx/raid6test.c b/crypto/async_tx/raid6test.c index dad95f45b88f..a5edaabae12a 100644 --- a/crypto/async_tx/raid6test.c +++ b/crypto/async_tx/raid6test.c @@ -81,11 +81,13 @@ static void raid6_dual_recov(int disks, size_t bytes, int faila, int failb, stru init_async_submit(&submit, 0, NULL, NULL, NULL, addr_conv); tx = async_gen_syndrome(ptrs, 0, disks, bytes, &submit); } else { - struct page *blocks[disks]; + struct page *blocks[NDISKS]; struct page *dest; int count = 0; int i; + BUG_ON(disks > NDISKS); + /* data+Q failure. Reconstruct data from P, * then rebuild syndrome */ |