* Build error in -next due to 'drivers/video/fbdev/atyfb: Replace MTRR UC hole with strong UC'
@ 2015-07-31 2:26 Guenter Roeck
2015-07-31 17:52 ` Paul Gortmaker
0 siblings, 1 reply; 6+ messages in thread
From: Guenter Roeck @ 2015-07-31 2:26 UTC (permalink / raw)
To: Luis R. Rodriguez; +Cc: linux-next, Ingo Molnar
Hi Luis,
probably the zero-day build already told you, but just in case:
Your commit 'drivers/video/fbdev/atyfb: Replace MTRR UC hole with strong UC'
causes build errors for architectures which don't support ioremap_uc.
This affects at least the alpha architecture, though there may be others.
Here is the build error:
drivers/video/fbdev/aty/atyfb_base.c: In function 'atyfb_setup_generic':
drivers/video/fbdev/aty/atyfb_base.c:3447:2: error: implicit declaration of function 'ioremap_uc'
Thanks,
Guenter
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: Build error in -next due to 'drivers/video/fbdev/atyfb: Replace MTRR UC hole with strong UC'
2015-07-31 2:26 Build error in -next due to 'drivers/video/fbdev/atyfb: Replace MTRR UC hole with strong UC' Guenter Roeck
@ 2015-07-31 17:52 ` Paul Gortmaker
2015-08-01 0:02 ` Luis R. Rodriguez
0 siblings, 1 reply; 6+ messages in thread
From: Paul Gortmaker @ 2015-07-31 17:52 UTC (permalink / raw)
To: Guenter Roeck; +Cc: Luis R. Rodriguez, linux-next, Ingo Molnar
On Thu, Jul 30, 2015 at 10:26 PM, Guenter Roeck <linux@roeck-us.net> wrote:
> Hi Luis,
>
> probably the zero-day build already told you, but just in case:
>
> Your commit 'drivers/video/fbdev/atyfb: Replace MTRR UC hole with strong UC'
> causes build errors for architectures which don't support ioremap_uc.
> This affects at least the alpha architecture, though there may be others.
Yep, parisc allmodconfig now sees it as well. Bisected before seeing
this message.
3cc2dac5be3f23414a4efdee0b26d79bed297cac is the first bad commit
commit 3cc2dac5be3f23414a4efdee0b26d79bed297cac
Author: Luis R. Rodriguez <mcgrof@suse.com>
Date: Thu Jul 9 18:24:58 2015 -0700
drivers/video/fbdev/atyfb: Replace MTRR UC hole with strong UC
http://kisskb.ellerman.id.au/kisskb/buildresult/12475234/
Paul.
--
>
> Here is the build error:
>
> drivers/video/fbdev/aty/atyfb_base.c: In function 'atyfb_setup_generic':
> drivers/video/fbdev/aty/atyfb_base.c:3447:2: error: implicit declaration of
> function 'ioremap_uc'
>
> Thanks,
> Guenter
> --
> To unsubscribe from this list: send the line "unsubscribe linux-next" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: Build error in -next due to 'drivers/video/fbdev/atyfb: Replace MTRR UC hole with strong UC'
2015-07-31 17:52 ` Paul Gortmaker
@ 2015-08-01 0:02 ` Luis R. Rodriguez
2015-08-01 0:09 ` Guenter Roeck
0 siblings, 1 reply; 6+ messages in thread
From: Luis R. Rodriguez @ 2015-08-01 0:02 UTC (permalink / raw)
To: Paul Gortmaker; +Cc: Guenter Roeck, linux-next, Ingo Molnar
On Fri, Jul 31, 2015 at 01:52:24PM -0400, Paul Gortmaker wrote:
> On Thu, Jul 30, 2015 at 10:26 PM, Guenter Roeck <linux@roeck-us.net> wrote:
> > Hi Luis,
> >
> > probably the zero-day build already told you, but just in case:
> >
> > Your commit 'drivers/video/fbdev/atyfb: Replace MTRR UC hole with strong UC'
> > causes build errors for architectures which don't support ioremap_uc.
> > This affects at least the alpha architecture, though there may be others.
>
> Yep, parisc allmodconfig now sees it as well. Bisected before seeing
> this message.
>
> 3cc2dac5be3f23414a4efdee0b26d79bed297cac is the first bad commit
> commit 3cc2dac5be3f23414a4efdee0b26d79bed297cac
> Author: Luis R. Rodriguez <mcgrof@suse.com>
> Date: Thu Jul 9 18:24:58 2015 -0700
>
> drivers/video/fbdev/atyfb: Replace MTRR UC hole with strong UC
>
> http://kisskb.ellerman.id.au/kisskb/buildresult/12475234/
I submitted a fix for it, can you tell me your linux-next tree?
git describe
Luis
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: Build error in -next due to 'drivers/video/fbdev/atyfb: Replace MTRR UC hole with strong UC'
2015-08-01 0:02 ` Luis R. Rodriguez
@ 2015-08-01 0:09 ` Guenter Roeck
2015-08-01 0:39 ` Luis R. Rodriguez
0 siblings, 1 reply; 6+ messages in thread
From: Guenter Roeck @ 2015-08-01 0:09 UTC (permalink / raw)
To: Luis R. Rodriguez, Paul Gortmaker; +Cc: linux-next, Ingo Molnar
On 07/31/2015 05:02 PM, Luis R. Rodriguez wrote:
> On Fri, Jul 31, 2015 at 01:52:24PM -0400, Paul Gortmaker wrote:
>> On Thu, Jul 30, 2015 at 10:26 PM, Guenter Roeck <linux@roeck-us.net> wrote:
>>> Hi Luis,
>>>
>>> probably the zero-day build already told you, but just in case:
>>>
>>> Your commit 'drivers/video/fbdev/atyfb: Replace MTRR UC hole with strong UC'
>>> causes build errors for architectures which don't support ioremap_uc.
>>> This affects at least the alpha architecture, though there may be others.
>>
>> Yep, parisc allmodconfig now sees it as well. Bisected before seeing
>> this message.
>>
>> 3cc2dac5be3f23414a4efdee0b26d79bed297cac is the first bad commit
>> commit 3cc2dac5be3f23414a4efdee0b26d79bed297cac
>> Author: Luis R. Rodriguez <mcgrof@suse.com>
>> Date: Thu Jul 9 18:24:58 2015 -0700
>>
>> drivers/video/fbdev/atyfb: Replace MTRR UC hole with strong UC
>>
>> http://kisskb.ellerman.id.au/kisskb/buildresult/12475234/
>
> I submitted a fix for it, can you tell me your linux-next tree?
>
http://server.roeck-us.net:8010/builders/next-alpha-next
next-20150729 to next-20150731.
Guenter
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: Build error in -next due to 'drivers/video/fbdev/atyfb: Replace MTRR UC hole with strong UC'
2015-08-01 0:09 ` Guenter Roeck
@ 2015-08-01 0:39 ` Luis R. Rodriguez
2015-08-01 1:05 ` Guenter Roeck
0 siblings, 1 reply; 6+ messages in thread
From: Luis R. Rodriguez @ 2015-08-01 0:39 UTC (permalink / raw)
To: Guenter Roeck; +Cc: Paul Gortmaker, linux-next, Ingo Molnar
On Fri, Jul 31, 2015 at 05:09:52PM -0700, Guenter Roeck wrote:
> On 07/31/2015 05:02 PM, Luis R. Rodriguez wrote:
> >On Fri, Jul 31, 2015 at 01:52:24PM -0400, Paul Gortmaker wrote:
> >>On Thu, Jul 30, 2015 at 10:26 PM, Guenter Roeck <linux@roeck-us.net> wrote:
> >>>Hi Luis,
> >>>
> >>>probably the zero-day build already told you, but just in case:
> >>>
> >>>Your commit 'drivers/video/fbdev/atyfb: Replace MTRR UC hole with strong UC'
> >>>causes build errors for architectures which don't support ioremap_uc.
> >>>This affects at least the alpha architecture, though there may be others.
> >>
> >>Yep, parisc allmodconfig now sees it as well. Bisected before seeing
> >>this message.
> >>
> >>3cc2dac5be3f23414a4efdee0b26d79bed297cac is the first bad commit
> >>commit 3cc2dac5be3f23414a4efdee0b26d79bed297cac
> >>Author: Luis R. Rodriguez <mcgrof@suse.com>
> >>Date: Thu Jul 9 18:24:58 2015 -0700
> >>
> >> drivers/video/fbdev/atyfb: Replace MTRR UC hole with strong UC
> >>
> >>http://kisskb.ellerman.id.au/kisskb/buildresult/12475234/
> >
> >I submitted a fix for it, can you tell me your linux-next tree?
> >
>
> http://server.roeck-us.net:8010/builders/next-alpha-next
>
> next-20150729 to next-20150731.
Great thanks, I sent a proposal general fix, that should be tested first.
If all the bots say go for it, then yay, otherwise a simple
#define ioremap_uc ioremap_nocache
would be the way to go, but I would prefer to fix this sort of thing for good.
Luis
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: Build error in -next due to 'drivers/video/fbdev/atyfb: Replace MTRR UC hole with strong UC'
2015-08-01 0:39 ` Luis R. Rodriguez
@ 2015-08-01 1:05 ` Guenter Roeck
0 siblings, 0 replies; 6+ messages in thread
From: Guenter Roeck @ 2015-08-01 1:05 UTC (permalink / raw)
To: Luis R. Rodriguez; +Cc: Paul Gortmaker, linux-next, Ingo Molnar
Hi Luis,
On 07/31/2015 05:39 PM, Luis R. Rodriguez wrote:
> On Fri, Jul 31, 2015 at 05:09:52PM -0700, Guenter Roeck wrote:
>> On 07/31/2015 05:02 PM, Luis R. Rodriguez wrote:
>>> On Fri, Jul 31, 2015 at 01:52:24PM -0400, Paul Gortmaker wrote:
>>>> On Thu, Jul 30, 2015 at 10:26 PM, Guenter Roeck <linux@roeck-us.net> wrote:
>>>>> Hi Luis,
>>>>>
>>>>> probably the zero-day build already told you, but just in case:
>>>>>
>>>>> Your commit 'drivers/video/fbdev/atyfb: Replace MTRR UC hole with strong UC'
>>>>> causes build errors for architectures which don't support ioremap_uc.
>>>>> This affects at least the alpha architecture, though there may be others.
>>>>
>>>> Yep, parisc allmodconfig now sees it as well. Bisected before seeing
>>>> this message.
>>>>
>>>> 3cc2dac5be3f23414a4efdee0b26d79bed297cac is the first bad commit
>>>> commit 3cc2dac5be3f23414a4efdee0b26d79bed297cac
>>>> Author: Luis R. Rodriguez <mcgrof@suse.com>
>>>> Date: Thu Jul 9 18:24:58 2015 -0700
>>>>
>>>> drivers/video/fbdev/atyfb: Replace MTRR UC hole with strong UC
>>>>
>>>> http://kisskb.ellerman.id.au/kisskb/buildresult/12475234/
>>>
>>> I submitted a fix for it, can you tell me your linux-next tree?
>>>
>>
>> http://server.roeck-us.net:8010/builders/next-alpha-next
>>
>> next-20150729 to next-20150731.
>
> Great thanks, I sent a proposal general fix, that should be tested first.
> If all the bots say go for it, then yay, otherwise a simple
>
> #define ioremap_uc ioremap_nocache
>
> would be the way to go, but I would prefer to fix this sort of thing for good.
>
I just replied to your patch - it doesn't work.
We already know that parisc is also affected, and with
some experimentation I found that ia64 is affected as well.
Guenter
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2015-08-01 1:05 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-07-31 2:26 Build error in -next due to 'drivers/video/fbdev/atyfb: Replace MTRR UC hole with strong UC' Guenter Roeck
2015-07-31 17:52 ` Paul Gortmaker
2015-08-01 0:02 ` Luis R. Rodriguez
2015-08-01 0:09 ` Guenter Roeck
2015-08-01 0:39 ` Luis R. Rodriguez
2015-08-01 1:05 ` Guenter Roeck
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).