On Tue, Feb 4, 2025 at 6:34 PM Maciej Wieczor-Retman <maciej.wieczor-retman@xxxxxxxxx> wrote: > > ======= Introduction > The patchset aims to add a KASAN tag-based mode for the x86 architecture > with the help of the new CPU feature called Linear Address Masking > (LAM). Main improvement introduced by the series is 4x lower memory > usage compared to KASAN's generic mode, the only currently available > mode on x86. > > There are two logical parts to this series. The first one attempts to > add a new memory saving mechanism called "dense mode" to the generic > part of the tag-based KASAN code. The second one focuses on implementing > and enabling the tag-based mode for the x86 architecture by using LAM. Hi Maciej, Awesome work! Great to see SW_TAGS mode supported on x86! I started reviewing the patches, but this is somewhat complicated, as the dense mode changes are squashed together with the generic ones for x86 support. Could you please split this series into 2? Or at least reorder the patches so that everything needed for basic x86 support comes first and can be reviewed and tested separately. I will post the comments for things I noted so far, including for the dense mode changes, but I'll take a closer look after the split. Also feel free to drop the dependency on that risc-v series, as it doesn't get updated very often. But up to you. And please also update all affected parts of Documentation/dev-tools/kasan.rst. Thank you!
