* 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@vger.kernel.org, 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@vger.kernel.org, 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@vger.kernel.org, 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@vger.kernel.org, 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@vger.kernel.org, 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@vger.kernel.org, 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 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.