From patchwork Fri Nov 11 05:10:09 2016 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Nicolas Pitre X-Patchwork-Id: 81776 Delivered-To: patch@linaro.org Received: by 10.140.97.165 with SMTP id m34csp1094056qge; Thu, 10 Nov 2016 21:11:02 -0800 (PST) X-Received: by 10.99.8.133 with SMTP id 127mr1691355pgi.76.1478841062706; Thu, 10 Nov 2016 21:11:02 -0800 (PST) Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id y1si6991889pal.140.2016.11.10.21.11.02; Thu, 10 Nov 2016 21:11:02 -0800 (PST) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE dis=NONE) header.from=linaro.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S936395AbcKKFK4 (ORCPT + 27 others); Fri, 11 Nov 2016 00:10:56 -0500 Received: from alt42.smtp-out.videotron.ca ([23.233.128.29]:26113 "EHLO alt42.smtp-out.videotron.ca" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752753AbcKKFKd (ORCPT ); Fri, 11 Nov 2016 00:10:33 -0500 Received: from yoda.home ([96.23.157.65]) by Videotron with SMTP id 546hcjxGCcMN9546iclTpt; Fri, 11 Nov 2016 00:10:32 -0500 X-Authority-Analysis: v=2.1 cv=YqOvP9sX c=1 sm=1 tr=0 a=keA3yYpnlypCNW5BNWqu+w==:117 a=keA3yYpnlypCNW5BNWqu+w==:17 a=L9H7d07YOLsA:10 a=9cW_t1CCXrUA:10 a=s5jvgZ67dGcA:10 a=L24OOQBejmoA:10 a=KKAkSRfTAAAA:8 a=EK0SQzbaxQmOMmQ328AA:9 a=0Jjz3aIMDOyf4vvM:21 a=KKSFXPerh3Vz4uLc:21 a=cvBusfyB2V15izCimMoJ:22 Received: from xanadu.home (xanadu.home [192.168.2.2]) by yoda.home (Postfix) with ESMTP id 8CD9F2DA0675; Fri, 11 Nov 2016 00:10:27 -0500 (EST) From: Nicolas Pitre To: John Stultz , Michal Marek Cc: Richard Cochran , Paul Bolle , Thomas Gleixner , Josh Triplett , Edward Cree , netdev@vger.kernel.org, linux-kbuild@vger.kernel.org, linux-kernel@vger.kernel.org Subject: [PATCH v4 5/6] posix_cpu_timers_exit: wrong place to collect entropy Date: Fri, 11 Nov 2016 00:10:09 -0500 Message-Id: <1478841010-28605-6-git-send-email-nicolas.pitre@linaro.org> X-Mailer: git-send-email 2.7.4 In-Reply-To: <1478841010-28605-1-git-send-email-nicolas.pitre@linaro.org> References: <1478841010-28605-1-git-send-email-nicolas.pitre@linaro.org> X-CMAE-Envelope: MS4wfHJ1soI4HcRY2bdbB8h6yUZcmagC4W8mlb7YV1SfC4Y25vV6BViZCzelyHNxtyYVFZjR8kQ/uTE7hN2ebYOAJj/SEvMKTWxl18PZ12pcK990Xx7ZUJUb gYIen6yO2rVzYSri6SB0Lx/UweGaMPAQEjCTGrDZpJ/XWaqBeS6FN0FsCL0SBTKtYUfWWpTIITvqTZm1IpyIFcrrlFlWTPq+oAuGDlCI0+CifEK6nD7mN9lL tD6uk/VWZbBICXnIuXoeivM55XIxA4hSPaRe3GKRjunI5cWE+Tlk9KkWXY9eoa7hUauP1KqH6zWMm+Ff1qv7okGjIQISkmh8IoCCIKrsUOf6Pg06BiiUR343 G1H6dHFKQPneMsuusP+0Cu8ir0o+otyIb+/edp9MQHO9rpO2C8JmPkrU87knzTDqMh2K93jRsYgDOnc8PIQ+4WYShpLjS8OYWRGsOqds1snf9G7xz98Iv6Z6 v7VewTRfpNi8CLp+ Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org There is no logical relation between add_device_randomness() and posix_cpu_timers_exit(). Let's move the former to where the later is called. This way, when posix-cpu-timers.c is compiled out, there is no need to worry about not losing a call to add_device_randomness(). Signed-off-by: Nicolas Pitre --- kernel/exit.c | 4 ++++ kernel/time/posix-cpu-timers.c | 4 ---- 2 files changed, 4 insertions(+), 4 deletions(-) -- 2.7.4 diff --git a/kernel/exit.c b/kernel/exit.c index 9d68c45ebb..d16bcdd89d 100644 --- a/kernel/exit.c +++ b/kernel/exit.c @@ -54,6 +54,7 @@ #include #include #include +#include #include #include @@ -116,6 +117,9 @@ static void __exit_signal(struct task_struct *tsk) sig->curr_target = next_thread(tsk); } + add_device_randomness((const void*) &tsk->se.sum_exec_runtime, + sizeof(unsigned long long)); + /* * Accumulate here the counters for all threads as they die. We could * skip the group leader because it is the last user of signal_struct, diff --git a/kernel/time/posix-cpu-timers.c b/kernel/time/posix-cpu-timers.c index 39008d7892..e582f20f47 100644 --- a/kernel/time/posix-cpu-timers.c +++ b/kernel/time/posix-cpu-timers.c @@ -9,7 +9,6 @@ #include #include #include -#include #include #include @@ -447,10 +446,7 @@ static void cleanup_timers(struct list_head *head) */ void posix_cpu_timers_exit(struct task_struct *tsk) { - add_device_randomness((const void*) &tsk->se.sum_exec_runtime, - sizeof(unsigned long long)); cleanup_timers(tsk->cpu_timers); - } void posix_cpu_timers_exit_group(struct task_struct *tsk) {