Bug#1067243: openssh: please build without -fzero-call-used-regs=used on m68k

2024-03-24 Thread Colin Watson
On Sat, Mar 23, 2024 at 01:49:19AM +, Thorsten Glaser wrote:
> Colin Watson dixit:
> >Could you try the somewhat further reduced patch in
> 
> The package made from that branch built fine in my cowbuilder,
> and I have all reason to assume it’ll do so in sbuild/buildd.

Thanks for the testing!

-- 
Colin Watson (he/him)  [cjwat...@debian.org]



Bug#1067243: openssh: please build without -fzero-call-used-regs=used on m68k

2024-03-23 Thread John Paul Adrian Glaubitz
Hi!

On Thu, 2024-03-21 at 16:26 +, Thorsten Glaser wrote:
> > please?  I wanted to use the mitchy.debian.net porterbox but I got
> > ECONNREFUSED.
> 
> Adrian, do you have an idea about that?

mitchy is back up again. The hosting server was forcefully rebooted
without me being notified. I assume that happened due to maintenance
reasons since the other Gandi machines were rebooted as well.

Adrian

-- 
 .''`.  John Paul Adrian Glaubitz
: :' :  Debian Developer
`. `'   Physicist
  `-GPG: 62FF 8A75 84E0 2956 9546  0006 7426 3B37 F5B5 F913



Bug#1067243: openssh: please build without -fzero-call-used-regs=used on m68k

2024-03-22 Thread Thorsten Glaser
Colin Watson dixit:

>Could you try the somewhat further reduced patch in

The package made from that branch built fine in my cowbuilder,
and I have all reason to assume it’ll do so in sbuild/buildd.

Thanks,
//mirabilos
-- 
 you introduced a merge commit│ % g rebase -i HEAD^^
 sorry, no idea and rebasing just fscked │ Segmentation
 should have cloned into a clean repo  │  fault (core dumped)
 if I rebase that now, it's really ugh │ wuahh



Bug#1067243: openssh: please build without -fzero-call-used-regs=used on m68k

2024-03-21 Thread Thorsten Glaser
Colin Watson dixit:

>Thanks!  No rush - I won't be at a proper computer until Sunday or so
>anyway.

OK sure… no rush is not the reason, the Atari VM I’m using having
only 98 MHz is the one here ;-)

But I already see:

checking if cc supports compile flag -fzero-call-used-regs=used and linking 
succeeds... no

So I guess it should work, but I’ll see tomorrow.

bye,
//mirabilos
-- 
16:47⎜«mika:#grml» .oO(mira ist einfach gut)  23:22⎜«mikap:#grml»
mirabilos: und dein bootloader ist geil :)23:29⎜«mikap:#grml» und ich
finds saugeil dass ich ein bsd zum booten mit grml hab, das muss ich dann
gleich mal auf usb-stick installieren   -- Michael Prokop über MirOS bsd4grml



Bug#1067243: openssh: please build without -fzero-call-used-regs=used on m68k

2024-03-21 Thread Colin Watson
On Thu, Mar 21, 2024 at 10:35:17PM +, Thorsten Glaser wrote:
> Colin Watson dixit:
> >Could you try the somewhat further reduced patch in
> 
> I’ve started a build and will let you know probably when I get
> back late tomorrow.

Thanks!  No rush - I won't be at a proper computer until Sunday or so
anyway.

-- 
Colin Watson (he/him)  [cjwat...@debian.org]



Bug#1067243: openssh: please build without -fzero-call-used-regs=used on m68k

2024-03-21 Thread Thorsten Glaser
Colin Watson dixit:

>Could you try the somewhat further reduced patch in

I’ve started a build and will let you know probably when I get
back late tomorrow.

bye,
//mirabilos
-- 
18:47⎜ well channels… you see, I see everything in the
same window anyway  18:48⎜ i know, you have some kind of
telnet with automatic pong 18:48⎜ haha, yes :D
18:49⎜ though that's more tinyirc – sirc is more comfy



Bug#1067243: openssh: please build without -fzero-call-used-regs=used on m68k

2024-03-21 Thread Thorsten Glaser
Colin Watson dixit:

>I don't love overriding snprintf here, since it seems possible that that

Agreed.

>could disturb the check on other architectures.  Could you try the
>somewhat further reduced patch in
>https://salsa.debian.org/ssh-team/openssh/-/tree/zero-call-used-regs-m68k,

Yes, but not sure I manage it tonight, and I’ll be gone all day
tomorrow.

