* [Qemu-devel] [PATCH] Fix : Enable qemu-img QED image commit support.
@ 2013-01-04 10:02 Onkar N Mahajan
2013-01-04 20:21 ` Anthony Liguori
0 siblings, 1 reply; 5+ messages in thread
From: Onkar N Mahajan @ 2013-01-04 10:02 UTC (permalink / raw)
To: qemu-devel; +Cc: Anthony Liguori, Stefan Hajnoczi
Running qemu-img commit on QED image gives this message even if the
image is never committed. This patch fixes this. Do we need to do any
L1, L2 table cleanup here ? I see that cleanup code is commented out in
the case of qcow2 (qcow2_make_empty).
before patching -
# qemu-img commit -f qed ss.qed
qemu-img: Image is already committed
even before committing the image.
after patching -
# qemu-img commit -f qed ss.qed
Image committed.
and image is actually successfully committed.
-------------------------------------------------------
Signed-off-by: Onkar N Mahajan/India/IBM
<onkar.n.mahajan@linux.vnet.ibm.com>
---
block/qed.c | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/block/qed.c b/block/qed.c
index 6c182ca..32f1d53 100644
--- a/block/qed.c
+++ b/block/qed.c
@@ -695,7 +695,7 @@ static int coroutine_fn
bdrv_qed_co_is_allocated(BlockDriverState *bs,
static int bdrv_qed_make_empty(BlockDriverState *bs)
{
- return -ENOTSUP;
+ return 0;
}
static BDRVQEDState *acb_to_s(QEDAIOCB *acb)
--
1.7.1
--
Onkar N Mahajan
System Software Engineer
IBM Linux Technology Center
Bangalore - India
^ permalink raw reply related [flat|nested] 5+ messages in thread
* [Qemu-devel] [PATCH] Fix : Enable qemu-img QED image commit support.
@ 2013-01-04 15:32 Onkar N Mahajan
2013-01-04 17:25 ` Stefan Hajnoczi
2013-01-04 20:21 ` Anthony Liguori
0 siblings, 2 replies; 5+ messages in thread
From: Onkar N Mahajan @ 2013-01-04 15:32 UTC (permalink / raw)
To: qemu-devel; +Cc: Anthony Liguori, Stefan Hajnoczi
Running qemu-img commit on QED image emits the below message even if the
image is never committed. This patch fixes this. Do we need to do any
L1, L2 table cleanup here ? I see that cleanup code is commented out in
the case of qcow2 (qcow2_make_empty).
before patching -
# qemu-img commit -f qed ss.qed
qemu-img: Image is already committed
even before committing the image.
after patching -
# qemu-img commit -f qed ss.qed
Image committed.
and image is actually successfully committed.
***
Signed-off-by: Onkar N Mahajan/India/IBM
<onkar.n.mahajan@linux.vnet.ibm.com>
---
block/qed.c | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/block/qed.c b/block/qed.c
index 6c182ca..32f1d53 100644
--- a/block/qed.c
+++ b/block/qed.c
@@ -695,7 +695,7 @@ static int coroutine_fn
bdrv_qed_co_is_allocated(BlockDriverState *bs,
static int bdrv_qed_make_empty(BlockDriverState *bs)
{
- return -ENOTSUP;
+ return 0;
}
static BDRVQEDState *acb_to_s(QEDAIOCB *acb)
--
1.7.1
--
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [Qemu-devel] [PATCH] Fix : Enable qemu-img QED image commit support.
2013-01-04 15:32 [Qemu-devel] [PATCH] Fix : Enable qemu-img QED image commit support Onkar N Mahajan
@ 2013-01-04 17:25 ` Stefan Hajnoczi
2013-01-04 20:21 ` Anthony Liguori
1 sibling, 0 replies; 5+ messages in thread
From: Stefan Hajnoczi @ 2013-01-04 17:25 UTC (permalink / raw)
To: Onkar N Mahajan; +Cc: Stefan Hajnoczi, qemu-devel, Anthony Liguori
On Fri, Jan 04, 2013 at 09:02:28PM +0530, Onkar N Mahajan wrote:
> @@ -695,7 +695,7 @@ static int coroutine_fn
> bdrv_qed_co_is_allocated(BlockDriverState *bs,
>
> static int bdrv_qed_make_empty(BlockDriverState *bs)
> {
> - return -ENOTSUP;
> + return 0;
> }
The only .bdrv_make_empty() caller is block.c:bdrv_commit():
if (drv->bdrv_make_empty) {
ret = drv->bdrv_make_empty(bs);
bdrv_flush(bs);
}
Perhaps it's best to drop bdrv_qed_make_empty() completely so
bdrv_commit() succeeds but we don't pretend to support
.bdrv_make_empty().
Stefan
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [Qemu-devel] [PATCH] Fix : Enable qemu-img QED image commit support.
2013-01-04 15:32 [Qemu-devel] [PATCH] Fix : Enable qemu-img QED image commit support Onkar N Mahajan
2013-01-04 17:25 ` Stefan Hajnoczi
@ 2013-01-04 20:21 ` Anthony Liguori
1 sibling, 0 replies; 5+ messages in thread
From: Anthony Liguori @ 2013-01-04 20:21 UTC (permalink / raw)
To: Onkar N Mahajan, qemu-devel; +Cc: Stefan Hajnoczi, Anthony Liguori
Hi,
This is an automated message generated from the QEMU Patches.
Thank you for submitting this patch. This patch no longer applies to qemu.git.
This may have occurred due to:
1) Changes in mainline requiring your patch to be rebased and re-tested.
2) Sending the mail using a tool other than git-send-email. Please use
git-send-email to send patches to QEMU.
3) Basing this patch off of a branch that isn't tracking the QEMU
master branch. If that was done purposefully, please include the name
of the tree in the subject line in the future to prevent this message.
For instance: "[PATCH block-next 1/10] qcow3: add fancy new feature"
4) You no longer wish for this patch to be applied to QEMU. No additional
action is required on your part.
Nacked-by: QEMU Patches <aliguori@us.ibm.com>
Below is the output from git-am:
Applying: Fix : Enable qemu-img QED image commit support.
fatal: corrupt patch at line 10
Repository lacks necessary blobs to fall back on 3-way merge.
Cannot fall back to three-way merge.
Patch failed at 0001 Fix : Enable qemu-img QED image commit support.
The copy of the patch that failed is found in:
/home/aliguori/.patches/qemu-working/.git/rebase-apply/patch
When you have resolved this problem run "git am --resolved".
If you would prefer to skip this patch, instead run "git am --skip".
To restore the original branch and stop patching run "git am --abort".
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [Qemu-devel] [PATCH] Fix : Enable qemu-img QED image commit support.
2013-01-04 10:02 Onkar N Mahajan
@ 2013-01-04 20:21 ` Anthony Liguori
0 siblings, 0 replies; 5+ messages in thread
From: Anthony Liguori @ 2013-01-04 20:21 UTC (permalink / raw)
To: Onkar N Mahajan, qemu-devel; +Cc: Stefan Hajnoczi, Anthony Liguori
Hi,
This is an automated message generated from the QEMU Patches.
Thank you for submitting this patch. This patch no longer applies to qemu.git.
This may have occurred due to:
1) Changes in mainline requiring your patch to be rebased and re-tested.
2) Sending the mail using a tool other than git-send-email. Please use
git-send-email to send patches to QEMU.
3) Basing this patch off of a branch that isn't tracking the QEMU
master branch. If that was done purposefully, please include the name
of the tree in the subject line in the future to prevent this message.
For instance: "[PATCH block-next 1/10] qcow3: add fancy new feature"
4) You no longer wish for this patch to be applied to QEMU. No additional
action is required on your part.
Nacked-by: QEMU Patches <aliguori@us.ibm.com>
Below is the output from git-am:
Applying: Fix : Enable qemu-img QED image commit support.
fatal: corrupt patch at line 10
Repository lacks necessary blobs to fall back on 3-way merge.
Cannot fall back to three-way merge.
Patch failed at 0001 Fix : Enable qemu-img QED image commit support.
The copy of the patch that failed is found in:
/home/aliguori/.patches/qemu-working/.git/rebase-apply/patch
When you have resolved this problem run "git am --resolved".
If you would prefer to skip this patch, instead run "git am --skip".
To restore the original branch and stop patching run "git am --abort".
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2013-01-04 20:21 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-01-04 15:32 [Qemu-devel] [PATCH] Fix : Enable qemu-img QED image commit support Onkar N Mahajan
2013-01-04 17:25 ` Stefan Hajnoczi
2013-01-04 20:21 ` Anthony Liguori
-- strict thread matches above, loose matches on Subject: below --
2013-01-04 10:02 Onkar N Mahajan
2013-01-04 20:21 ` Anthony Liguori
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).