linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* 2.4.23pre6aa2
@ 2003-10-04 10:57 Andrea Arcangeli
  2003-10-04 15:38 ` 2.4.23pre6aa2 - some problems [with patches] Eyal Lebedinsky
  0 siblings, 1 reply; 6+ messages in thread
From: Andrea Arcangeli @ 2003-10-04 10:57 UTC (permalink / raw)
  To: linux-kernel

URL:

	http://www.us.kernel.org/pub/linux/kernel/people/andrea/kernels/v2.4/2.4.23pre6aa2.gz
	http://www.us.kernel.org/pub/linux/kernel/people/andrea/kernels/v2.4/2.4.23pre6aa2/

Pending things for next releases:

	o libata for SATA
	o UML update

Changelog diff between 2.4.23pre6aa1 and 2.4.23pre6aa2:

Only in 2.4.23pre6aa2: 00_csum-trail-1

	Fixup overflow in p6 checksum routing that crashes sendfile, from
	Andi Kleen. IIRC I found and fixed (though fixed w/o fixmap) this same
	bug too in 1999, but at that time nobody listened about my claim that
	it would eventually crash machines due alignment issues.

Only in 2.4.23pre6aa1: 00_extraversion-30
Only in 2.4.23pre6aa2: 00_extraversion-31

	Rediffed.

Only in 2.4.23pre6aa2: 70_qsort-1
Only in 2.4.23pre6aa1: 71_qsort-1
Only in 2.4.23pre6aa2: 71_xfs-qsort-1

	Allow xfs to compile.

Only in 2.4.23pre6aa1: 91_zone_start_pfn-7
Only in 2.4.23pre6aa2: 91_zone_start_pfn-8

	Fix a NUMA bug introduced by this patch that was crashing my opteron
	desktop. From Andi Kleen.

Only in 2.4.23pre6aa1: 9999_zzz-dynamic-hz-1
Only in 2.4.23pre6aa2: 9999_zzz-dynamic-hz-2

	Fixup two miscompiles reported by Eyal Lebedinsky.

Andrea - If you prefer relying on open source software, check these links:
	    rsync.kernel.org::pub/scm/linux/kernel/bkcvs/linux-2.[45]/
	    http://www.cobite.com/cvsps/

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

* Re: 2.4.23pre6aa2 - some problems [with patches]
  2003-10-04 10:57 2.4.23pre6aa2 Andrea Arcangeli
@ 2003-10-04 15:38 ` Eyal Lebedinsky
  2003-10-05 10:40   ` Andrea Arcangeli
  0 siblings, 1 reply; 6+ messages in thread
From: Eyal Lebedinsky @ 2003-10-04 15:38 UTC (permalink / raw)
  To: Andrea Arcangeli; +Cc: linux-kernel

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

This is most unusual as -aa patches usually apply clean, but I am
encountering a number of build problems.

"PCMCIA SCSI adapter support" build is broken. I deselected the whole
section.

I now have this failure:

gcc -D__KERNEL__ -I/data2/usr/local/src/linux-2.4-pre-aa/include -Wall
-Wstrict-
prototypes -Wno-trigraphs -O2 -fno-strict-aliasing -fno-common 
-fomit-frame-poi
nter -pipe -msoft-float -mpreferred-stack-boundary=2 -march=i686
-malign-functio
ns=4 -DMODULE -DMODVERSIONS -include
/data2/usr/local/src/linux-2.4-pre-aa/inclu
de/linux/modversions.h  -nostdinc -iwithprefix include
-DKBUILD_BASENAME=ip_vs_c
onn  -c -o ip_vs_conn.o ip_vs_conn.c
ip_vs_conn.c: In function `ip_vs_tunnel_xmit':
ip_vs_conn.c:895: too many arguments to function
`ip_select_ident_Rd603b6c5'
make[2]: *** [ip_vs_conn.o] Error 1
make[2]: Leaving directory
`/data2/usr/local/src/linux-2.4-pre-aa/net/ipv4/ipvs'

Seems that ip_select_ident() only wants the first argument.


Next I get

gcc -D__KERNEL__ -I/data2/usr/local/src/linux-2.4-pre-aa/include -Wall
-Wstrict-
prototypes -Wno-trigraphs -O2 -fno-strict-aliasing -fno-common 
-fomit-frame-poi
nter -pipe -msoft-float -mpreferred-stack-boundary=2 -march=i686
-malign-functio
ns=4 -DMODULE -DMODVERSIONS -include
/data2/usr/local/src/linux-2.4-pre-aa/inclu
de/linux/modversions.h  -nostdinc -iwithprefix include
-DKBUILD_BASENAME=addrcon
f  -c -o addrconf.o addrconf.c
addrconf.c: In function `ipv6_store_devconf':
addrconf.c:1996: structure has no member named `rtr_solicit_interval'
addrconf.c:1997: structure has no member named `rtr_solicit_delay'
make[2]: *** [addrconf.o] Error 1
make[2]: Leaving directory
`/data2/usr/local/src/linux-2.4-pre-aa/net/ipv6'

