From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:41746) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ZyMd6-00052B-N0 for qemu-devel@nongnu.org; Mon, 16 Nov 2015 11:27:45 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1ZyMd2-0000hK-Fz for qemu-devel@nongnu.org; Mon, 16 Nov 2015 11:27:44 -0500 References: <1447052973-14513-1-git-send-email-pl@kamp.de> <20151113094509.GD6650@stefanha-x1.localdomain> <5645B2F1.4020404@kamp.de> <20151116032502.GA9498@stefanha-x1.localdomain> From: John Snow Message-ID: <564A03F2.3010802@redhat.com> Date: Mon, 16 Nov 2015 11:27:30 -0500 MIME-Version: 1.0 In-Reply-To: <20151116032502.GA9498@stefanha-x1.localdomain> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH V5] block/nfs: add support for setting debug level List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Stefan Hajnoczi , Peter Lieven Cc: kwolf@redhat.com, famz@redhat.com, qemu-block@nongnu.org, qemu-devel@nongnu.org, mreitz@redhat.com On 11/15/2015 10:25 PM, Stefan Hajnoczi wrote: > On Fri, Nov 13, 2015 at 10:52:49AM +0100, Peter Lieven wrote: >> Am 13.11.2015 um 10:45 schrieb Stefan Hajnoczi: >>> On Mon, Nov 09, 2015 at 08:09:33AM +0100, Peter Lieven wrote: >>>> recent libnfs versions support logging debug messages. Add >>>> support for it in qemu through an URL parameter. >>>> >>>> Example: >>>> qemu -cdrom nfs://127.0.0.1/iso/my.iso?debug=2 >>>> >>>> Signed-off-by: Peter Lieven >>>> --- >>>> v4->v5: add a comment in the code why we limit the debug level [Stefan] >>>> v3->v4: revert to the initial version, but limit max debug level >>>> v2->v3: use a per-drive option instead of a global one. [Stefan] >>>> v1->v2: reworked patch to accept the debug level as a cmdline >>>> parameter instead of an URI parameter [Stefan] >>>> >>>> block/nfs.c | 12 ++++++++++++ >>>> 1 file changed, 12 insertions(+) >>> Hi Peter, >>> Please use my official maintainer email address >>> when CCing me. I didn't spot the mail to GMail until after the QEMU 2.5 >>> hard freeze deadline. >> >> Okay. I think I used that email because you replayed to earlier versions >> of this Patch from your Gmail address. >> >>> >>> Only bug fixes are being merged for QEMU 2.5 now. I'm sorry that this >>> patch didn't make it. My block-next branch will be opening on Monday >>> and I'll merge this patch there for QEMU 2.6. >> >> Thats not critical in this case. I think same applies for the ATAPI stuff >> we worked on? There you used also your GMail account. Or can this >> be treated as a bugfix? > > The ATAPI patches go through John Snow. It's up to him whether they are > considered a bug fix suitable for 2.5 or too risky. > > Stefan > I've tested them pretty thoroughly; I think they're safe (ATAPI-wise) provided the buffered DMA helpers look sane (They look sane to me.) The modifications to the giant atapi-return-code blob are well understood at this point and are definitely fine. --js