Re: [Xen-devel] [PATCH v7 00/15] Load BIOS via toolstack instead of been embedded in hvmloader.

[Boris Ostrovsky]

On 07/29/2016 10:50 AM, Wei Liu wrote:
>
>>> You need to run ./autogen.sh. Anthony didn't commit the changes to
>>> ./configure.
>>
>> Yes, that did the trick. Except for the machine on which I actually
>> wanted it to run:
>>
>> root@ovs104> ./autogen.sh
>> configure.ac:4: error: Autoconf version 2.67 or higher is required
>> configure.ac:4: the top level
>> autom4te: /usr/bin/m4 failed with exit status: 63
>> root@ovs104> autoconf --version
>> autoconf (GNU Autoconf) 2.63
>> Copyright (C) 2008 Free Software Foundation, Inc.
>> License GPLv2+: GNU GPL version 2 or later
>> 
>> This is free software: you are free to change and redistribute it.
>> There is NO WARRANTY, to the extent permitted by law.
>>
>> Written by David J. MacKenzie and Akim Demaille.
>> root@ovs104>
>>
>> Is 2.67 really a pre-req?
>>
> It is not written down. But I guess there is a reason for it.
>
> Do you need me or Anthony to generate that for you?

No, I am good --- I can build on a system with software from this decade.

-boris


___
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel

Re: [Xen-devel] [PATCH v7 00/15] Load BIOS via toolstack instead of been embedded in hvmloader.

[Wei Liu]

On Fri, Jul 29, 2016 at 10:36:02AM -0400, Boris Ostrovsky wrote:
> On 07/29/2016 04:29 AM, Wei Liu wrote:
> > On Fri, Jul 29, 2016 at 01:28:02AM -0400, Boris Ostrovsky wrote:
> >>
> >> On 07/28/2016 06:49 AM, Anthony PERARD wrote:
> >>> Hi all,
> >>>
> >>> Changes in V7:
> >>>   - There is one new patch at the end to fix the doc.
> >>>   - Patch 6 as been change.
> >>>   that's it.
> >>>
> >>>   There is just a few missing ackes:
> >>> 6 xen: Move the hvm_start_info C representation from libxc to 
> >>> public/xen.h
> >>> 8 hvmloader: Locate the BIOS blob
> >>> 9 hvmloader: Check modules whereabouts in perform_tests
> >>>15 docs/misc/hvmlite: Point to the canonical definition of 
> >>> hvm_start_info
> >>>
> >>> Thanks.
> >>>
> >>> A git tree can be found here:
> >>> git://xenbits.xen.org/people/aperard/xen-unstable.git
> >>> tag: hvmloader-with-separated-bios-v7
> >>
> >> I am unable to build this:
> >>
> >> libxl_paths.c: In function ‘libxl__seabios_path’:
> >> libxl_paths.c:40: error: ‘SEABIOS_PATH’ undeclared (first use in this
> >> function)
> >> libxl_paths.c:40: error: (Each undeclared identifier is reported only once
> >> libxl_paths.c:40: error: for each function it appears in.)
> >> libxl_paths.c: In function ‘libxl__ovmf_path’:
> >> libxl_paths.c:45: error: ‘OVMF_PATH’ undeclared (first use in this 
> >> function)
> >>
> >> IIUIC these two are supposed to be generated into tools/config.h but they
> >> were not for me. I haven't looked any further yet.
> >>
> >> -boris
> >>
> > You need to run ./autogen.sh. Anthony didn't commit the changes to
> > ./configure.
> 
> 
> Yes, that did the trick. Except for the machine on which I actually
> wanted it to run:
> 
> root@ovs104> ./autogen.sh
> configure.ac:4: error: Autoconf version 2.67 or higher is required
> configure.ac:4: the top level
> autom4te: /usr/bin/m4 failed with exit status: 63
> root@ovs104> autoconf --version
> autoconf (GNU Autoconf) 2.63
> Copyright (C) 2008 Free Software Foundation, Inc.
> License GPLv2+: GNU GPL version 2 or later
> 
> This is free software: you are free to change and redistribute it.
> There is NO WARRANTY, to the extent permitted by law.
> 
> Written by David J. MacKenzie and Akim Demaille.
> root@ovs104>
> 
> Is 2.67 really a pre-req?
> 

It is not written down. But I guess there is a reason for it.

Do you need me or Anthony to generate that for you?

Wei.

> -boris
> 

___
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel

Re: [Xen-devel] [PATCH v7 00/15] Load BIOS via toolstack instead of been embedded in hvmloader.

[Boris Ostrovsky]

