All of lore.kernel.org
 help / color / mirror / Atom feed
* [KJ] [PATCH][RESUBMIT][0/21]  compile warning cleanup
@ 2005-02-14 22:02 Stephen Biggs
  0 siblings, 0 replies; only message in thread
From: Stephen Biggs @ 2005-02-14 22:02 UTC (permalink / raw)
  To: kernel-janitors

[-- Attachment #1: Mail message body --]
[-- Type: text/plain, Size: 1473 bytes --]

Hello all,

After taking into consideration all of the fine feedback I received from 
my first ever attempt at submitting a series of patches, I am now 
applying this feedback and am submitting patches that will hopefully pass 
the test and be applied.

I am resubmitting the entire series, even those that did not receive any 
feedback at all and are unchanged, since I received no notification that 
they were to be applied.

I have created these patches against the linux-2.6.11-rc3-mm2 tree since, 
while I am aware of the 2.6.11-rc4 patch, there is no corresponding patch 
for the -mm tree.  Is this the right way to go?  If not, please advise 
and I'll resubmit yet again.

Please note my solution (patch #17 in this series) to the unused table 
structures in some poorly written drivers that always assume that they 
will be compiled as modules.  I changed include/linux/modules.h to create 
an alias to remove the warning.  This works for all the compilation.  The 
advantages to this patch are 1:, that it removes the warnings, and 2:, 
the source files themselves do not have to be changed because of this 
problem.  The disadvantages are 1:, it removes the warning so a module 
writer will not be apprised of the fact that there is a minor problem, 
and 2, it pollutes the global name space. The patch also keeps the same 
status as before with the warning, that is, it still compiles the table 
into the code, so the kernel will drag around this unused data.



[-- Attachment #2: Type: text/plain, Size: 167 bytes --]

_______________________________________________
Kernel-janitors mailing list
Kernel-janitors@lists.osdl.org
http://lists.osdl.org/mailman/listinfo/kernel-janitors

^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2005-02-14 22:02 UTC | newest]

Thread overview: (only message) (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2005-02-14 22:02 [KJ] [PATCH][RESUBMIT][0/21] compile warning cleanup Stephen Biggs

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.