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

[xen staging] gnttab: drop dead local variable from gnttab_map_frame_begin()



commit 9ddb2a2132d01f21ebe7ade826eed58671c51b17
Author:     Jan Beulich <jbeulich@xxxxxxxx>
AuthorDate: Tue Jun 16 14:18:39 2026 +0200
Commit:     Jan Beulich <jbeulich@xxxxxxxx>
CommitDate: Tue Jun 16 14:18:39 2026 +0200

    gnttab: drop dead local variable from gnttab_map_frame_begin()
    
    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>
    Reviewed-by: Michal Orzel <michal.orzel@xxxxxxx>
    Release-Acked-by: Oleksii Kurochko <oleksii.kurochko@xxxxxxxxx>
---
 xen/common/grant_table.c | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/xen/common/grant_table.c b/xen/common/grant_table.c
index ac9fed6001..17e1af9fcf 100644
--- a/xen/common/grant_table.c
+++ b/xen/common/grant_table.c
@@ -4255,7 +4255,6 @@ int gnttab_map_frame_begin(
 {
     int rc = 0;
     struct grant_table *gt = d->grant_table;
-    bool status = false;
 
     if ( gfn_eq(gfn, INVALID_GFN) )
     {
@@ -4268,8 +4267,6 @@ int gnttab_map_frame_begin(
     if ( evaluate_nospec(gt->gt_version == 2) && (idx & 
XENMAPIDX_grant_table_status) )
     {
         idx &= ~XENMAPIDX_grant_table_status;
-        status = true;
-
         rc = gnttab_get_status_frame_mfn(d, idx, mfn);
     }
     else
--
generated by git-patchbot for /home/xen/git/xen.git#staging



 


Rackspace

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