All of lore.kernel.org
 help / color / mirror / Atom feed
From: Sahil Chandna <sahilchandna@linux.microsoft.com>
To: Ankur Arora <ankur.a.arora@oracle.com>
Cc: linux-kernel@vger.kernel.org, linux-arch@vger.kernel.org,
	linux-arm-kernel@lists.infradead.org, linux-pm@vger.kernel.org,
	bpf@vger.kernel.org, arnd@arndb.de, catalin.marinas@arm.com,
	will@kernel.org, peterz@infradead.org, akpm@linux-foundation.org,
	mark.rutland@arm.com, harisokn@amazon.com, cl@gentwo.org,
	ast@kernel.org, rafael@kernel.org, daniel.lezcano@kernel.org,
	memxor@gmail.com, zhenglifeng1@huawei.com,
	xueshuai@linux.alibaba.com, rdunlap@infradead.org,
	david.laight.linux@gmail.com, broonie@kernel.org,
	boris.ostrovsky@oracle.com, ashok.bhat@arm.com
Subject: Re: [PATCH v16 00/16] barrier: Add smp_cond_load_{relaxed,acquire}_timeout()
Date: Thu, 29 Oct 2026 23:37:54 -0500	[thread overview]
Message-ID: <auQfIuQdpd2i2IrX@kali> (raw)
In-Reply-To: <87v76lezeq.fsf@oracle.com>

