On Wed, 19 Jul 2023 at 17:06, Huacai Chen <chenhuacai@xxxxxxxxxx> wrote: > > Hi, Marco, > > On Wed, Jul 19, 2023 at 6:55 PM Marco Elver <elver@xxxxxxxxxx> wrote: > > > > On Wed, 19 Jul 2023 at 10:28, Enze Li <lienze@xxxxxxxxxx> wrote: > > > > > > The LoongArch architecture is different from other architectures. > > > It needs to update __kfence_pool during arch_kfence_init_pool. > > > > > > This patch modifies the assignment location of the local variable addr > > > in the kfence_init_pool function to support the case of updating > > > __kfence_pool in arch_kfence_init_pool. > > > > > > Signed-off-by: Enze Li <lienze@xxxxxxxxxx> > > > > I think it's fair to allow this use case. > > > > However, please make sure that when your arch_kfence_init_pool() > > fails, it is still possible to free the memblock allocated memory > > properly. > > > > Acked-by: Marco Elver <elver@xxxxxxxxxx> > Does Acked-by means this patch can go through loongarch tree together > with other patches? If this patch should go through kfence tree, then > the others should wait for some time. It can go through loongarch tree. I don't think there are conflicts with -mm around the patch's location right now. Thanks, -- Marco
