lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Date:	Thu, 11 Jun 2015 19:18:18 +0530
From:	Kishan Kumar <kishank@...eaurora.org>
To:	paulmck@...ux.vnet.ibm.com, josh@...htriplett.org,
	rostedt@...dmis.org, mathieu.desnoyers@...icios.com,
	laijs@...fujitsu.com, linux-kernel@...r.kernel.org,
	kaushalk@...eaurora.org
Cc:	Kishan Kumar <kishank@...eaurora.org>,
	Mohammed Khajapasha <mkhaja@...eaurora.org>,
	Vignesh Radhakrishnan <vigneshr@...eaurora.org>
Subject: [PATCH] rcu: Start grace period even for single callback

When we queue a callback for RCU, it goes and sits
on the nxttail of the per-cpu RCU data structure.
Callback is represented via struct callback_head
structure.

struct callback_head {
        struct callback_head *next;
        void (*func)(struct callback_head *head);
};

In case of a single callback queued in the nxttail,
the next field will be NULL. "next" happens to be
the zeroeth element of struct callback_head.

The condition "if(*rdp->nxttail[RCU_NEXT_READY_TAIL])"
in the function cpu_needs_another_gp(), essentially
checks if any callback is queued.

Since *rdp->nxttail[RCU_NEXT_READY_TAIL] dereferences
to the first element, the if condition will just turn
out to be if(NULL) in case there is a single callback
queued. This in turn causes cpu_needs_another_gp() to
return false even though we need a grace period to
process the single callback. This leads to writers
waiting until a second call_back gets queued, which can
cause undesirable effects like boot up delay upto 300
seconds, etc.

Fix this by performing this check on the "func" field
rather than the "next" field of the callback_head.

Signed-off-by: Kishan Kumar <kishank@...eaurora.org>
Signed-off-by: Mohammed Khajapasha <mkhaja@...eaurora.org>
Signed-off-by: Vignesh Radhakrishnan <vigneshr@...eaurora.org>
---
 kernel/rcu/tree.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/kernel/rcu/tree.c b/kernel/rcu/tree.c
index fc3abf1..394a4fa 100644
--- a/kernel/rcu/tree.c
+++ b/kernel/rcu/tree.c
@@ -324,7 +324,7 @@ cpu_needs_another_gp(struct rcu_state *rsp, struct rcu_data *rdp)
 		return 1;  /* Yes, a no-CBs CPU needs one. */
 	if (!rdp->nxttail[RCU_NEXT_TAIL])
 		return 0;  /* No, this is a no-CBs (or offline) CPU. */
-	if (*rdp->nxttail[RCU_NEXT_READY_TAIL])
+	if (((struct callback_head *)rdp->nxttail[RCU_NEXT_READY_TAIL])->func)
 		return 1;  /* Yes, this CPU has newly registered callbacks. */
 	for (i = RCU_WAIT_TAIL; i < RCU_NEXT_TAIL; i++)
 		if (rdp->nxttail[i - 1] != rdp->nxttail[i] &&
-- 
QUALCOMM INDIA, on behalf of Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum, hosted by The Linux Foundation.

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