[v4] file-posix: Simplifications on image locking

[Qemu-devel] [PATCH v4 0/3] file-posix: Simplifications on image locking

Fam Zheng posted 3 patches 7 years, 2 months ago

Diff against v3 v5
Download series mbox

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20180821005843.5146-1-famz@redhat.com

Test docker-clang@ubuntu failed

Test checkpatch passed

There is a newer version of this series

block/file-posix.c         |  83 ++++++++++++--------
tests/Makefile.include     |   2 +
tests/test-image-locking.c | 154 +++++++++++++++++++++++++++++++++++++
3 files changed, 209 insertions(+), 30 deletions(-)
create mode 100644 tests/test-image-locking.c

Expand all Fold all

[Qemu-devel] [PATCH v4 0/3] file-posix: Simplifications on image locking

Posted by Fam Zheng 7 years, 2 months ago

v4: Fix test on systems without OFD. [Patchew]

The first patch reduces chances of QEMU crash in unusual (but not unlikely)
cases especially when used by Libvirt (see commit message).

The second patch halves fd for images.

The third adds some more test for patch one (would have caught the regression
caused by v2).

Fam Zheng (3):
  file-posix: Skip effectiveless OFD lock operations
  file-posix: Drop s->lock_fd
  tests: Add unit tests for image locking

 block/file-posix.c         |  83 ++++++++++++--------
 tests/Makefile.include     |   2 +
 tests/test-image-locking.c | 154 +++++++++++++++++++++++++++++++++++++
 3 files changed, 209 insertions(+), 30 deletions(-)
 create mode 100644 tests/test-image-locking.c

-- 
2.17.1

Re: [Qemu-devel] [PATCH v4 0/3] file-posix: Simplifications on image locking

Posted by Fam Zheng 7 years, 1 month ago

On Tue, 08/21 08:58, Fam Zheng wrote:
> v4: Fix test on systems without OFD. [Patchew]

Ping?

> 
> The first patch reduces chances of QEMU crash in unusual (but not unlikely)
> cases especially when used by Libvirt (see commit message).
> 
> The second patch halves fd for images.
> 
> The third adds some more test for patch one (would have caught the regression
> caused by v2).
> 
> Fam Zheng (3):
>   file-posix: Skip effectiveless OFD lock operations
>   file-posix: Drop s->lock_fd
>   tests: Add unit tests for image locking
> 
>  block/file-posix.c         |  83 ++++++++++++--------
>  tests/Makefile.include     |   2 +
>  tests/test-image-locking.c | 154 +++++++++++++++++++++++++++++++++++++
>  3 files changed, 209 insertions(+), 30 deletions(-)
>  create mode 100644 tests/test-image-locking.c
> 
> -- 
> 2.17.1
> 
>