Re: [PATCH 0/2] x86: Use "er" asm constriant for add/sub

Re: [PATCH 0/2] x86: Use "er" asm constriant for add/sub

[Jonathan Cameron]

On Sun, 23 Aug 2026 23:35:36 +0100
Jonathan Cameron <jonathan.cameron@oss.qualcomm.com> wrote:

> On Mon, 3 Aug 2026 16:39:55 +0100
> David Laight <david.laight.linux@gmail.com> wrote:
> 
> > On Mon, 3 Aug 2026 10:49:40 -0400 (EDT)
> > Nicolas Pitre <nico@fluxnic.net> wrote:
> >   
> > > On Mon, 3 Aug 2026, David Laight wrote:
> > >     
> > > > The x86 instruction set only supports 32bit signed immediate values
> > > > for add/sub.
> > > > Replace the "ir" constraint with "er" to avoid build errors.
> > > > 
> > > > Found by a patch that added used mul_u64_add_u64_div_u64() to
> > > > do a rounding divide by 2^32 (perhaps not the best way to do this).
> > > > 
> > > > A quick grep only found one other affected file.      
> > > 
> > > Would be a good idea adding those cases to the test module.    
> > 
> > They fail to compile so it isn't a big deal.
> >   
> David, thanks for sorting this (and all for reviewing)
> 
> Seems like everyone is happy.  What route is this taking upstream?
> 
> I held Rodrigo's driver to avoid the build breakage and would like to
> get it queued up early in next cycle.  So not that urgent but nice
> to be able to tick it off the list!
> 

Given I have a driver queued up behind this, shall I just take it through
the drivers/iio tree?

I can do an immutable branch just in case anyone else needs it this cycle.

Jonathan


> Thanks,
> 
> Jonathan
> 
> > 	David
> >   
> > > 
> > > 
> > > Nicolas    
> >   
> 

Re: [PATCH 0/2] x86: Use "er" asm constriant for add/sub

[Jonathan Cameron]

On Tue, 1 Sep 2026 17:51:01 +0100
Jonathan Cameron <jic23@kernel.org> wrote:

> On Sun, 23 Aug 2026 23:35:36 +0100
> Jonathan Cameron <jonathan.cameron@oss.qualcomm.com> wrote:
> 
> > On Mon, 3 Aug 2026 16:39:55 +0100
> > David Laight <david.laight.linux@gmail.com> wrote:
> >   
> > > On Mon, 3 Aug 2026 10:49:40 -0400 (EDT)
> > > Nicolas Pitre <nico@fluxnic.net> wrote:
> > >     
> > > > On Mon, 3 Aug 2026, David Laight wrote:
> > > >       
> > > > > The x86 instruction set only supports 32bit signed immediate values
> > > > > for add/sub.
> > > > > Replace the "ir" constraint with "er" to avoid build errors.
> > > > > 
> > > > > Found by a patch that added used mul_u64_add_u64_div_u64() to
> > > > > do a rounding divide by 2^32 (perhaps not the best way to do this).
> > > > > 
> > > > > A quick grep only found one other affected file.        
> > > > 
> > > > Would be a good idea adding those cases to the test module.      
> > > 
> > > They fail to compile so it isn't a big deal.
> > >     
> > David, thanks for sorting this (and all for reviewing)
> > 
> > Seems like everyone is happy.  What route is this taking upstream?
> > 
> > I held Rodrigo's driver to avoid the build breakage and would like to
> > get it queued up early in next cycle.  So not that urgent but nice
> > to be able to tick it off the list!
> >   
> 
> Given I have a driver queued up behind this, shall I just take it through
> the drivers/iio tree?
> 
> I can do an immutable branch just in case anyone else needs it this cycle.
> 

I went ahead and picked these up on
https://git.kernel.org/pub/scm/linux/kernel/git/jic23/iio.git ib-iio-x86-asm-fix-7.3
and merged that into the main IIO branch. I want to give plenty of time in next
for the driver that was relying on them.

Currently this is all in my testing branch but I'll push that out as a non rebasing
(ideally) branch a bit later in the week.

Thanks

Jonathan

> Jonathan
> 
> 
> > Thanks,
> > 
> > Jonathan
> >   
> > > 	David
> > >     
> > > > 
> > > > 
> > > > Nicolas      
> > >     
> >   
> 
> 

