On Mon, 30 Nov 2020 at 18:52, Nicolas Pitre wrote:
>
> On Mon, 30 Nov 2020, Ard Biesheuvel wrote:
>
> > On Mon, 30 Nov 2020 at 16:51, Nicolas Pitre wrote:
> >
> > > Here's my version of the fix which should be correct. Warning: this
> > > is completely untested, but should in theory produce the s
On Mon, 30 Nov 2020, Ard Biesheuvel wrote:
> On Mon, 30 Nov 2020 at 16:51, Nicolas Pitre wrote:
>
> > Here's my version of the fix which should be correct. Warning: this
> > is completely untested, but should in theory produce the same code on
> > modern gcc.
> >
> > diff --git a/arch/arm/includ
On Mon, 30 Nov 2020 at 16:51, Nicolas Pitre wrote:
>
> On Mon, 30 Nov 2020, Ard Biesheuvel wrote:
>
> > (+ Nico)
> >
> > On Mon, 30 Nov 2020 at 11:11, Ard Biesheuvel wrote:
> > >
> > > On Mon, 23 Nov 2020 at 08:39, Antony Yu wrote:
> > > >
> > > > __do_div64 clobbers the input register r0 in lit
On Mon, 30 Nov 2020, Ard Biesheuvel wrote:
> (+ Nico)
>
> On Mon, 30 Nov 2020 at 11:11, Ard Biesheuvel wrote:
> >
> > On Mon, 23 Nov 2020 at 08:39, Antony Yu wrote:
> > >
> > > __do_div64 clobbers the input register r0 in little endian system.
> > > According to the inline assembly document, if
On Mon, Nov 30, 2020 at 01:58:27PM +, David Laight wrote:
> > And actually, the same applies on BE, but the other way around. So we
> > should mark __xl as an output register as well, as __xl will assume
> > the right value depending on the endianness.
>
> Why not use "+r" to indicate than an
> And actually, the same applies on BE, but the other way around. So we
> should mark __xl as an output register as well, as __xl will assume
> the right value depending on the endianness.
Why not use "+r" to indicate than an 'output' parameter is also
used as an input.
Rather cleaner than specif
On Mon, 30 Nov 2020 at 11:21, Russell King - ARM Linux admin
wrote:
>
> On Mon, Nov 30, 2020 at 11:12:33AM +0100, Ard Biesheuvel wrote:
> > (+ Nico)
> >
> > On Mon, 30 Nov 2020 at 11:11, Ard Biesheuvel wrote:
> > >
> > > On Mon, 23 Nov 2020 at 08:39, Antony Yu wrote:
> > > >
> > > > __do_div64 c
On Mon, Nov 30, 2020 at 11:12:33AM +0100, Ard Biesheuvel wrote:
> (+ Nico)
>
> On Mon, 30 Nov 2020 at 11:11, Ard Biesheuvel wrote:
> >
> > On Mon, 23 Nov 2020 at 08:39, Antony Yu wrote:
> > >
> > > __do_div64 clobbers the input register r0 in little endian system.
> > > According to the inline a
On Mon, 23 Nov 2020 at 08:39, Antony Yu wrote:
>
> __do_div64 clobbers the input register r0 in little endian system.
> According to the inline assembly document, if an input operand is
> modified, it should be tied to a output operand. This patch can
> prevent compilers from reusing r0 register a
(+ Nico)
On Mon, 30 Nov 2020 at 11:11, Ard Biesheuvel wrote:
>
> On Mon, 23 Nov 2020 at 08:39, Antony Yu wrote:
> >
> > __do_div64 clobbers the input register r0 in little endian system.
> > According to the inline assembly document, if an input operand is
> > modified, it should be tied to a ou
Hi Antony,
Thank you for the patch! Yet something to improve:
[auto build test ERROR on linus/master]
[also build test ERROR on v5.10-rc5 next-20201124]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
h
Thanks for the report, it probably was not fun to debug. I'll take a
closer look at this after the Thanksgiving holiday.
On Tue, Nov 24, 2020 at 2:14 AM Antony Yu wrote:
>
> Antony Yu 於 2020年11月24日 週二 下午3:43寫道:
> >
> > On Mon, Nov 23, 2020 at 11:16:02AM -0700, Nathan Chancellor wrote:
> > > On M
Antony Yu 於 2020年11月24日 週二 下午3:43寫道:
>
> On Mon, Nov 23, 2020 at 11:16:02AM -0700, Nathan Chancellor wrote:
> > On Mon, Nov 23, 2020 at 03:36:32PM +0800, Antony Yu wrote:
> > > __do_div64 clobbers the input register r0 in little endian system.
> > > According to the inline assembly document, if an
On Mon, Nov 23, 2020 at 03:36:32PM +0800, Antony Yu wrote:
> __do_div64 clobbers the input register r0 in little endian system.
> According to the inline assembly document, if an input operand is
> modified, it should be tied to a output operand. This patch can
> prevent compilers from reusing r0 r
__do_div64 clobbers the input register r0 in little endian system.
According to the inline assembly document, if an input operand is
modified, it should be tied to a output operand. This patch can
prevent compilers from reusing r0 register after asm statements.
Signed-off-by: Antony Yu
---
arch/
15 matches
Mail list logo