Bug 232883

Summary:	nvidia-drivers wrong paravirt check
Product:	Gentoo Linux	Reporter:	Patrizio Bassi <patrizio.bassi>
Component:	New packages	Assignee:	Tony Vroon (RETIRED) <chainsaw>
Status:	RESOLVED FIXED
Severity:	normal	CC:	dliana
Priority:	Highest
Version:	unspecified
Hardware:	All
OS:	Linux
Whiteboard:
Package list:		Runtime testing required:	---

Description Patrizio Bassi 2008-07-24 20:32:49 UTC

the paravirt check was added due to gpl-only exported symbols, but now needed ones are back to be loose included.

debian
http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=419943

and http://lwn.net/Articles/233315/
since 2.6.22

infact my 2.6.26 copy includes:
EXPORT_SYMBOL    (pv_mmu_ops);
paravirt.c


please remove the unneeded check from ebuild, it stops amd64 users from using vsmp option

Comment 1 Tony Vroon (RETIRED) gentoo-dev

2008-07-31 13:05:34 UTC

Agreed, this can go. I have applied your patch to both the existing 173.14.09 and to the new 173.14.12 committed for bug #233418. Thank you for your contribution to Gentoo Linux :)
You have been mentioned:
+*nvidia-drivers-173.14.12 (31 Jul 2008)
+
+  31 Jul 2008; <chainsaw@gentoo.org> nvidia-drivers-173.14.09.ebuild,
+  +nvidia-drivers-173.14.12.ebuild:
+  Version bump as requested by Wyatt Epp <epp.8@osu.edu> in bug #233418.
+  Ensure SYSVIPC is enabled as per cucu ionut <cuciferus@gmail.com> in bug
+  #213157. Patch to respect custom CFLAGS by boris64 <genzilla@boris64.net>
+  closes bug #230273. Paravirt exports now usable, delete unnecessary check
+  as per Patrizio Bassi <hetfield666@gmail.com> in bug #232883.

Comment 2 Patrizio Bassi 2008-07-31 16:10:19 UTC

great, thanks you for gentoo work, it's nice to contribute.