[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <CAKmqyKNteaaXWCt2WR_YOzFvWRFhkfZvnv+4vjkyo3qESL8SYQ@mail.gmail.com>
Date: Fri, 3 Jan 2025 21:21:53 +1000
From: Alistair Francis <alistair23@...il.com>
To: Andreas Hindborg <a.hindborg@...nel.org>
Cc: Alistair Francis <alistair@...stair23.me>, linux-kernel@...r.kernel.org,
benno.lossin@...ton.me, boqun.feng@...il.com, me@...enk.dev,
alex.gaynor@...il.com, gary@...yguo.net, aliceryhl@...gle.com,
alistair.francis@....com, bjorn3_gh@...tonmail.com, tmgross@...ch.edu,
rust-for-linux@...r.kernel.org, ojeda@...nel.org
Subject: Re: [PATCH v4 00/11] rust: bindings: Auto-generate inline static functions
On Fri, Dec 20, 2024 at 12:55 PM Alistair Francis <alistair23@...il.com> wrote:
>
> On Wed, Dec 18, 2024 at 7:45 PM Andreas Hindborg <a.hindborg@...nel.org> wrote:
> >
> > "Alistair Francis" <alistair23@...il.com> writes:
> >
> > > On Wed, Dec 4, 2024 at 11:14 PM Andreas Hindborg <a.hindborg@...nel.org> wrote:
> > >>
> > >>
> > >> Hi Alistair,
> > >>
> > >> "Alistair Francis" <alistair@...stair23.me> writes:
> > >> >
> > >> > This series adds support for bindgen generating wrappers for inline statics and
> > >> > then converts the existing helper functions to this new method. This doesn't
> > >> > work for C macros, so we can't reamove all of the helper functions, but we
> > >> > can remove most.
> > >>
> > >> Are you aware of the helper LTO patches by Gary [1]? Can you tell if
> > >
> > > I was not, but that's exciting to see
> > >
> > >> this series will compose with the LTO patches?
> > >
> > > I think it will still work, although it might take some extra effort.
> > >
> > > I assume my series isn't going to be accepted though, so I'm not going
> > > to try and get the LTO series to work on top of it. It's a lot of work
> > > to rebase this series as every new binding causes a conflict and it
> > > seems to have stalled
> >
> > I would assume that it _will_ be merged in some form. Why not? It's just
> > a matter of getting it reviewed and making sure that it is going to work
> > with the LTO inlining.
I have my patches building with the LTO inlining work. The kernel
still boots and it seems like the inlining is working
You can see it here if you are interested in testing:
https://github.com/alistair23/linux/commit/e6b847324b4f5e904e007c0e288c88d2483928a8
Alistair
Powered by blists - more mailing lists