* [Qemu-devel] [PATCH] xtensa_lx60: add missing #include "blockdev.h" @ 2012-06-08 9:50 Max Filippov 2012-06-08 10:08 ` Stefan Hajnoczi 0 siblings, 1 reply; 7+ messages in thread From: Max Filippov @ 2012-06-08 9:50 UTC (permalink / raw) To: qemu-devel; +Cc: stefanha, Max Filippov This should fix the following build failure: /home/buildbot/slave-public/block_mingw32/build/hw/xtensa_lx60.c: In function 'lx_init': /home/buildbot/slave-public/block_mingw32/build/hw/xtensa_lx60.c:212: warning: implicit declaration of function 'drive_get' /home/buildbot/slave-public/block_mingw32/build/hw/xtensa_lx60.c:212: warning: nested extern declaration of 'drive_get' /home/buildbot/slave-public/block_mingw32/build/hw/xtensa_lx60.c:212: error: 'IF_PFLASH' undeclared (first use in this function) /home/buildbot/slave-public/block_mingw32/build/hw/xtensa_lx60.c:212: error: (Each undeclared identifier is reported only once /home/buildbot/slave-public/block_mingw32/build/hw/xtensa_lx60.c:212: error: for each function it appears in.) /home/buildbot/slave-public/block_mingw32/build/hw/xtensa_lx60.c:216: error: dereferencing pointer to incomplete type Signed-off-by: Max Filippov <jcmvbkbc@gmail.com> --- hw/xtensa_lx60.c | 1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/hw/xtensa_lx60.c b/hw/xtensa_lx60.c index afdef49..cb759a9 100644 --- a/hw/xtensa_lx60.c +++ b/hw/xtensa_lx60.c @@ -34,6 +34,7 @@ #include "pc.h" #include "sysbus.h" #include "flash.h" +#include "blockdev.h" #include "xtensa_bootparam.h" typedef struct LxBoardDesc { -- 1.7.7.6 ^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [Qemu-devel] [PATCH] xtensa_lx60: add missing #include "blockdev.h" 2012-06-08 9:50 [Qemu-devel] [PATCH] xtensa_lx60: add missing #include "blockdev.h" Max Filippov @ 2012-06-08 10:08 ` Stefan Hajnoczi 2012-06-08 10:43 ` Stefan Weil 0 siblings, 1 reply; 7+ messages in thread From: Stefan Hajnoczi @ 2012-06-08 10:08 UTC (permalink / raw) To: Max Filippov; +Cc: qemu-devel On Fri, Jun 8, 2012 at 10:50 AM, Max Filippov <jcmvbkbc@gmail.com> wrote: > This should fix the following build failure: > > /home/buildbot/slave-public/block_mingw32/build/hw/xtensa_lx60.c: In function 'lx_init': > /home/buildbot/slave-public/block_mingw32/build/hw/xtensa_lx60.c:212: warning: implicit declaration of function 'drive_get' > /home/buildbot/slave-public/block_mingw32/build/hw/xtensa_lx60.c:212: warning: nested extern declaration of 'drive_get' > /home/buildbot/slave-public/block_mingw32/build/hw/xtensa_lx60.c:212: error: 'IF_PFLASH' undeclared (first use in this function) > /home/buildbot/slave-public/block_mingw32/build/hw/xtensa_lx60.c:212: error: (Each undeclared identifier is reported only once > /home/buildbot/slave-public/block_mingw32/build/hw/xtensa_lx60.c:212: error: for each function it appears in.) > /home/buildbot/slave-public/block_mingw32/build/hw/xtensa_lx60.c:216: error: dereferencing pointer to incomplete type > > Signed-off-by: Max Filippov <jcmvbkbc@gmail.com> > --- > hw/xtensa_lx60.c | 1 + > 1 files changed, 1 insertions(+), 0 deletions(-) Reviewed-by: Stefan Hajnoczi <stefanha@linux.vnet.ibm.com> ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [Qemu-devel] [PATCH] xtensa_lx60: add missing #include "blockdev.h" 2012-06-08 10:08 ` Stefan Hajnoczi @ 2012-06-08 10:43 ` Stefan Weil 2012-06-08 10:57 ` Stefan Hajnoczi 0 siblings, 1 reply; 7+ messages in thread From: Stefan Weil @ 2012-06-08 10:43 UTC (permalink / raw) To: Stefan Hajnoczi; +Cc: Max Filippov, qemu-devel Am 08.06.2012 12:08, schrieb Stefan Hajnoczi: > On Fri, Jun 8, 2012 at 10:50 AM, Max Filippov<jcmvbkbc@gmail.com> wrote: >> This should fix the following build failure: >> >> /home/buildbot/slave-public/block_mingw32/build/hw/xtensa_lx60.c: In function 'lx_init': >> /home/buildbot/slave-public/block_mingw32/build/hw/xtensa_lx60.c:212: warning: implicit declaration of function 'drive_get' >> /home/buildbot/slave-public/block_mingw32/build/hw/xtensa_lx60.c:212: warning: nested extern declaration of 'drive_get' >> /home/buildbot/slave-public/block_mingw32/build/hw/xtensa_lx60.c:212: error: 'IF_PFLASH' undeclared (first use in this function) >> /home/buildbot/slave-public/block_mingw32/build/hw/xtensa_lx60.c:212: error: (Each undeclared identifier is reported only once >> /home/buildbot/slave-public/block_mingw32/build/hw/xtensa_lx60.c:212: error: for each function it appears in.) >> /home/buildbot/slave-public/block_mingw32/build/hw/xtensa_lx60.c:216: error: dereferencing pointer to incomplete type >> >> Signed-off-by: Max Filippov<jcmvbkbc@gmail.com> >> --- >> hw/xtensa_lx60.c | 1 + >> 1 files changed, 1 insertions(+), 0 deletions(-) > Reviewed-by: Stefan Hajnoczi<stefanha@linux.vnet.ibm.com> > Strange, I don't see this problem here. xtensa_lx60.c includes pc.h which includes fdc.h which includes blockdev.h, so this patch should not be needed. What went wrong in your build, or are your sources different from qemu git master? Regards, Stefan Weil ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [Qemu-devel] [PATCH] xtensa_lx60: add missing #include "blockdev.h" 2012-06-08 10:43 ` Stefan Weil @ 2012-06-08 10:57 ` Stefan Hajnoczi 2012-06-08 12:05 ` Stefan Weil 0 siblings, 1 reply; 7+ messages in thread From: Stefan Hajnoczi @ 2012-06-08 10:57 UTC (permalink / raw) To: Stefan Weil; +Cc: Max Filippov, qemu-devel On Fri, Jun 8, 2012 at 11:43 AM, Stefan Weil <sw@weilnetz.de> wrote: > Am 08.06.2012 12:08, schrieb Stefan Hajnoczi: > >> On Fri, Jun 8, 2012 at 10:50 AM, Max Filippov<jcmvbkbc@gmail.com> wrote: >>> >>> This should fix the following build failure: >>> >>> /home/buildbot/slave-public/block_mingw32/build/hw/xtensa_lx60.c: In >>> function 'lx_init': >>> /home/buildbot/slave-public/block_mingw32/build/hw/xtensa_lx60.c:212: >>> warning: implicit declaration of function 'drive_get' >>> /home/buildbot/slave-public/block_mingw32/build/hw/xtensa_lx60.c:212: >>> warning: nested extern declaration of 'drive_get' >>> /home/buildbot/slave-public/block_mingw32/build/hw/xtensa_lx60.c:212: >>> error: 'IF_PFLASH' undeclared (first use in this function) >>> /home/buildbot/slave-public/block_mingw32/build/hw/xtensa_lx60.c:212: >>> error: (Each undeclared identifier is reported only once >>> /home/buildbot/slave-public/block_mingw32/build/hw/xtensa_lx60.c:212: >>> error: for each function it appears in.) >>> /home/buildbot/slave-public/block_mingw32/build/hw/xtensa_lx60.c:216: >>> error: dereferencing pointer to incomplete type >>> >>> Signed-off-by: Max Filippov<jcmvbkbc@gmail.com> >>> --- >>> hw/xtensa_lx60.c | 1 + >>> 1 files changed, 1 insertions(+), 0 deletions(-) >> >> Reviewed-by: Stefan Hajnoczi<stefanha@linux.vnet.ibm.com> >> > > > Strange, I don't see this problem here. > > xtensa_lx60.c includes pc.h which includes fdc.h which includes blockdev.h, > so this patch should not be needed. Please check the buildbot output for more details on the failure: http://buildbot.b1-systems.de/qemu/builders/block_mingw32/builds/237 I still think this patch is an improvement because relying on pc.h -> fdc.h -> blockdev.h to get drive_get() and IF_PFLASH is very obscure and could break in the future if those headers change. Stefan ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [Qemu-devel] [PATCH] xtensa_lx60: add missing #include "blockdev.h" 2012-06-08 10:57 ` Stefan Hajnoczi @ 2012-06-08 12:05 ` Stefan Weil 2012-06-08 12:43 ` Stefan Hajnoczi 2012-06-08 13:01 ` Kevin Wolf 0 siblings, 2 replies; 7+ messages in thread From: Stefan Weil @ 2012-06-08 12:05 UTC (permalink / raw) To: Kevin Wolf; +Cc: Stefan Hajnoczi, qemu-devel, Max Filippov Am 08.06.2012 12:57, schrieb Stefan Hajnoczi: > On Fri, Jun 8, 2012 at 11:43 AM, Stefan Weil<sw@weilnetz.de> wrote: >> Am 08.06.2012 12:08, schrieb Stefan Hajnoczi: >> >>> On Fri, Jun 8, 2012 at 10:50 AM, Max Filippov<jcmvbkbc@gmail.com> wrote: >>>> >>>> This should fix the following build failure: >>>> >>>> /home/buildbot/slave-public/block_mingw32/build/hw/xtensa_lx60.c: In >>>> function 'lx_init': >>>> /home/buildbot/slave-public/block_mingw32/build/hw/xtensa_lx60.c:212: >>>> warning: implicit declaration of function 'drive_get' >>>> /home/buildbot/slave-public/block_mingw32/build/hw/xtensa_lx60.c:212: >>>> warning: nested extern declaration of 'drive_get' >>>> /home/buildbot/slave-public/block_mingw32/build/hw/xtensa_lx60.c:212: >>>> error: 'IF_PFLASH' undeclared (first use in this function) >>>> /home/buildbot/slave-public/block_mingw32/build/hw/xtensa_lx60.c:212: >>>> error: (Each undeclared identifier is reported only once >>>> /home/buildbot/slave-public/block_mingw32/build/hw/xtensa_lx60.c:212: >>>> error: for each function it appears in.) >>>> /home/buildbot/slave-public/block_mingw32/build/hw/xtensa_lx60.c:216: >>>> error: dereferencing pointer to incomplete type >>>> >>>> Signed-off-by: Max Filippov<jcmvbkbc@gmail.com> >>>> --- >>>> hw/xtensa_lx60.c | 1 + >>>> 1 files changed, 1 insertions(+), 0 deletions(-) >>> >>> Reviewed-by: Stefan Hajnoczi<stefanha@linux.vnet.ibm.com> >>> >> >> >> Strange, I don't see this problem here. >> >> xtensa_lx60.c includes pc.h which includes fdc.h which includes blockdev.h, >> so this patch should not be needed. > > Please check the buildbot output for more details on the failure: > http://buildbot.b1-systems.de/qemu/builders/block_mingw32/builds/237 The build failure for all hosts is introduced by commit d39e6563312a8ea3fb436cd2c7e6893cf416f34e in Kevin's block-next. Kevin, please add Max' patch before that commit. Stefan H. already reviewed the patch, and if you want, you can now also add Reviewed-by: Stefan Weil <sw@weilnetz.de>. QEMU git master still uses the old code where blockdev.h is included by hw/fdc.h, that's why I did not get that problem. Regards, Stefan W. ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [Qemu-devel] [PATCH] xtensa_lx60: add missing #include "blockdev.h" 2012-06-08 12:05 ` Stefan Weil @ 2012-06-08 12:43 ` Stefan Hajnoczi 2012-06-08 13:01 ` Kevin Wolf 1 sibling, 0 replies; 7+ messages in thread From: Stefan Hajnoczi @ 2012-06-08 12:43 UTC (permalink / raw) To: Stefan Weil; +Cc: Kevin Wolf, Max Filippov, qemu-devel On Fri, Jun 8, 2012 at 1:05 PM, Stefan Weil <sw@weilnetz.de> wrote: > Am 08.06.2012 12:57, schrieb Stefan Hajnoczi: > >> On Fri, Jun 8, 2012 at 11:43 AM, Stefan Weil<sw@weilnetz.de> wrote: >>> >>> Am 08.06.2012 12:08, schrieb Stefan Hajnoczi: >>> >>>> On Fri, Jun 8, 2012 at 10:50 AM, Max Filippov<jcmvbkbc@gmail.com> >>>> wrote: >>>>> >>>>> >>>>> This should fix the following build failure: >>>>> >>>>> /home/buildbot/slave-public/block_mingw32/build/hw/xtensa_lx60.c: In >>>>> function 'lx_init': >>>>> /home/buildbot/slave-public/block_mingw32/build/hw/xtensa_lx60.c:212: >>>>> warning: implicit declaration of function 'drive_get' >>>>> /home/buildbot/slave-public/block_mingw32/build/hw/xtensa_lx60.c:212: >>>>> warning: nested extern declaration of 'drive_get' >>>>> /home/buildbot/slave-public/block_mingw32/build/hw/xtensa_lx60.c:212: >>>>> error: 'IF_PFLASH' undeclared (first use in this function) >>>>> /home/buildbot/slave-public/block_mingw32/build/hw/xtensa_lx60.c:212: >>>>> error: (Each undeclared identifier is reported only once >>>>> /home/buildbot/slave-public/block_mingw32/build/hw/xtensa_lx60.c:212: >>>>> error: for each function it appears in.) >>>>> /home/buildbot/slave-public/block_mingw32/build/hw/xtensa_lx60.c:216: >>>>> error: dereferencing pointer to incomplete type >>>>> >>>>> Signed-off-by: Max Filippov<jcmvbkbc@gmail.com> >>>>> --- >>>>> hw/xtensa_lx60.c | 1 + >>>>> 1 files changed, 1 insertions(+), 0 deletions(-) >>>> >>>> >>>> Reviewed-by: Stefan Hajnoczi<stefanha@linux.vnet.ibm.com> >>>> >>> >>> >>> Strange, I don't see this problem here. >>> >>> xtensa_lx60.c includes pc.h which includes fdc.h which includes >>> blockdev.h, >>> so this patch should not be needed. >> >> >> Please check the buildbot output for more details on the failure: >> http://buildbot.b1-systems.de/qemu/builders/block_mingw32/builds/237 > > > The build failure for all hosts is introduced by commit > d39e6563312a8ea3fb436cd2c7e6893cf416f34e in Kevin's block-next. > > Kevin, please add Max' patch before that commit. > Stefan H. already reviewed the patch, and if you want, > you can now also add > > Reviewed-by: Stefan Weil <sw@weilnetz.de>. > > QEMU git master still uses the old code where blockdev.h is > included by hw/fdc.h, that's why I did not get that problem. Thanks for finding the cause. This makes sense now. Stefan ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [Qemu-devel] [PATCH] xtensa_lx60: add missing #include "blockdev.h" 2012-06-08 12:05 ` Stefan Weil 2012-06-08 12:43 ` Stefan Hajnoczi @ 2012-06-08 13:01 ` Kevin Wolf 1 sibling, 0 replies; 7+ messages in thread From: Kevin Wolf @ 2012-06-08 13:01 UTC (permalink / raw) To: Stefan Weil; +Cc: Stefan Hajnoczi, qemu-devel, Anthony Liguori, Max Filippov Am 08.06.2012 14:05, schrieb Stefan Weil: > Am 08.06.2012 12:57, schrieb Stefan Hajnoczi: >> On Fri, Jun 8, 2012 at 11:43 AM, Stefan Weil<sw@weilnetz.de> wrote: >>> Am 08.06.2012 12:08, schrieb Stefan Hajnoczi: >>> >>>> On Fri, Jun 8, 2012 at 10:50 AM, Max Filippov<jcmvbkbc@gmail.com> wrote: >>>>> >>>>> This should fix the following build failure: >>>>> >>>>> /home/buildbot/slave-public/block_mingw32/build/hw/xtensa_lx60.c: In >>>>> function 'lx_init': >>>>> /home/buildbot/slave-public/block_mingw32/build/hw/xtensa_lx60.c:212: >>>>> warning: implicit declaration of function 'drive_get' >>>>> /home/buildbot/slave-public/block_mingw32/build/hw/xtensa_lx60.c:212: >>>>> warning: nested extern declaration of 'drive_get' >>>>> /home/buildbot/slave-public/block_mingw32/build/hw/xtensa_lx60.c:212: >>>>> error: 'IF_PFLASH' undeclared (first use in this function) >>>>> /home/buildbot/slave-public/block_mingw32/build/hw/xtensa_lx60.c:212: >>>>> error: (Each undeclared identifier is reported only once >>>>> /home/buildbot/slave-public/block_mingw32/build/hw/xtensa_lx60.c:212: >>>>> error: for each function it appears in.) >>>>> /home/buildbot/slave-public/block_mingw32/build/hw/xtensa_lx60.c:216: >>>>> error: dereferencing pointer to incomplete type >>>>> >>>>> Signed-off-by: Max Filippov<jcmvbkbc@gmail.com> >>>>> --- >>>>> hw/xtensa_lx60.c | 1 + >>>>> 1 files changed, 1 insertions(+), 0 deletions(-) >>>> >>>> Reviewed-by: Stefan Hajnoczi<stefanha@linux.vnet.ibm.com> >>>> >>> >>> >>> Strange, I don't see this problem here. >>> >>> xtensa_lx60.c includes pc.h which includes fdc.h which includes blockdev.h, >>> so this patch should not be needed. >> >> Please check the buildbot output for more details on the failure: >> http://buildbot.b1-systems.de/qemu/builders/block_mingw32/builds/237 > > The build failure for all hosts is introduced by commit > d39e6563312a8ea3fb436cd2c7e6893cf416f34e in Kevin's block-next. > > Kevin, please add Max' patch before that commit. > Stefan H. already reviewed the patch, and if you want, > you can now also add > > Reviewed-by: Stefan Weil <sw@weilnetz.de>. > > QEMU git master still uses the old code where blockdev.h is > included by hw/fdc.h, that's why I did not get that problem. I already included Markus' patch in a pull request and Anthony said that he pulled, even though the commit doesn't appear in master yet. I'll take the patch into the block branch just in case, but I really think this should be handled by Anthony directly, who can possibly still save bisectability. Kevin ^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2012-06-08 13:01 UTC | newest] Thread overview: 7+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2012-06-08 9:50 [Qemu-devel] [PATCH] xtensa_lx60: add missing #include "blockdev.h" Max Filippov 2012-06-08 10:08 ` Stefan Hajnoczi 2012-06-08 10:43 ` Stefan Weil 2012-06-08 10:57 ` Stefan Hajnoczi 2012-06-08 12:05 ` Stefan Weil 2012-06-08 12:43 ` Stefan Hajnoczi 2012-06-08 13:01 ` Kevin Wolf
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).