From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([140.186.70.92]:38819) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1RySLP-0001DX-H2 for qemu-devel@nongnu.org; Fri, 17 Feb 2012 13:15:44 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1RySLL-0002cS-Cq for qemu-devel@nongnu.org; Fri, 17 Feb 2012 13:15:43 -0500 Received: from mail-pw0-f45.google.com ([209.85.160.45]:45535) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1RySLL-0002cM-5c for qemu-devel@nongnu.org; Fri, 17 Feb 2012 13:15:39 -0500 Received: by pbbro12 with SMTP id ro12so4828348pbb.4 for ; Fri, 17 Feb 2012 10:15:38 -0800 (PST) Message-ID: <4F3E9945.4090701@codemonkey.ws> Date: Fri, 17 Feb 2012 12:15:33 -0600 From: Anthony Liguori MIME-Version: 1.0 References: <1326472445-25966-1-git-send-email-pbonzini@redhat.com> In-Reply-To: <1326472445-25966-1-git-send-email-pbonzini@redhat.com> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH v2 0/5] qemu-queue cleanups List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Paolo Bonzini Cc: qemu-devel@nongnu.org On 01/13/2012 10:34 AM, Paolo Bonzini wrote: > These patches slighly change the assortment of lists provided by > qemu-queue. QCIRCLEQ is dropped, since it provides no real advantage > over QTAILQ (and in fact is unused). QSLIST is introduced and used for > free lists. > > QSIMPLEQ is left in, since it provides some memory saving over QTAILQ. > > Stefan, these are a bit borderline for qemu-trivial. Let me know > if they're fine. Applied 1-4 since 5 introduced a regression. Thanks. Regards, Anthony Liguori > > Paolo Bonzini (5): > notifier: switch to QLIST > qemu-queue: add QSLIST > qemu-queue: drop QCIRCLEQ > coroutine: switch to QSLIST > block: use QSLIST for the AIO free list > > v1->v2: leave QSIMPLEQ in, add QSLIST (Peter) > > block.c | 9 +- > block_int.h | 4 +- > coroutine-ucontext.c | 10 +- > input.c | 2 +- > migration.c | 2 +- > notify.c | 10 +- > notify.h | 8 +- > qemu-coroutine-int.h | 2 +- > qemu-queue.h | 205 +++++++++++++++++++++----------------------------- > qemu-timer.c | 2 +- > vl.c | 2 +- > 11 files changed, 110 insertions(+), 146 deletions(-) >