I think that these members were renamed, my patch is just a guess.


Next
gcc -D__KERNEL__ -I/data2/usr/local/src/linux-2.4-pre-aa/include -Wall
-Wstrict-
prototypes -Wno-trigraphs -O2 -fno-strict-aliasing -fno-common 
-fomit-frame-poi
nter -pipe -msoft-float -mpreferred-stack-boundary=2 -march=i686
-malign-functio
ns=4 -DMODULE -DMODVERSIONS -include
/data2/usr/local/src/linux-2.4-pre-aa/inclu
de/linux/modversions.h  -nostdinc -iwithprefix include
-DKBUILD_BASENAME=protoco
l  -c -o protocol.o protocol.c
protocol.c: In function `sctp_v4_create_accept_sk':
protocol.c:537: structure has no member named `id'
make[2]: *** [protocol.o] Error 1
make[2]: Leaving directory
`/data2/usr/local/src/linux-2.4-pre-aa/net/sctp'

Member .id was removed, so my patch removes the offending line (a
guess).


Finally I get
if [ -r System.map ]; then /sbin/depmod -ae -F System.map 
2.4.23-pre6-aa2; fi
depmod: /lib/modules/2.4.23-pre6-aa2/ksyms is not an ELF file
depmod: /lib/modules/2.4.23-pre6-aa2/soundconf is not an ELF file
depmod: *** Unresolved symbols in
/lib/modules/2.4.23-pre6-aa2/kernel/drivers/vi
deo/sis/sisfb.o
depmod:         __floatsidf
depmod:         __divdf3
depmod:         __fixunsdfsi
depmod:         __muldf3
depmod:         __adddf3

And building i2c-2.7.0 (which I need for sensors) is failing.

depmod: *** Unresolved symbols in
/lib/modules/2.4.23-pre6-aa2/kernel/drivers/ie
ee1394/pcilynx.o
depmod:         i2c_bit_add_bus_Rca543f36
depmod:         i2c_transfer_R1dea91d1
depmod:         i2c_bit_del_bus_Rdf920b11
depmod: *** Unresolved symbols in
/lib/modules/2.4.23-pre6-aa2/kernel/drivers/me
dia/video/bttv.o
depmod:         i2c_bit_add_bus_Rca543f36
depmod:         i2c_master_recv_R67b29cc4
depmod:         i2c_bit_del_bus_Rdf920b11
depmod:         i2c_master_send_Rb692cb0e
depmod: *** Unresolved symbols in
/lib/modules/2.4.23-pre6-aa2/kernel/drivers/me
dia/video/msp3400.o
depmod:         i2c_probe_R4e2acbec
depmod:         i2c_add_driver_Racf22304
depmod:         i2c_transfer_R1dea91d1
depmod:         i2c_attach_client_Ra861362d
depmod:         i2c_master_send_Rb692cb0e
depmod:         i2c_detach_client_R0cfb40b4
depmod:         i2c_del_driver_R57837012
.... more following, all in  kernel/drivers/media/video/* ...

--
Eyal Lebedinsky (eyal@eyal.emu.id.au) <http://samba.org/eyal/>

[-- Attachment #2: 2.4.23-pre6-aa2.ip_vs_conn.patch --]
[-- Type: text/plain, Size: 364 bytes --]

--- linux/net/ipv4/ipvs/ip_vs_conn.c.orig	Sun Oct  5 01:14:39 2003
+++ linux/net/ipv4/ipvs/ip_vs_conn.c	Sun Oct  5 01:15:12 2003
@@ -892,7 +892,7 @@
 	iph->saddr		=	rt->rt_src;
 	iph->ttl		=	old_iph->ttl;
 	iph->tot_len		=	htons(skb->len);
-	ip_select_ident(iph, &rt->u.dst, NULL);
+	ip_select_ident(iph);
 	ip_send_check(iph);
 
 	skb->ip_summed = CHECKSUM_NONE;

[-- Attachment #3: 2.4.23-pre6-aa2.addrconf.patch --]
[-- Type: text/plain, Size: 678 bytes --]

--- linux/net/ipv6/addrconf.c.orig	Sun Oct  5 01:20:38 2003
+++ linux/net/ipv6/addrconf.c	Sun Oct  5 01:21:24 2003
@@ -1993,8 +1993,8 @@
 	array[DEVCONF_AUTOCONF] = cnf->autoconf;
 	array[DEVCONF_DAD_TRANSMITS] = cnf->dad_transmits;
 	array[DEVCONF_RTR_SOLICITS] = cnf->rtr_solicits;
-	array[DEVCONF_RTR_SOLICIT_INTERVAL] = cnf->rtr_solicit_interval;
-	array[DEVCONF_RTR_SOLICIT_DELAY] = cnf->rtr_solicit_delay;
+	array[DEVCONF_RTR_SOLICIT_INTERVAL] = cnf->__rtr_solicit_interval;
+	array[DEVCONF_RTR_SOLICIT_DELAY] = cnf->__rtr_solicit_delay;
 #ifdef CONFIG_IPV6_PRIVACY
 	array[DEVCONF_USE_TEMPADDR] = cnf->use_tempaddr;
 	array[DEVCONF_TEMP_VALID_LFT] = cnf->temp_valid_lft;

[-- Attachment #4: 2.4.23-pre6-aa2.protocol.patch --]
[-- Type: text/plain, Size: 366 bytes --]

--- linux/net/sctp/protocol.c.orig	Sun Oct  5 01:30:01 2003
+++ linux/net/sctp/protocol.c	Sun Oct  5 01:30:13 2003
@@ -534,7 +534,6 @@
 	newsk->dport = htons(asoc->peer.port);
 	newsk->daddr = asoc->peer.primary_addr.v4.sin_addr.s_addr;
 	newinet->pmtudisc = inet->pmtudisc;
-      	newinet->id = 0;
 
 	newinet->ttl = sysctl_ip_default_ttl;
 	newinet->mc_loop = 1;

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

* Re: 2.4.23pre6aa2 - some problems [with patches]
  2003-10-04 15:38 ` 2.4.23pre6aa2 - some problems [with patches] Eyal Lebedinsky
