[FTBFS] arm does not define __NR_migrate_pages
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
numactl (Debian) |
Fix Released
|
Unknown
|
|||
numactl (Ubuntu) |
Fix Released
|
Medium
|
dann frazier | ||
Bionic |
Fix Released
|
Medium
|
Unassigned |
Bug Description
[Impact]
* Due to arm not defining __NR_migrate_pages
Backport upstream changes to fix this as part of an ongoing SRU.
- https:/
[Test Case]
* Does it build in -proposed (and PPAs to prove it)
[Regression Potential]
* The change is minimal, therefore I hope that the regression risk
is rather low.
If anything then different libc/kernel behavior could make it regress,
but then IMHO that would apply to the current build in the archive as
well and not be part of this SRU.
[Other Info]
* Found to affect Bionic when working on SRU for bug 1817258
* Sorry for the short SRU template, but for FTBFS bugs I'm not sure if we need more
----
numactl currently FTBFS since arm does not define __NR_migrate_pages.
As described in the Debian bug comments [1] the function should just return -1 and set errno when __NR_migrate_pages is undefined. This prevents it from failing the build.
Failure from the buildlog [2]:
libtool: compile: gcc -DHAVE_CONFIG_H -I. -Wall -Wdate-time -D_FORTIFY_SOURCE=2 -g -O2 -fdebug-
syscall.c:113:2: error: #error "Add syscalls for your architecture or update kernel headers"
#error "Add syscalls for your architecture or update kernel headers"
^~~~~
syscall.c: In function ‘migrate_pages’:
syscall.c:209:17: error: ‘__NR_migrate_
return syscall(
syscall.c:209:17: note: each undeclared identifier is reported only once for each function it appears in
[1] https:/
[2] https:/
Related branches
- Robie Basak: Approve
- Canonical Server: Pending requested
- git-ubuntu developers: Pending requested
-
Diff: 223 lines (+197/-0)4 files modifieddebian/changelog (+12/-0)
debian/patches/Allow-building-on-ARM-systems.patch (+47/-0)
debian/patches/lp1817258-Segment-fault-when-numa-nodes-not-sequential-or-cont.patch (+136/-0)
debian/patches/series (+2/-0)
tags: | added: artful ftbfs |
tags: | added: patch |
Changed in numactl (Debian): | |
status: | Unknown → New |
tags: | added: bionic |
Changed in numactl (Debian): | |
status: | New → Confirmed |
Changed in numactl (Debian): | |
status: | Confirmed → Fix Released |
description: | updated |
Changed in numactl (Ubuntu): | |
importance: | Undecided → Medium |
Changed in numactl (Ubuntu Bionic): | |
importance: | Undecided → Medium |
Hi Tiago,
Thanks for also driving that to Debian - well the solution of the old Debian issue to be precise.
Do you have hope that with you doing the bulk of the work to get it uploaded in Debian soon and fix this with a sync?
Or do you want for somebody to review and sponsor this already without waiting for Debian.
I mean we can sync a week after FF as this is a bugfix-only change if the Debian upload would not change (much) anything else.
In general the debdiff LGTM but it needs a core-dev sponsor to upload, so my ack isn't worth a lot yet.