[v2,0/4] Various convenience fixes

Message

Andre Przywara Feb. 1, 2019, 12:37 p.m. UTC

As I found myself using kvmtool more often in the last weeks, I got more
and more annoyed by some smaller "ticks" that kvmtool shows.

So this is an attempt to post various smaller fixes I gathered up over
the years, but never found worth enough to send out.
Version 2 drops the two patches that have been merged already, and
amends the remaining ones.

The patches address:
- add /chosen/stdout-path node in .dtb
- honour make -s switch
- remove pointless kvmtool version number
- introduce autodetection of supported GIC type

Changelog v1..v2:
- stdout-path: Let hw/serial.c set the node path and use an alias
- make -s: adapt to newest version from the Linux kernel
- kvmtool "version" number: unchanged, but verified to be compatible
  with the Debian package
- GIC autodetection: ignore --force-pci, always try to have an MSI
  controller

Please have a look!

Cheers,
Andre.

Andre Przywara (4):
  arm: fdt: add stdout-path to /chosen node
  Makefile: support -s switch
  Makefile: Remove echoing of kvmtools version file
  arm: Auto-detect guest GIC type

 Makefile                     |  6 +++++-
 arm/fdt.c                    | 10 ++++++++++
 arm/gic.c                    | 25 +++++++++++++++++++++++++
 arm/include/arm-common/gic.h |  1 +
 hw/serial.c                  |  9 +++++++++
 include/kvm/fdt.h            |  2 ++
 util/KVMTOOLS-VERSION-GEN    |  1 -
 7 files changed, 52 insertions(+), 2 deletions(-)

Comments

Will Deacon Feb. 8, 2019, 4:16 p.m. UTC | #1

On Fri, Feb 01, 2019 at 12:37:12PM +0000, Andre Przywara wrote:
> As I found myself using kvmtool more often in the last weeks, I got more
> and more annoyed by some smaller "ticks" that kvmtool shows.
> 
> So this is an attempt to post various smaller fixes I gathered up over
> the years, but never found worth enough to send out.
> Version 2 drops the two patches that have been merged already, and
> amends the remaining ones.
> 
> The patches address:
> - add /chosen/stdout-path node in .dtb
> - honour make -s switch
> - remove pointless kvmtool version number
> - introduce autodetection of supported GIC type
> 
> Changelog v1..v2:
> - stdout-path: Let hw/serial.c set the node path and use an alias
> - make -s: adapt to newest version from the Linux kernel
> - kvmtool "version" number: unchanged, but verified to be compatible
>   with the Debian package
> - GIC autodetection: ignore --force-pci, always try to have an MSI
>   controller
> 
> Please have a look!
> 
> Cheers,
> Andre.
> 
> Andre Przywara (4):
>   arm: fdt: add stdout-path to /chosen node
>   Makefile: support -s switch
>   Makefile: Remove echoing of kvmtools version file

I've applied these three, and left a comment on the other one.

Will