From mboxrd@z Thu Jan 1 00:00:00 1970 From: Claude Code Review Bot To: dri-devel-reviews@example.com Subject: Claude review: drm/pagemap: Drop source_peer_migrates flag and assume true Date: Sun, 12 Apr 2026 12:09:35 +1000 Message-ID: In-Reply-To: <20260408201537.3580549-3-matthew.brost@intel.com> References: <20260408201537.3580549-1-matthew.brost@intel.com> <20260408201537.3580549-3-matthew.brost@intel.com> X-Mailer: Claude Code Patch Reviewer Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit MIME-Version: 1.0 Patch Review Straightforward cleanup. The only user (xe_svm.c) always set `source_peer_migrates = 1`. The conditional is removed and the body becomes unconditional: ```c - if (mdetails->source_peer_migrates) { - cur.dpagemap = src_zdd->dpagemap; - cur.ops = src_zdd->devmem_allocation->ops; - cur.device = cur.dpagemap->drm->dev; - pages[i] = src_page; - } + cur.dpagemap = src_zdd->dpagemap; + cur.ops = src_zdd->devmem_allocation->ops; + cur.device = cur.dpagemap->drm->dev; + pages[i] = src_page; ``` The kerneldoc for `can_migrate_same_pagemap` is properly updated to remove the stale reference to the dropped flag. Clean patch, no issues. --- Generated by Claude Code Patch Reviewer