On 07/29/2016 04:29 AM, Wei Liu wrote:
> On Fri, Jul 29, 2016 at 01:28:02AM -0400, Boris Ostrovsky wrote:
>>
>> On 07/28/2016 06:49 AM, Anthony PERARD wrote:
>>> Hi all,
>>>
>>> Changes in V7:
>>>   - There is one new patch at the end to fix the doc.
>>>   - Patch 6 as been change.
>>>   that's it.
>>>
>>>   There is just a few missing ackes:
>>> 6 xen: Move the hvm_start_info C representation from libxc to 
>>> public/xen.h
>>> 8 hvmloader: Locate the BIOS blob
>>> 9 hvmloader: Check modules whereabouts in perform_tests
>>>15 docs/misc/hvmlite: Point to the canonical definition of hvm_start_info
>>>
>>> Thanks.
>>>
>>> A git tree can be found here:
>>> git://xenbits.xen.org/people/aperard/xen-unstable.git
>>> tag: hvmloader-with-separated-bios-v7
>>
>> I am unable to build this:
>>
>> libxl_paths.c: In function ‘libxl__seabios_path’:
>> libxl_paths.c:40: error: ‘SEABIOS_PATH’ undeclared (first use in this
>> function)
>> libxl_paths.c:40: error: (Each undeclared identifier is reported only once
>> libxl_paths.c:40: error: for each function it appears in.)
>> libxl_paths.c: In function ‘libxl__ovmf_path’:
>> libxl_paths.c:45: error: ‘OVMF_PATH’ undeclared (first use in this function)
>>
>> IIUIC these two are supposed to be generated into tools/config.h but they
>> were not for me. I haven't looked any further yet.
>>
>> -boris
>>
> You need to run ./autogen.sh. Anthony didn't commit the changes to
> ./configure.


Yes, that did the trick. Except for the machine on which I actually
wanted it to run:

root@ovs104> ./autogen.sh
configure.ac:4: error: Autoconf version 2.67 or higher is required
configure.ac:4: the top level
autom4te: /usr/bin/m4 failed with exit status: 63
root@ovs104> autoconf --version
autoconf (GNU Autoconf) 2.63
Copyright (C) 2008 Free Software Foundation, Inc.
License GPLv2+: GNU GPL version 2 or later

This is free software: you are free to change and redistribute it.
There is NO WARRANTY, to the extent permitted by law.

Written by David J. MacKenzie and Akim Demaille.
root@ovs104>

Is 2.67 really a pre-req?

-boris


___
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel

Re: [Xen-devel] [PATCH v7 00/15] Load BIOS via toolstack instead of been embedded in hvmloader.

[Wei Liu]

On Fri, Jul 29, 2016 at 01:28:02AM -0400, Boris Ostrovsky wrote:
> 
> 
> On 07/28/2016 06:49 AM, Anthony PERARD wrote:
> >Hi all,
> >
> >Changes in V7:
> >   - There is one new patch at the end to fix the doc.
> >   - Patch 6 as been change.
> >   that's it.
> >
> >   There is just a few missing ackes:
> > 6 xen: Move the hvm_start_info C representation from libxc to 
> > public/xen.h
> > 8 hvmloader: Locate the BIOS blob
> > 9 hvmloader: Check modules whereabouts in perform_tests
> >15 docs/misc/hvmlite: Point to the canonical definition of hvm_start_info
> >
> >Thanks.
> >
> >A git tree can be found here:
> >git://xenbits.xen.org/people/aperard/xen-unstable.git
> >tag: hvmloader-with-separated-bios-v7
> 
> 
> I am unable to build this:
> 
> libxl_paths.c: In function ‘libxl__seabios_path’:
> libxl_paths.c:40: error: ‘SEABIOS_PATH’ undeclared (first use in this
> function)
> libxl_paths.c:40: error: (Each undeclared identifier is reported only once
> libxl_paths.c:40: error: for each function it appears in.)
> libxl_paths.c: In function ‘libxl__ovmf_path’:
> libxl_paths.c:45: error: ‘OVMF_PATH’ undeclared (first use in this function)
> 
> IIUIC these two are supposed to be generated into tools/config.h but they
> were not for me. I haven't looked any further yet.
> 
> -boris
> 

You need to run ./autogen.sh. Anthony didn't commit the changes to
./configure.

Wei.

> 

___
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel

Re: [Xen-devel] [PATCH v7 00/15] Load BIOS via toolstack instead of been embedded in hvmloader.

[Boris Ostrovsky]

On 07/28/2016 06:49 AM, Anthony PERARD wrote:

