Messages in this thread | | | Subject | Re: [PATCH] x86/iperm: remove unused pointers | From | Waiman Long <> | Date | Tue, 21 Jan 2020 09:16:01 -0500 |
| |
On 1/21/20 3:40 AM, Alex Shi wrote: > No one use the prev/next pointers in its function after commit 22fe5b0439dd > ("x86/ioperm: Move TSS bitmap update to exit to user work"). So better to > remove them. > > Signed-off-by: Alex Shi <alex.shi@linux.alibaba.com> > Cc: Thomas Gleixner <tglx@linutronix.de> > Cc: Ingo Molnar <mingo@redhat.com> > Cc: Borislav Petkov <bp@alien8.de> > Cc: "H. Peter Anvin" <hpa@zytor.com> > Cc: x86@kernel.org > Cc: Andy Lutomirski <luto@kernel.org> > Cc: Rik van Riel <riel@surriel.com> > Cc: Dave Hansen <dave.hansen@intel.com> > Cc: Waiman Long <longman@redhat.com> > Cc: Marcelo Tosatti <mtosatti@redhat.com> > Cc: linux-kernel@vger.kernel.org > --- > arch/x86/kernel/process.c | 4 ---- > 1 file changed, 4 deletions(-) > > diff --git a/arch/x86/kernel/process.c b/arch/x86/kernel/process.c > index 61e93a318983..839b5244e3b7 100644 > --- a/arch/x86/kernel/process.c > +++ b/arch/x86/kernel/process.c > @@ -615,12 +615,8 @@ void speculation_ctrl_update_current(void) > > void __switch_to_xtra(struct task_struct *prev_p, struct task_struct *next_p) > { > - struct thread_struct *prev, *next; > unsigned long tifp, tifn; > > - prev = &prev_p->thread; > - next = &next_p->thread; > - > tifn = READ_ONCE(task_thread_info(next_p)->flags); > tifp = READ_ONCE(task_thread_info(prev_p)->flags); >
Acked-by: Waiman Long <longman@redhat.com>
| |