755 scripts/simplebench/results_to_text.py
Reviewed-by: Max Reitz
ell += f' {col_j}{diff_pr:+}%'
Contrasting to v6, you added the "cell += ' '" line, dropped a space in
the "cell += '--'" line (was: "cell += ' --'"), but kept the space here.
I would have assumed that the leading space is drop
On 13.11.20 17:07, Alberto Garcia wrote:
On Fri 13 Nov 2020 12:49:04 PM CET, Max Reitz wrote:
On 11.11.20 17:53, Alberto Garcia wrote:
This simply calls bdrv_co_pwrite_zeroes() in all children
Signed-off-by: Alberto Garcia
---
block/quorum.c | 18 --
tests
lly agree is an improvement), I can’t
really give an R-b.
Perhaps this is OK:
Acked-by: Max Reitz
def result_to_text(result):
"""Return text representation of bench_one() returned dict."""
if 'average' in result:
-s = '{:.2f} +- {
eq.py | 3 +-
scripts/simplebench/results_to_text.py | 48 ++
scripts/simplebench/simplebench.py | 31 -
4 files changed, 52 insertions(+), 33 deletions(-)
create mode 100644 scripts/simplebench/results_to_text.py
Reviewed-by: Max Reitz
| 2 +-
scripts/simplebench/simplebench.py | 10 +-
3 files changed, 7 insertions(+), 7 deletions(-)
Reviewed-by: Max Reitz
(-)
Reviewed-by: Max Reitz
tions(+), 10 deletions(-)
Reviewed-by: Max Reitz
On 21.10.20 16:58, Vladimir Sementsov-Ogievskiy wrote:
Signed-off-by: Vladimir Sementsov-Ogievskiy
---
scripts/simplebench/simplebench.py | 12 ++--
1 file changed, 6 insertions(+), 6 deletions(-)
Reviewed-by: Max Reitz
mode 100644 tests/qemu-iotests/298
create mode 100644 tests/qemu-iotests/298.out
Reviewed-by: Max Reitz
(+), 1 deletion(-)
Reviewed-by: Max Reitz
if (prealloc == PREALLOC_MODE_FALLOC) {
+/*
+ * If offset <= s->file_end, the task is already done, just
+ * update s->file_end, to move part of "filter preallocation"
s/file_end/data_end/
With those fixed, and with %s/5\.2/6.0/:
Reviewed-by: Ma
| 7 +--
1 file changed, 5 insertions(+), 2 deletions(-)
Reviewed-by: Max Reitz
hat doesn't
resolve with "Base 'BASE' not found". Commit 6b33f3ae8b "qemu-img:
Improve commit invalid base message" improved this message in
qemu-img. Improve it here, too: "Can't find '%s' in the backing
chain".
QERR_BASE_NOT_FOUND is
((0x25)) $((0x1))" |
_filter_qemu_io
+
My gut would have preferred a test where the data region is larger than
the zeroed region (so we can see that the first write has done
something), but who cares about my gut.
I don’t mind not setting supported_zero_flags enough to warrant
insertions(+)
create mode 100755 tests/qemu-iotests/312
create mode 100644 tests/qemu-iotests/312.out
Reviewed-by: Max Reitz
On 11.11.20 13:43, Stefan Hajnoczi wrote:
Check that the sector number and byte count are valid.
Signed-off-by: Stefan Hajnoczi
---
block/export/vhost-user-blk-server.c | 14 ++
1 file changed, 14 insertions(+)
diff --git a/block/export/vhost-user-blk-server.c
b/block/export/vh
On 11.11.20 13:43, Stefan Hajnoczi wrote:
Exercise input validation code paths in
block/export/vhost-user-blk-server.c.
Signed-off-by: Stefan Hajnoczi
---
block/export/vhost-user-blk-server.c | 4 +-
tests/qtest/vhost-user-blk-test.c| 124 +++
2 files changed,
On 11.11.20 13:43, Stefan Hajnoczi wrote:
Validate discard/write zeroes the same way we do for virtio-blk. Some of
these checks are mandated by the VIRTIO specification, others are
internal to QEMU.
Signed-off-by: Stefan Hajnoczi
---
block/export/vhost-user-blk-server.c | 115
On 11.11.20 13:43, Stefan Hajnoczi wrote:
The function is used not just for image files but also for UNIX domain
sockets (QMP monitor and vhost-user-blk). Reflect that in the name.
Signed-off-by: Stefan Hajnoczi
---
tests/qtest/vhost-user-blk-test.c | 9 +
1 file changed, 5 insertion
On 09.11.20 18:38, Max Reitz wrote:
The following changes since commit 2a190a7256a3e0563b29ffd67e0164097b4a6dac:
Merge remote-tracking branch
'remotes/philmd-gitlab/tags/renesas-fixes-20201109' into staging (2020-11-09
11:20:25 +)
are available in the Git repository at:
The following changes since commit 2a190a7256a3e0563b29ffd67e0164097b4a6dac:
Merge remote-tracking branch
'remotes/philmd-gitlab/tags/renesas-fixes-20201109' into staging (2020-11-09
11:20:25 +)
are available in the Git repository at:
https://github.com/XanClic/qemu.git tags/pull-block
Alberto Garcia
Signed-off-by: Max Reitz
---
block.c | 54 --
1 file changed, 24 insertions(+), 30 deletions(-)
diff --git a/block.c b/block.c
index 9a945a058d..f1cedac362 100644
--- a/block.c
+++ b/block.c
@@ -4910,9 +4910,11 @@ int
h
also different kinds of "should not". Let's do it later.
Signed-off-by: Vladimir Sementsov-Ogievskiy
Message-Id: <20201106124241.16950-3-vsement...@virtuozzo.com>
Reviewed-by: Alberto Garcia
Signed-off-by: Max Reitz
---
block.c | 25 ++---
1 file changed,
From: Vladimir Sementsov-Ogievskiy
Signed-off-by: Vladimir Sementsov-Ogievskiy
Message-Id: <20201106124241.16950-2-vsement...@virtuozzo.com>
Reviewed-by: Alberto Garcia
Signed-off-by: Max Reitz
---
block.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/block.c b/block.c
index 2fd9
l reaches end of non-void function
[-Werror=return-type]
752 | }
| ^
On msys2/mingw, there is no st_blocks in struct _stat64 yet, we disable the
usage of it
on msys2/mingw, and create a typedef long long blkcnt_t; for further
implementation
Signed-off-by: Yonggang Luo
Message-Id: <
From: Yonggang Luo
Initially, libnfs has not been enabled, and now it's fixed, so enable it
on cirrus.
Signed-off-by: Yonggang Luo
Message-Id: <20201105123116.674-3-luoyongg...@gmail.com>
Signed-off-by: Max Reitz
---
.cirrus.yml | 1 +
1 file changed, 1 insertion(+)
diff --git a/
From: Klaus Jensen
Fix dereference after NULL check.
Reported-by: Coverity (CID 1436128)
Fixes: b20804946bce ("hw/block/nvme: update nsid when registered")
Signed-off-by: Klaus Jensen
Message-Id: <20201104102248.32168-2-...@irrelevant.dk>
Reviewed-by: Philippe Mathieu-Daudé
Si
From: shiliyang
There have some code style problems be found when read the block driver code.
So I fixes some problems of this error, ERROR: "foo* bar" should be "foo *bar".
Signed-off-by: Liyang Shi
Reported-by: Euler Robot
---
block/qcow2.h| 6 +++---
block/blkdebug.c | 2 +-
block/dm
1436131)
Signed-off-by: Klaus Jensen
Message-Id: <20201104102248.32168-4-...@irrelevant.dk>
Reviewed-by: Philippe Mathieu-Daudé
Signed-off-by: Max Reitz
---
hw/block/nvme.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/hw/block/nvme.c b/hw/block/nvme.c
index 2bdc50eb6f..01b657b1c5 10
From: AlexChen
The "qemu-common.h" include is not used, remove it.
Reported-by: Euler Robot
Signed-off-by: AlexChen
Message-Id: <5f8ffb94.3030...@huawei.com>
Signed-off-by: Max Reitz
---
block/dmg-lzfse.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/block/dmg-lzf
: Christian Borntraeger
Reviewed-by: Paolo Bonzini
Message-Id: <20201104185025.434703-3-mlevi...@redhat.com>
Signed-off-by: Max Reitz
---
tests/qemu-iotests/240 | 219 +++--
tests/qemu-iotests/240.out | 76 +++--
2 files changed, 130 insertions(+
irrelevant.dk>
Reviewed-by: Philippe Mathieu-Daudé
Signed-off-by: Max Reitz
---
hw/block/nvme.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/hw/block/nvme.c b/hw/block/nvme.c
index 080d782f1c..2bdc50eb6f 100644
--- a/hw/block/nvme.c
+++ b/hw/block/nvme.c
@@ -452,7 +452,7 @@
rto Garcia
Reviewed-by: Eric Blake
Reviewed-by: Vladimir Sementsov-Ogievskiy
Message-Id: <20201007161323.4667-1-be...@igalia.com>
Signed-off-by: Max Reitz
---
block/qcow2.h | 19 +++
block/qcow2-cluster.c | 5 +++--
block/qcow2.c | 19 +++
the original error. While we're at it, we can avoid the
confusing ?: by spelling the logic more directly.
Fixes: 4a9c9ea0d3
Reported-by: Guoyi Tu
Signed-off-by: Eric Blake
Message-Id: <20201105155122.60943-1-ebl...@redhat.com>
Reviewed-by: Alberto Garcia
Signed-off-by: Max Reitz
---
The following changes since commit 2a190a7256a3e0563b29ffd67e0164097b4a6dac:
Merge remote-tracking branch
'remotes/philmd-gitlab/tags/renesas-fixes-20201109' into staging (2020-11-09
11:20:25 +)
are available in the Git repository at:
https://github.com/XanClic/qemu.git tags/pull-block
From: Greg Kurz
This function is really an internal helper for bdrv_close(). Update its
doc comment to make this clear and make the function private.
Signed-off-by: Greg Kurz
Message-Id: <160387245480.131299.13430357162209598411.stgit@bahia>
Reviewed-by: Stefan Hajnoczi
Signed-off-b
: <20201104185025.434703-2-mlevi...@redhat.com>
Signed-off-by: Max Reitz
---
tests/qemu-iotests/iotests.py | 10 ++
1 file changed, 10 insertions(+)
diff --git a/tests/qemu-iotests/iotests.py b/tests/qemu-iotests/iotests.py
index 814804a4c6..bcd4fe5b6f 100644
--- a/tests/qemu-iotests/iotests.py
+++ b
[Cc-ing Stefan]
On 09.11.20 16:05, Peter Maydell wrote:
In vu_blk_discard_write_zeroes(), we read a 32-bit sector count from
the descriptor and convert it to a 64-bit byte count. Coverity warns
that the left shift is done with 32-bit arithmetic so it might
overflow before the conversion to 64-bi
On 06.11.20 13:42, Vladimir Sementsov-Ogievskiy wrote:
Hi all!
These series supersedes "Fix nested permission update" and includes one
more fix (patch 01) and more improvements.
I think patch 01 is good to have in 5.2, 02 is probably OK for 5.2 and
the others are OK for next release. Still all
block.c | 88 +++--
1 file changed, 17 insertions(+), 71 deletions(-)
Reviewed-by: Max Reitz
-by: Max Reitz
(because AFAIR *_set_perm is
used quite often).)
Reviewed-by: Max Reitz
On 30.10.20 04:35, shiliyang wrote:
There have some code style problems be found when read the block driver code.
So I fixes some problems of this error, ERROR: "foo* bar" should be "foo *bar".
Signed-off-by: Liyang Shi
Reported-by: Euler Robot
---
block/blkdebug.c | 2 +-
block/dmg.c
On 05.11.20 16:51, Eric Blake wrote:
Back in 2015, we attempted to fix error reporting for images that
claimed to have more than INT64_MAX/512 sectors, but due to the type
promotions caused by BDRV_SECTOR_SIZE being unsigned, this
inadvertently forces all negative ret values to be slammed into -E
On 05.11.20 13:31, Yonggang Luo wrote:
V3-V4
Use uint64_t instead of blkcnt_t
V2-V3
Revise the commit message of
* block: enable libnfs on msys2/mingw in cirrus.yml
V1-V2
Apply suggestion from Peter Lieven
Yonggang Luo (2):
block: Fixes nfs compiling error on msys2/mingw
block: enable li
On 05.11.20 13:32, 罗勇刚(Yonggang Luo) wrote:
On Thu, Nov 5, 2020 at 7:43 PM Max Reitz <mailto:mre...@redhat.com>> wrote:
>
> On 13.10.20 02:15, Yonggang Luo wrote:
> > At the begging libnfs are not enabled because of compiling error,
> > now it's fixed so e
ments, no?
(And, as you write, in the future potentially drop the parameters.)
Anyway:
Reviewed-by: Max Reitz
On 05.11.20 14:14, Eric Blake wrote:
On 11/5/20 2:31 AM, Max Reitz wrote:
On 05.11.20 06:40, Tuguoyi wrote:
As BDRV_SECTOR_SIZE is of type uint64_t, the expression will
automatically convert the @ret to uint64_t. When an error code
returned from bdrv_nb_sectors(), the promoted @ret will be a
On 05.11.20 14:22, Max Reitz wrote:
On 31.10.20 13:35, Vladimir Sementsov-Ogievskiy wrote:
First, permission update loop tries to do iterations transactionally,
but the whole update is not transactional: nobody roll-back successful
loop iterations when some iteration fails.
[...]
And
igned-off-by: Vladimir Sementsov-Ogievskiy
---
block.c | 36 +++-
1 file changed, 15 insertions(+), 21 deletions(-)
And it’s also much simpler and clearer now.
Reviewed-by: Max Reitz
On 05.11.20 12:46, 罗勇刚(Yonggang Luo) wrote:
On Thu, Nov 5, 2020 at 7:44 PM Max Reitz <mailto:mre...@redhat.com>> wrote:
>
> On 05.11.20 12:32, 罗勇刚(Yonggang Luo) wrote:
> >
> >
> > On Thu, Nov 5, 2020 at 7:29 PM Max Reitz <mailto:mre...@redhat.
On 13.10.20 02:15, Yonggang Luo wrote:
At the begging libnfs are not enabled because of compiling error,
now it's fixed so enable it
Signed-off-by: Yonggang Luo
---
.cirrus.yml | 1 +
1 file changed, 1 insertion(+)
diff --git a/.cirrus.yml b/.cirrus.yml
index f42ccb956a..2c6bf45e6d 100644
-
On 05.11.20 12:32, 罗勇刚(Yonggang Luo) wrote:
On Thu, Nov 5, 2020 at 7:29 PM Max Reitz <mailto:mre...@redhat.com>> wrote:
>
> On 13.10.20 02:15, Yonggang Luo wrote:
> > These compiling errors are fixed:
> > ../block/nfs.c:27:10: fatal error: poll.h: No such fi
On 13.10.20 02:15, Yonggang Luo wrote:
These compiling errors are fixed:
../block/nfs.c:27:10: fatal error: poll.h: No such file or directory
27 | #include
| ^~~~
compilation terminated.
../block/nfs.c:63:5: error: unknown type name 'blkcnt_t'
63 | blkcnt_t st_bl
On 04.11.20 19:50, Maxim Levitsky wrote:
While most of the patches in V1 of this series are already merged upstream,
the patch that fixes iotest 240 was broken on s390 and was not accepted.
This is an updated version of this patch, based on Paulo's suggestion,
that hopefully makes this iotest wo
On 05.11.20 06:40, Tuguoyi wrote:
As BDRV_SECTOR_SIZE is of type uint64_t, the expression will
automatically convert the @ret to uint64_t. When an error code
returned from bdrv_nb_sectors(), the promoted @ret will be a very
large number, as a result the -EFBIG will be returned which is not the
re
On 26.10.20 16:18, Vladimir Sementsov-Ogievskiy wrote:
> 23.07.2020 12:47, Max Reitz wrote:
>>> +static void coroutine_fn backup_set_speed(BlockJob *job, int64_t speed)
>>> +{
>>> + BackupBlockJob *s = container_of(job, BackupBlockJob, common);
>>> +
&g
On 22.10.20 23:10, Vladimir Sementsov-Ogievskiy wrote:
> 23.07.2020 11:03, Max Reitz wrote:
>> On 01.06.20 20:11, Vladimir Sementsov-Ogievskiy wrote:
>>> After introducing parallel async copy requests instead of plain
>>> cluster-by-cluster copying loop, backup
On 22.10.20 22:50, Vladimir Sementsov-Ogievskiy wrote:
> 22.07.2020 14:28, Max Reitz wrote:
>> On 01.06.20 20:11, Vladimir Sementsov-Ogievskiy wrote:
>>> Add function to cancel running async block-copy call. It will be used
>>> in backup.
>>>
>>>
spect BlockDeviceMapEntry hasn't bothered
> anyone enough to complain in almost four years. Get rid of it.
>
> Cc: Paolo Bonzini
> Cc: Eric Blake
> Reviewed-by: Eric Blake
> Signed-off-by: Markus Armbruster
> ---
> I found an old patch I neglected to
On 22.10.20 22:35, Vladimir Sementsov-Ogievskiy wrote:
> 22.07.2020 15:22, Max Reitz wrote:
>> On 01.06.20 20:11, Vladimir Sementsov-Ogievskiy wrote:
>>> Add new parameters to configure future backup features. The patch
>>> doesn't introduce aio backup reques
we can make
them even clearer.)
Signed-off-by: Max Reitz
---
qapi/block-core.json | 18 +-
1 file changed, 13 insertions(+), 5 deletions(-)
diff --git a/qapi/block-core.json b/qapi/block-core.json
index 1b8b4156b4..3f86675357 100644
--- a/qapi/block-core.json
+++ b/qapi/block
From: Markus Armbruster
BlockDeviceMapEntry has never been used. It was added in commit
facd6e2 "so that it is published through the introspection mechanism."
What exactly introspecting types that aren't used for anything could
accomplish isn't clear. What "introspection mechanism" to use is al
BlockDeviceMapEntry
Max Reitz (1):
qapi/block-core: Improve MapEntry documentation
qapi/block-core.json | 47
1 file changed, 13 insertions(+), 34 deletions(-)
--
2.28.0
On 04.11.20 11:22, Klaus Jensen wrote:
> From: Klaus Jensen
>
> Fix three issues reported by coverity (CIDs 1436128, 1436129 and
> 1436131).
>
> Klaus Jensen (3):
> hw/block/nvme: fix null ns in register namespace
> hw/block/nvme: fix uint16_t use of uint32_t sgls member
> hw/block/nvme: f
On 04.11.20 12:04, Klaus Jensen wrote:
> On Nov 4 11:59, Max Reitz wrote:
>> On 04.11.20 11:22, Klaus Jensen wrote:
>>> From: Klaus Jensen
>>>
>>> Since 7f0f1acedf15 ("hw/block/nvme: support multiple namespaces"), the
>>> namespaces
quot;hw/block/nvme: support multiple namespaces")
> Reported-by: Coverity (CID 1436131)
> Signed-off-by: Klaus Jensen
> ---
> hw/block/nvme.c | 1 -
> 1 file changed, 1 deletion(-)
Thanks! :)
Reviewed-by: Max Reitz
k/nvme.c | 3 +--
> 1 file changed, 1 insertion(+), 2 deletions(-)
Reviewed-by: Max Reitz
)
> Signed-off-by: Klaus Jensen
> ---
> hw/block/nvme.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
Reviewed-by: Max Reitz
On 27.10.20 11:49, Klaus Jensen wrote:
> From: Klaus Jensen
>
> This adds support for multiple namespaces by introducing a new 'nvme-ns'
> device model. The nvme device creates a bus named from the device name
> ('id'). The nvme-ns devices then connect to this and registers
> themselves with the
On 27.10.20 11:49, Klaus Jensen wrote:
> From: Klaus Jensen
>
> For now, support the Data Block, Segment and Last Segment descriptor
> types.
>
> See NVM Express 1.3d, Section 4.4 ("Scatter Gather List (SGL)").
>
> Signed-off-by: Klaus Jensen
> Reviewed-by: Keith Busch
> ---
> include/block/
On 27.10.20 11:49, Klaus Jensen wrote:
> From: Klaus Jensen
>
> If the user does not specify an nsid parameter on the nvme-ns device,
> nvme_register_namespace will find the first free namespace id and assign
> that.
>
> This fix makes sure the assigned id is saved.
>
> Signed-off-by: Klaus Jen
On 07.10.20 18:13, Alberto Garcia wrote:
> The QCowL2Meta structure is used to store information about a part of
> a write request that touches clusters that need changes in their L2
> entries. This happens with newly-allocated clusters or subclusters.
>
> This structure has changed a bit since it
On 28.10.20 09:07, Greg Kurz wrote:
> This function is really an internal helper for bdrv_close(). Update its
> doc comment to make this clear and make the function private.
>
> Signed-off-by: Greg Kurz
> ---
>
> As suggested by Stefan here:
>
> https://lists.gnu.org/archive/html/qemu-devel/202
On 21.10.20 11:12, AlexChen wrote:
> The "qemu-common.h" include is not used, remove it.
>
> Reported-by: Euler Robot
> Signed-off-by: AlexChen
> ---
> block/dmg-lzfse.c | 1 -
> 1 file changed, 1 deletion(-)
Thanks, I’ve applied this patch to my block branch:
https://git.xanclic.moe/XanClic/
On 01.11.20 17:15, Maxim Levitsky wrote:
> The recent changes that brought RCU delayed device deletion,
> broke few tests and this test breakage went unnoticed.
>
> Fix this test by rewriting it in python
> (which allows to wait for DEVICE_DELETED events before continuing).
>
> Signed-off-by: Max
We have good coverage of the normal I/O paths now, but what remains is a
test that tests some more special cases: Exporting an image on itself
(thus turning a formatted image into a raw one), some error cases, and
non-writable and non-growable exports.
Signed-off-by: Max Reitz
Reviewed-by: Kevin
the rather low-hanging fruits.
Note that 221 and 250 only pass when .lseek is correctly implemented,
which is only possible with a libfuse that is 3.8 or newer.
Signed-off-by: Max Reitz
Reviewed-by: Kevin Wolf
---
tests/qemu-iotests/025 | 2 +-
tests/qemu-iotests/026 | 2 +-
tests/qemu-iotests
This pretends FUSE exports are a kind of protocol. As such, they are
always tested under the format node. This is probably the best way to
test them, actually, because this will generate more I/O load and more
varied patterns.
Signed-off-by: Max Reitz
---
tests/qemu-iotests/check
Signed-off-by: Max Reitz
---
tests/qemu-iotests/check | 11 +++
tests/qemu-iotests/common.rc | 17 +
2 files changed, 28 insertions(+)
diff --git a/tests/qemu-iotests/check b/tests/qemu-iotests/check
index 678b6e4910..3c1fa4435a 100755
--- a/tests/qemu-iotests/check
This generally does not work on non-file protocols. It is better to
create the image with the final name from the start, and most tests do
this already. Let 046 follow suit.
Signed-off-by: Max Reitz
Reviewed-by: Kevin Wolf
---
tests/qemu-iotests/046 | 5 +++--
tests/qemu-iotests/046.out
If the test environment has some other child processes running (like a
storage daemon that provides a FUSE export), then "wait" will never
finish. Use wait=yes _cleanup_qemu instead.
(We need to discard the output so there is no change to the reference
output.)
Signed-off-by:
Most Python tests are restricted to the file protocol (without
explicitly saying so), but these are the ones that would break
./check -fuse -qcow2.
Signed-off-by: Max Reitz
Reviewed-by: Kevin Wolf
---
tests/qemu-iotests/206 | 3 ++-
tests/qemu-iotests/242 | 3 ++-
2 files changed, 4 insertions
Otherwise, exports and block devices are not properly shut down and
closed, unless the users explicitly issues blockdev-del and
block-export-del commands for each of them.
Signed-off-by: Max Reitz
Reviewed-by: Kevin Wolf
---
storage-daemon/qemu-storage-daemon.c | 3 +++
1 file changed, 3
is important, so
clean up the image before the state is lost.
Signed-off-by: Max Reitz
Reviewed-by: Kevin Wolf
---
tests/qemu-iotests/287 | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/tests/qemu-iotests/287 b/tests/qemu-iotests/287
index f98a4cadc1..036cc09e82 100755
Avoid creating images with custom filenames in $TEST_DIR, because
non-file protocols may want to keep $TEST_IMG (and all other test
images) in some other directory.
Signed-off-by: Max Reitz
Reviewed-by: Kevin Wolf
---
tests/qemu-iotests/200 | 3 +--
tests/qemu-iotests/200.out | 4
ss $TEST_IMG_FILE'
015/20:[] [--] 'iotests/287: Clean up subshell test image'
016/20:[] [--] 'storage-daemon: Call bdrv_close_all() on exit'
017/20:[0018] [FC] 'iotests: Give access to the qemu-storage-daemon'
018/20:[0004] [FC] 'iotests: Allow
alue and thus work around what technically is the caller
misbehaving.
This second solution is less clean, but it is robust against people
keeping their old habit of adjusting TEST_IMG only, and requires much
less changes. So this patch implements it.
Signed-off-by: Max Reitz
Reviewed-by: Kevin
, because at that point the
BlockBackend is destroyed altogether anyway.)
Signed-off-by: Max Reitz
---
qapi/block-export.json | 6 +-
block/export/fuse.c| 44 ++
2 files changed, 41 insertions(+), 9 deletions(-)
diff --git a/qapi/block-export.json
This allows allocating areas after the (old) EOF as part of a growing
resize, writing zeroes, and discarding.
Signed-off-by: Max Reitz
---
block/export/fuse.c | 84 +
1 file changed, 84 insertions(+)
diff --git a/block/export/fuse.c b/block/export
ation output.
Signed-off-by: Max Reitz
Reviewed-by: Kevin Wolf
---
tests/qemu-iotests/028 | 14 --
tests/qemu-iotests/028.out | 3 +++
tests/qemu-iotests/089 | 3 ++-
tests/qemu-iotests/089.out | 1 +
4 files changed, 10 insertions(+), 11 deletions(-)
diff --git a/tests
).
Signed-off-by: Max Reitz
Reviewed-by: Kevin Wolf
---
tests/qemu-iotests/071 | 19 +++
tests/qemu-iotests/174 | 10 +-
2 files changed, 24 insertions(+), 5 deletions(-)
diff --git a/tests/qemu-iotests/071 b/tests/qemu-iotests/071
index 88faebcc1d..18fe9054b0 100755
--- a/tests
This is a relatively new feature in libfuse (available since 3.8.0,
which was released in November 2019), so we have to add a dedicated
check whether it is available before making use of it.
Signed-off-by: Max Reitz
---
configure | 8 -
meson.build | 20
This makes the export actually useful instead of only producing errors
whenever it is accessed.
Signed-off-by: Max Reitz
---
block/export/fuse.c | 242
1 file changed, 242 insertions(+)
diff --git a/block/export/fuse.c b/block/export/fuse.c
index
In most cases, _make_test_img does not need a _filter_imgfmt on top. It
does that by itself.
(The exception is when IMGFMT has been overwritten but TEST_IMG has not.
In such cases, we do need a _filter_imgfmt on top to filter the test's
original IMGFMT from TEST_IMG.)
Signed-off-by: Max
settle. Therefore, keep track of mount point
paths to at least catch the most obvious instances of that problem.
Signed-off-by: Max Reitz
---
qapi/block-export.json | 23 ++-
include/block/fuse.h | 30
block.c | 1 +
block/export/export.c| 4 +
block/export
Signed-off-by: Max Reitz
---
configure | 7 +++
meson.build | 6 ++
meson_options.txt | 2 ++
3 files changed, 15 insertions(+)
diff --git a/configure b/configure
index 83610b0db8..fc315deebe 100755
--- a/configure
+++ b/configure
@@ -448,6 +448,7 @@ meson="&qu
FUSE
exports, because then the export (which is the image used by qemu-nbd)
will go away before qemu-nbd exits, which will lead to qemu-nbd
complaining that it cannot flush the bitmaps in the image.
Signed-off-by: Max Reitz
---
tests/qemu-iotests/291 | 2 ++
1 file changed, 2 insertions(+)
diff
into
two patches.)
Max Reitz (2):
iotests/291: Filter irrelevant parts of img-info
iotests/291: Stop NBD server
tests/qemu-iotests/291 | 16 +---
tests/qemu-iotests/291.out | 20
2 files changed, 13 insertions(+), 23 deletions(-)
--
2.26.2
701 - 800 of 9610 matches
Mail list logo