>please?  I wanted to use the mitchy.debian.net porterbox but I got
>ECONNREFUSED.

Adrian, do you have an idea about that?

>This configure check doesn't use the usual autoconf result caching
>arrangements, which makes it a bit more awkward to override from
>debian/rules.  There are options, but an extended configure check that I
>could send upstream would probably be best.

oic, yes, that’s definitely easier.

For now I’ve built the last upload with the flag manually removed,
so this isn’t a transition blocker any more, until the next upload
anyway and perhaps not even then…

bye,
//mirabilos
-- 
  "Using Lynx is like wearing a really good pair of shades: cuts out
   the glare and harmful UV (ultra-vanity), and you feel so-o-o COOL."
 -- Henry Nelson, March 1999



Bug#1067243: openssh: please build without -fzero-call-used-regs=used on m68k

2024-03-21 Thread Colin Watson
On Wed, Mar 20, 2024 at 07:14:42PM +, Thorsten Glaser wrote:
> I can confirm that appending…
> 
> int snprintf_eta;
> double snprintf_time_per_line;
> int snprintf(char *str, size_t size, const char *format, ...) {
>   snprintf_eta = snprintf_time_per_line;
> }
> 
> … (lightly changed from the above) to the program from
> m4/openssh.m4 OSSH_COMPILER_FLAG_TEST_PROGRAM fails with:
> 
> (pbuild-15711)root@ara2:/tmp# gcc -O2 -fPIE -fno-strict-aliasing 
> -fzero-call-used-regs=used t.c
> during RTL pass: zero_call_used_regs
> t.c: In function 'snprintf':
> t.c:51:1: internal compiler error: in change_address_1, at emit-rtl.cc:2287
>51 | }
>   | ^
> […]

I don't love overriding snprintf here, since it seems possible that that
could disturb the check on other architectures.  Could you try the
somewhat further reduced patch in
https://salsa.debian.org/ssh-team/openssh/-/tree/zero-call-used-regs-m68k,
please?  I wanted to use the mitchy.debian.net porterbox but I got
ECONNREFUSED.

> Alternatively, just hardcode disabling this flag on m68k for now,
> which we’ll eventually have to revert once GCC is on a fixed release
> (14 probably).

This configure check doesn't use the usual autoconf result caching
arrangements, which makes it a bit more awkward to override from
debian/rules.  There are options, but an extended configure check that I
could send upstream would probably be best.

Thanks,

-- 
Colin Watson (he/him)  [cjwat...@debian.org]



Bug#1067243: openssh: please build without -fzero-call-used-regs=used on m68k

2024-03-20 Thread Thorsten Glaser
Source: openssh
Version: 1:9.7p1-2
Severity: important
Justification: FTBFS on d-ports arch
Tags: ftbfs
X-Debbugs-Cc: t...@mirbsd.de, debian-...@lists.debian.org

On m68k, gcc-13 currently ICEs when -fzero-call-used-regs=used is
used (see #1066891) in moduli.c, packet.c, etc. but the configury
detects it and so it gets used. The upstream GCC bug comments say
there is no plan to backport the possible fix to releases, but it
has a short reproducer by doko:


$ cat moduli.i
int snprintf_eta;
double snprintf_time_per_line;
int snprintf(char *, int, char *, ...) {
  snprintf_eta = snprintf_time_per_line;
}

$ m68k-linux-gnu-gcc -c -O2 -fzero-call-used-regs=used -fPIE moduli.i
during RTL pass: zero_call_used_regs
moduli.i: In function ‘snprintf’:
moduli.i:5:1: internal compiler error: in change_address_1, at emit-rtl.cc:2287


Maybe this could be used in the configure script?

I can confirm that appending…

int snprintf_eta;
double snprintf_time_per_line;
int snprintf(char *str, size_t size, const char *format, ...) {
  snprintf_eta = snprintf_time_per_line;
}

… (lightly changed from the above) to the program from
m4/openssh.m4 OSSH_COMPILER_FLAG_TEST_PROGRAM fails with:

(pbuild-15711)root@ara2:/tmp# gcc -O2 -fPIE -fno-strict-aliasing 
-fzero-call-used-regs=used t.c
during RTL pass: zero_call_used_regs
t.c: In function 'snprintf':
t.c:51:1: internal compiler error: in change_address_1, at emit-rtl.cc:2287
   51 | }
  | ^
[…]


Alternatively, just hardcode disabling this flag on m68k for now,
which we’ll eventually have to revert once GCC is on a fixed release
(14 probably).

Thanks in advance!