[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH v3 2/2] block: move the bdrv_dev_change_media_cb

From: Stefan Hajnoczi
Subject: Re: [Qemu-devel] [PATCH v3 2/2] block: move the bdrv_dev_change_media_cb()
Date: Wed, 19 Jun 2013 12:16:58 +0200
User-agent: Mutt/1.5.21 (2010-09-15)

On Mon, Jun 17, 2013 at 03:21:41PM +0200, Pavel Hrdina wrote:
> The bdrv_dev_change_media_cb() should be called only for eject and change
> commands. We should call that function only if that command is successful.
> What this function does is that it calls the change_media_cb() and also emit
> If a password is not required, but user provides some, the error is used as
> warning.
> Signed-off-by: Pavel Hrdina <address@hidden>
> ---
>  block.c    | 8 --------
>  blockdev.c | 7 +++++++
>  2 files changed, 7 insertions(+), 8 deletions(-)

This commit description explains what the code changes do but it doesn't
explain why.  The cover letter mentions a regression without going into
detail, and that will not be commited to git.  Please add information
about the regression that this patch fixes so the git history has enough
information to justify this patch.

Markus posted a list of places that are affected by this change.  Have
you worked through them to show this patch is safe?


reply via email to

[Prev in Thread] Current Thread [Next in Thread]