Re: [RFC PATCH 2/4] rseq: Fix: Unregister rseq for CLONE_TLS

Of course, this patch title should read:

  rseq: Fix: Unregister rseq for CLONE_SETTLS

----- On Sep 11, 2019, at 1:27 AM, Mathieu Desnoyers mathieu.desnoyers@efficios.com wrote:

 

/*
 * If parent process has a registered restartable sequences area, the
- * child inherits. Only applies when forking a process, not a thread.
+ * child inherits. Unregister rseq for a clone with CLONE_TLS set.

and here CLONE_SETTLS as well.

 */
static inline void rseq_fork(struct task_struct *t, unsigned long clone_flags)
{
-	if (clone_flags & CLONE_THREAD) {
+	if (clone_flags & CLONE_TLS) {

.. and here.

Thanks,

Mathieu

		t->rseq = NULL;
		t->rseq_sig = 0;
		t->rseq_event_mask = 0;
--
2.17.1

-- 
Mathieu Desnoyers
EfficiOS Inc.
http://www.efficios.com