Received-SPF: pass (google.com: domain of linux-kernel+bounces-132227-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.48.161 as permitted sender) client-ip=147.75.48.161;
Date: Thu, 04 Apr 2024 15:56:43 -0700 (PDT)
Subject:     Re: [PATCH v3 0/3] page_owner: Fix refcount imbalance
In-Reply-To: <e0110a0a-9bde-4e83-8e2a-2631e1bad0cf@ghiti.fr>
CC: osalvador@suse.de, akpm@linux-foundation.org, linux-kernel@vger.kernel.org,
  linux-mm@kvack.org, mhocko@suse.com, vbabka@suse.cz, elver@google.com, andreyknvl@gmail.com,
  glider@google.com
From: Palmer Dabbelt <palmer@dabbelt.com>
To: alex@ghiti.fr
Message-ID: <mhng-4caed5c9-bc46-42fe-90d4-9d845376578f@palmer-ri-x1c9a>
Precedence: bulk
Mime-Version: 1.0 (MHng)
Content-Type: text/plain; charset=utf-8; format=flowed
Content-Transfer-Encoding: 8bit

On Thu, 28 Mar 2024 23:39:33 PDT (-0700), alex@ghiti.fr wrote:
> Hi Oscar,
>
> On 26/03/2024 07:30, Oscar Salvador wrote:
>> This series consists of a refactoring/correctness of updating the metadata
>> of tail pages and a couple of fixups for the refcounting part.
>>
>>  From this series on, instead of counting the stacks, we count the outstanding
>> nr_base_pages each stack has, which gives us a much better memory overview.
>> The other fixup is for the migration part.
>>
>> A more detailed explanation can be found in the changelog of the respective
>> patches.
>>
>> Oscar Salvador (3):
>>    mm,page_owner: Update metada for tail pages
>>    mm,page_owner: Fix refcount imbalance
>>    mm,page_owner: Fix accounting of pages when migrating
>>
>>   Documentation/mm/page_owner.rst |  73 +++++++------
>>   mm/page_owner.c                 | 184 +++++++++++++++++++-------------
>>   2 files changed, 146 insertions(+), 111 deletions(-)
>>
>
> This fixes the following report from syzbot:
> https://lore.kernel.org/linux-riscv/00000000000070a2660614b83885@google.com/T/#t
>
> So you can add:
>
> Tested-by: Alexandre Ghiti <alexghiti@rivosinc.com>

Acked-by: Palmer Dabbelt <palmer@rivosinc.com>

in case that helps any, but I think this is one for the MM folks  Thanks 
for chasing down the fix!

> Thanks,
>
> Alex