Re: [PATCH 0/2] x86: Use "er" asm constriant for add/sub

[Borislav Petkov]

On Mon, Sep 07, 2026 at 11:14:44PM +0100, Jonathan Cameron wrote:
> I went ahead and picked these up on
> https://git.kernel.org/pub/scm/linux/kernel/git/jic23/iio.git ib-iio-x86-asm-fix-7.3
> and merged that into the main IIO branch. I want to give plenty of time in next
> for the driver that was relying on them.
> 
> Currently this is all in my testing branch but I'll push that out as a non rebasing
> (ideally) branch a bit later in the week.

No, this is not how this works. x86 changes go through tip and if you need an
immutable branch, we can give you a tag.

-- 
Regards/Gruss,
    Boris.

https://people.kernel.org/tglx/notes-about-netiquette

Re: [PATCH 0/2] x86: Use "er" asm constriant for add/sub

[Uwe Kleine-König]

[-- Attachment #1: Type: text/plain, Size: 1046 bytes --]

On Mon, Sep 07, 2026 at 08:32:16PM -0700, Borislav Petkov wrote:
> On Mon, Sep 07, 2026 at 11:14:44PM +0100, Jonathan Cameron wrote:
> > I went ahead and picked these up on
> > https://git.kernel.org/pub/scm/linux/kernel/git/jic23/iio.git ib-iio-x86-asm-fix-7.3
> > and merged that into the main IIO branch. I want to give plenty of time in next
> > for the driver that was relying on them.
> > 
> > Currently this is all in my testing branch but I'll push that out as a non rebasing
> > (ideally) branch a bit later in the week.
> 
> No, this is not how this works. x86 changes go through tip and if you need an
> immutable branch, we can give you a tag.

While x86 usually goes through tip and none of the x86 maintainers
commented before Jonathan applied, it got positive feedback from hpa and
peterz a month ago and iio depends on it.

So I suggest to do the practical thing and pull Jonathan's branch into
tip. Technically that doesn't make a difference (apart from maybe branch
vs tag preferences).

Best regards
Uwe

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

Re: [PATCH 0/2] x86: Use "er" asm constriant for add/sub

[Borislav Petkov]

On Tue, Sep 08, 2026 at 12:02:42PM +0200, Uwe Kleine-König wrote:
> While x86 usually goes through tip and none of the x86 maintainers
> commented before Jonathan applied,

You mean it is easy to spot every question to x86 maintainers in the avalance
of mails?

> it got positive feedback from hpa and peterz a month ago and iio depends on
> it.

That's business as usual.
 
> So I suggest to do the practical thing and pull Jonathan's branch into tip.
> Technically that doesn't make a difference (apart from maybe branch vs tag
> preferences).

It does make a difference when the merge window comes and we have to
synchronize when to send what. And we send tip stuff very early.

Considering how early it is in the game, I'd prefer if I queue those into tip
now and give you an immutable branch or tag and not worry about adding more
stuff ontop later.

Thx.

-- 
Regards/Gruss,
    Boris.

https://people.kernel.org/tglx/notes-about-netiquette

Re: [PATCH 0/2] x86: Use "er" asm constriant for add/sub

[Uwe Kleine-König]

[-- Attachment #1: Type: text/plain, Size: 1230 bytes --]

On Tue, Sep 08, 2026 at 08:21:27AM -0700, Borislav Petkov wrote:
> On Tue, Sep 08, 2026 at 12:02:42PM +0200, Uwe Kleine-König wrote:
> > While x86 usually goes through tip and none of the x86 maintainers
> > commented before Jonathan applied,
> 
> You mean it is easy to spot every question to x86 maintainers in the avalance
> of mails?

No, there was no hidden offence, I only meant what I wrote.

> > So I suggest to do the practical thing and pull Jonathan's branch into tip.
> > Technically that doesn't make a difference (apart from maybe branch vs tag
> > preferences).
> 
> It does make a difference when the merge window comes and we have to
> synchronize when to send what. And we send tip stuff very early.

I don't see what you mean. If both the tip tree and the iio tree
contain the same commit (with or without a tag on it), it doesn't matter
who created that and when and in which repo it appeared first. The first
to send it to Linus makes it new for him and all later PRs using it only
introduce the follow-up commits. So if you pull Jonathan's branch you
still can send the branch/tag that contains it whenever you like
independant of when Jonathan sends out his PR.

