Hi Ram,
On Thu, Nov 09, 2017 at 03:37:46PM -0800, Ram Pai wrote:
On Thu, Nov 09, 2017 at 04:47:15PM -0200, Breno Leitao wrote:
On Mon, Nov 06, 2017 at 12:57:36AM -0800, Ram Pai wrote:
@@ -206,12 +209,14 @@ void signal_handler(int signum, siginfo_t *si, void *vucontext) trapno = uctxt->uc_mcontext.gregs[REG_TRAPNO]; ip = uctxt->uc_mcontext.gregs[REG_IP_IDX];
- fpregset = uctxt->uc_mcontext.fpregs;
- fpregs = (void *)fpregset;
Since you removed all references for fpregset now, you probably want to remove the declaration of the variable above.
fpregs is still needed.
Right, fpregs is still needed, but not fpregset. Every reference for this variable was removed with your patch.
Grepping this variable identifier on a tree with your patches, I see:
$ grep fpregset protection_keys.c fpregset_t fpregset; -- To unsubscribe from this list: send the line "unsubscribe linux-kselftest" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html