[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 0/5] qemu-timer: Clean code
From: |
Blue Swirl |
Subject: |
Re: [Qemu-devel] [PATCH 0/5] qemu-timer: Clean code |
Date: |
Sat, 28 Apr 2012 11:40:55 +0000 |
On Mon, Apr 23, 2012 at 20:39, Stefan Weil <address@hidden> wrote:
> 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.
Thanks, pulled.
>
> 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(-)
>
>
- [Qemu-devel] [PATCH 4/5] qemu-timer: Remove function alarm_has_dynticks, (continued)
- [Qemu-devel] [PATCH 4/5] qemu-timer: Remove function alarm_has_dynticks, Stefan Weil, 2012/04/20
- [Qemu-devel] [PATCH 2/5] qemu-timer: Remove unused function qemu_alarm_pending, Stefan Weil, 2012/04/20
- [Qemu-devel] [PATCH 5/5] qemu-timer: Optimize data structures, Stefan Weil, 2012/04/20
- [Qemu-devel] [PATCH 3/5] qemu-timer: Use bool, false, true for boolean values, Stefan Weil, 2012/04/20
- [Qemu-devel] [PATCH 1/5] qemu-timer: Remove redundant include statements, Stefan Weil, 2012/04/20
- Re: [Qemu-devel] [PATCH 0/5] qemu-timer: Clean code, Paolo Bonzini, 2012/04/20
- Re: [Qemu-devel] [PATCH 0/5] qemu-timer: Clean code, Anthony Liguori, 2012/04/23