Best regards
Uwe

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

Re: [PATCH 0/2] x86: Use "er" asm constriant for add/sub

[Borislav Petkov]

On Tue, Sep 08, 2026 at 05:49:17PM +0200, Uwe Kleine-König wrote:
> I don't see what you mean. If both the tip tree and the iio tree
> contain the same commit (with or without a tag on it), it doesn't matter
> who created that and when and in which repo it appeared first. The first
> to send it to Linus makes it new for him and all later PRs using it only
> introduce the follow-up commits. So if you pull Jonathan's branch you
> still can send the branch/tag that contains it whenever you like
> independant of when Jonathan sends out his PR.

Yes, I'm going to have to merge that branch into tip now so I'd need the
assurance from Jonathan that this branch is not going to change.

Also, those patches are not tip-conform, but whatever, not that important for
a one-time thing.

What I really don't understand is, why this rush needs to happen now so that
we completely do a one-off on the agreed upon process and we have to do all
this unnecessary gymnastics?

We have plenty of time.

So why do we even have to debate this instead of you folks dropping this and
we merge it through tip and then everything's going the usual way?

When people start merging x86 patches in their trees, when do you think the
process will break down and we'll have to redo branches and go into a crazy
dependency madness?

We're not merging other stuff through tip without an Ack - why can't you
adhere to the process we've all agreed upon?

Yes, this can work now. Is it necessary? Absolutely not because it is
a one-off and will get everyone involved, confused.

So please do not do this in the future. If we don't react to the mail, you can
always ping us on IRC like other folks do.

Thx.

-- 
Regards/Gruss,
    Boris.

https://people.kernel.org/tglx/notes-about-netiquette

Re: [PATCH 0/2] x86: Use "er" asm constriant for add/sub

[Jonathan Cameron]

On Tue, 8 Sep 2026 09:55:01 -0700
Borislav Petkov <bp@alien8.de> wrote:

> On Tue, Sep 08, 2026 at 05:49:17PM +0200, Uwe Kleine-König wrote:
> > I don't see what you mean. If both the tip tree and the iio tree
> > contain the same commit (with or without a tag on it), it doesn't matter
> > who created that and when and in which repo it appeared first. The first
> > to send it to Linus makes it new for him and all later PRs using it only
> > introduce the follow-up commits. So if you pull Jonathan's branch you
> > still can send the branch/tag that contains it whenever you like
> > independant of when Jonathan sends out his PR.  
> 
> Yes, I'm going to have to merge that branch into tip now so I'd need the
> assurance from Jonathan that this branch is not going to change.

That's what an immutable branch means to everyone I've ever shared them
with in the past, but sure - it wouldn't change.  Anyhow, easier solution
below.

> 
> Also, those patches are not tip-conform, but whatever, not that important for
> a one-time thing.
> 
> What I really don't understand is, why this rush needs to happen now so that
> we completely do a one-off on the agreed upon process and we have to do all
> this unnecessary gymnastics?
> 
> We have plenty of time.
> 
> So why do we even have to debate this instead of you folks dropping this and
> we merge it through tip and then everything's going the usual way?
> 
> When people start merging x86 patches in their trees, when do you think the
> process will break down and we'll have to redo branches and go into a crazy
> dependency madness?
> 
> We're not merging other stuff through tip without an Ack - why can't you
> adhere to the process we've all agreed upon?
> 
> Yes, this can work now. Is it necessary? Absolutely not because it is
> a one-off and will get everyone involved, confused.
> 
> So please do not do this in the future. If we don't react to the mail, you can
> always ping us on IRC like other folks do.
> 
> Thx.

I've dropped the branch from IIO. That's fine because it was still being
tested and hasn't gone out as non-rebasing yet.

Next time I'll maybe find an IRC gateway and ping you on that.  I can't
find any reference to that being your preference in the kernel docs.
Given you have a helpful page on tip, might be worth adding a note.
I might of course be missing it.

If it goes into a reasonably early rc, I'll just merge the relevant rc
rather than needing an immutable. If next cycle, immutable please.

