ramosian-glider
commented
4 years ago LGTM (sorry, couldn't log in from phone)
On Fri, Oct 9, 2020, 18:59 Marco Elver notifications@github.com wrote:
@melver https://github.com/melver requested your review on: #160 https://github.com/google/kasan/pull/160 kfence: Do not limit KFENCE_NUM_OBJECTS in Kconfig.
— You are receiving this because your review was requested. Reply to this email directly, view it on GitHub https://github.com/google/kasan/pull/160#event-3861042599, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAG6Z4YFLRVWQZIOWQ2CBHTSJ46QRANCNFSM4SKKASKQ .
Changes to KFENCE_NUM_OBJECTS handling. FYI: Turns out actual max order is MAX_ORDER-1 (🤔). So in reality the max number of objects on e.g. x86 is only 511.
New KASAN problems, because of using page_alloc.
set_directmap{invalid,default}_noflush() does not work due to potential deadlock.