* seabios regression in staging
@ 2018-03-29 11:42 Olaf Hering
2018-03-29 14:29 ` Wei Liu
0 siblings, 1 reply; 9+ messages in thread
From: Olaf Hering @ 2018-03-29 11:42 UTC (permalink / raw)
To: xen-devel
[-- Attachment #1.1: Type: text/plain, Size: 916 bytes --]
It seems the latest seabios that was pulled into staging recently fails
to compile, at least in SLE_11:
[ 86s] Compile checking out/src/hw/blockcmd.o
[ 86s] src/hw/blockcmd.c: In function 'scsi_rep_luns_scan':
[ 86s] src/hw/blockcmd.c:229: error: unknown field 'cdbcmd' specified in initializer
[ 86s] src/hw/blockcmd.c:229: warning: missing braces around initializer
[ 86s] src/hw/blockcmd.c:229: warning: (near initialization for 'op.<anonymous>')
[ 86s] src/hw/blockcmd.c:229: warning: initialization makes integer from pointer without a cast
[ 86s] make[7]: *** [out/src/hw/blockcmd.o] Error 1
I can eventually trick the build to use gcc48 not only for xen but also
for tools, but I wonder if there is a way to fix this.
Upstream may have not fixed that yet:
https://github.com/coreboot/seabios/blame/master/src/block.h
https://github.com/coreboot/seabios/blame/master/src/hw/blockcmd.c
Olaf
[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 195 bytes --]
[-- Attachment #2: Type: text/plain, Size: 157 bytes --]
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xenproject.org
https://lists.xenproject.org/mailman/listinfo/xen-devel
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: seabios regression in staging
2018-03-29 11:42 seabios regression in staging Olaf Hering
@ 2018-03-29 14:29 ` Wei Liu
2018-03-29 14:53 ` Olaf Hering
2018-03-29 15:36 ` Olaf Hering
0 siblings, 2 replies; 9+ messages in thread
From: Wei Liu @ 2018-03-29 14:29 UTC (permalink / raw)
To: Olaf Hering; +Cc: Wei Liu, xen-devel
On Thu, Mar 29, 2018 at 01:42:43PM +0200, Olaf Hering wrote:
> It seems the latest seabios that was pulled into staging recently fails
> to compile, at least in SLE_11:
>
> [ 86s] Compile checking out/src/hw/blockcmd.o
> [ 86s] src/hw/blockcmd.c: In function 'scsi_rep_luns_scan':
> [ 86s] src/hw/blockcmd.c:229: error: unknown field 'cdbcmd' specified in initializer
> [ 86s] src/hw/blockcmd.c:229: warning: missing braces around initializer
> [ 86s] src/hw/blockcmd.c:229: warning: (near initialization for 'op.<anonymous>')
> [ 86s] src/hw/blockcmd.c:229: warning: initialization makes integer from pointer without a cast
> [ 86s] make[7]: *** [out/src/hw/blockcmd.o] Error 1
>
> I can eventually trick the build to use gcc48 not only for xen but also
> for tools, but I wonder if there is a way to fix this.
>
> Upstream may have not fixed that yet:
> https://github.com/coreboot/seabios/blame/master/src/block.h
> https://github.com/coreboot/seabios/blame/master/src/hw/blockcmd.c
>
I think this is a problem with seabios upstream. We should ask them to
fix it and do another release.
Do you use a non-default seabios configuration? Osstest seems to be
happy with the update.
Wei.
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xenproject.org
https://lists.xenproject.org/mailman/listinfo/xen-devel
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: seabios regression in staging
2018-03-29 14:29 ` Wei Liu
@ 2018-03-29 14:53 ` Olaf Hering
2018-03-29 15:04 ` Wei Liu
2018-03-29 17:56 ` Doug Goldstein
2018-03-29 15:36 ` Olaf Hering
1 sibling, 2 replies; 9+ messages in thread
From: Olaf Hering @ 2018-03-29 14:53 UTC (permalink / raw)
To: Wei Liu; +Cc: xen-devel
[-- Attachment #1.1: Type: text/plain, Size: 372 bytes --]
On Thu, Mar 29, Wei Liu wrote:
> Do you use a non-default seabios configuration? Osstest seems to be
> happy with the update.
Not sure how I would create a non-default seabios or toolchain build.
osstest does not use SLE11, so it can not possibly spot such compile
errors. It would certainly be cool if there would be a test with very
old and very new toolchain.
Olaf
[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 195 bytes --]
[-- Attachment #2: Type: text/plain, Size: 157 bytes --]
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xenproject.org
https://lists.xenproject.org/mailman/listinfo/xen-devel
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: seabios regression in staging
2018-03-29 14:53 ` Olaf Hering
@ 2018-03-29 15:04 ` Wei Liu
2018-03-29 17:56 ` Doug Goldstein
1 sibling, 0 replies; 9+ messages in thread
From: Wei Liu @ 2018-03-29 15:04 UTC (permalink / raw)
To: Olaf Hering; +Cc: Wei Liu, xen-devel
On Thu, Mar 29, 2018 at 04:53:41PM +0200, Olaf Hering wrote:
> On Thu, Mar 29, Wei Liu wrote:
>
> > Do you use a non-default seabios configuration? Osstest seems to be
> > happy with the update.
>
> Not sure how I would create a non-default seabios or toolchain build.
>
> osstest does not use SLE11, so it can not possibly spot such compile
> errors. It would certainly be cool if there would be a test with very
> old and very new toolchain.
>
My reading of the log was that the code was actually wrong. But looking
at the code itself it should be a problem with older toolchain.
There is work on-going to use docker to test build Xen. See
.gitlab-ci.yml in xen.git. I think it would be helpful to add SLES 11 to
the list, too.
Wei.
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xenproject.org
https://lists.xenproject.org/mailman/listinfo/xen-devel
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: seabios regression in staging
2018-03-29 14:29 ` Wei Liu
2018-03-29 14:53 ` Olaf Hering
@ 2018-03-29 15:36 ` Olaf Hering
2018-03-29 15:40 ` Wei Liu
1 sibling, 1 reply; 9+ messages in thread
From: Olaf Hering @ 2018-03-29 15:36 UTC (permalink / raw)
To: Wei Liu; +Cc: xen-devel
[-- Attachment #1.1: Type: text/plain, Size: 242 bytes --]
On Thu, Mar 29, Wei Liu wrote:
> I think this is a problem with seabios upstream. We should ask them to
> fix it and do another release.
https://mail.coreboot.org/pipermail/seabios/2017-November/011932.html
gcc-4.6+ is now required.
Olaf
[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 195 bytes --]
[-- Attachment #2: Type: text/plain, Size: 157 bytes --]
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xenproject.org
https://lists.xenproject.org/mailman/listinfo/xen-devel
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: seabios regression in staging
2018-03-29 15:36 ` Olaf Hering
@ 2018-03-29 15:40 ` Wei Liu
0 siblings, 0 replies; 9+ messages in thread
From: Wei Liu @ 2018-03-29 15:40 UTC (permalink / raw)
To: Olaf Hering; +Cc: Wei Liu, xen-devel
On Thu, Mar 29, 2018 at 05:36:32PM +0200, Olaf Hering wrote:
> On Thu, Mar 29, Wei Liu wrote:
>
> > I think this is a problem with seabios upstream. We should ask them to
> > fix it and do another release.
>
> https://mail.coreboot.org/pipermail/seabios/2017-November/011932.html
>
> gcc-4.6+ is now required.
>
Yeah. I saw that.
We should either bump our requirement or have a separate entry for
seabios. I prefer the former.
Wei.
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xenproject.org
https://lists.xenproject.org/mailman/listinfo/xen-devel
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: seabios regression in staging
2018-03-29 14:53 ` Olaf Hering
2018-03-29 15:04 ` Wei Liu
@ 2018-03-29 17:56 ` Doug Goldstein
2018-03-29 20:00 ` Olaf Hering
1 sibling, 1 reply; 9+ messages in thread
From: Doug Goldstein @ 2018-03-29 17:56 UTC (permalink / raw)
To: Olaf Hering, Wei Liu; +Cc: xen-devel
[-- Attachment #1.1.1: Type: text/plain, Size: 594 bytes --]
On 3/29/18 9:53 AM, Olaf Hering wrote:
> On Thu, Mar 29, Wei Liu wrote:
>
>> Do you use a non-default seabios configuration? Osstest seems to be
>> happy with the update.
>
> Not sure how I would create a non-default seabios or toolchain build.
>
> osstest does not use SLE11, so it can not possibly spot such compile
> errors. It would certainly be cool if there would be a test with very
> old and very new toolchain.
>
> Olaf
If you'd be willing to create a SLE11 Docker container, we can add that
to the tree and start doing builds against it.
--
Doug Goldstein
[-- Attachment #1.2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 963 bytes --]
[-- Attachment #2: Type: text/plain, Size: 157 bytes --]
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xenproject.org
https://lists.xenproject.org/mailman/listinfo/xen-devel
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: seabios regression in staging
2018-03-29 17:56 ` Doug Goldstein
@ 2018-03-29 20:00 ` Olaf Hering
2018-03-31 1:40 ` Doug Goldstein
0 siblings, 1 reply; 9+ messages in thread
From: Olaf Hering @ 2018-03-29 20:00 UTC (permalink / raw)
To: Doug Goldstein; +Cc: Wei Liu, xen-devel
[-- Attachment #1.1: Type: text/plain, Size: 213 bytes --]
On Thu, Mar 29, Doug Goldstein wrote:
> If you'd be willing to create a SLE11 Docker container, we can add that
> to the tree and start doing builds against it.
I do not know how to do that. Any pointers?
Olaf
[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 195 bytes --]
[-- Attachment #2: Type: text/plain, Size: 157 bytes --]
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xenproject.org
https://lists.xenproject.org/mailman/listinfo/xen-devel
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: seabios regression in staging
2018-03-29 20:00 ` Olaf Hering
@ 2018-03-31 1:40 ` Doug Goldstein
0 siblings, 0 replies; 9+ messages in thread
From: Doug Goldstein @ 2018-03-31 1:40 UTC (permalink / raw)
To: Olaf Hering; +Cc: Wei Liu, xen-devel
[-- Attachment #1.1.1: Type: text/plain, Size: 637 bytes --]
On 3/29/18 3:00 PM, Olaf Hering wrote:
> On Thu, Mar 29, Doug Goldstein wrote:
>
>> If you'd be willing to create a SLE11 Docker container, we can add that
>> to the tree and start doing builds against it.
>
> I do not know how to do that. Any pointers?
>
> Olaf
>
Find the base image for SLE11. I've for the openSUSE ones at
https://hub.docker.com/r/library/opensuse/ so using those you would get
the line:
FROM opensuse:42.3
Then take one of the existing ones in the tree under automation/build
and change the FROM line and then update the commands that are
installing dependencies.
--
Doug Goldstein
[-- Attachment #1.2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 963 bytes --]
[-- Attachment #2: Type: text/plain, Size: 157 bytes --]
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xenproject.org
https://lists.xenproject.org/mailman/listinfo/xen-devel
^ permalink raw reply [flat|nested] 9+ messages in thread
end of thread, other threads:[~2018-03-31 1:40 UTC | newest]
Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-03-29 11:42 seabios regression in staging Olaf Hering
2018-03-29 14:29 ` Wei Liu
2018-03-29 14:53 ` Olaf Hering
2018-03-29 15:04 ` Wei Liu
2018-03-29 17:56 ` Doug Goldstein
2018-03-29 20:00 ` Olaf Hering
2018-03-31 1:40 ` Doug Goldstein
2018-03-29 15:36 ` Olaf Hering
2018-03-29 15:40 ` Wei Liu
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.