Am 23.04.2012 22:13, schrieb Anthony Liguori:
On 04/20/2012 05:03 AM, Stefan Weil wrote:
These patches remove timer code which is no longer needed
and try to improve the remaining code.

[PATCH 1/5] qemu-timer: Remove redundant include statements
[PATCH 2/5] qemu-timer: Remove unused function qemu_alarm_pending
[PATCH 3/5] qemu-timer: Use bool, false, true for boolean values
[PATCH 4/5] qemu-timer: Remove function alarm_has_dynticks
[PATCH 5/5] qemu-timer: Optimize data structures



If you're goign to rev a patch, please resubmit the full series. Otherwise it's a pain in the ass to apply because the patches have to be reordered by hand. It's not so bad if it's the last patch in a series but when it's in the middle, it sucks.

Regards,

Anthony Liguori

The patches 4 and 5 don't depend on each other, so they can be reordered.
Then patch 4 (v2) would be the last patch.

To avoid further pain, it's now possible to pull this series.

Regards,

Stefan Weil


The following changes since commit 092dfc7714ad7983aeb0cada5d5983e9fde8d84c:

Merge remote-tracking branch 'origin/master' into staging (2012-04-23 15:02:53 -0500)

are available in the git repository at:


  git://qemu.weilnetz.de/qemu.git timer

for you to fetch changes up to 475d53efedeaf3444e78088939afd376e8843d09:

  qemu-timer: Optimize data structures (2012-04-23 22:37:12 +0200)

----------------------------------------------------------------
Stefan Weil (5):
      qemu-timer: Remove redundant include statements
      qemu-timer: Remove unused function qemu_alarm_pending
      qemu-timer: Use bool, false, true for boolean values
      qemu-timer: Remove function alarm_has_dynticks
      qemu-timer: Optimize data structures

qemu-timer.c | 94 ++++++++++++++++++---------------------------------------
 qemu-timer.h |   13 ++------
 2 files changed, 33 insertions(+), 74 deletions(-)


Reply via email to