[Qemu-devel] [PATCH v3 0/7] fw_cfg_test refactor and add two test cases

Philippe Mathieu-Daudé posted 7 patches 4 years, 11 months ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20190520213700.12620-1-philmd@redhat.com
Test docker-clang@ubuntu passed
Test asan passed
Test checkpatch passed
Test docker-mingw@fedora passed
Maintainers: Gerd Hoffmann <kraxel@redhat.com>, Paolo Bonzini <pbonzini@redhat.com>, Thomas Huth <thuth@redhat.com>, "Philippe Mathieu-Daudé" <philmd@redhat.com>, Laszlo Ersek <lersek@redhat.com>, Laurent Vivier <lvivier@redhat.com>
hw/nvram/fw_cfg.c        |   4 +-
tests/fw_cfg-test.c      | 127 +++++++++++++++++++++++++++++++++++----
tests/libqos/fw_cfg.c    |  55 +++++++++++++++++
tests/libqos/fw_cfg.h    |   9 +++
tests/libqos/malloc-pc.c |   2 +-
5 files changed, 184 insertions(+), 13 deletions(-)
[Qemu-devel] [PATCH v3 0/7] fw_cfg_test refactor and add two test cases
Posted by Philippe Mathieu-Daudé 4 years, 11 months ago
This is the work from Li Qiang, I added few changes while preparing
the pull request. If Li is OK with it, I'll send as it.

Since v2:
- split 1st patch
- clarified commit description based on review comments on list

Since v1:
- Add a patch to store the reboot_timeout as little endian

v2: https://lists.gnu.org/archive/html/qemu-devel/2019-04/msg04064.html
v1: https://lists.gnu.org/archive/html/qemu-devel/2019-03/msg05679.html

Li Qiang (5):
  tests: refactor fw_cfg_test
  tests: fw_cfg: add a function to get the fw_cfg file
  hw/nvram/fw_cfg: Store 'reboot-timeout' as little endian
  tests: fw_cfg: add 'reboot-timeout' test case
  tests: fw_cfg: add 'splash-time' test case

Philippe Mathieu-Daudé (2):
  tests/libqos: Add io_fw_cfg_uninit() and mm_fw_cfg_uninit()
  tests/libqos: Add pc_fw_cfg_uninit() and use it

 hw/nvram/fw_cfg.c        |   4 +-
 tests/fw_cfg-test.c      | 127 +++++++++++++++++++++++++++++++++++----
 tests/libqos/fw_cfg.c    |  55 +++++++++++++++++
 tests/libqos/fw_cfg.h    |   9 +++
 tests/libqos/malloc-pc.c |   2 +-
 5 files changed, 184 insertions(+), 13 deletions(-)

-- 
2.20.1


Re: [Qemu-devel] [PATCH v3 0/7] fw_cfg_test refactor and add two test cases
Posted by Philippe Mathieu-Daudé 4 years, 11 months ago
On 5/20/19 11:36 PM, Philippe Mathieu-Daudé wrote:
> This is the work from Li Qiang, I added few changes while preparing
> the pull request. If Li is OK with it, I'll send as it.
> 
> Since v2:
> - split 1st patch
> - clarified commit description based on review comments on list
> 
> Since v1:
> - Add a patch to store the reboot_timeout as little endian
> 
> v2: https://lists.gnu.org/archive/html/qemu-devel/2019-04/msg04064.html
> v1: https://lists.gnu.org/archive/html/qemu-devel/2019-03/msg05679.html
> 
> Li Qiang (5):
>   tests: refactor fw_cfg_test
>   tests: fw_cfg: add a function to get the fw_cfg file
>   hw/nvram/fw_cfg: Store 'reboot-timeout' as little endian
>   tests: fw_cfg: add 'reboot-timeout' test case
>   tests: fw_cfg: add 'splash-time' test case
> 
> Philippe Mathieu-Daudé (2):
>   tests/libqos: Add io_fw_cfg_uninit() and mm_fw_cfg_uninit()
>   tests/libqos: Add pc_fw_cfg_uninit() and use it
> 
>  hw/nvram/fw_cfg.c        |   4 +-
>  tests/fw_cfg-test.c      | 127 +++++++++++++++++++++++++++++++++++----
>  tests/libqos/fw_cfg.c    |  55 +++++++++++++++++
>  tests/libqos/fw_cfg.h    |   9 +++
>  tests/libqos/malloc-pc.c |   2 +-
>  5 files changed, 184 insertions(+), 13 deletions(-)
> 

Series queued (with Laszlo's comment on patch #2 addressed).