• Important packages that are broken ia64

    From John Paul Adrian Glaubitz@21:1/5 to All on Sun Aug 6 10:50:02 2023
    Hello!

    The following important packages are broken on ia64:

    - grub (git master) does not boot on ia64, crashes when loading stage2 (https://lists.gnu.org/archive/html/grub-devel/2023-07/msg00106.html)
    - kernel FTBFS with gcc-13 (https://buildd.debian.org/status/fetch.php?pkg=linux&arch=ia64&ver=6.4.4-2&stamp=1690708282&raw=0)
    - glibc testsuite failures, in particular math functions (https://buildd.debian.org/status/fetch.php?pkg=glibc&arch=ia64&ver=2.37-7&stamp=1691155973&raw=0)
    - webkit2gtk (https://gitlab.gnome.org/GNOME/gobject-introspection/-/issues/471)

    At least the grub problem could certainly be easily bisected, but alas I don't really
    have the time and motivation do that. So it would be nice if someone else could at
    least bisect grub to find the issue and report it on the grub-devel mailing list.

    Adrian

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

    --- SoupGate-Win32 v1.05
    * Origin: fsxNet Usenet Gateway (21:1/5)
  • From John Paul Adrian Glaubitz@21:1/5 to Pedro Miguel Justo on Sun Aug 6 19:20:01 2023
    Hi!

    On Sun, 2023-08-06 at 15:08 +0000, Pedro Miguel Justo wrote:
    ow much automation can be used for bisecting a problem on grub?

    My rx2660 is considerably up to date and still booting.

    What do you recommend for best bisecting the offending step?

    You cannot automate this as the machine needs to be rebooted all the time.

    Basically, you need to do the following:

    # apt build-dep grub2
    # git clone git://git.savannah.gnu.org/grub.git
    # cd grub
    # git bisect start
    # git bisect good grub-2.06
    # git bisect bad HEAD

    Then build grub, install and testboot:

    # ./bootstrap && ./configure && make && make install
    # /usr/local/sbin/grub-install /dev/sda (when sda is your primary disk)

    Reboot the machine and chose the "grub" boot entry, "debian" will be your fallback entry in case you have run into the bug.

    If the machine boot successfully using the "grub" boot entry in EFI, go back into the "grub" source directory and run "git bisect good". Run "make clean" and repeat the build and install step above.

    If a boot fails, reset the machine, boot the "debian" entry and then go back into the "grub" source directory and run "git bisect bad". Run "make clean"
    and repeat the build and install step above.

    Repeat this procedure until git shows you the offending commit.

    Adrian

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

    --- SoupGate-Win32 v1.05
    * Origin: fsxNet Usenet Gateway (21:1/5)
  • From Jessica Clarke@21:1/5 to Pedro Miguel Justo on Mon Aug 7 01:20:01 2023
    On 7 Aug 2023, at 00:00, Pedro Miguel Justo <pmsjt@texair.net> wrote:



    On Aug 6, 2023, at 10:13, John Paul Adrian Glaubitz <glaubitz@physik.fu-berlin.de> wrote:

    Hi!

    On Sun, 2023-08-06 at 15:08 +0000, Pedro Miguel Justo wrote:
    ow much automation can be used for bisecting a problem on grub?

    My rx2660 is considerably up to date and still booting.

    What do you recommend for best bisecting the offending step?

    You cannot automate this as the machine needs to be rebooted all the time.

    That is what I was expecting.


    Basically, you need to do the following:

    # apt build-dep grub2

    Sourt of embarrassing question but… what is the deb-src right entry for debian ports?

    Same as non-ports; there’s no ports copy of the unstable sources.

    Jess

    --- SoupGate-Win32 v1.05
    * Origin: fsxNet Usenet Gateway (21:1/5)
  • From John Paul Adrian Glaubitz@21:1/5 to Pedro Miguel Justo on Mon Aug 7 10:10:01 2023
    On Sun, 2023-08-06 at 23:34 +0000, Pedro Miguel Justo wrote:
    # git clone git://git.savannah.gnu.org/grub.git

    This leads me to another question: Has anyone been able to install git these days?

    # wget http://snapshot.debian.org/archive/debian/20220510T034305Z/pool/main/g/git/git-man_2.36.1-1_all.deb
    # dpkg -i git-man_2.36.1-1_all.deb

    Then install git normally with apt-get.

    Adrian

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

    --- SoupGate-Win32 v1.05
    * Origin: fsxNet Usenet Gateway (21:1/5)
  • From John Paul Adrian Glaubitz@21:1/5 to Pedro Miguel Justo on Wed Aug 9 08:50:01 2023
    On Tue, 2023-08-08 at 23:27 +0000, Pedro Miguel Justo wrote:

    On Aug 7, 2023, at 01:03, John Paul Adrian Glaubitz <glaubitz@physik.fu-berlin.de> wrote:

    # wget http://snapshot.debian.org/archive/debian/20220510T034305Z/pool/main/g/git/git-man_2.36.1-1_all.deb
    # dpkg -i git-man_2.36.1-1_all.deb

    Then install git normally with apt-get.


    Thanks Adrian.

    From there, all good till the grub-install step, which failed:

    pmsjt@itanium:~/grub$ sudo /usr/local/sbin/grub-install /dev/sda
    Installing for ia64-efi platform.
    /usr/local/sbin/grub-install: warning: cannot open directory `/usr/local/share/locale': No such file or directory.

    Unless you care about localization (which you hopefully don't during debugging),
    you can safely ignore this warning.

    /usr/local/sbin/grub-install: error: efibootmgr: not found.

    Well, you need to install efibootmgr. Not sure why it isn't installed on your machine in the first place since it's needed for grub-install to update the boot manager settings in firmware and create the "grub" entry.

    Just install it with:

    # apt install efibootmgr

    Adrian

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

    --- SoupGate-Win32 v1.05
    * Origin: fsxNet Usenet Gateway (21:1/5)
  • From Jason Duerstock@21:1/5 to pmsjt@texair.net on Thu Aug 10 03:50:01 2023
    Hi Pedro,

    Did you finish the git bisect? What was the commit that broke it?

    Thanks,

    Jason

    On Wed, Aug 9, 2023 at 9:41 PM Pedro Miguel Justo <pmsjt@texair.net> wrote:


    Well, you need to install efibootmgr. Not sure why it isn't installed on your
    machine in the first place since it's needed for grub-install to update the boot manager settings in firmware and create the "grub" entry.

    Just install it with:

    # apt install efibootmgr

    Adrian


    Thanks Adrian. It is a mystery why it was previously installed. After that I was able to go through 3 or 4 bisections with successful boots. Then, finally, I hit the problem. The trouble is that it seems to have also affected my “debian” entry, so
    no safe boot:

    Loading.: debian
    Starting: debian
    Welcome to GRUB!

    7 0 0x00006B 0x000000000000001E unexpected trap
    7 0 0x000066 0x000000000000001E trap taken, number in ext PE
    7 0 0x00003C 0x0000000000005A00 trap taken, offset in ext PE

    --- SoupGate-Win32 v1.05
    * Origin: fsxNet Usenet Gateway (21:1/5)
  • From John Paul Adrian Glaubitz@21:1/5 to Pedro Miguel Justo on Thu Aug 10 08:20:01 2023
    Hi Pedro!

    On Thu, 2023-08-10 at 00:08 +0000, Pedro Miguel Justo wrote:

    Thanks Adrian. It is a mystery why it was previously installed. After that
    I was able to go through 3 or 4 bisections with successful boots. Then, finally, I hit the problem. The trouble is that it seems to have also affected
    my “debian” entry, so no safe boot:

    Are you sure you didn't overwrite the "debian" entry?

    Either way, you should be able to restore the bootloader by booting into rescue mode using the installation media.

    Adrian

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

    --- SoupGate-Win32 v1.05
    * Origin: fsxNet Usenet Gateway (21:1/5)
  • From John Paul Adrian Glaubitz@21:1/5 to Pedro Miguel Justo on Thu Aug 10 14:40:01 2023
    Hi Pedro!

    On Thu, 2023-08-10 at 12:04 +0000, Pedro Miguel Justo wrote:
    Here it is:

    06edd40db76bb78457ac26156ed5f7b62381bbe8 is the first bad commit
    commit 06edd40db76bb78457ac26156ed5f7b62381bbe8
    Author: Oliver Steffen <osteffen@redhat.com>
    Date: Fri May 26 13:35:43 2023 +0200

    guid: Unify GUID types

    There are 3 implementations of a GUID in GRUB. Replace them with
    a common one, placed in types.h.

    It uses the "packed" flavor of the GUID structs, the alignment attribute
    is dropped, since it is not required.

    Signed-off-by: Oliver Steffen <osteffen@redhat.com>
    Reviewed-by: Daniel Kiper <daniel.kiper@oracle.com>

    Did you verify that reverting this commit on git master makes GRUB work again?

    If yes, please report this finding to the grub-devel mailing list, CC-ing Oliver Steffen.

    Adrian

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

    --- SoupGate-Win32 v1.05
    * Origin: fsxNet Usenet Gateway (21:1/5)
  • From Frank Scheiner@21:1/5 to Pedro Miguel Justo on Thu Aug 10 19:40:02 2023
    Hi Pedro, all,

    On 10.08.23 14:04, Pedro Miguel Justo wrote:
    Here it is:

    06edd40db76bb78457ac26156ed5f7b62381bbe8 is the first bad commit
    commit 06edd40db76bb78457ac26156ed5f7b62381bbe8
    Author: Oliver Steffen <osteffen@redhat.com>
    Date: Fri May 26 13:35:43 2023 +0200

    guid: Unify GUID types

    There are 3 implementations of a GUID in GRUB. Replace them with
    a common one, placed in types.h.

    It uses the "packed" flavor of the GUID structs, the alignment attribute
    is dropped, since it is not required.

    Signed-off-by: Oliver Steffen <osteffen@redhat.com>
    Reviewed-by: Daniel Kiper <daniel.kiper@oracle.com>


    I had a quick look into this and the ia64 "related" part ([1]) switches
    from an `__attribute__ ((aligned(8)))` struct "grub_efi_guid_t" to an `__attribute__ ((packed))` struct "grub_guid_t" (see [2], [3] and
    below), which might not work as expected on ia64.

    `include/grub/types.h` defines:

    ```
    #define GRUB_PACKED __attribute__ ((packed))
    ```

    [1]: https://git.savannah.gnu.org/cgit/grub.git/diff/grub-core/loader/ia64/efi/linux.c?id=06edd40db76bb78457ac26156ed5f7b62381bbe8

    [2]: https://git.savannah.gnu.org/cgit/grub.git/diff/include/grub/efi/api.h?id=06edd40db76bb78457ac26156ed5f7b62381bbe8

    [3]: https://git.savannah.gnu.org/cgit/grub.git/diff/include/grub/types.h?id=06edd40db76bb78457ac26156ed5f7b62381bbe8

    Not sure if it could help to just replace `GRUB_PACKED` with
    `__attribute__ ((aligned(8)))` for "grub_guid_t" in [3] or if this
    breaks the cases that originally used "grub_efi_packed_guid_t".

    Cheers,
    Frank

    --- SoupGate-Win32 v1.05
    * Origin: fsxNet Usenet Gateway (21:1/5)
  • From Frank Scheiner@21:1/5 to All on Fri Aug 11 08:40:01 2023
    SGkgUGVkcm8sDQoNCk9uIDEwLjA4LjIzIDE5OjMxLCBGcmFuayBTY2hlaW5lciB3cm90ZToN Cj4gSGkgUGVkcm8sIGFsbCwNCj4gDQo+IE9uIDEwLjA4LjIzIDE0OjA0LCBQZWRybyBNaWd1 ZWwgSnVzdG8gd3JvdGU6DQo+PiBIZXJlIGl0IGlzOg0KPj4NCj4+IDA2ZWRkNDBkYjc2YmI3 ODQ1N2FjMjYxNTZlZDVmN2I2MjM4MWJiZTggaXMgdGhlIGZpcnN0IGJhZCBjb21taXQNCj4+ IGNvbW1pdCAwNmVkZDQwZGI3NmJiNzg0NTdhYzI2MTU2ZWQ1ZjdiNjIzODFiYmU4DQo+PiBB dXRob3I6IE9saXZlciBTdGVmZmVuIDxvc3RlZmZlbkByZWRoYXQuY29tPg0KPj4gRGF0ZTrC oMKgIEZyaSBNYXkgMjYgMTM6MzU6NDMgMjAyMyArMDIwMA0KPj4NCj4+IMKgwqDCoMKgIGd1 aWQ6IFVuaWZ5IEdVSUQgdHlwZXMNCj4+DQo+PiDCoMKgwqDCoCBUaGVyZSBhcmUgMyBpbXBs ZW1lbnRhdGlvbnMgb2YgYSBHVUlEIGluIEdSVUIuIFJlcGxhY2UgdGhlbSB3aXRoDQo+PiDC oMKgwqDCoCBhIGNvbW1vbiBvbmUsIHBsYWNlZCBpbiB0eXBlcy5oLg0KPj4NCj4+IMKgwqDC oMKgIEl0IHVzZXMgdGhlICJwYWNrZWQiIGZsYXZvciBvZiB0aGUgR1VJRCBzdHJ1Y3RzLCB0 aGUgYWxpZ25tZW50IA0KPj4gYXR0cmlidXRlDQo+PiDCoMKgwqDCoCBpcyBkcm9wcGVkLCBz aW5jZSBpdCBpcyBub3QgcmVxdWlyZWQuDQo+Pg0KPj4gwqDCoMKgwqAgU2lnbmVkLW9mZi1i eTogT2xpdmVyIFN0ZWZmZW4gPG9zdGVmZmVuQHJlZGhhdC5jb20+DQo+PiDCoMKgwqDCoCBS ZXZpZXdlZC1ieTogRGFuaWVsIEtpcGVyIDxkYW5pZWwua2lwZXJAb3JhY2xlLmNvbT4NCj4+ DQo+IA0KPiBJIGhhZCBhIHF1aWNrIGxvb2sgaW50byB0aGlzIGFuZCB0aGUgaWE2NCAicmVs YXRlZCIgcGFydCAoWzFdKSBzd2l0Y2hlcw0KPiBmcm9tIGFuIGBfX2F0dHJpYnV0ZV9fICgo YWxpZ25lZCg4KSkpYCBzdHJ1Y3QgImdydWJfZWZpX2d1aWRfdCIgdG8gYW4NCj4gYF9fYXR0 cmlidXRlX18gKChwYWNrZWQpKWAgc3RydWN0ICJncnViX2d1aWRfdCIgKHNlZSBbMl0sIFsz XSBhbmQNCj4gYmVsb3cpLCB3aGljaCBtaWdodCBub3Qgd29yayBhcyBleHBlY3RlZCBvbiBp YTY0Lg0KDQpEYW1uIGl0LCBJIHNob3VsZCBoYXZlIHJlYWQgdGhlIGNvbW1pdCBtZXNzYWdl IGluc3RlYWQgb2YgdGhlIGNvZGUgDQpjaGFuZ2VzLiBJdCB0ZWxscyB0aGUgLSBtYXliZSAt IGltcG9ydGFudCB0aGluZyBhbHJlYWR5Lg0KDQpVbmZvcnR1bmF0ZWx5IHRoZXJlJ3Mgbm8g aW5mbyBpbiBbNF0gKHdoZXJlIHRoaXMgd2FzIGludHJvZHVjZWQpIGFib3V0IA0Kd2h5IGV4 YWN0bHkgdGhlIGFsaWdtZW50IHdhcyBpbnRlbmRlZCBpbiB0aGUgZmlyc3QgcGxhY2UuDQoN Cls0XTogDQpodHRwczovL2dpdC5zYXZhbm5haC5nbnUub3JnL2NnaXQvZ3J1Yi5naXQvY29t bWl0Lz9pZD04MzcwOTEyNThkN2U5ZjRhZjNjYzMzM2VjNzc1MjcxZjFiNzY3ZDExDQoNCkxp bnV4IGFjdHVhbGx5IHVzZXMgYSBtaW5pbXVtIGFsaWdubWVudCBvZiAzMiBiaXRzLCBzZWUg Zm9yIGV4YW1wbGUgWzVdLg0KDQpbNV06IA0KaHR0cHM6Ly9naXRodWIuY29tL3RvcnZhbGRz L2xpbnV4L2Jsb2IvbWFzdGVyL2luY2x1ZGUvbGludXgvZWZpLmgjTDYwLUw3OA0KDQpAUGVk cm86DQpEb2VzIGl0IHdvcmsgYWdhaW4gd2l0aCBgX19hdHRyaWJ1dGVfXyAoKGFsaWduZWQo OCkpKWAgaW5zdGVhZCBvZiANCmBfX2F0dHJpYnV0ZV9fICgocGFja2VkKSlgIGZvciAiZ3J1 Yl9ndWlkIj8NCg0KQ2hlZXJzLA0KRnJhbmsNCg==

    --- SoupGate-Win32 v1.05
    * Origin: fsxNet Usenet Gateway (21:1/5)
  • From Frank Scheiner@21:1/5 to John Paul Adrian Glaubitz on Fri Sep 15 13:50:01 2023
    Hi Adrian,

    On 06.08.23 10:44, John Paul Adrian Glaubitz wrote:
    Hello!

    The following important packages are broken on ia64:

    - grub (git master) does not boot on ia64, crashes when loading stage2 (https://lists.gnu.org/archive/html/grub-devel/2023-07/msg00106.html)
    - kernel FTBFS with gcc-13 (https://buildd.debian.org/status/fetch.php?pkg=linux&arch=ia64&ver=6.4.4-2&stamp=1690708282&raw=0)

    I created a bug report for this, see [1]. In short this happens (1) also
    for cross-compilation on amd64 (most likely also for other
    cross-compilation host arches), (2) for me always for the same file (`net/ipv4/fib_semantics.c`) and function ("fib_create_info()"), but
    there are others, too, see buildd link, (3) for different kernel
    versions and (3) is gone when providing `-fsanitize=undefined` during compilation.

    Details in [1].

    [1]: https://gcc.gnu.org/bugzilla/show_bug.cgi?id=111425

    I hope someone picks that up.

    Cheers,
    Frank

    --- SoupGate-Win32 v1.05
    * Origin: fsxNet Usenet Gateway (21:1/5)
  • From Frank Scheiner@21:1/5 to Frank Scheiner on Mon Sep 18 11:50:01 2023
    Hi again,

    On 15.09.23 13:47, Frank Scheiner wrote:
    Hi Adrian,

    On 06.08.23 10:44, John Paul Adrian Glaubitz wrote:
    Hello!

    The following important packages are broken on ia64:

    - grub (git master) does not boot on ia64, crashes when loading stage2
    (https://lists.gnu.org/archive/html/grub-devel/2023-07/msg00106.html)
    - kernel FTBFS with gcc-13
    (https://buildd.debian.org/status/fetch.php?pkg=linux&arch=ia64&ver=6.4.4-2&stamp=1690708282&raw=0)

    I created a bug report for this, see [1]. In short this happens (1) also
    for cross-compilation on amd64 (most likely also for other
    cross-compilation host arches), (2) for me always for the same file (`net/ipv4/fib_semantics.c`) and function ("fib_create_info()"), but
    there are others, too, see buildd link, (3) for different kernel
    versions and (3) is gone when providing `-fsanitize=undefined` during compilation.

    Details in [1].

    [1]: https://gcc.gnu.org/bugzilla/show_bug.cgi?id=111425

    I hope someone picks that up.

    Richard Biener suggested `-fno-var-tracking` or `-g0` as workaround and
    both indeed workaround the problem (see [2]). I don't yet know how to
    limit the addition of `-fno-var-tracking` to KBUILD_CFLAGS to `net/ipv4/fib_semantics.c` but the following patch:

    ```
    root@dl380-g7:/usr/src# cat workaround-for-gcc-13.patch
    diff --git a/net/ipv4/Makefile b/net/ipv4/Makefile
    index b18ba8ef93ad..38f8586ebcc6 100644
    --- a/net/ipv4/Makefile
    +++ b/net/ipv4/Makefile
    @@ -3,6 +3,8 @@
    # Makefile for the Linux TCP/IP (INET) layer.
    #

    +KBUILD_CFLAGS += -fno-var-tracking
    +
    obj-y := route.o inetpeer.o protocol.o \
    ip_input.o ip_fragment.o ip_forward.o ip_options.o \
    ip_output.o ip_sockglue.o inet_hashtables.o \

    ```

    ...makes compilation work for the used kernel configuration ([3]):

    ```
    root@dl380-g7:/usr/src# time ./make-kernel.bash rx2620-rx2660-rx2800-i2-combined-localmodconfig ia64 linux-on-ramdisk/torvalds-linux/ w-gcc13
    [...]
    Making kernel...
    time make -j24 LOCALVERSION="-ce9ecca0238b140b88f43859b211c9fdfd8e5b70-ia64-w-gcc13"
    ARCH=ia64 CROSS_COMPILE=ia64-linux- all
    [...]
    CC security/integrity/ima/ima_fs.o
    CC net/ipv4/fib_semantics.o
    CC security/integrity/ima/ima_queue.o
    [...]
    No errors detected in 26726 functions.

    real 3m41.183s
    user 69m23.751s
    sys 6m57.374s

    0
    [...]
    done
    Build artifacts:
    /boot/vmlinux-6.6.0-rc2-ce9ecca0238b140b88f438
  • From Frank Scheiner@21:1/5 to Frank Scheiner on Mon Sep 18 13:10:01 2023
    Hi again,

    On 18.09.23 11:43, Frank Scheiner wrote:
    The resulting kernel and modules aren't yet tested, though. I plan that
    for today and tomorrow.

    v6.6-rc2 with the acpi build fix (I used the patch from [1]) and the
    workaround for gcc-13 (see my prior email) and built with gcc 13.2.0
    from [2] works fine (boot to login prompt) on:

    * rx2620
    * rx2800 i2

    More test(ed) systems to follow tomorrow.

    Cheers,
    Frank

    [1]: https://lore.kernel.org/lkml/CAJZ5v0hnNK4O_HyinvTp01YxXR7V4vzpMhf85yW9M2=52-O2Fg@mail.gmail.com/

    [2]: https://mirrors.edge.kernel.org/pub/tools/crosstool/

    --- SoupGate-Win32 v1.05
    * Origin: fsxNet Usenet Gateway (21:1/5)
  • From Frank Scheiner@21:1/5 to Frank Scheiner on Mon Sep 18 13:50:01 2023
    On 18.09.23 11:43, Frank Scheiner wrote:
    Richard Biener suggested `-fno-var-tracking` or `-g0` as workaround and
    both indeed workaround the problem (see [2]). I don't yet know how to
    limit the addition of `-fno-var-tracking` to KBUILD_CFLAGS to `net/ipv4/fib_semantics.c` [...]

    Ok, found what I need on [1]:

    ```
    $(CFLAGS_$@) specifies per-file options for $(CC). The @ part has a
    literal value which specifies the file that it is for.
    [...]
    Example:

    # drivers/scsi/Makefile
    CFLAGS_aha152x.o = -DAHA152X_STAT -DAUTOCONF
    ```

    [1]: https://github.com/torvalds/linux/blob/master/Documentation/kbuild/makefiles.rst

    I'll give that a try and if it works create a MR on [2] so kernel builds
    for ia64 can succeed with gcc-13 until that one is fixed.

    [2]: https://salsa.debian.org/kernel-team/linux

    Cheers,
    Frank

    --- SoupGate-Win32 v1.05
    * Origin: fsxNet Usenet Gateway (21:1/5)
  • From John Paul Adrian Glaubitz@21:1/5 to Frank Scheiner on Mon Sep 18 22:40:01 2023
    Hello Frank!

    On Mon, 2023-09-18 at 22:14 +0200, Frank Scheiner wrote:
    Worked for me, MR is here:

    https://salsa.debian.org/kernel-team/linux/-/merge_requests/852

    Looking forward to new ia64 kernels for Sid.

    I don't think this patch is acceptable in its current form as it
    modifies the Makefile globally so that the flag is passed on to
    the host compiler for all architectures.

    Adrian

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

    --- SoupGate-Win32 v1.05
    * Origin: fsxNet Usenet Gateway (21:1/5)
  • From Frank Scheiner@21:1/5 to All on Mon Sep 18 22:20:02 2023
    T24gMTguMDkuMjMgMTM6NDMsIEZyYW5rIFNjaGVpbmVyIHdyb3RlOg0KPiBPbiAxOC4wOS4y MyAxMTo0MywgRnJhbmsgU2NoZWluZXIgd3JvdGU6DQo+PiBSaWNoYXJkIEJpZW5lciBzdWdn ZXN0ZWQgYC1mbm8tdmFyLXRyYWNraW5nYCBvciBgLWcwYCBhcyB3b3JrYXJvdW5kIGFuZA0K Pj4gYm90aCBpbmRlZWQgd29ya2Fyb3VuZCB0aGUgcHJvYmxlbSAoc2VlIFsyXSkuIEkgZG9u J3QgeWV0IGtub3cgaG93IHRvDQo+PiBsaW1pdCB0aGUgYWRkaXRpb24gb2YgYC1mbm8tdmFy LXRyYWNraW5nYCB0byBLQlVJTERfQ0ZMQUdTIHRvDQo+PiBgbmV0L2lwdjQvZmliX3NlbWFu dGljcy5jYCBbLi4uXQ0KPiANCj4gT2ssIGZvdW5kIHdoYXQgSSBuZWVkIG9uIFsxXToNCj4g DQo+IGBgYA0KPiAkKENGTEFHU18kQCkgc3BlY2lmaWVzIHBlci1maWxlIG9wdGlvbnMgZm9y ICQoQ0MpLiBUaGUgQCBwYXJ0IGhhcyBhDQo+IGxpdGVyYWwgdmFsdWUgd2hpY2ggc3BlY2lm aWVzIHRoZSBmaWxlIHRoYXQgaXQgaXMgZm9yLg0KPiBbLi4uXQ0KPiBFeGFtcGxlOg0KPiAN Cj4gIyBkcml2ZXJzL3Njc2kvTWFrZWZpbGUNCj4gQ0ZMQUdTX2FoYTE1MngubyA9wqDCoCAt REFIQTE1MlhfU1RBVCAtREFVVE9DT05GDQo+IGBgYA0KPiANCj4gWzFdOg0KPiBodHRwczov L2dpdGh1Yi5jb20vdG9ydmFsZHMvbGludXgvYmxvYi9tYXN0ZXIvRG9jdW1lbnRhdGlvbi9r YnVpbGQvbWFrZWZpbGVzLnJzdA0KPiANCj4gSSdsbCBnaXZlIHRoYXQgYSB0cnkgYW5kIGlm IGl0IHdvcmtzIGNyZWF0ZSBhIE1SIG9uIFsyXSBzbyBrZXJuZWwgYnVpbGRzDQo+IGZvciBp YTY0IGNhbiBzdWNjZWVkIHdpdGggZ2NjLTEzIHVudGlsIHRoYXQgb25lIGlzIGZpeGVkLg0K PiANCj4gWzJdOiBodHRwczovL3NhbHNhLmRlYmlhbi5vcmcva2VybmVsLXRlYW0vbGludXgN Cg0KV29ya2VkIGZvciBtZSwgTVIgaXMgaGVyZToNCg0KaHR0cHM6Ly9zYWxzYS5kZWJpYW4u b3JnL2tlcm5lbC10ZWFtL2xpbnV4Ly0vbWVyZ2VfcmVxdWVzdHMvODUyDQoNCkxvb2tpbmcg Zm9yd2FyZCB0byBuZXcgaWE2NCBrZXJuZWxzIGZvciBTaWQuDQoNCkNoZWVycywNCkZyYW5r
    DQo=

    --- SoupGate-Win32 v1.05
    * Origin: fsxNet Usenet Gateway (21:1/5)
  • From John Paul Adrian Glaubitz@21:1/5 to Frank Scheiner on Mon Sep 18 22:50:01 2023
    On Mon, 2023-09-18 at 22:36 +0200, Frank Scheiner wrote:
    I don't think this patch is acceptable in its current form as it
    modifies the Makefile globally so that the flag is passed on to
    the host compiler for all architectures.

    Yes, thought about that, too. Know a better solution? Is there a way to
    make this only affecting compilations for ia64?

    You could modify CFLAGS globally in debian/rules specifically for ia64:

    ifneq (,$(filter $(DEB_HOST_ARCH),ia64))
    export DEB_CFLAGS_MAINT_APPEND += -fno-var-tracking
    endif

    Adrian

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

    --- SoupGate-Win32 v1.05
    * Origin: fsxNet Usenet Gateway (21:1/5)
  • From Frank Scheiner@21:1/5 to John Paul Adrian Glaubitz on Mon Sep 18 23:00:01 2023
    On 18.09.23 22:42, John Paul Adrian Glaubitz wrote:
    On Mon, 2023-09-18 at 22:36 +0200, Frank Scheiner wrote:
    I don't think this patch is acceptable in its current form as it
    modifies the Makefile globally so that the flag is passed on to
    the host compiler for all architectures.

    Yes, thought about that, too. Know a better solution? Is there a way to
    make this only affecting compilations for ia64?

    You could modify CFLAGS globally in debian/rules specifically for ia64:

    ifneq (,$(filter $(DEB_HOST_ARCH),ia64))
    export DEB_CFLAGS_MAINT_APPEND += -fno-var-tracking
    endif

    That's the other extreme then, everything for ia64 gets compiled with `-fno-var-tracking`, or? Is that more acceptable?

    Maybe the per-file CFLAGS are also effective when included in the
    architecture Makefile in `arch/ia64/`?

    Or maybe we could query the arch from within the two Makefiles and only
    apply `-fno-var-tracking` when we compile for ia64.

    I'll have a look into that tomorrow.

    Cheers,
    Frank

    --- SoupGate-Win32 v1.05
    * Origin: fsxNet Usenet Gateway (21:1/5)
  • From John Paul Adrian Glaubitz@21:1/5 to Frank Scheiner on Mon Sep 18 23:10:01 2023
    On Mon, 2023-09-18 at 22:56 +0200, Frank Scheiner wrote:
    That's the other extreme then, everything for ia64 gets compiled with `-fno-var-tracking`, or? Is that more acceptable?

    It's certainly more acceptable than messing around with other targets.

    Maybe the per-file CFLAGS are also effective when included in the architecture Makefile in `arch/ia64/`?

    No idea.

    Or maybe we could query the arch from within the two Makefiles and only
    apply `-fno-var-tracking` when we compile for ia64.

    I'll have a look into that tomorrow.

    Adrian

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

    --- SoupGate-Win32 v1.05
    * Origin: fsxNet Usenet Gateway (21:1/5)
  • From Frank Scheiner@21:1/5 to Frank Scheiner on Tue Sep 19 20:30:01 2023
    On 18.09.23 22:56, Frank Scheiner wrote:
    On 18.09.23 22:42, John Paul Adrian Glaubitz wrote:
    On Mon, 2023-09-18 at 22:36 +0200, Frank Scheiner wrote:
    I don't think this patch is acceptable in its current form as it
    modifies the Makefile globally so that the flag is passed on to
    the host compiler for all architectures.

    Yes, thought about that, too. Know a better solution? Is there a way to
    make this only affecting compilations for ia64?

    You could modify CFLAGS globally in debian/rules specifically for ia64:

    ifneq (,$(filter $(DEB_HOST_ARCH),ia64))
    export DEB_CFLAGS_MAINT_APPEND += -fno-var-tracking
    endif

    That's the other extreme then, everything for ia64 gets compiled with `-fno-var-tracking`, or? Is that more acceptable?

    Maybe the per-file CFLAGS are also effective when included in the architecture Makefile in `arch/ia64/`?

    Or maybe we could query the arch from within the two Makefiles and only
    apply `-fno-var-tracking` when we compile for ia64.

    I'll have a look into that tomorrow.

    The MR ([1]) was updated and now uses `-fno-var-tracking` for the
    respective files only when the target architecture is ia64. This works
    like so for example:

    ```
    ifeq ($(ARCH),ia64)
    CFLAGS_bnx2x_sp.o += -fno-var-tracking
    endif
    ```

    Tested via a local kernel build where one case was using a wrong target architecture (i386) at first to see it failing and then working after
    changing that back to the correct target architecture (ia64).

    [1]: https://salsa.debian.org/kernel-team/linux/-/merge_requests/852

    Cheers,
    Frank

    --- SoupGate-Win32 v1.05
    * Origin: fsxNet Usenet Gateway (21:1/5)
  • From Frank Scheiner@21:1/5 to John Paul Adrian Glaubitz on Tue Sep 19 20:40:01 2023
    On 19.09.23 20:33, John Paul Adrian Glaubitz wrote:
    On Tue, 2023-09-19 at 20:26 +0200, Frank Scheiner wrote:
    The MR ([1]) was updated and now uses `-fno-var-tracking` for the
    respective files only when the target architecture is ia64. This works
    like so for example:

    ```
    ifeq ($(ARCH),ia64)
    CFLAGS_bnx2x_sp.o += -fno-var-tracking
    endif
    ```

    Tested via a local kernel build where one case was using a wrong target
    architecture (i386) at first to see it failing and then working after
    changing that back to the correct target architecture (ia64).

    [1]: https://salsa.debian.org/kernel-team/linux/-/merge_requests/852

    This looks reasonable to me and I think we can have this merged as a temporary
    workaround so that we can get 6.5 and 6.6 kernels built for ia64 on the buildds.

    You're welcome.

    Cheers,
    Frank

    --- SoupGate-Win32 v1.05
    * Origin: fsxNet Usenet Gateway (21:1/5)