From mboxrd@z Thu Jan 1 00:00:00 1970 From: Chintan Pandya Subject: Re: [PATCH v3 2/2] ksm: provide support to use deferrable timers for scanner thread Date: Wed, 20 Aug 2014 17:24:51 +0530 Message-ID: <53F48C8B.4020707@codeaurora.org> References: <1406793591-26793-2-git-send-email-cpandya@codeaurora.org> <1406793591-26793-3-git-send-email-cpandya@codeaurora.org> <53EA3FF5.1050709@codeaurora.org> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <53EA3FF5.1050709@codeaurora.org> Sender: owner-linux-mm@kvack.org To: Hugh Dickins Cc: Andrew Morton , linux-kernel@vger.kernel.org, linux-mm@kvack.org, Thomas Gleixner , John Stultz , Peter Zijlstra , Ingo Molnar , linux-arm-msm@vger.kernel.org List-Id: linux-arm-msm@vger.kernel.org Hi Hugh, >>> + unsigned long enable; >>> + int err; >>> + >>> + err = kstrtoul(buf, 10,&enable); >>> + if (err< 0) >>> + return err; >>> + if (enable>= 1) >>> + return -EINVAL; >> >> I haven't studied the patch itself, I'm still worrying about the concept. >> But this caught my eye just before hitting Send: I don't think we need >> a tunable which only accepts the value 0 ;) > > Okay. I can correct this to accept any non-zero value. Is that okay ? I missed that to reply earlier. This was suggested by Andrew. And I think that is okay as displaying any non-zero value to user via this knob may not be completely right. > >> >>> + use_deferrable_timer = enable; > -- Chintan Pandya QUALCOMM INDIA, on behalf of Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum, hosted by The Linux Foundation -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@kvack.org. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: email@kvack.org