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: KVM: use rest_of_page() macro where appropriate
Date: Thu, 05 Mar 2026 13:59:11 +1000
Message-ID: <review-patch6-20260304012717.201797-7-ynorov@nvidia.com>
In-Reply-To: <20260304012717.201797-7-ynorov@nvidia.com>
References: <20260304012717.201797-1-ynorov@nvidia.com> <20260304012717.201797-7-ynorov@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

- `arch/s390/kvm/gaccess.c`: Three correct replacements.
- `arch/x86/kvm/emulate.c`: Two replacements. **Minor nit:** the first hunk changes the type specifier from `unsigned` to `unsigned int`:
  ```c
  -	size = min_t(unsigned, size, PAGE_SIZE - offset_in_page(linear));
  +	size = min_t(unsigned int, size, rest_of_page(linear));
  ```
  While `unsigned` and `unsigned int` are identical, this is a gratuitous change beyond what the patch description claims ("No functional changes intended" / use rest_of_page). It should either be dropped or mentioned in the commit message.

---
Generated by Claude Code Patch Reviewer