All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] PARISC: provide __ucmpdi2
@ 2013-04-20 19:41 John David Anglin
  2013-04-20 19:51 ` Rolf Eike Beer
  0 siblings, 1 reply; 5+ messages in thread
From: John David Anglin @ 2013-04-20 19:41 UTC (permalink / raw)
  To: linux-parisc List; +Cc: Helge Deller, James E.J. Bottomley

[-- Attachment #1: Type: text/plain, Size: 543 bytes --]

Provide __ucmpdi2 to resolve undefined references in 32 bit builds.

The Debian experimental linux source package (3.8.5-1) build fails  
with the following errors:
...
MODPOST 2016 modules
ERROR: "__ucmpdi2" [fs/btrfs/btrfs.ko] undefined!
ERROR: "__ucmpdi2" [drivers/md/dm-verity.ko] undefined!

The attached patch resolves this problem.  It is based on the s390  
implementation
of ucmpdi2.c.

Signed-off-by: John David Anglin <dave.anglin@bell.net>
Cc: "James E.J. Bottomley" <jejb@parisc-linux.org>
Cc: Helge Deller <deller@gmx.de>
---




[-- Attachment #2: ucmpdi2.diff.txt --]
[-- Type: text/plain, Size: 1635 bytes --]

diff --git a/arch/parisc/kernel/parisc_ksyms.c b/arch/parisc/kernel/parisc_ksyms.c
index 6795dc6..568b2c6 100644
--- a/arch/parisc/kernel/parisc_ksyms.c
+++ b/arch/parisc/kernel/parisc_ksyms.c
@@ -120,11 +120,13 @@ extern void __ashrdi3(void);
 extern void __ashldi3(void);
 extern void __lshrdi3(void);
 extern void __muldi3(void);
+extern void __ucmpdi2(void);
 
 EXPORT_SYMBOL(__ashrdi3);
 EXPORT_SYMBOL(__ashldi3);
 EXPORT_SYMBOL(__lshrdi3);
 EXPORT_SYMBOL(__muldi3);
+EXPORT_SYMBOL(__ucmpdi2);
 
 asmlinkage void * __canonicalize_funcptr_for_compare(void *);
 EXPORT_SYMBOL(__canonicalize_funcptr_for_compare);
diff --git a/arch/parisc/lib/Makefile b/arch/parisc/lib/Makefile
index 5f2e690..5651536 100644
--- a/arch/parisc/lib/Makefile
+++ b/arch/parisc/lib/Makefile
@@ -2,6 +2,7 @@
 # Makefile for parisc-specific library files
 #
 
-lib-y	:= lusercopy.o bitops.o checksum.o io.o memset.o fixup.o memcpy.o
+lib-y	:= lusercopy.o bitops.o checksum.o io.o memset.o fixup.o memcpy.o \
+	   ucmpdi2.o
 
 obj-y	:= iomap.o
--- /dev/null	2013-04-18 22:44:06.316000000 -0400
+++ linux/arch/parisc/lib/ucmpdi2.c	2013-04-20 10:50:01.000000000 -0400
@@ -0,0 +1,25 @@
+#include <linux/module.h>
+
+union ull_union {
+	unsigned long long ull;
+	struct {
+		unsigned int high;
+		unsigned int low;
+	} ui;
+};
+
+int __ucmpdi2(unsigned long long a, unsigned long long b)
+{
+	union ull_union au = {.ull = a};
+	union ull_union bu = {.ull = b};
+
+	if (au.ui.high < bu.ui.high)
+		return 0;
+	else if (au.ui.high > bu.ui.high)
+		return 2;
+	if (au.ui.low < bu.ui.low)
+		return 0;
+	else if (au.ui.low > bu.ui.low)
+		return 2;
+	return 1;
+}

[-- Attachment #3: Type: text/plain, Size: 50 bytes --]



Dave
--
John David Anglin	dave.anglin@bell.net


^ permalink raw reply related	[flat|nested] 5+ messages in thread

* Re: [PATCH] PARISC: provide __ucmpdi2
  2013-04-20 19:41 [PATCH] PARISC: provide __ucmpdi2 John David Anglin
@ 2013-04-20 19:51 ` Rolf Eike Beer
  2013-04-20 19:52   ` John David Anglin
  0 siblings, 1 reply; 5+ messages in thread
From: Rolf Eike Beer @ 2013-04-20 19:51 UTC (permalink / raw)
  To: linux-parisc List

[-- Attachment #1: Type: text/plain, Size: 696 bytes --]

Am Samstag 20 April 2013, 15:41:06 schrieb John David Anglin:
> Provide __ucmpdi2 to resolve undefined references in 32 bit builds.
> 
> The Debian experimental linux source package (3.8.5-1) build fails
> with the following errors:
> ...
> MODPOST 2016 modules
> ERROR: "__ucmpdi2" [fs/btrfs/btrfs.ko] undefined!
> ERROR: "__ucmpdi2" [drivers/md/dm-verity.ko] undefined!
> 
> The attached patch resolves this problem.  It is based on the s390
> implementation
> of ucmpdi2.c.
> 
> Signed-off-by: John David Anglin <dave.anglin@bell.net>
> Cc: "James E.J. Bottomley" <jejb@parisc-linux.org>
> Cc: Helge Deller <deller@gmx.de>

If this is in 3.8.5 this should also get a Cc:stable@kernel.org

Eike

[-- Attachment #2: This is a digitally signed message part. --]
[-- Type: application/pgp-signature, Size: 198 bytes --]

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH] PARISC: provide __ucmpdi2
  2013-04-20 19:51 ` Rolf Eike Beer
@ 2013-04-20 19:52   ` John David Anglin
  2013-04-20 20:02     ` Rolf Eike Beer
  0 siblings, 1 reply; 5+ messages in thread
