From: Andres Freund <andres@anarazel.de>
To: David Hildenbrand <david@redhat.com>
Cc: Matthew Wilcox <willy@infradead.org>,
linux-mm@kvack.org, linux-block@vger.kernel.org,
Muchun Song <muchun.song@linux.dev>,
Jane Chu <jane.chu@oracle.com>
Subject: Re: Direct I/O performance problems with 1GB pages
Date: Mon, 27 Jan 2025 14:36:53 -0500 [thread overview]
Message-ID: <dj45nz5vspupjdeig53p6ynn226fzwfrj5ee2orfdu2q3krhpy@fff6c2poz7bw> (raw)
In-Reply-To: <4a75d25f-bcb9-42b6-aa9e-1e63e4be98e3@redhat.com>
Hi,
On 2025-01-27 20:20:41 +0100, David Hildenbrand wrote:
> On 27.01.25 18:25, Andres Freund wrote:
> > On 2025-01-27 15:09:23 +0100, David Hildenbrand wrote:
> > Unfortunately for the VMs with those disks I don't have access to hardware
> > performance counters :(.
> > >
> > > Maybe there is a link to the report you could share, thanks.
> >
> > A profile of the "original" case where I hit this, without the patch that
> > Willy linked to:
> >
> > Note this is a profile *not* using hardware perf counters, thus likely to be
> > rather skewed:
> > https://gist.github.com/anarazel/304aa6b81d05feb3f4990b467d02dabc
> > (this was on Debian Sid's 6.12.6)
> >
> > Without the patch I achieved ~18GB/s with 1GB pages and ~35GB/s with 2MB
> > pages.
>
> Out of interest, did you ever compare it to 4k?
I didn't. Postgres will always do at least 8kB (unless compiled with
non-default settings). But I also don't think I tested just doing 8kB on that
VM. I doubt I'd have gotten close to the max, even with 2MB huge pages. At
least not without block-layer-level merging of IOs.
If it's particularly interesting, I can bring a similar VM up and run that
comparison.
> > This time it's actual hardware perf counters...
> >
> > Relevant details about the c2c report, excerpted from IRC:
> >
> > andres | willy: Looking at a bit more detail into the c2c report, it looks
> > like the dirtying is due to folio->_pincount and folio->_refcount in
> > about equal measure and folio->flags being modified in
> > gup_fast_fallback(). The modifications then, unsurprisingly, cause a
> > lot of cache misses for reads (like in bio_set_pages_dirty() and
> > bio_check_pages_dirty()).
> >
> > willy | andres: that makes perfect sense, thanks
> > willy | really, the only way to fix that is to split it up
> > willy | and either we can split it per-cpu or per-physical-address-range
>
> As discussed, even better is "not repeatedly pinning/unpinning" at all :)
Indeed ;)
> I'm curious, are multiple processes involved, or is this all within a single
> process?
In the test case here multiple processes are involved, I was testing a
parallel sequential scan, with a high limit to the paralellism.
There are cases in which a fair bit of read IO is done from a single proccess
(e.g. to prerewarm the buffer pool after a restart, that's currently done by a
single process), but it's more common for high throughput to happen across
multiple processes. With modern drives a single task won't be able to execute
non-trivial queries at full disk speed.
Greetings,
Andres Freund
next prev parent reply other threads:[~2025-01-27 19:36 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-01-26 0:46 Matthew Wilcox
2025-01-27 14:09 ` David Hildenbrand
2025-01-27 16:02 ` Matthew Wilcox
2025-01-27 16:09 ` David Hildenbrand
2025-01-27 16:20 ` David Hildenbrand
2025-01-27 16:56 ` Matthew Wilcox
2025-01-27 16:59 ` David Hildenbrand
2025-01-27 18:21 ` Andres Freund
2025-01-27 18:54 ` Jens Axboe
2025-01-27 19:07 ` David Hildenbrand
2025-01-27 21:32 ` Pavel Begunkov
2025-01-27 16:24 ` Keith Busch
2025-01-27 17:25 ` Andres Freund
2025-01-27 19:20 ` David Hildenbrand
2025-01-27 19:36 ` Andres Freund [this message]
2025-01-28 5:56 ` Christoph Hellwig
2025-01-28 9:47 ` David Hildenbrand
2025-01-29 6:03 ` Christoph Hellwig
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=dj45nz5vspupjdeig53p6ynn226fzwfrj5ee2orfdu2q3krhpy@fff6c2poz7bw \
--to=andres@anarazel.de \
--cc=david@redhat.com \
--cc=jane.chu@oracle.com \
--cc=linux-block@vger.kernel.org \
--cc=linux-mm@kvack.org \
--cc=muchun.song@linux.dev \
--cc=willy@infradead.org \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox