From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752116AbbE0Bcl (ORCPT ); Tue, 26 May 2015 21:32:41 -0400 Received: from mailout1.samsung.com ([203.254.224.24]:41091 "EHLO mailout1.samsung.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751333AbbE0Bci (ORCPT ); Tue, 26 May 2015 21:32:38 -0400 X-AuditID: cbfee68e-f79c56d000006efb-2a-55651eb3315d Date: Wed, 27 May 2015 01:32:35 +0000 (GMT) From: EunTaik Lee Subject: Re: Re: [RFC PATCH] suspend/resume performance improvement To: "Rafael J. Wysocki" Cc: "pavel@ucw.cz" , "len.brown@intel.com" , "linux-pm@vger.kernel.org" , "linux-kernel@vger.kernel.org" Reply-to: eun.taik.lee@samsung.com MIME-version: 1.0 X-MTR: 20150527012455913@eun.taik.lee Msgkey: 20150527012455913@eun.taik.lee X-EPLocale: ko_KR.euc-kr X-Priority: 3 X-EPWebmail-Msg-Type: personal X-EPWebmail-Reply-Demand: 0 X-EPApproval-Locale: X-EPHeader: ML X-MLAttribute: X-RootMTR: 20150527012455913@eun.taik.lee X-ParentMTR: X-ArchiveUser: EV X-CPGSPASS: N X-ConfirmMail: N,general Content-type: text/plain; charset=euc-kr MIME-version: 1.0 Message-id: <310063525.576351432690353260.JavaMail.weblogic@epmlwas01d> X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFprDJsWRmVeSWpSXmKPExsVy+t8zXd3NcqmhBosvSVhc3jWHzYHR4/Mm uQDGqAZGm8Si5IzMslSF1Lzk/JTMvHRbpdAQN10LJYWM/OISW6VoIwNjPSNTEz0jE3M9S4NY KyNTJYW8xNxUW6UKXaheJYWi5AKg2tzKYqABOal6UHG94tS8FIes/FKQS/SKE3OLS/PS9ZLz c5UUyhJzSoFGKOknTGXM2PtyO2vBAvaKzd07WBsYW9i7GDk5hATUJU7sXsMCYksImEh8P3OS CcIWk7hwbz1bFyMXUM0yRon2e/vYYIqmL3vJCJGYwyjxvns1WDeLgKrE6juzwaayCehK/P/Y BWYLCzhLdHW+AGsWEdCWmNtzihmkmVngFKPErDetUGcoScw/3AA2iFdAUOLkzCdQJ6lKfPy7 nBkiribxY387I0RcQmLW9AusEDavxIz2p1D1chLTvq5hhrClJc7P2sAI887i74+h4vwSx27v gHpTQGLqmYNQNVoSm3afh7L5JNYsfAs1U1Di9LVuZphd97fMZYK5YWvLE7AbmAUUJaZ0P2SH sLUkvvyAhBbCLxxAtrvEm5dmIL9LCPRySLy5dJ5pAqPSLCRls5CMmoVkFLKaBYwsqxhFUwuS C4qT0ouMkON7EyMkGfbtYLx5wPoQowAHoxIPb4ZkaqgQa2JZcWXuIcZkYDxNZJYSTc4Hpty8 knhDYzMjC1MTU2Mjc0szDGETUwsLEyMcwkrivAlSP4OFBNITS1KzU1MLUovii0pzUosPMTJx cEo1MOrwVp/R2pHe0beyLSMsXu5V1uHznjv/1+jL9oifMPRS4Pv4iq/Latt0n+Q4GxH9W1HC cg3GMmYBmq4vGxjEWQ/r7nrLtPPnHPEZN2KLVJ8aFEen7tO8v3Xxv1fHP/tNiFiky7n9A0vc pDtCn/izs74bcsx/7+76rHUiozDzjPV808wd+x4JKrEUZyQaajEXFScCAHKpLbevAwAA X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFjrJKsWRmVeSWpSXmKPExsVy+t/tXt3NcqmhBo0/RS0u75rD5sDo8XmT XABjVIZNRmpiSmqRQmpecn5KZl66rZJ3cLxzvKmZgaGuoaWFuZJCXmJuqq2Si0+ArltmDtBQ JYWyxJxSoFBAYnGxkr6dTVF+aUmqQkZ+cYmtUrSRgbGekamJnpGxgZ6JQayVoYGBkSlQVUJG xt6X21kLFrBXbO7ewdrA2MLexcjJISSgLnFi9xoWEFtCwERi+rKXjBC2mMSFe+vZuhi5gGrm MEq8714NVsQioCqx+s5ssGY2AV2J/x+7wGxhAWeJrs4XbCC2iIC2xNyeU8wgzcwCpxglZr1p hdqmJDH/cAPYIF4BQYmTM59AbVaV+Ph3OTNEXE3ix/52qCskJGZNv8AKYfNKzGh/ClUvJzHt 6xpmCFta4vysDXBXL/7+GCrOL3Hs9g4mCFtAYuqZg1A1WhKbdp+Hsvkk1ix8CzVTUOL0tW5m mF33t8xlgrlha8sTsBuYBRQlpnQ/ZIewtSS+/NjHhuoXDiDbXeLNS7MJjLKzkGRmIemehaQb Wc0CRpZVjKKpBckFxUnpFcZ6xYm5xaV56XrJ+bmbGMHJ6NniHYz/z1sfYhTgYFTi4T0gnRoq xJpYVlyZe4hRgoNZSYS3SRwoxJuSWFmVWpQfX1Sak1p8iNEUGGsTmaVEk/OBiTKvJN7Q2MDY 0NDS3MDU0MhCSZz3/7ncECGB9MSS1OzU1ILUIpg+Jg5OqQbGSb/5uuZmJHH3+dTdysjp67CI arz1mu1mtee5pRMWzRLpWXbef/1qN6al1QKvf7oeSX23/hKnnOmNddvyziyKqeU+ImDzpiB7 mRWHzgwr9UW7bJTvb5hufN2qYM0X/9wUo2ztJ/PeTPAMX37krcBCxuO19l1yt/90R9p/imPc zi09R0xLf3mdEktxRqKhFnNRcSIAbIk1qlwDAAA= DLP-Filter: Pass X-CFilter-Loop: Reflected Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Transfer-Encoding: 8bit X-MIME-Autoconverted: from base64 to 8bit by nfs id t4R1WlQ1016122 >> When a task that calls state_store() to suspend >> the device has used up most of its time slice, >> suspend sometimes take too long. (User noticeable) >> >> Suspend/resume is a system wide operation. >> So, instead of depending on a userspace task's time >> slice, let kworker do the work to avoid a long wait >> on the runqueue. > All of that seems to duplicate the existing autosleep.c functionality. Yes, the effect of using the kworker in autosleep will be same performance-wise. But if a platform doesn't use the autosleep functionality then we still have the problem. So doesn't it make sense to implement it for the autosleep-disabled use cases?{.n++%ݶw{.n+{G{ayʇڙ,jfhz_(階ݢj"mG?&~iOzv^m ?I