Re: linux-next: build failure after merge of the final tree (stagingrelated)

From: Greg KH
Date: Wed Jun 15 2011 - 11:47:18 EST


On Wed, Jun 15, 2011 at 02:05:29PM +0200, Geert Uytterhoeven wrote:
> On Fri, Jun 10, 2011 at 08:05, Stephen Rothwell <sfr@xxxxxxxxxxxxxxxx> wrote:
> > On Thu, 9 Jun 2011 12:22:43 -0700 Greg KH <greg@xxxxxxxxx> wrote:
> >> On Thu, Jun 09, 2011 at 11:41:27AM -0700, Greg KH wrote:
> >> > > Also in several other files in drivers/staging/brcm80211/
> >> > >
> >> > > Also:
> >> > >
> >> > > drivers/staging/comedi/drivers.c: In function 'comedi_buf_alloc':
> >> > > drivers/staging/comedi/drivers.c:505:37: error: 'PAGE_KERNEL_NOCACHE' undeclared (first use in this function)
>
> I have my doubts
>
> #ifdef PAGE_KERNEL_NOCACHE
> vmap(pages, n_pages, VM_MAP, PAGE_KERNEL_NOCACHE);
> #else
> vmap(pages, n_pages, VM_MAP, PAGE_KERNEL);
> #endif
>
> is the right "fix" for this...
>
> BTW, PAGE_KERNEL_NOCACHE is defined on frv, m32r, mn10300, sh, and x86 only.
>
> Strangely, both sparc and tile have
>
> /*
> * Some hardware wants to get fixmapped without caching.
> */
> #define set_fixmap_nocache(idx, phys) \
> __set_fixmap(idx, phys, PAGE_KERNEL_NOCACHE)
>
> in their <asm/fixmap.h>, without having PAGE_KERNEL_NOCACHE...

So, what should the correct fix here be?

confused,

greg k-h
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/