Commit 8be3a53e authored by Linus Torvalds's avatar Linus Torvalds

Merge tag 'erofs-for-5.8-rc3-fixes' of git://git.kernel.org/pub/scm/linux/kernel/git/xiang/erofs

Pull erofs fix from Gao Xiang:
 "Fix a regression which uses potential uninitialized high 32-bit value
  unexpectedly recently observed with specific compiler options"

* tag 'erofs-for-5.8-rc3-fixes' of git://git.kernel.org/pub/scm/linux/kernel/git/xiang/erofs:
  erofs: fix partially uninitialized misuse in z_erofs_onlinepage_fixup
parents fc10807d 3c597282
...@@ -144,22 +144,22 @@ static inline void z_erofs_onlinepage_init(struct page *page) ...@@ -144,22 +144,22 @@ static inline void z_erofs_onlinepage_init(struct page *page)
static inline void z_erofs_onlinepage_fixup(struct page *page, static inline void z_erofs_onlinepage_fixup(struct page *page,
uintptr_t index, bool down) uintptr_t index, bool down)
{ {
unsigned long *p, o, v, id; union z_erofs_onlinepage_converter u = { .v = &page_private(page) };
repeat: int orig, orig_index, val;
p = &page_private(page);
o = READ_ONCE(*p);
id = o >> Z_EROFS_ONLINEPAGE_INDEX_SHIFT; repeat:
if (id) { orig = atomic_read(u.o);
orig_index = orig >> Z_EROFS_ONLINEPAGE_INDEX_SHIFT;
if (orig_index) {
if (!index) if (!index)
return; return;
DBG_BUGON(id != index); DBG_BUGON(orig_index != index);
} }
v = (index << Z_EROFS_ONLINEPAGE_INDEX_SHIFT) | val = (index << Z_EROFS_ONLINEPAGE_INDEX_SHIFT) |
((o & Z_EROFS_ONLINEPAGE_COUNT_MASK) + (unsigned int)down); ((orig & Z_EROFS_ONLINEPAGE_COUNT_MASK) + (unsigned int)down);
if (cmpxchg(p, o, v) != o) if (atomic_cmpxchg(u.o, orig, val) != orig)
goto repeat; goto repeat;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment