Re: [Qemu-devel] [PATCH 00/59] RAMState + qdev
no-re...@patchew.org wrote: > Hi, > ERROR: do not initialise globals to 0 or NULL > #24: FILE: hw/core/qdev.c:42: > +bool qdev_hotplug = false; > > total: 1 errors, 0 warnings, 24 lines checked I am just removing one static here. This is maintainer preference. there are three bools together, but I don't feel like changing this. How are we supposed to fix this kind of errors? Later, Juan.
Re: [Qemu-devel] [PATCH 00/59] RAMState + qdev
Hi,
This series seems to have some coding style problems. See output below for
more information:
Type: series
Message-id: 20170419205923.8808-1-quint...@redhat.com
Subject: [Qemu-devel] [PATCH 00/59] RAMState + qdev
=== TEST SCRIPT BEGIN ===
#!/bin/bash
BASE=base
n=1
total=$(git log --oneline $BASE.. | wc -l)
failed=0
# Useful git options
git config --local diff.renamelimit 0
git config --local diff.renames True
commits="$(git log --format=%H --reverse $BASE..)"
for c in $commits; do
echo "Checking PATCH $n/$total: $(git log -n 1 --format=%s $c)..."
if ! git show $c --format=email | ./scripts/checkpatch.pl --mailback -; then
failed=1
echo
fi
n=$((n+1))
done
exit $failed
=== TEST SCRIPT END ===
Updating 3c8cf5a9c21ff8782164d1def7f44bd888713384
From https://github.com/patchew-project/qemu
* [new tag] patchew/20170419205923.8808-1-quint...@redhat.com ->
patchew/20170419205923.8808-1-quint...@redhat.com
Switched to a new branch 'test'
6d30711 ram: Remove migration_bitmap_extend()
123c47a migration: Disable hotplug/unplug during migration
df49a84 qdev: Move qdev_unplug() to qdev-monitor.c
9c87aac qdev: Export qdev_hot_removed
8d1cce2 qdev: qdev_hotplug is really a bool
ea0729a migration: Remove MigrationState parameter from migration_is_idle()
e13e7c0 ram: Use RAMBitmap type for coherence
3178a8b ram: rename last_ram_offset() last_ram_pages()
0954f82 ram: Use ramblock and page offset instead of absolute offset
f097dac ram: Change offset field in PageSearchStatus to page
5cd2c21 ram: Remember last_page instead of last_offset
bd26e98 ram: Use page number instead of an address for the bitmap operations
32451bd ram: reorganize last_sent_block
7f5e04a ram: ram_discard_range() don't use the mis parameter
801f4ba ram: Pass RAMBlock to bitmap_sync
22fd5ab ram: Add page-size to output in 'info migrate'
db3f4d1 ram: Rename qemu_target_page_bits() to qemu_target_page_size()
7bf9556 ram: We don't need MigrationState parameter anymore
700c985 migration: Remove MigrationState from migration_in_postcopy
f029ef9 ram: Remove compression_switch and inline its logic
c646ad4 ram: Move QEMUFile into RAMState
939622e ram: Add QEMUFile to RAMState
6f20a3d ram: Move postcopy_requests into RAMState
048ab66 ram: Move dirty_pages_rate to RAMState
b0d70d8 ram: Remove dirty_bytes_rate
3f41071 ram: Create ram_dirty_sync_count()
377a2ed ram: Move src_page_req* to RAMState
89103e7 ram: Move last_req_rb to RAMState
7801c98 ram: Remove ram_save_remaining
8043ead ram: Use the RAMState bytes_transferred parameter
09ee03e ram: Move bytes_transferred into RAMState
6a9cf97 ram: Move migration_bitmap_rcu into RAMState
ea114cd ram: Move migration_bitmap_mutex into RAMState
3d75400 ram: Everything was init to zero, so use memset
3e21ead ram: Move migration_dirty_pages to RAMState
cda540b ram: Move xbzrle_overflows into RAMState
dc059c2 ram: Move xbzrle_cache_miss_rate into RAMState
293ce6f ram: Move xbzrle_cache_miss into RAMState
6502c29 ram: Move xbzrle_pages into RAMState
e4f0f58 ram: Move xbzrle_bytes into RAMState
98fe82c ram: Move iterations into RAMState
34be799 ram: Remove norm_mig_bytes_transferred
8e345bc ram: Move norm_pages to RAMState
a4ffc19 ram: Remove unused pages_skipped variable
8953f8b ram: Remove unused dup_mig_bytes_transferred()
280281f ram: Move dup_pages into RAMState
b3ccc2e ram: Move iterations_prev into RAMState
bdf3604 ram: Move xbzrle_cache_miss_prev into RAMState
1d3322b ram: Change num_dirty_pages_period type to uint64_t
a6254ab ram: Move num_dirty_pages_period into RAMState
aacfe07 ram: Change byte_xfer_{prev, now} type to uint64_t
49f2369 ram: Move bytes_xfer_prev into RAMState
9bb96a0 ram: Move start time into RAMState
e0c33c4 ram: Move bitmap_sync_count into RAMState
ffb4fda ram: Add dirty_rate_high_cnt to RAMState
e2435c3 ram: Create RAMState
8392453 ram: Rename block_name to rbname
f68407c ram: Rename flush_page_queue() to migration_page_queue_free()
c12f2ff ram: Update all functions comments
=== OUTPUT BEGIN ===
Checking PATCH 1/59: ram: Update all functions comments...
Checking PATCH 2/59: ram: Rename flush_page_queue() to
migration_page_queue_free()...
Checking PATCH 3/59: ram: Rename block_name to rbname...
Checking PATCH 4/59: ram: Create RAMState...
Checking PATCH 5/59: ram: Add dirty_rate_high_cnt to RAMState...
Checking PATCH 6/59: ram: Move bitmap_sync_count into RAMState...
Checking PATCH 7/59: ram: Move start time into RAMState...
Checking PATCH 8/59: ram: Move bytes_xfer_prev into RAMState...
Checking PATCH 9/59: ram: Change byte_xfer_{prev, now} type to uint64_t...
Checking PATCH 10/59: ram: Move num_dirty_pages_period into RAMState...
Checking PATCH 11/59: ram: Change num_dirty_pages_period type to uint64_t...
Checking PATCH 12/59: ram: Move xbzrle_cache_miss_prev into RAMState...
Checking PATCH 13/59: ram: Move iterations_prev into RAMState...
Checking PATCH 14/59: ram: Move dup_pages into RAMState...
Checking PATCH 15/59: ram: Remove unused dup_mig_
