[v2] Remove res_compatible parameter

[PATCH v2 0/3] Remove res_compatible parameter

Juan Quintela posted 3 patches 1 year, 2 months ago

Diff against v1
Download series mbox

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20230215170345.6220-1-quintela@redhat.com

Maintainers: Richard Henderson <richard.henderson@linaro.org>, David Hildenbrand <david@redhat.com>, Ilya Leoshkevich <iii@linux.ibm.com>, Halil Pasic <pasic@linux.ibm.com>, Christian Borntraeger <borntraeger@linux.ibm.com>, Eric Farman <farman@linux.ibm.com>, Thomas Huth <thuth@redhat.com>, Alex Williamson <alex.williamson@redhat.com>, Juan Quintela <quintela@redhat.com>, "Dr. David Alan Gilbert" <dgilbert@redhat.com>, Stefan Hajnoczi <stefanha@redhat.com>, Fam Zheng <fam@euphon.net>, Eric Blake <eblake@redhat.com>, Vladimir Sementsov-Ogievskiy <vsementsov@yandex-team.ru>, John Snow <jsnow@redhat.com>

include/migration/register.h   | 30 +++++++++++++++---------------
migration/savevm.h             | 10 ++++------
hw/s390x/s390-stattrib.c       |  8 +++-----
hw/vfio/migration.c            | 11 ++++-------
migration/block-dirty-bitmap.c |  7 +++----
migration/block.c              |  8 +++-----
migration/migration.c          | 20 +++++++++-----------
migration/ram.c                | 20 ++++++++------------
migration/savevm.c             | 28 ++++++++++------------------
hw/vfio/trace-events           |  2 +-
migration/trace-events         |  4 ++--
11 files changed, 62 insertions(+), 86 deletions(-)

Expand all Fold all

[PATCH v2 0/3] Remove res_compatible parameter

Posted by Juan Quintela 1 year, 2 months ago

Hi

In this v2:
- rename res_postocopy_only ->can_postcopy
- rename res_precopy_only -> must_precopy
- Rewrite comments after discussions with Vladimir.
- 2nd patch is reviewd-by Vladimir.

Please, review.

[v1]
This series are the redo of the series from vfio.  Vladimir
Sementsov-Ogievskiy <vsementsov@yandex-team.ru> asked that I split the
change in ram.c (only place that set res_compatible), and the rest of
the patches.
So I ended:
- change ram.c to put the memory in res_postcopy
- remove res_compatible
- rename res_postcopy/precopy_only to not have the _only suffix.

Please review.

Juan Quintela (3):
  migration: In case of postcopy, the memory ends in res_postcopy_only
  migration: Remove unused res_compatible
  migration: Rename res_{postcopy,precopy}_only

 include/migration/register.h   | 30 +++++++++++++++---------------
 migration/savevm.h             | 10 ++++------
 hw/s390x/s390-stattrib.c       |  8 +++-----
 hw/vfio/migration.c            | 11 ++++-------
 migration/block-dirty-bitmap.c |  7 +++----
 migration/block.c              |  8 +++-----
 migration/migration.c          | 20 +++++++++-----------
 migration/ram.c                | 20 ++++++++------------
 migration/savevm.c             | 28 ++++++++++------------------
 hw/vfio/trace-events           |  2 +-
 migration/trace-events         |  4 ++--
 11 files changed, 62 insertions(+), 86 deletions(-)

-- 
2.39.1