On Tuesday, October 3, 2017 10:50:11 PM CEST Rafael J. Wysocki wrote: > On Tuesday, October 3, 2017 9:41:23 PM CEST Moore, Robert wrote: > > > > > -----Original Message----- > > > From: rjwysocki@xxxxxxxxx [mailto:rjwysocki@xxxxxxxxx] On Behalf Of > > > Rafael J. Wysocki > > > Sent: Tuesday, October 3, 2017 11:21 AM > > > To: Moore, Robert <robert.moore@xxxxxxxxx> > > > Cc: Rafael J. Wysocki <rjw@xxxxxxxxxxxxx>; Zheng, Lv > > > <lv.zheng@xxxxxxxxx>; Wu, Fengguang <fengguang.wu@xxxxxxxxx>; kbuild- > > > all@xxxxxx; linux-acpi@xxxxxxxxxxxxxxx; devel@xxxxxxxxxx; linux- > > > pm@xxxxxxxxxxxxxxx > > > Subject: Re: [pm:bleeding-edge 12/41] utstrsuppt.c:undefined reference > > > to `__udivdi3' > > > > > > On Tue, Oct 3, 2017 at 6:43 PM, Moore, Robert <robert.moore@xxxxxxxxx> > > > wrote: > > > > > > > > > > > >> -----Original Message----- > > > >> From: Rafael J. Wysocki [mailto:rjw@xxxxxxxxxxxxx] > > > >> Sent: Tuesday, October 3, 2017 4:32 AM > > > >> To: Moore, Robert <robert.moore@xxxxxxxxx>; Zheng, Lv > > > >> <lv.zheng@xxxxxxxxx> > > > >> Cc: Wu, Fengguang <fengguang.wu@xxxxxxxxx>; kbuild-all@xxxxxx; linux- > > > >> acpi@xxxxxxxxxxxxxxx; devel@xxxxxxxxxx; linux-pm@xxxxxxxxxxxxxxx > > > >> Subject: Re: [pm:bleeding-edge 12/41] utstrsuppt.c:undefined > > > >> reference to `__udivdi3' > > > >> > > > >> On Tuesday, October 3, 2017 4:25:52 AM CEST kbuild test robot wrote: > > > >> > tree: > > > https://git.kernel.org/pub/scm/linux/kernel/git/rafael/linux- > > > >> pm.git bleeding-edge > > > >> > head: 09b089a247ea7272df93a2116368bf24b656cd65 > > > >> > commit: 05545b3fb6eab35290413211617793bc274e136b [12/41] ACPICA: > > > >> Restructure/cleanup all string-to-integer conversion functions > > > >> > config: i386-defconfig (attached as .config) > > > >> > compiler: gcc-6 (Debian 6.2.0-3) 6.2.0 20160901 > > > >> > reproduce: > > > >> > git checkout 05545b3fb6eab35290413211617793bc274e136b > > > >> > # save the attached .config to linux build tree > > > >> > make ARCH=i386 > > > >> > > > > >> > All errors (new ones prefixed by >>): > > > >> > > > > >> > drivers/acpi/acpica/utstrsuppt.o: In function > > > >> `acpi_ut_insert_digit': > > > >> > >> utstrsuppt.c:(.text+0x33): undefined reference to `__udivdi3' > > > >> > > > >> __udivdi3() doesn't work on 32-bit x86 AFAICS. > > > >> > > > >> Can we avoid using it or at least put it under #ifdef somehow? > > > > [Moore, Robert] > > > > > > > > > > > > I take it that __udivdi3 is a 64-bit divide? > > > > > > Practically, yes (AFAICS). > > > > > > > We do have a local divide implementation(s) for things like this. > > > > > > > > However, I don't see a divide in ut_insert_digit; There is a 64-bit > > > divide in ut_strtoul_add_64, however. > > [Moore, Robert] > > > > Oops, sorry. The divided is in strtoul_multiply_64. We are checking for an > > overflow condition up front. > > But this is a plain division, so it looks like a build system or compiler issue > to me. Well, OK. Plain u64 / u64 divisions need to be avoided for stuff to build on 32-bit x86. Thanks, Rafael -- To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