@ 2003-10-05 10:40   ` Andrea Arcangeli
  2003-10-05 11:02     ` Thomas Winischhofer
                       ` (2 more replies)
  0 siblings, 3 replies; 6+ messages in thread
From: Andrea Arcangeli @ 2003-10-05 10:40 UTC (permalink / raw)
  To: Eyal Lebedinsky; +Cc: linux-kernel, thomas

On Sun, Oct 05, 2003 at 01:38:20AM +1000, Eyal Lebedinsky wrote:
> This is most unusual as -aa patches usually apply clean, but I am
> encountering a number of build problems.
> 
> "PCMCIA SCSI adapter support" build is broken. I deselected the whole
> section.

can you double check you can reproduce it in 2.4.23pre6 vanilla? It
doesn't seem introduced by my patches.

> I now have this failure:
> 
> gcc -D__KERNEL__ -I/data2/usr/local/src/linux-2.4-pre-aa/include -Wall
> -Wstrict-
> prototypes -Wno-trigraphs -O2 -fno-strict-aliasing -fno-common 
> -fomit-frame-poi
> nter -pipe -msoft-float -mpreferred-stack-boundary=2 -march=i686
> -malign-functio
> ns=4 -DMODULE -DMODVERSIONS -include
> /data2/usr/local/src/linux-2.4-pre-aa/inclu
> de/linux/modversions.h  -nostdinc -iwithprefix include
> -DKBUILD_BASENAME=ip_vs_c
> onn  -c -o ip_vs_conn.o ip_vs_conn.c
> ip_vs_conn.c: In function `ip_vs_tunnel_xmit':
> ip_vs_conn.c:895: too many arguments to function
> `ip_select_ident_Rd603b6c5'
> make[2]: *** [ip_vs_conn.o] Error 1
> make[2]: Leaving directory
> `/data2/usr/local/src/linux-2.4-pre-aa/net/ipv4/ipvs'
> 
> Seems that ip_select_ident() only wants the first argument.

this isn't new, we simplified the ip_select_ident a long time ago.
Not sure it triggers only now. Your fix is correct thanks.

> Next I get
> 
> gcc -D__KERNEL__ -I/data2/usr/local/src/linux-2.4-pre-aa/include -Wall
> -Wstrict-
> prototypes -Wno-trigraphs -O2 -fno-strict-aliasing -fno-common 
> -fomit-frame-poi
> nter -pipe -msoft-float -mpreferred-stack-boundary=2 -march=i686
> -malign-functio
> ns=4 -DMODULE -DMODVERSIONS -include
> /data2/usr/local/src/linux-2.4-pre-aa/inclu
> de/linux/modversions.h  -nostdinc -iwithprefix include
> -DKBUILD_BASENAME=addrcon
> f  -c -o addrconf.o addrconf.c
> addrconf.c: In function `ipv6_store_devconf':
> addrconf.c:1996: structure has no member named `rtr_solicit_interval'
> addrconf.c:1997: structure has no member named `rtr_solicit_delay'
> make[2]: *** [addrconf.o] Error 1
> make[2]: Leaving directory
> `/data2/usr/local/src/linux-2.4-pre-aa/net/ipv6'
> 
> I think that these members were renamed, my patch is just a guess.

