[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-trivial] [PATCH 0/8] qemu-queue cleanups
From: |
Stefan Hajnoczi |
Subject: |
Re: [Qemu-trivial] [PATCH 0/8] qemu-queue cleanups |
Date: |
Tue, 3 Jan 2012 12:07:32 +0000 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
On Mon, Jan 02, 2012 at 07:00:29PM +0100, Paolo Bonzini wrote:
> These patches simplify the jungle of lists provided by qemu-queue from
> 4 to 3. QCIRCLEQ is dropped, since it provides no real advantage over
> QTAILQ. QSIMPLEQ is simplified to no longer permit insertion at the
> tail, with the advantage that it is more suited for simple free lists.
>
> Stefan, these are a bit borderline for qemu-trivial. Let me know
> if they're fine.
Please have them merged by a qemu.git committer.
The changes seem fine. No list implementation is perfect for all use
cases and I see why you want the new QSIMPLEQ.
Reviewed-by: Stefan Hajnoczi <address@hidden>
- Re: [Qemu-trivial] [Qemu-devel] [PATCH 1/8] notifier: switch to QLIST, (continued)
- [Qemu-trivial] [PATCH 5/8] qemu-queue: really simplify QSIMPLEQ, Paolo Bonzini, 2012/01/02
- [Qemu-trivial] [PATCH 8/8] block: use QSIMPLEQ for the AIO free list, Paolo Bonzini, 2012/01/02
- [Qemu-trivial] [PATCH 3/8] qed: switch to QTAILQ, Paolo Bonzini, 2012/01/02
- [Qemu-trivial] [PATCH 4/8] ccid: switch to QTAILQ, Paolo Bonzini, 2012/01/02
- [Qemu-trivial] [PATCH 6/8] qemu-queue: drop QCIRCLEQ, Paolo Bonzini, 2012/01/02
- [Qemu-trivial] [PATCH 2/8] block-migration: switch to QTAILQ, Paolo Bonzini, 2012/01/02
- [Qemu-trivial] [PATCH 7/8] coroutine: switch to QSIMPLEQ, Paolo Bonzini, 2012/01/02
- Re: [Qemu-trivial] [PATCH 0/8] qemu-queue cleanups,
Stefan Hajnoczi <=