* test12-pre4 drivers/net/dummy
@ 2000-12-04 6:07 Garst R. Reese
2000-12-04 6:19 ` Mohammad A. Haque
0 siblings, 1 reply; 6+ messages in thread
From: Garst R. Reese @ 2000-12-04 6:07 UTC (permalink / raw)
To: linux-kernel
Fails to compile module at line 103,
invalid type argument of ->
Sorry if this well known.
Garst
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
Please read the FAQ at http://www.tux.org/lkml/
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: test12-pre4 drivers/net/dummy
2000-12-04 6:07 test12-pre4 drivers/net/dummy Garst R. Reese
@ 2000-12-04 6:19 ` Mohammad A. Haque
2000-12-04 8:05 ` Jeff Garzik
0 siblings, 1 reply; 6+ messages in thread
From: Mohammad A. Haque @ 2000-12-04 6:19 UTC (permalink / raw)
To: Garst R. Reese; +Cc: linux-kernel
Patch posted here...
http://marc.theaimsgroup.com/?l=linux-kernel&m=97590235825341&w=2
"Garst R. Reese" wrote:
>
> Fails to compile module at line 103,
> invalid type argument of ->
> Sorry if this well known.
> Garst
> -
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to majordomo@vger.kernel.org
> Please read the FAQ at http://www.tux.org/lkml/
--
=====================================================================
Mohammad A. Haque http://www.haque.net/
mhaque@haque.net
"Alcohol and calculus don't mix. Project Lead
Don't drink and derive." --Unknown http://wm.themes.org/
batmanppc@themes.org
=====================================================================
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
Please read the FAQ at http://www.tux.org/lkml/
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: test12-pre4 drivers/net/dummy
2000-12-04 6:19 ` Mohammad A. Haque
@ 2000-12-04 8:05 ` Jeff Garzik
2000-12-04 14:39 ` Mohammad A. Haque
0 siblings, 1 reply; 6+ messages in thread
From: Jeff Garzik @ 2000-12-04 8:05 UTC (permalink / raw)
To: Mohammad A. Haque; +Cc: Garst R. Reese, linux-kernel
the fix is in module.h which needs extra parens in the def of
set_module_owner...
Jeff
On Mon, 4 Dec 2000, Mohammad A. Haque wrote:
> Patch posted here...
> http://marc.theaimsgroup.com/?l=linux-kernel&m=97590235825341&w=2
>
> "Garst R. Reese" wrote:
> >
> > Fails to compile module at line 103,
> > invalid type argument of ->
> > Sorry if this well known.
> > Garst
> > -
> > To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> > the body of a message to majordomo@vger.kernel.org
> > Please read the FAQ at http://www.tux.org/lkml/
>
> --
>
> =====================================================================
> Mohammad A. Haque http://www.haque.net/
> mhaque@haque.net
>
> "Alcohol and calculus don't mix. Project Lead
> Don't drink and derive." --Unknown http://wm.themes.org/
> batmanppc@themes.org
> =====================================================================
> -
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to majordomo@vger.kernel.org
> Please read the FAQ at http://www.tux.org/lkml/
>
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
Please read the FAQ at http://www.tux.org/lkml/
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: test12-pre4 drivers/net/dummy
2000-12-04 8:05 ` Jeff Garzik
@ 2000-12-04 14:39 ` Mohammad A. Haque
2000-12-05 4:19 ` Garst R. Reese
0 siblings, 1 reply; 6+ messages in thread
From: Mohammad A. Haque @ 2000-12-04 14:39 UTC (permalink / raw)
To: Jeff Garzik; +Cc: Garst R. Reese, linux-kernel
[-- Attachment #1: Type: TEXT/PLAIN, Size: 704 bytes --]
Ok, so here's the proper patch for those who dont want to wait for t5 =)
Ignore previous.
On Mon, 4 Dec 2000, Jeff Garzik wrote:
> the fix is in module.h which needs extra parens in the def of
> set_module_owner...
>
> Jeff
--
=====================================================================
Mohammad A. Haque http://www.haque.net/
mhaque@haque.net
"Alcohol and calculus don't mix. Project Lead
Don't drink and derive." --Unknown http://wm.themes.org/
batmanppc@themes.org
=====================================================================
[-- Attachment #2: Type: TEXT/PLAIN, Size: 451 bytes --]
--- linux/include/linux/module.h.orig Mon Dec 4 09:15:38 2000
+++ linux/include/linux/module.h Mon Dec 4 09:35:25 2000
@@ -345,7 +345,7 @@
#endif /* MODULE */
#ifdef CONFIG_MODULES
-#define SET_MODULE_OWNER(some_struct) do { some_struct->owner = THIS_MODULE; } while (0)
+#define SET_MODULE_OWNER(some_struct) do { (some_struct)->owner = THIS_MODULE; } while (0)
#else
#define SET_MODULE_OWNER(some_struct) do { } while (0)
#endif
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: test12-pre4 drivers/net/dummy
2000-12-04 14:39 ` Mohammad A. Haque
@ 2000-12-05 4:19 ` Garst R. Reese
2000-12-05 5:25 ` Mohammad A. Haque
0 siblings, 1 reply; 6+ messages in thread
From: Garst R. Reese @ 2000-12-05 4:19 UTC (permalink / raw)
To: Mohammad A. Haque; +Cc: Jeff Garzik, linux-kernel
Did you send it Linus?
It is not in pre5
Garst
"Mohammad A. Haque" wrote:
> Ok, so here's the proper patch for those who dont want to wait for t5 =)
> Ignore previous.
>
> On Mon, 4 Dec 2000, Jeff Garzik wrote:
>
> > the fix is in module.h which needs extra parens in the def of
> > set_module_owner...
> >
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
Please read the FAQ at http://www.tux.org/lkml/
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: test12-pre4 drivers/net/dummy
2000-12-05 4:19 ` Garst R. Reese
@ 2000-12-05 5:25 ` Mohammad A. Haque
0 siblings, 0 replies; 6+ messages in thread
From: Mohammad A. Haque @ 2000-12-05 5:25 UTC (permalink / raw)
To: Garst R. Reese; +Cc: Jeff Garzik, linux-kernel
Alan has it and Linus hasn't applied Alan's patch yet...
Linus said.....
"[ Alan - I ahve your patches in my incoming queue still, I wanted to
get
an interim version out to check with Al on the block list and the VM
stuff with Rik and people. ]"
"Garst R. Reese" wrote:
>
> Did you send it Linus?
> It is not in pre5
> Garst
--
=====================================================================
Mohammad A. Haque http://www.haque.net/
mhaque@haque.net
"Alcohol and calculus don't mix. Project Lead
Don't drink and derive." --Unknown http://wm.themes.org/
batmanppc@themes.org
=====================================================================
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
Please read the FAQ at http://www.tux.org/lkml/
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2000-12-05 5:56 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2000-12-04 6:07 test12-pre4 drivers/net/dummy Garst R. Reese
2000-12-04 6:19 ` Mohammad A. Haque
2000-12-04 8:05 ` Jeff Garzik
2000-12-04 14:39 ` Mohammad A. Haque
2000-12-05 4:19 ` Garst R. Reese
2000-12-05 5:25 ` Mohammad A. Haque
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox