diff options
| author | Matthew Wilcox <willy@infradead.org> | 2019-02-21 23:54:44 +0100 |
|---|---|---|
| committer | Matthew Wilcox <willy@infradead.org> | 2019-02-21 23:54:44 +0100 |
| commit | 4a5c8d898948d1ac876522cdd62f07a78104bfe9 (patch) | |
| tree | f7b60e242dc970b9f4c6524f6882a4eb9261c388 /lib/test_xarray.c | |
| parent | XArray: Fix xa_erase of 2-byte aligned entries (diff) | |
| download | linux-4a5c8d898948d1ac876522cdd62f07a78104bfe9.tar.xz linux-4a5c8d898948d1ac876522cdd62f07a78104bfe9.zip | |
XArray: Fix xa_reserve for 2-byte aligned entries
If we reserve index 0, the next entry to be stored there might be 2-byte
aligned. That means we have to create the root xa_node at the time of
reserving the initial entry.
Signed-off-by: Matthew Wilcox <willy@infradead.org>
Diffstat (limited to 'lib/test_xarray.c')
| -rw-r--r-- | lib/test_xarray.c | 10 |
1 files changed, 10 insertions, 0 deletions
diff --git a/lib/test_xarray.c b/lib/test_xarray.c index bc202d468a6b..5d4bad8bd96a 100644 --- a/lib/test_xarray.c +++ b/lib/test_xarray.c @@ -1355,6 +1355,10 @@ static void check_align_1(struct xarray *xa, char *name) xa_destroy(xa); } +/* + * We should always be able to store without allocating memory after + * reserving a slot. + */ static void check_align_2(struct xarray *xa, char *name) { int i; @@ -1366,6 +1370,12 @@ static void check_align_2(struct xarray *xa, char *name) xa_erase(xa, 0); } + for (i = 0; i < 8; i++) { + XA_BUG_ON(xa, xa_reserve(xa, 0, GFP_KERNEL) != 0); + XA_BUG_ON(xa, xa_store(xa, 0, name + i, 0) != NULL); + xa_erase(xa, 0); + } + XA_BUG_ON(xa, !xa_empty(xa)); } |