* linux-next: pci tree build warning
@ 2009-10-08 4:27 Stephen Rothwell
2009-10-08 9:59 ` [PATCH pci-2.6#linux-next] pci: pci_dfl_cache_line_size is __devinitdata Tejun Heo
0 siblings, 1 reply; 10+ messages in thread
From: Stephen Rothwell @ 2009-10-08 4:27 UTC (permalink / raw)
To: Jesse Barnes; +Cc: linux-next, linux-kernel, Tejun Heo
[-- Attachment #1: Type: text/plain, Size: 681 bytes --]
Hi Jesse,
Today's linux-next build (powerpc allnoconfig) produced this warning:
WARNING: drivers/pci/built-in.o(.devinit.text+0x86a): Section mismatch in reference from the function pci_init() to the variable .init.data:pci_dfl_cache_line_size
The function __devinit pci_init() references
a variable __initdata pci_dfl_cache_line_size.
If pci_dfl_cache_line_size is only used by pci_init then
annotate pci_dfl_cache_line_size with a matching annotation.
Introduced by commit b20dd110d21aac94b3f4063cffbcf9718656ce51 ("PCI:
determine CLS more intelligently").
--
Cheers,
Stephen Rothwell sfr@canb.auug.org.au
http://www.canb.auug.org.au/~sfr/
[-- Attachment #2: Type: application/pgp-signature, Size: 198 bytes --]
^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH pci-2.6#linux-next] pci: pci_dfl_cache_line_size is __devinitdata
2009-10-08 4:27 linux-next: pci tree build warning Stephen Rothwell
@ 2009-10-08 9:59 ` Tejun Heo
2009-10-08 17:25 ` Jesse Barnes
2009-10-26 20:40 ` Jesse Barnes
0 siblings, 2 replies; 10+ messages in thread
From: Tejun Heo @ 2009-10-08 9:59 UTC (permalink / raw)
To: Stephen Rothwell; +Cc: Jesse Barnes, linux-next, linux-kernel
pci_dfl_cache_line_size is marked as __initdata but referenced by
pci_init() which is __devinit. Make it __devinitdata instead of
__initdata.
Signed-off-by: Tejun Heo <tj@kernel.org>
Reported-by: Stephen Rothwell <sfr@canb.auug.org.au>
---
drivers/pci/pci.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/pci/pci.c b/drivers/pci/pci.c
index 1ae95df..a6e22fd 100644
--- a/drivers/pci/pci.c
+++ b/drivers/pci/pci.c
@@ -53,7 +53,7 @@ unsigned long pci_hotplug_mem_size = DEFAULT_HOTPLUG_MEM_SIZE;
* the dfl or actual value as it sees fit. Don't forget this is
* measured in 32-bit words, not bytes.
*/
-u8 pci_dfl_cache_line_size __initdata = L1_CACHE_BYTES >> 2;
+u8 pci_dfl_cache_line_size __devinitdata = L1_CACHE_BYTES >> 2;
u8 pci_cache_line_size;
/**
^ permalink raw reply related [flat|nested] 10+ messages in thread
* Re: [PATCH pci-2.6#linux-next] pci: pci_dfl_cache_line_size is __devinitdata
2009-10-08 9:59 ` [PATCH pci-2.6#linux-next] pci: pci_dfl_cache_line_size is __devinitdata Tejun Heo
@ 2009-10-08 17:25 ` Jesse Barnes
2009-10-26 20:40 ` Jesse Barnes
1 sibling, 0 replies; 10+ messages in thread
From: Jesse Barnes @ 2009-10-08 17:25 UTC (permalink / raw)
To: Tejun Heo; +Cc: Stephen Rothwell, linux-next, linux-kernel
On Thu, 08 Oct 2009 18:59:53 +0900
Tejun Heo <tj@kernel.org> wrote:
> pci_dfl_cache_line_size is marked as __initdata but referenced by
> pci_init() which is __devinit. Make it __devinitdata instead of
> __initdata.
>
> Signed-off-by: Tejun Heo <tj@kernel.org>
> Reported-by: Stephen Rothwell <sfr@canb.auug.org.au>
> ---
> drivers/pci/pci.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/pci/pci.c b/drivers/pci/pci.c
> index 1ae95df..a6e22fd 100644
> --- a/drivers/pci/pci.c
> +++ b/drivers/pci/pci.c
> @@ -53,7 +53,7 @@ unsigned long pci_hotplug_mem_size =
> DEFAULT_HOTPLUG_MEM_SIZE;
> * the dfl or actual value as it sees fit. Don't forget this is
> * measured in 32-bit words, not bytes.
> */
> -u8 pci_dfl_cache_line_size __initdata = L1_CACHE_BYTES >> 2;
> +u8 pci_dfl_cache_line_size __devinitdata = L1_CACHE_BYTES >> 2;
> u8 pci_cache_line_size;
Fix pushed, thanks for the quick turnaround Tejun.
--
Jesse Barnes, Intel Open Source Technology Center
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH pci-2.6#linux-next] pci: pci_dfl_cache_line_size is __devinitdata
2009-10-08 9:59 ` [PATCH pci-2.6#linux-next] pci: pci_dfl_cache_line_size is __devinitdata Tejun Heo
2009-10-08 17:25 ` Jesse Barnes
@ 2009-10-26 20:40 ` Jesse Barnes
1 sibling, 0 replies; 10+ messages in thread
From: Jesse Barnes @ 2009-10-26 20:40 UTC (permalink / raw)
To: Tejun Heo; +Cc: Stephen Rothwell, Jesse Barnes, linux-next, linux-kernel
On Thu, 08 Oct 2009 18:59:53 +0900
Tejun Heo <tj@kernel.org> wrote:
> pci_dfl_cache_line_size is marked as __initdata but referenced by
> pci_init() which is __devinit. Make it __devinitdata instead of
> __initdata.
>
> Signed-off-by: Tejun Heo <tj@kernel.org>
> Reported-by: Stephen Rothwell <sfr@canb.auug.org.au>
> ---
Applied to my linux-next tree, thanks.
--
Jesse Barnes, Intel Open Source Technology Center
^ permalink raw reply [flat|nested] 10+ messages in thread
* linux-next: pci tree build warning
@ 2009-08-13 2:47 Stephen Rothwell
2009-08-13 3:32 ` Benjamin Herrenschmidt
0 siblings, 1 reply; 10+ messages in thread
From: Stephen Rothwell @ 2009-08-13 2:47 UTC (permalink / raw)
To: Jesse Barnes
Cc: linux-next, linux-kernel, Benjamin Herrenschmidt, Tiago Vignatti,
Dave Airlie
[-- Attachment #1: Type: text/plain, Size: 804 bytes --]
Hi Jesse,
Today's linux-next build (x86_64 allmodconfig) produced these warnings:
drivers/gpu/vga/vgaarb.c: In function 'vga_arb_write':
drivers/gpu/vga/vgaarb.c:834: warning: cast from pointer to integer of different size
drivers/gpu/vga/vgaarb.c:870: warning: cast from pointer to integer of different size
drivers/gpu/vga/vgaarb.c:920: warning: cast from pointer to integer of different size
drivers/gpu/vga/vgaarb.c:963: warning: cast from pointer to integer of different size
drivers/gpu/vga/vgaarb.c:1017: warning: cast from pointer to integer of different size
Introduced by commit a49c9c1b728fa8f50816d96a0f135e15a359e448 ("PCI/GPU:
implement VGA arbitration on Linux").
--
Cheers,
Stephen Rothwell sfr@canb.auug.org.au
http://www.canb.auug.org.au/~sfr/
[-- Attachment #2: Type: application/pgp-signature, Size: 197 bytes --]
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: linux-next: pci tree build warning
2009-08-13 2:47 linux-next: pci tree build warning Stephen Rothwell
@ 2009-08-13 3:32 ` Benjamin Herrenschmidt
2009-08-13 8:38 ` Vignatti Tiago (Nokia-D/Helsinki)
0 siblings, 1 reply; 10+ messages in thread
From: Benjamin Herrenschmidt @ 2009-08-13 3:32 UTC (permalink / raw)
To: Stephen Rothwell
Cc: Jesse Barnes, linux-next, linux-kernel, Tiago Vignatti, Dave Airlie
On Thu, 2009-08-13 at 12:47 +1000, Stephen Rothwell wrote:
> Hi Jesse,
>
> Today's linux-next build (x86_64 allmodconfig) produced these warnings:
>
> drivers/gpu/vga/vgaarb.c: In function 'vga_arb_write':
> drivers/gpu/vga/vgaarb.c:834: warning: cast from pointer to integer of different size
> drivers/gpu/vga/vgaarb.c:870: warning: cast from pointer to integer of different size
> drivers/gpu/vga/vgaarb.c:920: warning: cast from pointer to integer of different size
> drivers/gpu/vga/vgaarb.c:963: warning: cast from pointer to integer of different size
> drivers/gpu/vga/vgaarb.c:1017: warning: cast from pointer to integer of different size
>
> Introduced by commit a49c9c1b728fa8f50816d96a0f135e15a359e448 ("PCI/GPU:
> implement VGA arbitration on Linux").
Right, somebody weirdly decided to cast pointers to int just to printk
them with %X :-) I hope it wasn't me ...
Just remove the casts and use %p instead.
Cheers,
Ben.
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: linux-next: pci tree build warning
2009-08-13 3:32 ` Benjamin Herrenschmidt
@ 2009-08-13 8:38 ` Vignatti Tiago (Nokia-D/Helsinki)
0 siblings, 0 replies; 10+ messages in thread
From: Vignatti Tiago (Nokia-D/Helsinki) @ 2009-08-13 8:38 UTC (permalink / raw)
To: Benjamin Herrenschmidt
Cc: Stephen Rothwell, Jesse Barnes, linux-next, linux-kernel, Dave Airlie
On Thu, Aug 13, 2009 at 05:32:40AM +0200, Benjamin Herrenschmidt wrote:
> On Thu, 2009-08-13 at 12:47 +1000, Stephen Rothwell wrote:
> > Hi Jesse,
> >
> > Today's linux-next build (x86_64 allmodconfig) produced these warnings:
> >
> > drivers/gpu/vga/vgaarb.c: In function 'vga_arb_write':
> > drivers/gpu/vga/vgaarb.c:834: warning: cast from pointer to integer of different size
> > drivers/gpu/vga/vgaarb.c:870: warning: cast from pointer to integer of different size
> > drivers/gpu/vga/vgaarb.c:920: warning: cast from pointer to integer of different size
> > drivers/gpu/vga/vgaarb.c:963: warning: cast from pointer to integer of different size
> > drivers/gpu/vga/vgaarb.c:1017: warning: cast from pointer to integer of different size
> >
> > Introduced by commit a49c9c1b728fa8f50816d96a0f135e15a359e448 ("PCI/GPU:
> > implement VGA arbitration on Linux").
>
> Right, somebody weirdly decided to cast pointers to int just to printk
> them with %X :-) I hope it wasn't me ...
Ahhh oops.. looks like someone really newbie trying to play developing kernel
code :p
Tiago
^ permalink raw reply [flat|nested] 10+ messages in thread
* linux-next: pci tree build warning
@ 2009-05-06 4:30 Stephen Rothwell
2009-05-06 7:21 ` Yinghai Lu
0 siblings, 1 reply; 10+ messages in thread
From: Stephen Rothwell @ 2009-05-06 4:30 UTC (permalink / raw)
To: Jesse Barnes; +Cc: linux-next, Yinghai Lu
[-- Attachment #1: Type: text/plain, Size: 471 bytes --]
Hi Jesse,
Today's linux-next build (powerpc ppc64_defconfig) produced this warning:
drivers/pci/probe.c: In function '__pci_read_base':
drivers/pci/probe.c:196: warning: large integer implicitly truncated to unsigned type
Probably introduced by commit 82160fd142cdf6956a677120426bf5baefcc7cf9
("PCI/x86: don't assume prefetchable ranges are 64bit").
--
Cheers,
Stephen Rothwell sfr@canb.auug.org.au
http://www.canb.auug.org.au/~sfr/
[-- Attachment #2: Type: application/pgp-signature, Size: 197 bytes --]
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: linux-next: pci tree build warning
2009-05-06 4:30 Stephen Rothwell
@ 2009-05-06 7:21 ` Yinghai Lu
0 siblings, 0 replies; 10+ messages in thread
From: Yinghai Lu @ 2009-05-06 7:21 UTC (permalink / raw)
To: Stephen Rothwell; +Cc: Jesse Barnes, linux-next
Stephen Rothwell wrote:
> Hi Jesse,
>
> Today's linux-next build (powerpc ppc64_defconfig) produced this warning:
>
> drivers/pci/probe.c: In function '__pci_read_base':
> drivers/pci/probe.c:196: warning: large integer implicitly truncated to unsigned type
>
> Probably introduced by commit 82160fd142cdf6956a677120426bf5baefcc7cf9
> ("PCI/x86: don't assume prefetchable ranges are 64bit").
>
--- a/drivers/pci/probe.c
+++ b/drivers/pci/probe.c
@@ -193,7 +193,7 @@ int __pci_read_base(struct pci_dev *dev, enum pci_bar_type type,
res->flags |= pci_calc_resource_flags(l) | IORESOURCE_SIZEALIGN;
if (type == pci_bar_io) {
l &= PCI_BASE_ADDRESS_IO_MASK;
- mask = PCI_BASE_ADDRESS_IO_MASK & 0xffff;
+ mask = PCI_BASE_ADDRESS_IO_MASK & IO_SPACE_LIMIT;
and
for x86:
#define IO_SPACE_LIMIT 0xffff
for powerpc
arch/powerpc/include/asm/io.h:#define IO_SPACE_LIMIT ~(0UL)
maybe we need to change back that line...
YH
^ permalink raw reply [flat|nested] 10+ messages in thread
* linux-next: pci tree build warning
@ 2009-02-18 6:31 Stephen Rothwell
0 siblings, 0 replies; 10+ messages in thread
From: Stephen Rothwell @ 2009-02-18 6:31 UTC (permalink / raw)
To: Jesse Barnes; +Cc: linux-next, Andrew Morton
[-- Attachment #1: Type: text/plain, Size: 510 bytes --]
Hi Jesse,
Today's linux-next build (powerpc ppc64_defconfig) produced this warning:
drivers/pci/setup-bus.c: In function 'pci_bus_assign_resources':
drivers/pci/setup-bus.c:503: warning: passing argument 1 of 'pbus_assign_resources_sorted' discards qualifiers from pointer target type
Caused by commit 1454fec9f61d92514bf3300f8ae99ba7749ccd0c ("PCI: constify
pci_bus_assign_resources()").
--
Cheers,
Stephen Rothwell sfr@canb.auug.org.au
http://www.canb.auug.org.au/~sfr/
[-- Attachment #2: Type: application/pgp-signature, Size: 197 bytes --]
^ permalink raw reply [flat|nested] 10+ messages in thread
end of thread, other threads:[~2009-10-26 20:40 UTC | newest]
Thread overview: 10+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2009-10-08 4:27 linux-next: pci tree build warning Stephen Rothwell
2009-10-08 9:59 ` [PATCH pci-2.6#linux-next] pci: pci_dfl_cache_line_size is __devinitdata Tejun Heo
2009-10-08 17:25 ` Jesse Barnes
2009-10-26 20:40 ` Jesse Barnes
-- strict thread matches above, loose matches on Subject: below --
2009-08-13 2:47 linux-next: pci tree build warning Stephen Rothwell
2009-08-13 3:32 ` Benjamin Herrenschmidt
2009-08-13 8:38 ` Vignatti Tiago (Nokia-D/Helsinki)
2009-05-06 4:30 Stephen Rothwell
2009-05-06 7:21 ` Yinghai Lu
2009-02-18 6:31 Stephen Rothwell
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).