diff options
| author | Vincent Palatin <vpalatin@chromium.org> | 2011-03-10 15:47:46 -0500 |
|---|---|---|
| committer | Anthony Liguori <aliguori@us.ibm.com> | 2011-03-10 16:12:21 -0600 |
| commit | 7d82af38b78305155553013c6fd709dc50404199 (patch) | |
| tree | 3c221451fa30e4dc65ad26c49665683ad07d51c0 /ui | |
| parent | d48751ed4f8368d3fdca99fbcd241d9efeedccbc (diff) | |
| download | qemu-7d82af38b78305155553013c6fd709dc50404199.tar.gz | |
Fix performance regression in qemu_get_ram_ptr
When the commit f471a17e9d869df3c6573f7ec02c4725676d6f3a converted the
ram_blocks structure to QLIST, it also removed the conditional check before
switching the current block at the beginning of the list.
In the common use case where ram_blocks has a few blocks with only one
frequently accessed (the main RAM), this has a performance impact as it
performs the useless list operations on each call (which are on a really
hot path).
On my machine emulation (ARM on amd64), this patch reduces the
percentage of CPU time spent in qemu_get_ram_ptr from 6.3% to 2.1% in the
profiling of a full boot.
Signed-off-by: Vincent Palatin <vpalatin@chromium.org>
Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>
Diffstat (limited to 'ui')
0 files changed, 0 insertions, 0 deletions