From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([208.118.235.92]:58270) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1UNhJh-0000Vc-Nq for qemu-devel@nongnu.org; Thu, 04 Apr 2013 06:22:54 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1UNhJc-0001At-N6 for qemu-devel@nongnu.org; Thu, 04 Apr 2013 06:22:49 -0400 Received: from mail-lb0-f170.google.com ([209.85.217.170]:60583) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1UNhJc-0001Ah-G6 for qemu-devel@nongnu.org; Thu, 04 Apr 2013 06:22:44 -0400 Received: by mail-lb0-f170.google.com with SMTP id x11so2544964lbi.1 for ; Thu, 04 Apr 2013 03:22:43 -0700 (PDT) MIME-Version: 1.0 In-Reply-To: References: <1363283830-22717-1-git-send-email-peter.maydell@linaro.org> From: Peter Maydell Date: Thu, 4 Apr 2013 11:22:22 +0100 Message-ID: Content-Type: text/plain; charset=UTF-8 Subject: Re: [Qemu-devel] [PATCH v2 0/2] configure: fix coroutine backend selection logic List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Cc: Paolo Bonzini , Anthony Liguori , patches@linaro.org Ping^2! PS: I note that the patches script doesn't list a patch in the 'to:foo' list if foo was forgotten on initial send but cc'd on subsequent pings... thanks -- PMM On 25 March 2013 13:50, Peter Maydell wrote: > Ping! > > thanks > -- PMM > > On 14 March 2013 17:57, Peter Maydell wrote: >> The main aim of this patchset is patch 2, which changes the coroutine >> backend selection logic so that it goes 'ucontext -> sigaltstack' >> rather than 'ucontext -> gthread', since the gthread backend is >> broken. To do this properly on all platforms we have to refactor >> the code a bit (which it needed anyway). >> >> The first patch here is a generic configure cleanup to how we print >> error messages, which I wanted for the third patch because otherwise >> the various error paths would have been rather verbose. >> >> Changes v1->v2: >> * fix the tests makefile too >> * CONFIG_COROUTINE_BACKEND=foo rather than CONFIG_FOO_COROUTINE=y; >> this simplifies both the configure code and the makefiles >> * dropped the 'move upper()' patch as no longer needed (in this >> series, anyway) >> >> Peter Maydell (2): >> configure: Provide and use convenience error reporting function >> configure: Don't fall back to gthread coroutine backend >> >> Makefile.objs | 11 +-- >> configure | 257 ++++++++++++++++++++++++++------------------------------ >> tests/Makefile | 14 +-- >> 3 files changed, 119 insertions(+), 163 deletions(-) >> >> -- >> 1.7.9.5