Received-SPF: pass (google.com: domain of linux-kernel+bounces-128926-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.48.161 as permitted sender) client-ip=147.75.48.161;
Date: Wed, 3 Apr 2024 10:21:32 +0900
From: Bongkyu Kim <bongkyu7.kim@samsung.com>
To: John Stultz <jstultz@google.com>
Cc: peterz@infradead.org, mingo@redhat.com, will@kernel.org,
	longman@redhat.com, boqun.feng@gmail.com, linux-kernel@vger.kernel.org,
	gregkh@linuxfoundation.org, bongkyu7.kim@samsung.com
Subject: Re: [PATCH v2 0/2] Make reader optimistic spinning optional
Message-ID: <20240403012132.GA460@KORCO045595.samsungds.net>
Precedence: bulk
MIME-Version: 1.0
In-Reply-To: <CANDhNCoFRr=qizswLm-dzxJK0fHuCx98Z2B1pUspdwGqBEejYg@mail.gmail.com>
Content-Type: multipart/mixed;
	boundary="----vz81d6m7j.py-bGO0GPJ08-4LvSGVDQp9Yqo3MxnNZXK7Ax7=_b700b_"
CMS-TYPE: 102P
DLP-Filter: Pass
References: <CGME20230901010734epcas2p4aadced02d68d3db407fda23de34601d2@epcas2p4.samsung.com>
	<20230901010704.18493-1-bongkyu7.kim@samsung.com>
	<CANDhNCoFRr=qizswLm-dzxJK0fHuCx98Z2B1pUspdwGqBEejYg@mail.gmail.com>

------vz81d6m7j.py-bGO0GPJ08-4LvSGVDQp9Yqo3MxnNZXK7Ax7=_b700b_
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 8bit
Content-Disposition: inline

On Tue, Apr 02, 2024 at 04:46:06PM -0700, John Stultz wrote:
> On Thu, Aug 31, 2023 at 6:07 PM Bongkyu Kim <bongkyu7.kim@samsung.com> wrote:
> >
> > This is rework of the following discussed patch.
> > https://lore.kernel.org/all/20230613043308.GA1027@KORCO045595.samsungds.net/
> >
> > Changes from the previous patch
> > - Split to revert and modify patches
> > - Change according to Waiman Long's review
> >     More wording to documentation part
> >     Change module_param to early_param
> >     Code change by Waiman Long's suggestion
> >
> > In mobile environment, reader optimistic spinning is still useful
> > because there're not many readers. In my test result at android device,
> > it improves application startup time about 3.8%
> > App startup time is most important factor for android user expriences.
> > So, re-enable reader optimistic spinning by this commit. And,
> > make it optional feature by cmdline.
> >
> > Test result:
> > This is 15 application startup performance in our exynos soc.
> > - Cortex A78*2 + Cortex A55*6
> > - unit: ms (lower is better)
> >
> > Application             base  opt_rspin  Diff  Diff(%)
> > --------------------  ------  ---------  ----  -------
> > * Total(geomean)         343        330   -13    +3.8%
> > --------------------  ------  ---------  ----  -------
> > helloworld               110        108    -2    +1.8%
> > Amazon_Seller            397        388    -9    +2.3%
> > Whatsapp                 311        304    -7    +2.3%
> > Simple_PDF_Reader        500        463   -37    +7.4%
> > FaceApp                  330        317   -13    +3.9%
> > Timestamp_Camera_Free    451        443    -8    +1.8%
> > Kindle                   629        597   -32    +5.1%
> > Coinbase                 243        233   -10    +4.1%
> > Firefox                  425        399   -26    +6.1%
> > Candy_Crush_Soda         552        538   -14    +2.5%
> > Hill_Climb_Racing        245        230   -15    +6.1%
> > Call_Recorder            437        426   -11    +2.5%
> > Color_Fill_3D            190        180   -10    +5.3%
> > eToro                    512        505    -7    +1.4%
> > GroupMe                  281        266   -15    +5.3%
> >
> 
> Hey Bongkyu,
>   I wanted to reach out to see what the current status of this patch
> set? I'm seeing other parties trying to work around the loss of the
> optimistic spinning functionality since commit 617f3ef95177
> ("locking/rwsem: Remove reader optimistic spinning") as well, with
> their own custom variants (providing some substantial gains), and
> would really like to have a common solution.
> 
> thanks
> -john
> 

Hi John,

I didn't get an reply, so I've been waiting.
Could you let me know about their patch?
I will try to comparing the app startup performance with reader optimistic spinning.

Thanks,
Bongkyu

------vz81d6m7j.py-bGO0GPJ08-4LvSGVDQp9Yqo3MxnNZXK7Ax7=_b700b_
Content-Type: text/plain; charset="utf-8"


------vz81d6m7j.py-bGO0GPJ08-4LvSGVDQp9Yqo3MxnNZXK7Ax7=_b700b_--