Hi all,

Changes in V7:
   - There is one new patch at the end to fix the doc.
   - Patch 6 as been change.
   that's it.

   There is just a few missing ackes:
 6 xen: Move the hvm_start_info C representation from libxc to public/xen.h
 8 hvmloader: Locate the BIOS blob
 9 hvmloader: Check modules whereabouts in perform_tests
15 docs/misc/hvmlite: Point to the canonical definition of hvm_start_info

Thanks.

A git tree can be found here:
git://xenbits.xen.org/people/aperard/xen-unstable.git
tag: hvmloader-with-separated-bios-v7



I am unable to build this:

libxl_paths.c: In function ‘libxl__seabios_path’:
libxl_paths.c:40: error: ‘SEABIOS_PATH’ undeclared (first use in this 
function)

libxl_paths.c:40: error: (Each undeclared identifier is reported only once
libxl_paths.c:40: error: for each function it appears in.)
libxl_paths.c: In function ‘libxl__ovmf_path’:
libxl_paths.c:45: error: ‘OVMF_PATH’ undeclared (first use in this function)

IIUIC these two are supposed to be generated into tools/config.h but 
they were not for me. I haven't looked any further yet.


-boris



___
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel

[Xen-devel] [PATCH v7 00/15] Load BIOS via toolstack instead of been embedded in hvmloader.

[Anthony PERARD]

Hi all,

Changes in V7:
  - There is one new patch at the end to fix the doc.
  - Patch 6 as been change.
  that's it.

  There is just a few missing ackes:
6 xen: Move the hvm_start_info C representation from libxc to public/xen.h
8 hvmloader: Locate the BIOS blob
9 hvmloader: Check modules whereabouts in perform_tests
   15 docs/misc/hvmlite: Point to the canonical definition of hvm_start_info

Thanks.

A git tree can be found here:
git://xenbits.xen.org/people/aperard/xen-unstable.git
tag: hvmloader-with-separated-bios-v7

Cc: Ian Jackson 
Cc: Wei Liu 
Cc: Andrew Cooper 
Cc: Jan Beulich 

Regards,

Anthony PERARD (15):
  libxc: Rework extra module initialisation
  libxc: Prepare a start info structure for hvmloader
  configure: #define SEABIOS_PATH and OVMF_PATH
  firmware/makefile: install BIOS blob ...
  libxl: Load guest BIOS from file
  xen: Move the hvm_start_info C representation to the public headers
  hvmloader: Grab the hvm_start_info pointer
  hvmloader: Locate the BIOS blob
  hvmloader: Check modules whereabouts in perform_tests
  hvmloader: Load SeaBIOS from hvm_start_info modules
  hvmloader: Load OVMF from modules
  hvmloader: bios->bios_load() now needs to be defined
  hvmloader: Always build-in SeaBIOS and OVMF loader
  configure: do not depend on SEABIOS_PATH or OVMF_PATH ...
  docs/misc/hvmlite: Point to the canonical definition of hvm_start_info

 docs/man/xl.cfg.pod.5.in |   9 ++
 docs/misc/hvmlite.markdown   |  20 +---
 tools/configure.ac   |  12 ++-
 tools/firmware/Makefile  |  10 +-
 tools/firmware/hvmloader/Makefile|  39 +--
 tools/firmware/hvmloader/config.h|   2 +-
 tools/firmware/hvmloader/hvmloader.c |  82 --
 tools/firmware/hvmloader/ovmf.c  |  34 +++---
 tools/firmware/hvmloader/rombios.c   |   3 +-
 tools/firmware/hvmloader/seabios.c   |  25 +++--
 tools/firmware/hvmloader/tests.c |  76 -
 tools/firmware/hvmloader/util.h  |   3 +
 tools/libxc/include/xc_dom.h |  34 +-
 tools/libxc/xc_dom_hvmloader.c   | 136 
 tools/libxc/xc_dom_x86.c | 153 +++
 tools/libxl/libxl.h  |   8 ++
 tools/libxl/libxl_dom.c  |  61 +++
 tools/libxl/libxl_internal.h |   2 +
 tools/libxl/libxl_paths.c|  10 ++
 tools/libxl/libxl_types.idl  |   1 +
 tools/libxl/xl_cmdimpl.c |  11 +-
 xen/include/public/arch-x86/hvm/start_info.h |  99 +
 xen/include/public/xen.h |  46 
 23 files changed, 558 insertions(+), 318 deletions(-)
 create mode 100644 xen/include/public/arch-x86/hvm/start_info.h

-- 
Anthony PERARD


___
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel