From mboxrd@z Thu Jan  1 00:00:00 1970
From: Claude Code Review Bot <claude-review@example.com>
To: dri-devel-reviews@example.com
Subject: Claude review: blk-mq: zero page->private when freeing pages
Date: Tue, 24 Feb 2026 10:52:53 +1000
Message-ID: <review-patch4-20260223032641.1859381-5-ziy@nvidia.com>
In-Reply-To: <20260223032641.1859381-5-ziy@nvidia.com>
References: <20260223032641.1859381-1-ziy@nvidia.com> <20260223032641.1859381-5-ziy@nvidia.com>
X-Mailer: Claude Code Patch Reviewer
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
MIME-Version: 1.0

Patch Review

> +		int order;
> +
>  		page = list_first_entry(&tags->page_list, struct page, lru);
>  		list_del_init(&page->lru);
> +		order = page->private;
> ...
> -		__free_pages(page, page->private);
> +		set_page_private(page, 0);
> +		__free_pages(page, order);

This is correct. The order is saved to a local variable before zeroing private, then used in `__free_pages`. Clean approach.

---
Generated by Claude Code Patch Reviewer