this one is related to dynamic-hz so you're right it's a new thing.
But careful, your fix is wrong. The rename was done for a reason, that
is exactly to catch those at compile time. This will fix it right:

--- xx/net/ipv6/addrconf.c.~1~	2003-10-05 12:23:08.000000000 +0200
+++ xx/net/ipv6/addrconf.c	2003-10-05 12:23:33.000000000 +0200
@@ -1993,8 +1993,8 @@ static void inline ipv6_store_devconf(st
 	array[DEVCONF_AUTOCONF] = cnf->autoconf;
 	array[DEVCONF_DAD_TRANSMITS] = cnf->dad_transmits;
 	array[DEVCONF_RTR_SOLICITS] = cnf->rtr_solicits;
-	array[DEVCONF_RTR_SOLICIT_INTERVAL] = cnf->rtr_solicit_interval;
-	array[DEVCONF_RTR_SOLICIT_DELAY] = cnf->rtr_solicit_delay;
+	array[DEVCONF_RTR_SOLICIT_INTERVAL] = user_to_kernel_hz_overflow(cnf->__rtr_solicit_interval);
+	array[DEVCONF_RTR_SOLICIT_DELAY] = user_to_kernel_hz_overflow(cnf->__rtr_solicit_delay);
 #ifdef CONFIG_IPV6_PRIVACY
 	array[DEVCONF_USE_TEMPADDR] = cnf->use_tempaddr;
 	array[DEVCONF_TEMP_VALID_LFT] = cnf->temp_valid_lft;

> Next
> gcc -D__KERNEL__ -I/data2/usr/local/src/linux-2.4-pre-aa/include -Wall
> -Wstrict-
> prototypes -Wno-trigraphs -O2 -fno-strict-aliasing -fno-common 
> -fomit-frame-poi
> nter -pipe -msoft-float -mpreferred-stack-boundary=2 -march=i686
> -malign-functio
> ns=4 -DMODULE -DMODVERSIONS -include
> /data2/usr/local/src/linux-2.4-pre-aa/inclu
> de/linux/modversions.h  -nostdinc -iwithprefix include
> -DKBUILD_BASENAME=protoco
> l  -c -o protocol.o protocol.c
> protocol.c: In function `sctp_v4_create_accept_sk':
> protocol.c:537: structure has no member named `id'
> make[2]: *** [protocol.o] Error 1
> make[2]: Leaving directory
> `/data2/usr/local/src/linux-2.4-pre-aa/net/sctp'
> 
> Member .id was removed, so my patch removes the offending line (a
> guess).

Well SCPT is a new feature, introduced in 2.4.23pre, this had nothing to
do with my tree, but thanks for the fix, merged.

> Finally I get
> if [ -r System.map ]; then /sbin/depmod -ae -F System.map 
> 2.4.23-pre6-aa2; fi
> depmod: /lib/modules/2.4.23-pre6-aa2/ksyms is not an ELF file
> depmod: /lib/modules/2.4.23-pre6-aa2/soundconf is not an ELF file
> depmod: *** Unresolved symbols in
> /lib/modules/2.4.23-pre6-aa2/kernel/drivers/vi
> deo/sis/sisfb.o
> depmod:         __floatsidf
> depmod:         __divdf3
> depmod:         __fixunsdfsi
> depmod:         __muldf3
> depmod:         __adddf3

those drivers are buggy, and had always been buggy, it's just that you
couldn't notice it before. I added an option in my new tree to catch
those longstanding bugs. They *must* not compile. Driver
authors can audit their 2.4 drivers by grabbing my tree or by applying
this patch alone:

	http://www.us.kernel.org/pub/linux/kernel/people/andrea/kernels/v2.4/2.4.23pre6aa2/9999900_soft-float-1

see drivers/video/sis/init.c:

      divider = (sr2b & 0x80) ? 2.0 : 1.0;
      postscalar = (sr2c & 0x80) ?
              ( (((sr2c >> 5) & 0x03) == 0x02) ? 6.0 : 8.0) : (((sr2c >> 5) & 0x03) + 1.0);
      num = (sr2b & 0x7f) + 1.0;
      denum = (sr2c & 0x1f) + 1.0;

there would be a slight chance to find false positives too, but those
aren't false positives, there's no fpu save at all there.


Ironically all the float numbers seems to end with .0, I fixed it a bit
but I've no idea if I catched all them, so Eyal, you can try to apply
this patch and follow the code to see if you can spot more of these
longstanding bugs (those could corrupt the fpu state and segfault
userspace at the very least):

--- xx/drivers/video/sis/init.c.~1~	2003-10-02 00:09:44.000000000 +0200
+++ xx/drivers/video/sis/init.c	2003-10-05 12:36:03.000000000 +0200
@@ -3940,11 +3940,11 @@ SiSBuildBuiltInModeList(ScrnInfoPtr pScr
       sr2b = pSiS->SiS_Pr->SiS_VCLKData[vclkindex].SR2B;
       sr2c = pSiS->SiS_Pr->SiS_VCLKData[vclkindex].SR2C;
 
-      divider = (sr2b & 0x80) ? 2.0 : 1.0;
+      divider = (sr2b & 0x80) ? 2 : 1;
       postscalar = (sr2c & 0x80) ?
-              ( (((sr2c >> 5) & 0x03) == 0x02) ? 6.0 : 8.0) : (((sr2c >> 5) & 0x03) + 1.0);
-      num = (sr2b & 0x7f) + 1.0;
-      denum = (sr2c & 0x1f) + 1.0;
+              ( (((sr2c >> 5) & 0x03) == 0x02) ? 6 : 8) : (((sr2c >> 5) & 0x03) + 1);
+      num = (sr2b & 0x7f) + 1;
+      denum = (sr2c & 0x1f) + 1;
 
 #ifdef TWDEBUG
       xf86DrvMsg(0, X_INFO, "------------\n");

> And building i2c-2.7.0 (which I need for sensors) is failing.
> 
> depmod: *** Unresolved symbols in
> /lib/modules/2.4.23-pre6-aa2/kernel/drivers/ie
> ee1394/pcilynx.o
> depmod:         i2c_bit_add_bus_Rca543f36
> depmod:         i2c_transfer_R1dea91d1
> depmod:         i2c_bit_del_bus_Rdf920b11
> depmod: *** Unresolved symbols in
> /lib/modules/2.4.23-pre6-aa2/kernel/drivers/me
> dia/video/bttv.o
> depmod:         i2c_bit_add_bus_Rca543f36
> depmod:         i2c_master_recv_R67b29cc4
> depmod:         i2c_bit_del_bus_Rdf920b11
> depmod:         i2c_master_send_Rb692cb0e
> depmod: *** Unresolved symbols in
> /lib/modules/2.4.23-pre6-aa2/kernel/drivers/me
> dia/video/msp3400.o
> depmod:         i2c_probe_R4e2acbec
> depmod:         i2c_add_driver_Racf22304
> depmod:         i2c_transfer_R1dea91d1
> depmod:         i2c_attach_client_Ra861362d
> depmod:         i2c_master_send_Rb692cb0e
> depmod:         i2c_detach_client_R0cfb40b4
> depmod:         i2c_del_driver_R57837012
> .... more following, all in  kernel/drivers/media/video/* ...

this looks like if you didn't compile the needed i2c (or maybe it was
due the lack of a `make dep` first), the above modules (pcilynx bttv
msp3400) looks innocent.

Thanks for the help in tracking and fixing those bits!

Andrea - If you prefer relying on open source software, check these links:
	    rsync.kernel.org::pub/scm/linux/kernel/bkcvs/linux-2.[45]/
	    http://www.cobite.com/cvsps/

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

* Re: 2.4.23pre6aa2 - some problems [with patches]
  2003-10-05 10:40   ` Andrea Arcangeli
@ 2003-10-05 11:02     ` Thomas Winischhofer
  2003-10-06  0:43     ` Eyal Lebedinsky
  2003-10-06  0:55     ` Eyal Lebedinsky
  2 siblings, 0 replies; 6+ messages in thread
From: Thomas Winischhofer @ 2003-10-05 11:02 UTC (permalink / raw)
  To: Andrea Arcangeli; +Cc: Eyal Lebedinsky, linux-kernel

Andrea Arcangeli wrote:
>>depmod: *** Unresolved symbols in
>>/lib/modules/2.4.23-pre6-aa2/kernel/drivers/vi
>>deo/sis/sisfb.o
>>depmod:         __floatsidf
>>depmod:         __divdf3
>>depmod:         __fixunsdfsi
>>depmod:         __muldf3
>>depmod:         __adddf3
> 
> 
> those drivers are buggy, and had always been buggy, it's just that you
> couldn't notice it before. I added an option in my new tree to catch
> those longstanding bugs. They *must* not compile. Driver
> authors can audit their 2.4 drivers by grabbing my tree or by applying
> this patch alone:
> 
> 	http://www.us.kernel.org/pub/linux/kernel/people/andrea/kernels/v2.4/2.4.23pre6aa2/9999900_soft-float-1
> 
> see drivers/video/sis/init.c:
> 
>       divider = (sr2b & 0x80) ? 2.0 : 1.0;
>       postscalar = (sr2c & 0x80) ?
>               ( (((sr2c >> 5) & 0x03) == 0x02) ? 6.0 : 8.0) : (((sr2c >> 5) & 0x03) + 1.0);
>       num = (sr2b & 0x7f) + 1.0;
>       denum = (sr2c & 0x1f) + 1.0;
> 
> there would be a slight chance to find false positives too, but those
> aren't false positives, there's no fpu save at all there.
> 
> 
> Ironically all the float numbers seems to end with .0, I fixed it a bit
> but I've no idea if I catched all them, so Eyal, you can try to apply
> this patch and follow the code to see if you can spot more of these
> longstanding bugs (those could corrupt the fpu state and segfault
> userspace at the very least):
> 
> --- xx/drivers/video/sis/init.c.~1~	2003-10-02 00:09:44.000000000 +0200
> +++ xx/drivers/video/sis/init.c	2003-10-05 12:36:03.000000000 +0200
> @@ -3940,11 +3940,11 @@ SiSBuildBuiltInModeList(ScrnInfoPtr pScr
>        sr2b = pSiS->SiS_Pr->SiS_VCLKData[vclkindex].SR2B;
>        sr2c = pSiS->SiS_Pr->SiS_VCLKData[vclkindex].SR2C;
>  
> -      divider = (sr2b & 0x80) ? 2.0 : 1.0;
> +      divider = (sr2b & 0x80) ? 2 : 1;
>        postscalar = (sr2c & 0x80) ?
> -              ( (((sr2c >> 5) & 0x03) == 0x02) ? 6.0 : 8.0) : (((sr2c >> 5) & 0x03) + 1.0);
> -      num = (sr2b & 0x7f) + 1.0;
> -      denum = (sr2c & 0x1f) + 1.0;
> +              ( (((sr2c >> 5) & 0x03) == 0x02) ? 6 : 8) : (((sr2c >> 5) & 0x03) + 1);
> +      num = (sr2b & 0x7f) + 1;
> +      denum = (sr2c & 0x1f) + 1;
>  
>  #ifdef TWDEBUG
>        xf86DrvMsg(0, X_INFO, "------------\n");
> 

That part of init.c doesn't get compiled in the linux kernel, it's for 
XFree86. I think the xf86DrvMsg statement should make that clear...

Thomas

-- 
Thomas Winischhofer
Vienna/Austria
thomas AT winischhofer DOT net          http://www.winischhofer.net/
twini AT xfree86 DOT org


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

* Re: 2.4.23pre6aa2 - some problems [with patches]
  2003-10-05 10:40   ` Andrea Arcangeli
  2003-10-05 11:02     ` Thomas Winischhofer
@ 2003-10-06  0:43     ` Eyal Lebedinsky
  2003-10-06  0:55     ` Eyal Lebedinsky
  2 siblings, 0 replies; 6+ messages in thread
From: Eyal Lebedinsky @ 2003-10-06  0:43 UTC (permalink / raw)
  To: Andrea Arcangeli; +Cc: linux-kernel

Andrea Arcangeli wrote:
> 
> On Sun, Oct 05, 2003 at 01:38:20AM +1000, Eyal Lebedinsky wrote:
> > This is most unusual as -aa patches usually apply clean, but I am
> > encountering a number of build problems.
> >
> > "PCMCIA SCSI adapter support" build is broken. I deselected the whole
> > section.
> 
> can you double check you can reproduce it in 2.4.23pre6 vanilla? It
> doesn't seem introduced by my patches.

I am reasonably sure that vanilla -pre6 built without any complaint.

> > And building i2c-2.7.0 (which I need for sensors) is failing.
> >
> > depmod: *** Unresolved symbols in
> > /lib/modules/2.4.23-pre6-aa2/kernel/drivers/ie
> > ee1394/pcilynx.o
> > depmod:         i2c_bit_add_bus_Rca543f36
> > depmod:         i2c_transfer_R1dea91d1
> > depmod:         i2c_bit_del_bus_Rdf920b11
> > depmod: *** Unresolved symbols in
> > /lib/modules/2.4.23-pre6-aa2/kernel/drivers/me
> > dia/video/bttv.o
> > depmod:         i2c_bit_add_bus_Rca543f36
> > depmod:         i2c_master_recv_R67b29cc4
> > depmod:         i2c_bit_del_bus_Rdf920b11
> > depmod:         i2c_master_send_Rb692cb0e
> > depmod: *** Unresolved symbols in
> > /lib/modules/2.4.23-pre6-aa2/kernel/drivers/me
> > dia/video/msp3400.o
> > depmod:         i2c_probe_R4e2acbec
> > depmod:         i2c_add_driver_Racf22304
> > depmod:         i2c_transfer_R1dea91d1
> > depmod:         i2c_attach_client_Ra861362d
> > depmod:         i2c_master_send_Rb692cb0e
> > depmod:         i2c_detach_client_R0cfb40b4
> > depmod:         i2c_del_driver_R57837012
> > .... more following, all in  kernel/drivers/media/video/* ...
> 
> this looks like if you didn't compile the needed i2c (or maybe it was
> due the lack of a `make dep` first), the above modules (pcilynx bttv
> msp3400) looks innocent.

I have a script that goes into the i2c-2.7.0 tree and builds it. This
worked just fine with vanilla -pre6 (and older ones for a long time
now) but fails with -pre6-aa2.


Nevertheless, I am now building vanilla -pre6 just to confirm the
above claims... built just fine. Retry -aa2... failed.

Here is the diff in .config between my vanilla -pre6 and -aa2

@@ -44,8 +44,7 @@
 CONFIG_X86_XADD=y
 CONFIG_X86_BSWAP=y
 CONFIG_X86_POPAD_OK=y
-# CONFIG_RWSEM_GENERIC_SPINLOCK is not set
-CONFIG_RWSEM_XCHGADD_ALGORITHM=y
+CONFIG_X86_CMPXCHG8=y
 CONFIG_X86_L1_CACHE_SHIFT=6
 CONFIG_X86_HAS_TSC=y
 CONFIG_X86_GOOD_APIC=y
@@ -56,6 +55,7 @@
 CONFIG_X86_MCE=y
 CONFIG_TOSHIBA=m
 CONFIG_I8K=m
+CONFIG_THINKPAD=m
 CONFIG_MICROCODE=m
 CONFIG_X86_MSR=m
 CONFIG_X86_CPUID=m
@@ -63,6 +63,11 @@
 # CONFIG_HIGHMEM4G is not set
 # CONFIG_HIGHMEM64G is not set
 # CONFIG_HIGHMEM is not set
+CONFIG_FORCE_MAX_ZONEORDER=11
+CONFIG_1GB=y
+# CONFIG_2GB is not set
+# CONFIG_3GB is not set
+# CONFIG_05GB is not set
 # CONFIG_MATH_EMULATION is not set
 CONFIG_MTRR=y
 # CONFIG_SMP is not set
@@ -105,6 +110,8 @@
 CONFIG_SYSVIPC=y
 # CONFIG_BSD_PROCESS_ACCT is not set
 CONFIG_SYSCTL=y
+CONFIG_MAX_USER_RT_PRIO=100
+CONFIG_MAX_RT_PRIO=0
 CONFIG_KCORE_ELF=y
 # CONFIG_KCORE_AOUT is not set
 CONFIG_BINFMT_AOUT=m
@@ -316,6 +323,7 @@
 CONFIG_MD_RAID5=m
 CONFIG_MD_MULTIPATH=m
 CONFIG_BLK_DEV_LVM=m
+CONFIG_BLK_DEV_DM=m
 
 #
 # Networking options
@@ -328,6 +336,10 @@
 CONFIG_FILTER=y
 CONFIG_UNIX=m
 CONFIG_INET=y
+CONFIG_TUX=m
+CONFIG_TUX_EXTCGI=y
+# CONFIG_TUX_EXTENDED_LOG is not set
+# CONFIG_TUX_DEBUG is not set
 # CONFIG_IP_MULTICAST is not set
 # CONFIG_IP_ADVANCED_ROUTER is not set
 # CONFIG_IP_PNP is not set
@@ -715,6 +727,10 @@
 CONFIG_SCSI_QLOGIC_FC=m
 # CONFIG_SCSI_QLOGIC_FC_FIRMWARE is not set
 CONFIG_SCSI_QLOGIC_1280=m
+CONFIG_SCSI_QLOGIC_QLA2XXX=y
+CONFIG_SCSI_QLOGIC_QLA2XXX_QLA2100=m
+CONFIG_SCSI_QLOGIC_QLA2XXX_QLA2200=m
+CONFIG_SCSI_QLOGIC_QLA2XXX_QLA2300=m
 CONFIG_SCSI_SEAGATE=m
 CONFIG_SCSI_SIM710=m
 CONFIG_SCSI_SYM53C416=m
@@ -731,11 +747,7 @@
 #
 # PCMCIA SCSI adapter support
 #
-CONFIG_SCSI_PCMCIA=y
-CONFIG_PCMCIA_AHA152X=m
-CONFIG_PCMCIA_FDOMAIN=m
-CONFIG_PCMCIA_NINJA_SCSI=m
-CONFIG_PCMCIA_QLOGIC=m
+# CONFIG_SCSI_PCMCIA is not set
 
 #
 # Fusion MPT device support
@@ -909,6 +921,7 @@
 CONFIG_FDDI=y
 CONFIG_DEFXX=m
 CONFIG_SKFP=m
+CONFIG_NETCONSOLE=m
 CONFIG_HIPPI=y
 CONFIG_ROADRUNNER=m
 # CONFIG_ROADRUNNER_LARGE_RINGS is not set
@@ -1479,6 +1492,7 @@
 # CONFIG_AGP_SWORKS is not set
 # CONFIG_AGP_NVIDIA is not set
 # CONFIG_AGP_ATI is not set
+CONFIG_ECC=m
 
 #
 # Direct Rendering Manager (XFree86 DRI support)
@@ -1619,6 +1633,13 @@
 # CONFIG_UDF_RW is not set
 CONFIG_UFS_FS=m
 # CONFIG_UFS_FS_WRITE is not set
+CONFIG_XFS_FS=m
+# CONFIG_XFS_POSIX_ACL is not set
+# CONFIG_XFS_RT is not set
+# CONFIG_XFS_QUOTA is not set
+# CONFIG_XFS_DMAPI is not set
+# CONFIG_XFS_DEBUG is not set
+# CONFIG_PAGEBUF_DEBUG is not set
 
 #
 # Network File Systems
@@ -2061,3 +2082,4 @@
 CONFIG_ZLIB_INFLATE=m
 CONFIG_ZLIB_DEFLATE=m
 CONFIG_FW_LOADER=m
+CONFIG_QSORT=m


--
Eyal Lebedinsky (eyal@eyal.emu.id.au) <http://samba.org/eyal/>

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

* Re: 2.4.23pre6aa2 - some problems [with patches]
  2003-10-05 10:40   ` Andrea Arcangeli
  2003-10-05 11:02     ` Thomas Winischhofer
  2003-10-06  0:43     ` Eyal Lebedinsky
@ 2003-10-06  0:55     ` Eyal Lebedinsky
  2 siblings, 0 replies; 6+ messages in thread
From: Eyal Lebedinsky @ 2003-10-06  0:55 UTC (permalink / raw)
  To: Andrea Arcangeli; +Cc: linux-kernel

Andrea Arcangeli wrote:
> 
> On Sun, Oct 05, 2003 at 01:38:20AM +1000, Eyal Lebedinsky wrote:
> > This is most unusual as -aa patches usually apply clean, but I am
> > encountering a number of build problems.
> 
> > And building i2c-2.7.0 (which I need for sensors) is failing.
> >
> > depmod: *** Unresolved symbols in
> > /lib/modules/2.4.23-pre6-aa2/kernel/drivers/ie
> > ee1394/pcilynx.o
> > depmod:         i2c_bit_add_bus_Rca543f36
> > depmod:         i2c_transfer_R1dea91d1
> > depmod:         i2c_bit_del_bus_Rdf920b11
> > depmod: *** Unresolved symbols in
[trimmed]
> 
> this looks like if you didn't compile the needed i2c (or maybe it was
> due the lack of a `make dep` first), the above modules (pcilynx bttv
> msp3400) looks innocent.

OK, carefully inspecting the logs explains the above. -aa2 breaks the
build
of i2c-2.7.0, but my script already removed the original i2c modules by
then. The failure is just another static HZ initializer situation.

I think that we need an option to revert HZ to a constant for people
that do not want to fight with this change for now (I am sure there
are other drivers in the wild that will take a while to catch up
with this change even after it makes mainline).

gcc -I/usr/src/linux/include -O2 -DLM_SENSORS -D__KERNEL__ -DMODULE
-fomit-frame
-pointer -DEXPORT_SYMTAB -DMODVERSIONS -include
/usr/src/linux/include/linux/mod
versions.h -c kernel/i2c-philips-par.c -o kernel/i2c-philips-par.o
kernel/i2c-philips-par.c:163: initializer element is not constant
kernel/i2c-philips-par.c:163: (near initialization for
`bit_lp_data.timeout')
kernel/i2c-philips-par.c:172: initializer element is not constant
kernel/i2c-philips-par.c:172: (near initialization for
`bit_lp_data2.timeout')

--
Eyal Lebedinsky (eyal@eyal.emu.id.au) <http://samba.org/eyal/>

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

end of thread, other threads:[~2003-10-06  0:55 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2003-10-04 10:57 2.4.23pre6aa2 Andrea Arcangeli
2003-10-04 15:38 ` 2.4.23pre6aa2 - some problems [with patches] Eyal Lebedinsky
2003-10-05 10:40   ` Andrea Arcangeli
2003-10-05 11:02     ` Thomas Winischhofer
2003-10-06  0:43     ` Eyal Lebedinsky
2003-10-06  0:55     ` Eyal Lebedinsky

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).