Oh, right, got it. Sorry for the confusion, I’ll take that out of the patch.
Thanks,
Lucian Petrut
From: Paolo Bonzini<mailto:pbonz...@redhat.com>
Sent: Wednesday, May 16, 2018 1:42 PM
To: Lucian Petrut<mailto:lpet...@cloudbasesolutions.com>;
petrutlucia...@gmail.com<mai
explicitly mentioned this in the commit
message.
Regards,
Lucian Petrut
From: Paolo Bonzini<mailto:pbonz...@redhat.com>
Sent: Wednesday, May 16, 2018 1:13 PM
To: petrutlucia...@gmail.com<mailto:petrutlucia...@gmail.com>
Cc: Lucian Petrut<mailto:lpet...@cloudbasesolutions.com>; A
compatibility with
Azure, which requires the MB alignment to be preserved.
Signed-off-by: Lucian Petrut
---
Proposed fix for https://bugs.launchpad.net/qemu/+bug/1490611
block/vpc.c | 32
1 file changed, 8 insertions(+), 24 deletions(-)
diff --git a/block/vpc.c b/block
error is returned if the data blocks are
layed on the image otherwise.
Tests covering this feature have been added as well.
Signed-off-by: Lucian Petrut
---
Changes since the last version: Fixed the issues pointed out by
Stefan Hajnoczi, among which are: wrong conversions to/from BE,
switched to
Thanks for reviewing this patch! I’ll send a new one, fixing the issues you’ve
pointed out, along with some tests.
Please see my inline comments bellow.
Regards,
Lucian Petrut
>From: Stefan Hajnoczi
>Sent: Monday, September 22, 2014 3:17 PM
>To: Petrut Lucian
>Cc
.
Signed-off-by: Lucian Petrut
---
block/vpc.c | 186 +++-
1 file changed, 184 insertions(+), 2 deletions(-)
Changes from previous version: fixed a few coding style nits
underlined by the checkpatch.pl script. I forgot to run this
script
.
Signed-off-by: Lucian Petrut
---
block/vpc.c | 179 +++-
1 file changed, 177 insertions(+), 2 deletions(-)
diff --git a/block/vpc.c b/block/vpc.c
index 055efc4..f3895cb 100644
--- a/block/vpc.c
+++ b/block/vpc.c
@@ -417,7 +417,7 @@ static
