Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754670AbbGXQ45 (ORCPT ); Fri, 24 Jul 2015 12:56:57 -0400 Received: from mail-db3on0058.outbound.protection.outlook.com ([157.55.234.58]:45654 "EHLO emea01-db3-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1752909AbbGXQ4z (ORCPT ); Fri, 24 Jul 2015 12:56:55 -0400 Authentication-Results: redhat.com; dkim=none (message not signed) header.d=none; Subject: Re: [PATCH 07/10] sched: Migrate sched to use new tick dependency mask model To: Frederic Weisbecker , LKML References: <1437669735-8786-1-git-send-email-fweisbec@gmail.com> <1437669735-8786-8-git-send-email-fweisbec@gmail.com> CC: Peter Zijlstra , Thomas Gleixner , Preeti U Murthy , Christoph Lameter , Ingo Molnar , Viresh Kumar , Rik van Riel From: Chris Metcalf Message-ID: <55B26E4A.80209@ezchip.com> Date: Fri, 24 Jul 2015 12:56:42 -0400 User-Agent: Mozilla/5.0 (X11; Linux i686 on x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.1.0 MIME-Version: 1.0 In-Reply-To: <1437669735-8786-8-git-send-email-fweisbec@gmail.com> Content-Type: text/plain; charset="windows-1252"; format=flowed Content-Transfer-Encoding: 7bit X-Originating-IP: [12.216.194.146] X-ClientProxiedBy: BLUPR0601CA0011.namprd06.prod.outlook.com (25.163.210.21) To DB5PR02MB0776.eurprd02.prod.outlook.com (25.161.243.147) X-Microsoft-Exchange-Diagnostics: 1;DB5PR02MB0776;2:ahWk5hJjmbYtFAZhsnt4Gi4AzfOYMxB3DLVTp/YGT8ao44omL+xxuf+dbda5vrGT;3:o4yk76CqPa6ghIhqWNkgZrzPGGdjXBrTKe2G+7/Mbp2krBrhApSv6J8jX7ukxmS7MtvUiIisKDp7x2YQ41pikpGhxGQ6Xr6qUeFS3ksosoelr6RTWpX/stxMJUZYX9btVJwJxU2iJ0KFBpAnXHbqbg==;25:TEpKNzqjIgTApdyqx/vPVGmSPwR3/jOcNLcuY8XP3j1vKGSm9I/lIQDPsrXpcasJQoH1YYyvupbpVdwbV8H2r8S1DuAprzh8/2IOz4LNN/Ya95ykXW66IwBXjHpHu+4Q21JVOESGj6Fz8YjgFzzkfzqTK1VDAT887xTiQyn1OvC51s9fWdDEpzcV9Lqyh6qMsbTdK5bBxGLFaAujX6XmCeHBr5iD37yTrFDPDRClHQZdalNC5dR5LtoIcmiz/rMj;20:C4RfKPpmoAc68q+rUXt2PX6Xc9LLEOj7uf1oXmppeczmxqSoKRRQ3IzPxVIWtdutuzWEecEwuQiVYx+q1blPT87vZdP6AC2pddMCCQVVsSPKVd+5VO8sTE0hMYBklQa8/rXlRklpCD1f0f30cB2XxTkeXQ2gxb/b7cG85vUsplY=;4:Gpu7ik9PlUJX8OaFkA8V9yla9qnVPoVqg5e6GRYblHG4FqnX9USBulKduH01RsCLNLlId+W2P0xxaarsneRMDfateoN6ztQRihDPZKgtT/81OaEdhFpMZowdWSkDBs8Hljn9CysBV6BjpLNpgrOJN54xcJqRAm6NBm+p94EM6C+O+yFn7P0w4/iQXAk5UDUiHWOzalFoDYe/b11CRmHQ4rd8B8cJtJE8HRIXgkuoaDk/N+Ly1JMfi0DWnsqbAfoXnaS1g85SJ7mwzyePRRRAjQKV27yyw4MoqE4anbE/WU0= X-Microsoft-Antispam: UriScan:;BCL:0;PCL:0;RULEID:;SRVR:DB5PR02MB0776; DB5PR02MB0776: X-MS-Exchange-Organization-RulesExecuted X-Microsoft-Antispam-PRVS: X-Exchange-Antispam-Report-Test: UriScan:; X-Exchange-Antispam-Report-CFA-Test: BCL:0;PCL:0;RULEID:(601004)(5005006)(3002001);SRVR:DB5PR02MB0776;BCL:0;PCL:0;RULEID:;SRVR:DB5PR02MB0776; X-Forefront-PRVS: 0647963F84 X-Forefront-Antispam-Report: SFV:NSPM;SFS:(10009020)(6009001)(6049001)(24454002)(479174004)(377454003)(46102003)(64126003)(62966003)(87976001)(77156002)(76176999)(65816999)(54356999)(50986999)(23746002)(65956001)(47776003)(65806001)(40100003)(36756003)(83506001)(122386002)(92566002)(66066001)(77096005)(33656002)(4001350100001)(5001960100002)(15975445007)(50466002)(5001770100001)(2950100001)(86362001)(19580395003)(189998001)(42186005)(18886065003);DIR:OUT;SFP:1101;SCL:1;SRVR:DB5PR02MB0776;H:[10.7.0.41];FPR:;SPF:None;MLV:sfv;LANG:en; X-Microsoft-Exchange-Diagnostics: =?Windows-1252?Q?1;DB5PR02MB0776;23:T7TtkToE+3Re0yoRvoZowEHWhM/o+i8XBbIwe?= =?Windows-1252?Q?QpdeHmV9bspcrM966zhP1fNEtEWNAevViZY1YNsQhYI/vjnCCnWoetAj?= =?Windows-1252?Q?g13cTkLzIHyXdcX9lgG2ItzWC3UHwusvsAINAzRiMRseyFguJGxyd5Et?= =?Windows-1252?Q?XUXPheM/3cymb0uPPXtDrZuvl20r0Rk1L7cXv6sT73vCBk0n+1AQOV5k?= =?Windows-1252?Q?QBcqf+PqTnRLQukzLZ/E4oS9zRHg3a7x5T2d+rS9rtBSrueZtJegKjLV?= =?Windows-1252?Q?WQFtnxfEc2cfDhsZOTChOl5m3Eszs1TMq9KSs8YL7KibbUjTVR75se5v?= =?Windows-1252?Q?8CVjqNvM6/AP9q23sdXdu0n4S306tkDNc9+uREz/dTihPSRYUOlYhu8w?= =?Windows-1252?Q?45ykvClhQaRSh5njgiTtKusUb6I4f2Jb7nNeZhV68iDa7i7+3rjK1i2T?= =?Windows-1252?Q?gOTemfkf4v4aYkUI+3vA4V9qj1KfQWt1HOovvrN/FyHgAkbC6TWw7DcG?= =?Windows-1252?Q?I+2wuX9YQndCADnTOU7/sApNxE8+OiCjxganTzZmfB4Oep3OsWBml+HX?= =?Windows-1252?Q?TqkYaX1CK8zTy/GPjL/zVZD85VqYFNC8mlqIrObtkyWTgrFFecpfiXPa?= =?Windows-1252?Q?KE7EMHcGG0NFFowdD4hKACXO06/G2ot0FKP0ZYzBb2hTrlgRXt7Fd+DG?= =?Windows-1252?Q?w76Ca5CSYC0pPQTaBejcV6rX2MkzjZu2ycneBrq2a2PXnqFgNciWS+31?= =?Windows-1252?Q?s6a3D9JUr+N8F0ISkr7Auvc/ptMQvz1SGOYfYRiUxpvz9CiG/3XkP64l?= =?Windows-1252?Q?QPbGuFRyHL6pST0zNu2q7f70Q/UatuRFosxmEDb5cpcy+nLr84WP1/kJ?= =?Windows-1252?Q?iJiQg5AbMcKZxZTXwoVdT/BCi932ZBRzl5z+UZktjn7ZcQxQ3gz1tgNa?= =?Windows-1252?Q?NKRqtA+26FelQ/Gp3JxTaUyfRvXJMorgq40GvP6GYLD8Zky7e+2DgHF1?= =?Windows-1252?Q?qASjlLm1crsh8+tuhvWMbXxclM5D0Xd1Y5AUxTBwsLJHVX2HtZe3nFdE?= =?Windows-1252?Q?PYUAmMK5oRAJNbJw9HOVZyCrE3xPX3gI3+l?= X-Microsoft-Exchange-Diagnostics: 1;DB5PR02MB0776;5:StSgFIYrc9v+Nk7j8L7E/DySLIugAdF+sgrgp4zUTncaObd3rvQd33DwBsMZsOgHqu5+ijSs8RSH5ktIO5gOvhDTJ7mbo4YqsBLQsNLPwNqFg4Ial85AJmFVGF66UA9Y0iaNBCEM7W15/v3cKZ8zlg==;24:B4rTVtoEQqPoFdpfDLtVCTrmmvHKzztad+mER4iUxdakFvjEDKnAUZcXuVBX955fosGxvfxsZwb369DkCuKjBQ3u7T1fV7ekNYd4+mMcfrg=;20:uoqkK/cCqfJbSOUCPRrXDw72RdWmoyGE0hdwPqcNLHwotIhZjUrOhic38MmS2ahY+0eBYaBsSv2HBEiIFOLr/A== SpamDiagnosticOutput: 1:23 SpamDiagnosticMetadata: NSPM X-OriginatorOrg: ezchip.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 24 Jul 2015 16:56:51.4477 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-Transport-CrossTenantHeadersStamped: DB5PR02MB0776 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 892 Lines: 29 On 07/23/2015 12:42 PM, Frederic Weisbecker wrote: > +static inline void sched_update_tick_dependency(struct rq *rq) > +{ > + int cpu; > + > + if (!tick_nohz_full_enabled()) > + return; > + > + cpu = cpu_of(rq); > + > + if (!tick_nohz_full_cpu(rq->cpu)) > + return; > + > + if (sched_can_stop_tick(rq)) > + tick_nohz_clear_tick_dependency_cpu(TICK_SCHED_BIT, cpu); > + else > + tick_nohz_set_tick_dependency_cpu(TICK_SCHED_BIT, cpu); > +} Is it worth asserting that the rq is locked at this point? Presumably that's a requirement so that the cpu can't change out from under you. -- Chris Metcalf, EZChip Semiconductor http://www.ezchip.com -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/