[PATCH v7 0/2] memory: Have 'info mtree' remove duplicated Address Space information

Philippe Mathieu-Daudé posted 2 patches 2 years, 6 months ago
Test checkpatch passed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20210904231101.1071929-1-philmd@redhat.com
softmmu/memory.c | 150 ++++++++++++++++++++++++++++++++++-------------
1 file changed, 108 insertions(+), 42 deletions(-)
[PATCH v7 0/2] memory: Have 'info mtree' remove duplicated Address Space information
Posted by Philippe Mathieu-Daudé 2 years, 6 months ago
Series fully reviewed.

Follow Peter Maydell suggestions:
- Split mtree_info() as mtree_info_flatview() + mtree_info_as()
- Remove duplicated Address Space information

Since v6:
- Added missing vertical whitespace (rth)
- Added rth's R-b

Since v5:
- Fixed messed up during v3->v4 (peterx)
  . Restore format
  . Remove unused 'int counter'

Since v4:
- Merged patches 1 & 2 (David)
- Remove unnecessary return void (David)
- Added David R-b

Since v3:
- Fix typos
- Split mtree_info_flatview() + mtree_info_as() first
- Rebased last patch keeping Peter's R-b tag

Since v2:
- Removed unused AddressSpaceInfo::counter

Since v1:
- List AS similarly to 'info mtree -f' (peterx)

checkpatch warning (81 chars):

  WARNING: line over 80 characters
  #86: FILE: softmmu/memory.c:3359:
  +                                  address_space_compare_name);

Philippe Mathieu-Daudé (2):
  memory: Split mtree_info() as mtree_info_flatview() + mtree_info_as()
  memory: Have 'info mtree' remove duplicated Address Space information

 softmmu/memory.c | 150 ++++++++++++++++++++++++++++++++++-------------
 1 file changed, 108 insertions(+), 42 deletions(-)

-- 
2.31.1


Re: [PATCH v7 0/2] memory: Have 'info mtree' remove duplicated Address Space information
Posted by Philippe Mathieu-Daudé 2 years, 4 months ago
Ping?

On 9/5/21 01:10, Philippe Mathieu-Daudé wrote:
> Series fully reviewed.
> 
> Follow Peter Maydell suggestions:
> - Split mtree_info() as mtree_info_flatview() + mtree_info_as()
> - Remove duplicated Address Space information
> 
> Since v6:
> - Added missing vertical whitespace (rth)
> - Added rth's R-b
> 
> Since v5:
> - Fixed messed up during v3->v4 (peterx)
>   . Restore format
>   . Remove unused 'int counter'
> 
> Since v4:
> - Merged patches 1 & 2 (David)
> - Remove unnecessary return void (David)
> - Added David R-b
> 
> Since v3:
> - Fix typos
> - Split mtree_info_flatview() + mtree_info_as() first
> - Rebased last patch keeping Peter's R-b tag
> 
> Since v2:
> - Removed unused AddressSpaceInfo::counter
> 
> Since v1:
> - List AS similarly to 'info mtree -f' (peterx)
> 
> checkpatch warning (81 chars):
> 
>   WARNING: line over 80 characters
>   #86: FILE: softmmu/memory.c:3359:
>   +                                  address_space_compare_name);
> 
> Philippe Mathieu-Daudé (2):
>   memory: Split mtree_info() as mtree_info_flatview() + mtree_info_as()
>   memory: Have 'info mtree' remove duplicated Address Space information
> 
>  softmmu/memory.c | 150 ++++++++++++++++++++++++++++++++++-------------
>  1 file changed, 108 insertions(+), 42 deletions(-)
> 


Re: [PATCH v7 0/2] memory: Have 'info mtree' remove duplicated Address Space information
Posted by Philippe Mathieu-Daudé via 2 years, 2 months ago
On 9/5/21 01:10, Philippe Mathieu-Daudé wrote:

> Philippe Mathieu-Daudé (2):
>   memory: Split mtree_info() as mtree_info_flatview() + mtree_info_as()
>   memory: Have 'info mtree' remove duplicated Address Space information

Queued via memory-api.