From: John David Anglin @ 2013-04-20 19:52 UTC (permalink / raw)
  To: Rolf Eike Beer; +Cc: linux-parisc List

On 20-Apr-13, at 3:51 PM, Rolf Eike Beer wrote:

> Am Samstag 20 April 2013, 15:41:06 schrieb John David Anglin:
>> Provide __ucmpdi2 to resolve undefined references in 32 bit builds.
>>
>> The Debian experimental linux source package (3.8.5-1) build fails
>> with the following errors:
>> ...
>> MODPOST 2016 modules
>> ERROR: "__ucmpdi2" [fs/btrfs/btrfs.ko] undefined!
>> ERROR: "__ucmpdi2" [drivers/md/dm-verity.ko] undefined!
>>
>> The attached patch resolves this problem.  It is based on the s390
>> implementation
>> of ucmpdi2.c.
>>
>> Signed-off-by: John David Anglin <dave.anglin@bell.net>
>> Cc: "James E.J. Bottomley" <jejb@parisc-linux.org>
>> Cc: Helge Deller <deller@gmx.de>
>
> If this is in 3.8.5 this should also get a Cc:stable@kernel.org


It is not in anything.

Dave
--
John David Anglin	dave.anglin@bell.net




^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH] PARISC: provide __ucmpdi2
  2013-04-20 19:52   ` John David Anglin
@ 2013-04-20 20:02     ` Rolf Eike Beer
  2013-04-20 20:26       ` John David Anglin
  0 siblings, 1 reply; 5+ messages in thread
From: Rolf Eike Beer @ 2013-04-20 20:02 UTC (permalink / raw)
  To: linux-parisc List

[-- Attachment #1: Type: text/plain, Size: 1069 bytes --]

Am Samstag 20 April 2013, 15:52:59 schrieb John David Anglin:
> On 20-Apr-13, at 3:51 PM, Rolf Eike Beer wrote:
> > Am Samstag 20 April 2013, 15:41:06 schrieb John David Anglin:
> >> Provide __ucmpdi2 to resolve undefined references in 32 bit builds.
> >> 
> >> The Debian experimental linux source package (3.8.5-1) build fails
> >> with the following errors:
> >> ...
> >> MODPOST 2016 modules
> >> ERROR: "__ucmpdi2" [fs/btrfs/btrfs.ko] undefined!
> >> ERROR: "__ucmpdi2" [drivers/md/dm-verity.ko] undefined!
> >> 
> >> The attached patch resolves this problem.  It is based on the s390
> >> implementation
> >> of ucmpdi2.c.
> >> 
> >> Signed-off-by: John David Anglin <dave.anglin@bell.net>
> >> Cc: "James E.J. Bottomley" <jejb@parisc-linux.org>
> >> Cc: Helge Deller <deller@gmx.de>
> > 
> > If this is in 3.8.5 this should also get a Cc:stable@kernel.org
> 
> It is not in anything.

The function? Or the error? If the function is missing, i.e. the error will 
happen once enabling e.g. btrfs, in a already released kernel we should get 
this into stable.

Eike

[-- Attachment #2: This is a digitally signed message part. --]
[-- Type: application/pgp-signature, Size: 198 bytes --]

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH] PARISC: provide __ucmpdi2
  2013-04-20 20:02     ` Rolf Eike Beer
@ 2013-04-20 20:26       ` John David Anglin
  0 siblings, 0 replies; 5+ messages in thread
From: John David Anglin @ 2013-04-20 20:26 UTC (permalink / raw)
  To: Rolf Eike Beer; +Cc: linux-parisc List

On 20-Apr-13, at 4:02 PM, Rolf Eike Beer wrote:

> Am Samstag 20 April 2013, 15:52:59 schrieb John David Anglin:
>> On 20-Apr-13, at 3:51 PM, Rolf Eike Beer wrote:
>>> Am Samstag 20 April 2013, 15:41:06 schrieb John David Anglin:
>>>> Provide __ucmpdi2 to resolve undefined references in 32 bit builds.
>>>>
>>>> The Debian experimental linux source package (3.8.5-1) build fails
>>>> with the following errors:
>>>> ...
>>>> MODPOST 2016 modules
>>>> ERROR: "__ucmpdi2" [fs/btrfs/btrfs.ko] undefined!
>>>> ERROR: "__ucmpdi2" [drivers/md/dm-verity.ko] undefined!
>>>>
>>>> The attached patch resolves this problem.  It is based on the s390
>>>> implementation
>>>> of ucmpdi2.c.
>>>>
>>>> Signed-off-by: John David Anglin <dave.anglin@bell.net>
>>>> Cc: "James E.J. Bottomley" <jejb@parisc-linux.org>
>>>> Cc: Helge Deller <deller@gmx.de>
>>>
>>> If this is in 3.8.5 this should also get a Cc:stable@kernel.org
>>
>> It is not in anything.
>
> The function? Or the error? If the function is missing, i.e. the  
> error will
> happen once enabling e.g. btrfs, in a already released kernel we  
> should get
> this into stable.

The error is probably in all active stable branches given that most  
ports provide the function.
GCC generates a call to __ucmpdi2 if the backend doesn't support  
unsigned 64-bit compares.
Normally, the function is resolved by libgcc.  I have no objection to  
pushing the patch back
but Helge and James usually decide this.

Dave
--
John David Anglin	dave.anglin@bell.net




^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2013-04-20 20:26 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-04-20 19:41 [PATCH] PARISC: provide __ucmpdi2 John David Anglin
2013-04-20 19:51 ` Rolf Eike Beer
2013-04-20 19:52   ` John David Anglin
2013-04-20 20:02     ` Rolf Eike Beer
2013-04-20 20:26       ` John David Anglin

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.