[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH for-4.22] gnttab: drop dead local variable from gnttab_map_frame_begin()

To: Jan Beulich <jbeulich@xxxxxxxx>, "xen-devel@xxxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxxx>
From: Oleksii Kurochko <oleksii.kurochko@xxxxxxxxx>
Date: Tue, 16 Jun 2026 09:13:53 +0200
Authentication-results: eu.smtp.expurgate.cloud; dkim=pass header.s=20251104 header.d=gmail.com header.i="@gmail.com" header.h="Content-Transfer-Encoding:In-Reply-To:From:Content-Language:References:Cc:To:Subject:User-Agent:MIME-Version:Date:Message-ID"
Cc: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>, Julien Grall <julien@xxxxxxx>, Stefano Stabellini <sstabellini@xxxxxxxxxx>, Anthony PERARD <anthony.perard@xxxxxxxxxx>, Michal Orzel <michal.orzel@xxxxxxx>, Roger Pau Monné <roger.pau@xxxxxxxxxx>
Delivery-date: Tue, 16 Jun 2026 07:14:06 +0000
List-id: Xen developer discussion <xen-devel.lists.xenproject.org>



On 6/16/26 9:02 AM, Jan Beulich wrote:

As Michal had indicated in review, status is now a variable that is set
but never read. I made the resulting change locally, but then committed a
stale version of the patch (also omitting Michal's R-b).

Amends: eff88c4d3543 ("gnttab: simplify (really: drop) gnttab_set_frame_gfn()")
Reported-by: Michal Orzel <michal.orzel@xxxxxxx>
Signed-off-by: Jan Beulich <jbeulich@xxxxxxxx>


Release-Acked-by: Oleksii Kurochko <oleksii.kurochko@xxxxxxxxx>

Thanks.

~ Oleksii

References:
- [PATCH for-4.22] gnttab: drop dead local variable from gnttab_map_frame_begin()
  - From: Jan Beulich

Prev by Date: Re: Assertion '!is_idle_vcpu(v)' failed after 'Remove fully_eager_fpu' commit on EFI
Next by Date: [ANNOUNCEMENT] Xen 4.22.0-rc2 is tagged
Previous by thread: [PATCH for-4.22] gnttab: drop dead local variable from gnttab_map_frame_begin()
Next by thread: Re: [PATCH for-4.22] gnttab: drop dead local variable from gnttab_map_frame_begin()
Index(es):
- Date
- Thread

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.