qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
* [Qemu-devel] [PATCH for-2.10? v2 0/2] block: Do OFD lock check at runtime
@ 2017-08-11  9:53 Fam Zheng
  2017-08-11  9:53 ` [Qemu-devel] [PATCH for-2.10? v2 1/2] osdep: Add runtime OFD lock detection Fam Zheng
                   ` (2 more replies)
  0 siblings, 3 replies; 4+ messages in thread
From: Fam Zheng @ 2017-08-11  9:53 UTC (permalink / raw)
  To: qemu-devel
  Cc: berrange, qemu-block, eblake, Kevin Wolf, Max Reitz,
	christian.ehrhardt, Andrew Baumann, jsnow

v2: Probe /dev/null to save LOC. [Eric]
    Mention "new glibc + old kernel" in commit message. [Kevin, Daniel, Eric,
    Christian]

This fixes the image opening failure reported by Andrew Baumann:

> I'm running a recent Linux build of qemu on Windows Subsystem for Linux (WSL)
> which doesn't appear to implement file locking:
>
> $ qemu-system-aarch64 ... -drive file=test.vhdx,if=none,id=hd0 -device virtio-blk-pci,drive=hd0
> qemu-system-aarch64: -drive file=test.vhdx,if=none,id=hd0: Failed to unlock byte 100
> qemu-system-aarch64: -drive file=test.vhdx,if=none,id=hd0: Failed to unlock byte 100
> qemu-system-aarch64: -drive file=test.vhdx,if=none,id=hd0: Failed to lock byte 100

It appears to be that the binary is built for Linux targets, but the WSL
runtime doesn't recognize the ops (-EINVAL).

This is also a practical problem for Linux considering it's possible to run
QEMU which is built against a new glibc on an old kernel that doesn't have OFD
API.

Convert to runtime check to cope with that.

Fam Zheng (2):
  osdep: Add runtime OFD lock detection
  file-posix: Do runtime check for ofd lock API

 block/file-posix.c   | 19 +++++++---------
 include/qemu/osdep.h |  1 +
 util/osdep.c         | 63 ++++++++++++++++++++++++++++++++++++++++++++--------
 3 files changed, 63 insertions(+), 20 deletions(-)

-- 
2.13.4

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2017-08-11 10:12 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-08-11  9:53 [Qemu-devel] [PATCH for-2.10? v2 0/2] block: Do OFD lock check at runtime Fam Zheng
2017-08-11  9:53 ` [Qemu-devel] [PATCH for-2.10? v2 1/2] osdep: Add runtime OFD lock detection Fam Zheng
2017-08-11  9:53 ` [Qemu-devel] [PATCH for-2.10? v2 2/2] file-posix: Do runtime check for ofd lock API Fam Zheng
2017-08-11 10:11 ` [Qemu-devel] [PATCH for-2.10? v2 0/2] block: Do OFD lock check at runtime no-reply

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).