269 pointsby crcastle9 hours ago15 comments
  • lfittl8 hours ago
    Its worth reading this follow-up LKML post by Andres Freund (who works on Postgres): https://lore.kernel.org/lkml/yr3inlzesdb45n6i6lpbimwr7b25kqk...
    • aftbit6 hours ago
      >If this somehow does end up being a reproducible performance issue (I still suspect something more complicated is going on), I don't see how userspace could be expected to mitigate a substantial perf regression in 7.0 that can only be mitigated by a default-off non-trivial functionality also introduced in 7.0.
    • justinclift6 hours ago
      Note that it's just not a single post, and there's additional further information in following the full thread. :)
      • adrian_ban hour ago
        Yes, and in the following messages the conclusion was that the regression is mitigated when using huge pages.
        • jeltz10 minutes ago
          Which you always should use anyway if you can.
    • anal_reactoran hour ago
      > Maybe we should, but requiring the use of a new low level facility that was introduced in the 7.0 kernel, to address a regression that exists only in 7.0+, seems not great.

      Completely right. This sounds like a communication failure. Maybe Linux maintainers should pick a few applications that have "priority support" and problems with these applications are also problems with Linux itself. Breaking Postgres is a serious regression.

      Reminds me of a situation where Fedora couldn't be updated if you had Wine installed and one side of the argument was "user applications are user problem" while the other was "it's Wine, like come on".

    • jeffbee7 hours ago
      Funny how "use hugepages" is right there on the table and 99% of users ignore it.
      • bombcar7 hours ago
        I’m absolutely flabbergasted by the performance left on the table; even by myself - just yesterday I learned Gentoo’s emerge can use git and be a billion times faster.
      • GandalfHN20 minutes ago
        [dead]
    • TacticalCoder7 hours ago
      AIUI in that thread they're saying "0.51x" the perf on a 96-core arm64 machine and they're also saying they cannot reproduce it on a 96-core amd64 machine.

      So it's not going to affect everybody both running PostgreSQL and upgrading to the latest kernel. Conditions seems to be: arm64, shitloads of core, kernel 7.0, current version of PostgreSQL.

      That is not going to be 100% of the installed PostgreSQL DBs out there in the wild when 7.0 lands in a few weeks.

      • torginusan hour ago
        It's a huge issue of ARM based systems, that hardly anyone uses or tests things on them (in production).

        Yes, Macs going ARM has been a huge boon, but I've also seen crazy regressions on AWS Graviton (compared to how its supposed to perform), on .NET (and node as well), which frankly I have no expertise or time digging into.

        Which was the main reason we ultimately cancelled our migration.

        I'm sure this is the same reason why its important to AWS.

      • zamalek4 hours ago
        It was later reproduced on the same machine without huge pages enabled. PICNIC?
        • anarazel4 hours ago
          Yes, I did reproduce it (to a much smaller degree, but it's just a 48c/96t machine). But it's an absurd workload in an insane configuration. Not using huge pages hurts way more than the regression due to PREEMPT_LAZY does.

          With what we know so far, I expect that there are just about no real world workloads that aren't already completely falling over that will be affected.

      • MBCook5 hours ago
        So perhaps this is a regression specifically in the arm64 code, or said differently maybe it’s a performance bug that has been there for a long time but covered up by the scheduler part that was removed?
        • adrian_ban hour ago
          The following messages concluded that using huge pages mitigates the regression, while not using huge pages reproduces it.
        • db48x4 hours ago
          Could be either of those, or something else entirely. Or even measurement error.
      • master_crab6 hours ago
        For production Postgres, i would assume it’s close to almost no effect?

        If someone is running postgres in a serious backend environment, i doubt they are using Ubuntu or even touching 7.x for months (or years). It’ll be some flavor of Debian or Red Hat still on 6.x (maybe even 5?). Those same users won’t touch 7.x until there has been months of testing by distros.

        • crcastle6 hours ago
          Ubuntu is used in many serious backend environments. Heroku runs tens of thousands (if not more) instances of Ubuntu on its fleet. Or at least it did through the teens and early 2020s.

          https://devcenter.heroku.com/articles/stack

          • rixed2 hours ago
            There is serious as in "corporate-serious" and serious as in "engineer-serious".
          • nine_k6 hours ago
            Do they upgrade to the new LTS the day it is released?
            • crcastle5 hours ago
              Not historically.
              • rvnx5 hours ago
                and they are right, this is because a lot of junior sysadmins believe that newer = better.

                But the reality:

                  a) may get irreversible upgrades (e.g. new underlying database structure) 
                  b) permanent worse performance / regression (e.g. iOS 26)
                  c) added instability
                  d) new security issues (litellm)
                  e) time wasted migrating / debugging
                  f) may need rewrite of consumers / users of APIs / sys calls
                  g) potential new IP or licensing issues
                
                etc.

                A couple of the few reasons to upgrade something is:

                  a) new features provide genuine comfort or performance upgrade (or... some revert)
                  b) there is an extremely critical security issue
                  c) you do not care about stability because reverting is uneventful and production impact is nil (e.g. Claude Code)
                
                but 99% of the time, if ain't broke, don't fix it.

                https://en.wikipedia.org/wiki/2024_CrowdStrike-related_IT_ou...

                • miki1232112 hours ago
                  On the other hand, I suspect LLMs will dramatically decrease the window between a vulnerability being discovered and that vulnerability being exploited in the wild, especially for open-source projects.

                  Even if the vulnerability itself is discovered through other means than by an LLM, it's trivial to ask a SOTA model to "monitor all new commits to project X and decide which ones are likely patching an exploitable vulnerability, and then write a PoC." That's a lot easier than finding the vulnerable itself.

                  I won't be surprised if update windows (for open source networked services) shrink to ~10 minutes within a year or two. It's going to be a brutal world.

        • pmontra3 hours ago
          A customer of mine is running on Ubuntu 22.04 and the plan is to upgrade to 26.04 in Q1 2027. We'll have to add performance regression to the plan.
          • wongogue37 minutes ago
            Are you running ARM servers?
  • dsr_8 hours ago
    Nobody sensible runs the latest kernel; nobody running PG in production should be afraid of setting a non-default at either boot time or as a sysctl. So this will, most likely, be another step in building a PG database server (turn off pre-emption if your kernel is 7.0 or later and PG is pre-whatever-version).

    At worst it might become a permanent part of building a PG server and a FAQ... but if it affects one thing this badly, it will affect others.

    • Meekro7 hours ago
      > Nobody sensible runs the latest kernel

      From the article: "Linux 7.0 stable is due out in about two weeks. This is also the kernel version powering Ubuntu 26.04 LTS to be released later in April."

      Unfortunately, lots of people will be running it in less than a month. At the moment, it'll take a kernel patch (not a sysctl) to undo this-- hopefully something changes.

      • teekert5 minutes ago
        I think most people on enterprise-y systems wait for (at least) 26.04.1, the window is 3 years (when on 24.04, which is supported until ~2029-04-30, it's 1 year when on 22.04) starting now, hardly anyone switches immediately.
      • Neywiny7 hours ago
        Not nobody but not everybody upgrades to the newest distros immediately. That's the advantage of LTS. I've even found that a lot of programs have poorer support on 24.04 than 22.04 due to security changes, so I'm fine sticking with 22.04 as my main dev system.
        • justinclift6 hours ago
          > ... not everybody upgrades to the newest distros immediately.

          While that's true, for new deployments the story is often "deploy on the latest release of things available at the time".

          So, there will probably be a substantial deployment of new projects / testing projects using the Linux 7.0 kernel along with the latest available software packages in a few weeks.

        • stingraycharles7 hours ago
          This seems to be brushing off a major performance regression just because you personally don’t upgrade for 4 years. I don’t think that’s common at all.
        • vascoan hour ago
          Someone said "its fine nobody uses this" and someone else gave the world's biggest slam dunk of "Ubuntu in 1 month" and your reply is that "not everyone does it". How far from the point can you be!

          In the Linux world this is the worst possible scenario, distro with the largest adoption, LTS.

        • esafak6 hours ago
          That's the advantage of LTS? 24.04 is the LTS, not the one you use, 22.04.
          • SoftTalker6 hours ago
            22.04 is also an LTS release, supported for another year still.

            https://ubuntu.com/about/release-cycle

            We're just now looking at moving production machines to 24.04.

            • apelapan3 minutes ago
              If you are on a maintenance contract with Ubuntu, 22.04 is supported until 2032.

              If it aint broken, don't fix it.

          • 6 hours ago
            undefined
          • cortesoft6 hours ago
            All even number .04 releases are LTS in Ubuntu
      • 9999000009995 hours ago
        Depends on your shop.

        As someone with a heavy QA/Dev Opps background I don't think we have enough details.

        Is it only ARM64 ? How many ARM64 PG DBs are running 96 cores?

        However...

        This is the most popular database in the world. Odds are this will effect a bunch of other lesser known applications.

    • stingraycharles7 hours ago
      That may be the case, but it’s still not a great situation to be in and one has to wonder: if PostgreSQL is affected, what else is?
      • bombcar7 hours ago
        That's the big thing - PSQL will be tested, noticed, and fixed (and likely have a version that handles 7.0 by the time it's in common use).

        But other software won't and may not even be noticed, except as a (I hate using the term) enshittification.

        Better to introduce the "correct way" in 7.0 but not regress the old (translate the "correct" into the old if necessary) - and then in 8.0 or some future release implement the regression.

        • stingraycharles6 hours ago
          Exactly, this is how it’s usually done. As the developer on the mailing list mentions, implementing a new low level construct in 7.0 and a performance regression that requires said new low level construct to mitigate is not great. You need a grace period in which both old and new approach is fast.
    • bombcar7 hours ago
      We need some sensible people running the latest and greatest or we won't catch things like this.
    • Seattle35035 hours ago
      If you're running in a docker container you share the host kernel. You might not have a choice.
    • cwillu6 hours ago
      The option to set PREEMPT_NONE was removed for basically all platforms.
    • GandalfHN2 hours ago
      [dead]
    • GandalfHN3 hours ago
      [dead]
  • bob1029an hour ago
    I'm struggling a bit with why we need all these fancy dynamic preemption modes. Is this about hyperscalars shoving more VMs per physical machine? What does a person trying to host a software solution gain from this kernel change?

    If a user wants to spin in an infinite loop all day every day, I don't see the problem with that. Even if the spinning will provably never do any useful work.

  • monocasa7 hours ago
    I feel like using spinlocks in user space at all without kernel support like rseq is just asking for weird performance degradations.
    • jeltz4 minutes ago
      PostgreSQL is old and had to support kernels which did not support spinlocks. But, yes, maybe PostgreSQL should stop doing so now that kernels do.
    • jcalvinowens6 hours ago
      > I feel like using spinlocks in user space at all without kernel support like rseq is just asking for weird performance degradations.

      Yeah, exactly. "Doctor, help, somebody replaced my wooden hammer with a metal one, and now I can't hit myself in the face with it as many times."

      If you use spinlocks in userspace, you're gonna have a bad time.

      • mgaunard2 hours ago
        Most people looking for performance will reach for the spinlock.

        The expectation is that the kernel should somehow detect applications that are spinning, and avoid preempting them early.

  • harshreality7 hours ago
    Background on PREEMPT_LAZY:

    https://lwn.net/Articles/994322/

  • longislandguido7 hours ago
    Anyone check to see if Jia Tan has submitted any kernel patches lately?
    • rs_rs_rs_rs_rs3 hours ago
      They don't need to, there's about a billion bugs they can exploit.
  • cperciva7 hours ago
    This makes me feel better about the 10% performance regression I just measured between FreeBSD 14 and FreeBSD 15.0.
    • db48x4 hours ago
      Heh. Did they at least add useful features to balance out that cost?
  • Deeg9rie9usi2 hours ago
    Once again phoronix shoot out an article without further researching nor letting the mail thread in question cool down. The follow up mails make clear that the issue is more or less a non-issue since the benchmark is wrong.
    • adrian_b41 minutes ago
      The following up mails conclude that the regression happens only when huge pages are not used.

      While using huge pages whenever possible is the right solution and this should be enough for PostgreSQL, perhaps there are applications that cannot use huge pages and which are affected by the regression.

      So I do not think that it is right to just ignore what happened.

      • Deeg9rie9usia few seconds ago
        I agree with you. The lurid headlines of phoronix.com just annoy me...
  • galbar8 hours ago
    It's not a good look to break userspace applications without a deprecation period where both old and new solutions exist, allowing for a transition period.
    • 6 hours ago
      undefined
  • anal_reactor3 hours ago
    Can someone explain to me what's the problem? I have very little knowledge of Linux kernel, but I'm curious. I've tried reading a little, but it's jargon over jargon.
    • alienchow2 hours ago
      I'm not familiar with the jargon either, but based on some reading it comes down to how the latest kernel treats process preempts.

      Postgres uses spinlocks to hold shared memory for very critical processes. Spinlocks are an infinite loop with no sleep to attempt to hold a lock, thus "spinning". Previous kernels allowed spinlocking processes to run with PREEMPT_NONE. This flag tells the kernel to let the locking process complete their work before doing anything. Now the latest kernel removed this functionality and is interrupting spinlocking processes. So if a process that is holding a lock gets interrupted, all other postgres spinlocks processes that need the same lock spin in place for way longer times, leading to performance degradation.

      • anal_reactoran hour ago
        Why does it only appear on arm64 and not x86?
        • adrian_b37 minutes ago
          It was not architecture-related. Not using huge pages also reproduced the regression on x86.

          I do not know why using huge pages mitigates the regression, but it could be just because when the application uses huge pages it uses spinlocks much less frequently so the additional delays do not accumulate enough to cause a significant performance reduction.

    • tijsvd37 minutes ago
      From what I understand in the follow up: postgres uses shared memory for buffers. This shared memory is read by a new connection while locked.

      In postgres, connections are handled with a process fork, not a new thread. If such a fork first reads memory, even if it already exists, that causes a minor page fault, which goes back to the kernel so it can update memory mapping tables.

      The operation under lock is only a few instructions, but if it takes longer than expected, then that causes lock contention. Regression in the kernel handling minor faults?

      The whole thing is then made worse because it's a spinlock, causing all waiting processes to contend over the cpus which adds to kernel processing.

      Mitigated by using huge pages, which dramatically reduces the number of mapping entries and faults. I reckon that it could also be mitigated in postgres by pre-faulting all shared memory early?

  • FireBeyond8 hours ago
    Once upon a time, Linus would shout and yell about how the kernel should never "break" userspace (and I see in some places, some arguments of "It's not broken, it's just a performance regression" - personally I'd argue a 50% hit to performance of a pre-eminent database engine is ... quite the regression).

    Now, the kernel engineer who introduced the brand new mechanism (introduced in Linux 7.0) for handling pre-emption says the "fix" is for Postgres to start using this new mechanism (I think the sister comment below links to what one of the Postgres engineers thinks of that, and I'm inclined mostly to agree).

    • shakna2 hours ago
      Freund seems to suggest that hugepages is the right way to run a system under this sort of load - which is the fix.

      > Hah. I had reflexively used huge_pages=on - as that is the only sane thing to do with 10s to 100s of GB of shared memory and thus part of all my benchmarking infrastructure - during the benchmark runs mentioned above.

      > Turns out, if I disable huge pages, I actually can reproduce the contention that Salvatore reported (didn't see whether it's a regression for me though). Not anywhere close to the same degree, because the bottleneck for me is the writes.

      But, they can speak for themselves here [0].

      [0] https://news.ycombinator.com/item?id=47646332

    • perching_aix7 hours ago
      Entertaining perspective - I thought that the whole "it's not an outage it's a (horizontal or vertical) degradation" thing was exclusive to web services, but thinking about it, I guess it does apply even in cases like this.
    • MBCook5 hours ago
      It wouldn’t be the first time one of the other maintainers ran afoul of “Linus’s law“.

      He may simply be waiting until more is known on exactly what’s causing it.

    • bear86427 hours ago
      > I'd argue a 50% hit to performance [...] is ... quite the regression

      Indeed! Especially if said regression happens to impact anything trade/market related...

    • arjie5 hours ago
      Well, the reason he'd yell about it is that someone did it. If no one ever did it, he'd never yell and we'd never have the rule. So one can only imagine that this is one of those things where someone has to keep holding the line rather than one of those things where you set some rule and it self-holds.

      Doubtless someone will have to do the yelling.

    • quietsegfault6 hours ago
      This was my immediate thought - kernel doesn’t break software, or at least it didn’t used to.
  • up2isomorphism2 hours ago
    Not sure why people have to upgrade to the newest major kernel version as soon as it is released.
    • conradludgate2 hours ago
      It's the performance team's job to test these things. Doesn't mean they're going to deploy it immediately.

      Someone should be testing these things and reporting regressions

    • IshKebaban hour ago
      Don't make excuses.
  • lossotha few seconds ago
    [dead]
  • 6 hours ago
    undefined