On Thu, Oct 29, 2026 at 12:31:09AM -0700, Ankur Arora wrote:
>
> Sahil Chandna <sahilchandna@linux.microsoft.com> writes:
>
> > On Fri, Oct 16, 2026 at 12:08:21AM -0700, Ankur Arora wrote:
> >> Hi,
> >>
> >> Changes in this version address sashiko comments [17]. The major change
> >> is the change in the exit path in __smp_cond_load_relaxed_timeout()
> >> which allows us to simplify providing ACQUIRE semantics in
> >> smp_cond_load_acquire_timeout():
> >>
> [ ... ]
> >>
> > Hi Ankur,
> >
> > I tested this patch series on an Azure Cobalt 100
> > (neoverse-n2 on 64-vCPU Hyper-V guest (Standard_D64ps_v6),
> > on v7.2.12 + this series + arm64 cpuidle-haltpoll enablement[1]
> >
> > cpuidle-haltpoll loaded with force=1, guest_halt_poll_ns=200000.
> >
> > Baseline: no cpuidle driver(only WFI).
> >
> > Workload: PostgreSQL 16.15/pgbench scale 100, client and server in
> > the same VM over a unix socket, with single run:
> >
> >             with (-S)                         TPC-B-like r/w
> > clients  base TPS  haltpoll lat avg   base TPS   haltpoll  lat avg
> >    2       38658     +89%   52->27us     5299     +86%    378->203us
> >    4       72987     +99%   55->28      10305     +87%    388->208
> >   16      300895     +91%   53->28      42035     +83%    381->208
> >   32      590987     +93%   55->28      78319     +85%    410->221
> >   64      987749      +0%   65->65     130675      +3%    490->474
> >
> > Tested-by: Sahil Chandna <sahilchandna@linux.microsoft.com>
>
> Thanks Sahil. That's very helpful. Btw what's the (-S) option with
> in your first heading?
>
Hi Ankur,

This is the -Select Only option running one statement per transaction,
more info captured in [1] [2].

TPC_B like runs small bank transfer transaction.
> Just to double check: was the VM  created with any special option
> such as kvm-arm.wfe_trap_policy?
>
No, the host in my case is Hyper-V. The cmdline does not have any idle
related parameters.
console=tty1 console=ttyAMA0 earlycon=pl011,0xeffec000 panic=-1
> --
> ankur
>
Regards,
Sahil

[1] https://www.postgresql.org/docs/16/pgbench.html#PGBENCH-OPTION-SELECT-ONLY
[2] https://www.postgresql.org/docs/16/pgbench.html#TRANSACTIONS-AND-SCRIPTS

      reply	other threads:[~2026-10-30  4:38 UTC|newest]

Thread overview: 37+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-10-16  7:08 [PATCH v16 00/16] barrier: Add smp_cond_load_{relaxed,acquire}_timeout() Ankur Arora
2026-10-16  7:08 ` [PATCH v16 01/16] asm-generic: barrier: Add smp_cond_load_relaxed_timeout() Ankur Arora
2026-10-16  7:21   ` sashiko-bot
2026-10-17  7:18     ` Ankur Arora
2026-10-16  7:55   ` bot+bpf-ci
2026-10-16  7:08 ` [PATCH v16 02/16] arm64: barrier: Support smp_cond_load_relaxed_timeout() Ankur Arora
2026-10-16  7:24   ` sashiko-bot
2026-10-17  7:37     ` Ankur Arora
2026-10-16  7:08 ` [PATCH v16 03/16] arm64/delay: move, fixup usecs_to_cycles() Ankur Arora
2026-10-16  7:16   ` sashiko-bot
2026-10-16  7:55   ` bot+bpf-ci
2026-10-16 12:22   ` Catalin Marinas
2026-10-17  7:19     ` Ankur Arora
2026-10-16  7:08 ` [PATCH v16 04/16] arm64: support WFET in smp_cond_load_relaxed_timeout() Ankur Arora
2026-10-16  7:27   ` sashiko-bot
2026-10-16  7:55   ` bot+bpf-ci
2026-10-16  7:08 ` [PATCH v16 05/16] arm64: rqspinlock: Remove private copy of smp_cond_load_acquire_timewait() Ankur Arora
2026-10-16  7:08 ` [PATCH v16 06/16] asm-generic: barrier: Add smp_cond_load_acquire_timeout() Ankur Arora
2026-10-16  7:21   ` sashiko-bot
2026-10-16  7:08 ` [PATCH v16 07/16] atomic: Add atomic_cond_read_*_timeout() Ankur Arora
2026-10-16  7:25   ` sashiko-bot
2026-10-16  7:08 ` [PATCH v16 08/16] locking/atomic: scripts: build atomic_long_cond_read_*_timeout() Ankur Arora
2026-10-16  7:08 ` [PATCH v16 09/16] bpf/rqspinlock: switch check_timeout() to a clock interface Ankur Arora
2026-10-16  7:08 ` [PATCH v16 10/16] bpf/rqspinlock: Use smp_cond_load_acquire_timeout() Ankur Arora
2026-10-16  7:08 ` [PATCH v16 11/16] sched: add need-resched timed wait interface Ankur Arora
2026-10-16  7:08 ` [PATCH v16 12/16] cpuidle/poll_state: Wait for need-resched via tif_need_resched_relaxed_wait() Ankur Arora
2026-10-16  7:08 ` [PATCH v16 13/16] arm64/delay: enable testing smp_cond_load_relaxed_timeout() Ankur Arora
2026-10-16  7:20   ` sashiko-bot
2026-10-16  7:08 ` [PATCH v16 14/16] barrier: add tests for smp_cond_load_*_timeout() Ankur Arora
2026-10-16  7:55   ` bot+bpf-ci
2026-10-16  7:08 ` [PATCH v16 15/16] barrier: timeout validity checks for smp_cond_load_relaxed_timeout() Ankur Arora
2026-10-16  7:19   ` sashiko-bot
2026-10-16  7:55   ` bot+bpf-ci
2026-10-16  7:08 ` [PATCH v16 16/16] barrier: timeout validity checks for smp_cond_load_acquire_timeout() Ankur Arora
2026-10-29  4:11 ` [PATCH v16 00/16] barrier: Add smp_cond_load_{relaxed,acquire}_timeout() Sahil Chandna
2026-10-29  7:31   ` Ankur Arora
2026-10-30  4:37     ` Sahil Chandna [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=auQfIuQdpd2i2IrX@kali \
    --to=sahilchandna@linux.microsoft.com \
    --cc=akpm@linux-foundation.org \
    --cc=ankur.a.arora@oracle.com \
    --cc=arnd@arndb.de \
    --cc=ashok.bhat@arm.com \
    --cc=ast@kernel.org \
    --cc=boris.ostrovsky@oracle.com \
    --cc=bpf@vger.kernel.org \
    --cc=broonie@kernel.org \
    --cc=catalin.marinas@arm.com \
    --cc=cl@gentwo.org \
    --cc=daniel.lezcano@kernel.org \
    --cc=david.laight.linux@gmail.com \
    --cc=harisokn@amazon.com \
    --cc=linux-arch@vger.kernel.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-pm@vger.kernel.org \
    --cc=mark.rutland@arm.com \
    --cc=memxor@gmail.com \
    --cc=peterz@infradead.org \
    --cc=rafael@kernel.org \
    --cc=rdunlap@infradead.org \
    --cc=will@kernel.org \
    --cc=xueshuai@linux.alibaba.com \
    --cc=zhenglifeng1@huawei.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.