From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:40465) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1bAEFp-0002WC-GM for qemu-devel@nongnu.org; Tue, 07 Jun 2016 06:29:02 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1bAEFn-0008GU-KS for qemu-devel@nongnu.org; Tue, 07 Jun 2016 06:29:00 -0400 Date: Tue, 7 Jun 2016 12:28:50 +0200 From: Kevin Wolf Message-ID: <20160607102850.GD4684@noname.str.redhat.com> References: <1465242006-22509-1-git-send-email-clord@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1465242006-22509-1-git-send-email-clord@redhat.com> Subject: Re: [Qemu-devel] [PATCH] Report error when opening device with locked tray List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Colin Lord Cc: qemu-devel@nongnu.org, armbru@redhat.com, qemu-block@nongnu.org, mreitz@redhat.com Am 06.06.2016 um 21:40 hat Colin Lord geschrieben: > This commit causes qmp_blockdev_change_medium to report an error if an > attempt is made to open a device with a locked tray. The old behaviour is that the command seemingly succeeds, but the medium isn't actually changed. Correct? Should this be mentioned in the commit message? You just describe what you change, but not why. > Signed-off-by: Colin Lord > This is based off my previous patch regarding the do_open_tray function > (currently at v3). Probably should have been submitted as a patch set > but I wasn't thinking that far ahead when I submitted the first patch. > --- > blockdev.c | 7 +++++-- > 1 file changed, 5 insertions(+), 2 deletions(-) Yes, would probably have made sense as a series, but as long as it's only two patches, it's not really a problem. Please make sure to put such comments below the "---" line, though, i.e. comments that make sense for the review, but not as part of the commit log. Then git-am automatically removes that part from the commit message while applying the patch. I did it manually for this one now. Kevin