On 08/05/18 18:05, George Dunlap wrote: > On Tue, May 8, 2018 at 2:03 PM, Andrew Cooper <[email protected]> > wrote: >> c/s 4c5d78a10d "x86/pagewalk: Re-implement the pagetable walker" dropped the >> consumer of smap_policy. Looking at c/s 31ae587e6f which introduced the >> smap_check logic, it exists only to work around a bug in guest_walk_tables() >> was resolved by the aformentioned commit. >> >> Remove the unused variables and associated infrastructure. >> >> Reported-by: Jason Andryuk <[email protected]> >> Signed-off-by: Andrew Cooper <[email protected]> > > Reviewed-by: George Dunlap <[email protected]> > >> --- >> CC: Jan Beulich <[email protected]> >> CC: Wei Liu <[email protected]> >> CC: Roger Pau Monné <[email protected]> >> CC: Juergen Gross <[email protected]> >> CC: Jason Andryuk <[email protected]> >> >> I'm on the fence as to whether to suggest this for 4.11 at this point. Its >> probably not something to be backported, but it is a nice bit of cleanup, and >> removes a particularly gross hack. > > It looks like the commit that made this code vestigal was introduced > in March 2017? So we've already had two releases with this flag not > doing anything, and no ill effects reported. > > I'd be in favor of accepting a patch like this for 4.11, and also for > backporting it to 4.10 and 4.9
Okay, so: Release-acked-by: Juergen Gross <[email protected]> Juergen _______________________________________________ Xen-devel mailing list [email protected] https://lists.xenproject.org/mailman/listinfo/xen-devel
