| commit | beaec533fc2701a28a4d667f67c9f59c6e4e0d13 | |
| author | Alexander Potapenko <glider@google.com> | |
| Wed, 19 Jul 2017 18:27:30 +0000 (19 20:27 +0200) | ||
| committer | Linus Torvalds <torvalds@linux-foundation.org> | |
| Wed, 19 Jul 2017 22:33:50 +0000 (19 15:33 -0700) | ||
| tree | d865dda1b1bf9c50ada3b4a8344aba60186c42fe | treesnapshot (tar.gz zip) |
| parent | e06fdaf40a5c021dd4a2ec797e8b724f07360070 | commitdiff |
llist: clang: introduce member_address_is_nonnull()
Currently llist_for_each_entry() and llist_for_each_entry_safe() iterate
until &pos->member != NULL. But when building the kernel with Clang,
the compiler assumes &pos->member cannot be NULL if the member's offset
is greater than 0 (which would be equivalent to the object being
non-contiguous in memory). Therefore the loop condition is always true,
and the loops become infinite.
To work around this, introduce the member_address_is_nonnull() macro,
which casts object pointer to uintptr_t, thus letting the member pointer
to be NULL.
Signed-off-by: Alexander Potapenko <glider@google.com>
Tested-by: Sodagudi Prasad <psodagud@codeaurora.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Currently llist_for_each_entry() and llist_for_each_entry_safe() iterate
until &pos->member != NULL. But when building the kernel with Clang,
the compiler assumes &pos->member cannot be NULL if the member's offset
is greater than 0 (which would be equivalent to the object being
non-contiguous in memory). Therefore the loop condition is always true,
and the loops become infinite.
To work around this, introduce the member_address_is_nonnull() macro,
which casts object pointer to uintptr_t, thus letting the member pointer
to be NULL.
Signed-off-by: Alexander Potapenko <glider@google.com>
Tested-by: Sodagudi Prasad <psodagud@codeaurora.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
| include/linux/llist.h | diffblobblamehistory |