I started out a bit grumpy that I had to hold a non trivial series for
a cycle because of an x86 bug (having been very happy when David
fixed it incredibly quickly!), that got reviewed then disappeared
down a hole and didn't look like it would ever be seen again.
Meh, got me on a belligerent day.

Jonathan

Re: [PATCH 0/2] x86: Use "er" asm constriant for add/sub

[Borislav Petkov]

On Thu, Sep 10, 2026 at 04:20:29AM +0100, Jonathan Cameron wrote:
> I've dropped the branch from IIO. That's fine because it was still being
> tested and hasn't gone out as non-rebasing yet.

Ack, will do first thing tomorrow.

> Next time I'll maybe find an IRC gateway and ping you on that.  I can't
> find any reference to that being your preference in the kernel docs.
> Given you have a helpful page on tip, might be worth adding a note.
> I might of course be missing it.

I'll send you a private note after this.

> If it goes into a reasonably early rc, I'll just merge the relevant rc
> rather than needing an immutable. If next cycle, immutable please.

First one can go to Linus now, second one would go next cycle. Do you want to
base ontop of both?

> I started out a bit grumpy that I had to hold a non trivial series for
> a cycle because of an x86 bug (having been very happy when David
> fixed it incredibly quickly!), that got reviewed then disappeared
> down a hole and didn't look like it would ever be seen again.
> Meh, got me on a belligerent day.

No worries, happens to all of us.

Thanks!

-- 
Regards/Gruss,
    Boris.

https://people.kernel.org/tglx/notes-about-netiquette

Re: [PATCH 0/2] x86: Use "er" asm constriant for add/sub

[Jonathan Cameron]

On Wed, 9 Sep 2026 20:45:27 -0700
Borislav Petkov <bp@alien8.de> wrote:

> On Thu, Sep 10, 2026 at 04:20:29AM +0100, Jonathan Cameron wrote:
> > I've dropped the branch from IIO. That's fine because it was still being
> > tested and hasn't gone out as non-rebasing yet.  
> 
> Ack, will do first thing tomorrow.
> 
> > Next time I'll maybe find an IRC gateway and ping you on that.  I can't
> > find any reference to that being your preference in the kernel docs.
> > Given you have a helpful page on tip, might be worth adding a note.
> > I might of course be missing it.  
> 
> I'll send you a private note after this.
> 
> > If it goes into a reasonably early rc, I'll just merge the relevant rc
> > rather than needing an immutable. If next cycle, immutable please.  
> 
> First one can go to Linus now, second one would go next cycle. Do you want to
> base ontop of both?

Only need the first patch, so I'll just pick it up by merging
an RC once it this is upstream.

Thanks,

Jonathan

> 
> > I started out a bit grumpy that I had to hold a non trivial series for
> > a cycle because of an x86 bug (having been very happy when David
> > fixed it incredibly quickly!), that got reviewed then disappeared
> > down a hole and didn't look like it would ever be seen again.
> > Meh, got me on a belligerent day.  
> 
> No worries, happens to all of us.
> 
> Thanks!
> 

Re: [PATCH 0/2] x86: Use "er" asm constriant for add/sub

[Borislav Petkov]

On Thu, Sep 10, 2026 at 05:09:13PM +0100, Jonathan Cameron wrote:
> Only need the first patch, so I'll just pick it up by merging
> an RC once it this is upstream.

Pushed out into tip:x86/urgent. That branch should be mainline with -rc3.

If something last-minute changes, lemme know and I'll do an immutable branch.

Thanks!

-- 
Regards/Gruss,
    Boris.

https://people.kernel.org/tglx/notes-about-netiquette

Re: [PATCH 0/2] x86: Use "er" asm constriant for add/sub

[Borislav Petkov]

On Thu, Sep 10, 2026 at 11:00:02AM -0700, Borislav Petkov wrote:
> Pushed out into tip:x86/urgent. That branch should be mainline with -rc3.
> 
> If something last-minute changes, lemme know and I'll do an immutable
> branch.

It changed - -rc3 released and we didn't send the pull in time:

https://lore.kernel.org/r/20260913225356.1033865-1-dave.hansen@linux.intel.com

I can let you know when Linus merges it.

-- 
Regards/Gruss,
    Boris.

https://people.kernel.org/tglx/notes-about-netiquette