On 1/25/21 10:08 PM, John David Anglin wrote:
> I would suggest the following for this hunk:
>
> +    ldil    L%intr_restore, %r2
> +    BL    preempt_schedule_irq
> +    ldo     R%intr_restore(%r2), %r2
>
>     ldil    L%intr_restore, %r1
>     b,l    preempt_schedule_irq,%r2
>     ldo     R%intr_restore(%r1), %r2
>
> On PA 2.0 hardware that gives a 22-bit call.

"BL" is already using "b,l", see #define in arch/parisc/include/asm/assembly.h

The 22-bit weren't sufficient, that's why I changed it too.

Helge

Reply via email to