Please create an account to participate in the Slashdot moderation system

 



Forgot your password?
typodupeerror
×
Linux

Linus Torvalds Cleaned Up the Intel LAM Code for Linux 6.4 (phoronix.com) 27

Last week Linus Torvalds personally cleaned up the x86 memory copy code for Linux 6.4, Phoronix reports — and this week "he's merged more of his own code as he took issue with some of the code merged by Intel engineers as part of their Linear Address Masking enabling." Back during the Linux 6.2 days at the end of last year, Linus rejected the Intel LAM code at the time for various technical issues. Intel then reworked it for Linux 6.4. This time around Linus merged Intel LAM into Linux 6.4 as this new CPU feature for letting user-space store metadata within some bits of pointers without masking it out before use. Intel LAM — like Arm TBI — can be of use to virtual machines, profiling / sanitizers / tagging, and other applications. But this time around there were some less than ideal code that he personally took to sprucing up...

Torvalds reworked around one hundred lines of code for cleaning it up.

It's fun to read Torvalds' commit messages (included in both Phoronix articles). Torvalds begins by writing that the LAM updates "made me unhappy about how 'access_ok()' was done, and it actually turned out to have a couple of small bugs in it too..."
This discussion has been archived. No new comments can be posted.

Linus Torvalds Cleaned Up the Intel LAM Code for Linux 6.4

Comments Filter:
  • I have concerns. Linus should be working on appointing a successor that he can trust at this point.
  • by Gravis Zero ( 934156 ) on Sunday May 07, 2023 @11:59AM (#63504061)

    A: Intel Linear Address Masking (LAM) allows user-space applications to make use of untranslated address bits of 64-bit linear addresses for metadata. Linear addresses use either 48-bits (4-level paging) or 57-bits (5-level paging) while LAM allows the remaining space of the 64-bit linear addresses to be used for metadata. It can be of use to virtual machines, profiling / sanitizers / tagging, and other applications.

    • by _merlin ( 160982 )

      They've reinvented the classic Macintosh 24-bit addressing mode using the extra bits to store metadata.

  • Too many coders tune out polite.
    • by gweihir ( 88907 )

      Indeed. Anything important has to be communicated in a way that gets _heard_. This is not a fluffy, bunny, we-are-all-happy-together clusterfuck, because those cannot produce quality.

  • Unix == memcpy() (Score:4, Interesting)

    by smoot123 ( 1027084 ) on Sunday May 07, 2023 @01:45PM (#63504317)

    Back in the day I remember hearing the aphorism "to a first approximation, a Unix kernel is memcpy()", the reasoning being that Unix kernels spent most of their time copying memory buffers from user space to kernel space to I/O devices.

    Sounds like it hasn't changed as much as one might have expected.

  • I understand that Phoronix is clickbait for nerds, but people falling for it is totally unjustified.

    Since when "Linus Torvalds wrote some code" is news? This is just a cleanup, and it's totally uninteresting. Even the fact that Linus Torvalds did it is uninteresting. Even though he is mostly a maintainer these days he still writing some code all the time, check the git logs.

  • by dltaylor ( 7510 ) on Sunday May 07, 2023 @07:12PM (#63505065)

    The programming guide for the Amiga cautioned not to use the upper 8 bits of an address for ANYTHING, because, although the MC68000 only bonded out the lower address bits, it was still a 32-bit processor and future versions might bring out all the address lines. The never-to-be-forgiven (for this among other things) clowns at Microsoft ignored the directive and used to upper address bits in their BASIC, which promptly broke everything when the 68020 (all lines bonded out) came out.

    Unless someone shows some intelligence and reverts that disaster-in-the-making, we will be stuck with incompatibilities and restrictions for however long Linux/X86 lasts.

    • by Junta ( 36770 )

      Couple of things, it looks like this is specifically a mode influenced by addressing mode, with one allowing for 48 bits of addressing, the other for 57 bit. So should a bigger addressing mode comes along, presumably the inappropriate LAM scheme would be disabled.

      Also, as we go up in bits, the less likely we will *ever* need it. The 16MB limitation of 24 bit addressing was obviously going to come up, and the 4G of 32 bit addressing did come up... eventually though it took way longer. As we add bits we ad

      • You shouldn't look at the addressing capability as the capacity to address RAM.
        We're at the end days of of PCIe's awkward access windows, and there is much value to disks being mapped directly to address space (and we're seeing this in specialized cases already, since it's a natural operation for NVMe).
        That being said, x86 is no stranger to multiple addressing modes (even x86-64 has multiple), so the parent's concern isn't realistic.
        Intel/AMD will come up with a new addressing mode when it becomes necess
        • by Junta ( 36770 )

          Somewhat true, though the tricks to move the access windows around barely have a noticeable performance impact on interacting with even pcie device RAM, so it's hard to imagine that NAND access would really require superseding the traditional aperture approach. So sure, we have the address space we might as well provide the means to memory-map the whole devices even up to many many petabytes, but if there ever was a crunch for address space, it's probably easier to limit pcie to aperture approach and the s

          • Currently, they have a pretty massive performance impact, actually.
            But that's to be expected. The performance of any kind of access window is going to be determined on how quickly it takes you to fill it with data vs. how quickly you can modify it. If you are talking to a very slow device through the window, the cost of moving the window is not going to be a large percentage of the total cost of access.

            As PCIe gets faster and faster, and the devices plugged into it do, the need to interrupt a user proces
    • Na.
      x86 has multiple addressing mode. LAM works in 48-bit addressing mode.
      Some future that allows bigger than 48-bit addresses (on your 64-bit pointers) will operate your code in 48-bit addressing mode, still with full LAM support. New code that needed the bigger pointers would work in a new addressing mode.
      Arm64 does the same thing.

      Your historical example is not analogous, because M68k made no promise that those lines would continue to be ignored in the ISA.
      • by gweihir ( 88907 )

        The historic example still shows that this has to be done very carefully as a special case and kept under close monitoring with regards to future changes in the hardware. Oh, and it must be easy to switch off.

        • LAM (and Arm equivalents) require OS support.
          The binary will specify whether they're allowed or not. If they are not, they will be masked.
          It is the technology that allows for things like pointer-signing. It requires no kind of special monitoring with regard to future changes in the hardware, as 48/57-bit addressing is literally part of the specification.
    • by tlhIngan ( 30335 )

      The programming guide for the Amiga cautioned not to use the upper 8 bits of an address for ANYTHING, because, although the MC68000 only bonded out the lower address bits, it was still a 32-bit processor and future versions might bring out all the address lines. The never-to-be-forgiven (for this among other things) clowns at Microsoft ignored the directive and used to upper address bits in their BASIC, which promptly broke everything when the 68020 (all lines bonded out) came out.

      Unless someone shows some

Suggest you just sit there and wait till life gets easier.

Working...