1
The following changes since commit 248b23735645f7cbb503d9be6f5bf825f2a603ab:
1
The following changes since commit 8e6c70b9d4a1b1f3011805947925cfdb31642f7f:
2
2
3
Update version for v2.10.0-rc4 release (2017-08-24 17:34:26 +0100)
3
Merge tag 'kraxel-20220614-pull-request' of git://git.kraxel.org/qemu into staging (2022-06-14 06:21:46 -0700)
4
4
5
are available in the git repository at:
5
are available in the Git repository at:
6
6
7
git://github.com/stefanha/qemu.git tags/block-pull-request
7
https://gitlab.com/stefanha/qemu.git tags/block-pull-request
8
8
9
for you to fetch changes up to 3e4c705212abfe8c9882a00beb2d1466a8a53cec:
9
for you to fetch changes up to 99b969fbe105117f5af6060d3afef40ca39cc9c1:
10
10
11
qcow2: allocate cluster_cache/cluster_data on demand (2017-08-30 18:02:10 +0100)
11
linux-aio: explain why max batch is checked in laio_io_unplug() (2022-06-15 16:43:42 +0100)
12
13
----------------------------------------------------------------
14
Pull request
15
16
This pull request includes an important aio=native I/O stall fix, the
17
experimental vifo-user server, the io_uring_register_ring_fd() optimization for
18
aio=io_uring, and an update to Vladimir Sementsov-Ogievskiy's maintainership
19
details.
12
20
13
----------------------------------------------------------------
21
----------------------------------------------------------------
14
22
15
----------------------------------------------------------------
23
Jagannathan Raman (14):
24
qdev: unplug blocker for devices
25
remote/machine: add HotplugHandler for remote machine
26
remote/machine: add vfio-user property
27
vfio-user: build library
28
vfio-user: define vfio-user-server object
29
vfio-user: instantiate vfio-user context
30
vfio-user: find and init PCI device
31
vfio-user: run vfio-user context
32
vfio-user: handle PCI config space accesses
33
vfio-user: IOMMU support for remote device
34
vfio-user: handle DMA mappings
35
vfio-user: handle PCI BAR accesses
36
vfio-user: handle device interrupts
37
vfio-user: handle reset of remote device
16
38
17
Alberto Garcia (8):
39
Sam Li (1):
18
throttle: Fix wrong variable name in the header documentation
40
Use io_uring_register_ring_fd() to skip fd operations
19
throttle: Update the throttle_fix_bucket() documentation
20
throttle: Make throttle_is_valid() a bit less verbose
21
throttle: Remove throttle_fix_bucket() / throttle_unfix_bucket()
22
throttle: Make LeakyBucket.avg and LeakyBucket.max integer types
23
throttle: Make burst_length 64bit and add range checks
24
throttle: Test the valid range of config values
25
misc: Remove unused Error variables
26
41
27
Dan Aloni (1):
42
Stefan Hajnoczi (2):
28
nvme: Fix get/set number of queues feature, again
43
linux-aio: fix unbalanced plugged counter in laio_io_unplug()
44
linux-aio: explain why max batch is checked in laio_io_unplug()
29
45
30
Eduardo Habkost (1):
46
Vladimir Sementsov-Ogievskiy (1):
31
oslib-posix: Print errors before aborting on qemu_alloc_stack()
47
MAINTAINERS: update Vladimir's address and repositories
32
48
33
Fred Rolland (1):
49
MAINTAINERS | 27 +-
34
qemu-doc: Add UUID support in initiator name
50
meson_options.txt | 2 +
35
51
qapi/misc.json | 31 +
36
Stefan Hajnoczi (4):
52
qapi/qom.json | 20 +-
37
scripts: add argparse module for Python 2.6 compatibility
53
configure | 17 +
38
docker.py: Python 2.6 argparse compatibility
54
meson.build | 24 +-
39
tests: migration/guestperf Python 2.6 argparse compatibility
55
include/exec/memory.h | 3 +
40
qcow2: allocate cluster_cache/cluster_data on demand
56
include/hw/pci/msi.h | 1 +
41
57
include/hw/pci/msix.h | 1 +
42
include/qemu/throttle.h | 8 +-
58
include/hw/pci/pci.h | 13 +
43
block/qcow.c | 12 +-
59
include/hw/qdev-core.h | 29 +
44
block/qcow2-cluster.c | 17 +
60
include/hw/remote/iommu.h | 40 +
45
block/qcow2.c | 20 +-
61
include/hw/remote/machine.h | 4 +
46
dump.c | 4 +-
62
include/hw/remote/vfio-user-obj.h | 6 +
47
hw/block/nvme.c | 4 +-
63
block/io_uring.c | 12 +-
48
tests/test-throttle.c | 80 +-
64
block/linux-aio.c | 10 +-
49
util/oslib-posix.c | 2 +
65
hw/core/qdev.c | 24 +
50
util/throttle.c | 86 +-
66
hw/pci/msi.c | 49 +-
51
COPYING.PYTHON | 270 ++++
67
hw/pci/msix.c | 35 +-
52
qemu-doc.texi | 5 +-
68
hw/pci/pci.c | 13 +
53
scripts/argparse.py | 2406 ++++++++++++++++++++++++++++++++++++
69
hw/remote/iommu.c | 131 ++++
54
tests/docker/docker.py | 4 +-
70
hw/remote/machine.c | 88 ++-
55
tests/migration/guestperf/shell.py | 8 +-
71
hw/remote/vfio-user-obj.c | 958 ++++++++++++++++++++++++
56
14 files changed, 2831 insertions(+), 95 deletions(-)
72
softmmu/physmem.c | 4 +-
57
create mode 100644 COPYING.PYTHON
73
softmmu/qdev-monitor.c | 4 +
58
create mode 100644 scripts/argparse.py
74
stubs/vfio-user-obj.c | 6 +
75
tests/qtest/fuzz/generic_fuzz.c | 9 +-
76
.gitlab-ci.d/buildtest.yml | 1 +
77
.gitmodules | 3 +
78
Kconfig.host | 4 +
79
hw/remote/Kconfig | 4 +
80
hw/remote/meson.build | 4 +
81
hw/remote/trace-events | 11 +
82
scripts/meson-buildoptions.sh | 4 +
83
stubs/meson.build | 1 +
84
subprojects/libvfio-user | 1 +
85
tests/docker/dockerfiles/centos8.docker | 2 +
86
37 files changed, 1565 insertions(+), 31 deletions(-)
87
create mode 100644 include/hw/remote/iommu.h
88
create mode 100644 include/hw/remote/vfio-user-obj.h
89
create mode 100644 hw/remote/iommu.c
90
create mode 100644 hw/remote/vfio-user-obj.c
91
create mode 100644 stubs/vfio-user-obj.c
92
create mode 160000 subprojects/libvfio-user
59
93
60
--
94
--
61
2.13.5
95
2.36.1
62
63
diff view generated by jsdifflib
1
From: Fred Rolland <rollandf@gmail.com>
1
From: Vladimir Sementsov-Ogievskiy <vsementsov@yandex-team.ru>
2
2
3
Update doc with the usage of UUID for initiator name.
3
Signed-off-by: Vladimir Sementsov-Ogievskiy <vsementsov@yandex-team.ru>
4
4
Message-id: 20220526115432.138384-1-vsementsov@yandex-team.ru
5
Related-To: https://bugzilla.redhat.com/1006468
6
Signed-off-by: Fred Rolland <frolland@redhat.com>
7
Message-id: 20170823084830.30500-1-frolland@redhat.com
8
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
5
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
9
---
6
---
10
qemu-doc.texi | 5 +++--
7
MAINTAINERS | 22 ++++++++++++----------
11
1 file changed, 3 insertions(+), 2 deletions(-)
8
1 file changed, 12 insertions(+), 10 deletions(-)
12
9
13
diff --git a/qemu-doc.texi b/qemu-doc.texi
10
diff --git a/MAINTAINERS b/MAINTAINERS
14
index XXXXXXX..XXXXXXX 100644
11
index XXXXXXX..XXXXXXX 100644
15
--- a/qemu-doc.texi
12
--- a/MAINTAINERS
16
+++ b/qemu-doc.texi
13
+++ b/MAINTAINERS
17
@@ -XXX,XX +XXX,XX @@ in a configuration file provided via '-readconfig' or directly on the
14
@@ -XXX,XX +XXX,XX @@ F: scsi/*
18
command line.
15
19
16
Block Jobs
20
If the initiator-name is not specified qemu will use a default name
17
M: John Snow <jsnow@redhat.com>
21
-of 'iqn.2008-11.org.linux-kvm[:<name>'] where <name> is the name of the
18
-M: Vladimir Sementsov-Ogievskiy <v.sementsov-og@mail.ru>
22
+of 'iqn.2008-11.org.linux-kvm[:<uuid>'] where <uuid> is the UUID of the
19
+M: Vladimir Sementsov-Ogievskiy <vsementsov@yandex-team.ru>
23
+virtual machine. If the UUID is not specified qemu will use
20
L: qemu-block@nongnu.org
24
+'iqn.2008-11.org.linux-kvm[:<name>'] where <name> is the name of the
21
S: Supported
25
virtual machine.
22
F: blockjob.c
26
23
@@ -XXX,XX +XXX,XX @@ F: block/aio_task.c
27
-
24
F: util/qemu-co-shared-resource.c
28
@example
25
F: include/qemu/co-shared-resource.h
29
Setting a specific initiator name to use when logging in to the target
26
T: git https://gitlab.com/jsnow/qemu.git jobs
30
-iscsi initiator-name=iqn.qemu.test:my-initiator
27
-T: git https://src.openvz.org/scm/~vsementsov/qemu.git jobs
28
+T: git https://gitlab.com/vsementsov/qemu.git block
29
30
Block QAPI, monitor, command line
31
M: Markus Armbruster <armbru@redhat.com>
32
@@ -XXX,XX +XXX,XX @@ F: include/hw/cxl/
33
34
Dirty Bitmaps
35
M: Eric Blake <eblake@redhat.com>
36
-M: Vladimir Sementsov-Ogievskiy <v.sementsov-og@mail.ru>
37
+M: Vladimir Sementsov-Ogievskiy <vsementsov@yandex-team.ru>
38
R: John Snow <jsnow@redhat.com>
39
L: qemu-block@nongnu.org
40
S: Supported
41
@@ -XXX,XX +XXX,XX @@ F: util/hbitmap.c
42
F: tests/unit/test-hbitmap.c
43
F: docs/interop/bitmaps.rst
44
T: git https://repo.or.cz/qemu/ericb.git bitmaps
45
+T: git https://gitlab.com/vsementsov/qemu.git block
46
47
Character device backends
48
M: Marc-André Lureau <marcandre.lureau@redhat.com>
49
@@ -XXX,XX +XXX,XX @@ F: scripts/*.py
50
F: tests/*.py
51
52
Benchmark util
53
-M: Vladimir Sementsov-Ogievskiy <v.sementsov-og@mail.ru>
54
+M: Vladimir Sementsov-Ogievskiy <vsementsov@yandex-team.ru>
55
S: Maintained
56
F: scripts/simplebench/
57
-T: git https://src.openvz.org/scm/~vsementsov/qemu.git simplebench
58
+T: git https://gitlab.com/vsementsov/qemu.git simplebench
59
60
Transactions helper
61
-M: Vladimir Sementsov-Ogievskiy <v.sementsov-og@mail.ru>
62
+M: Vladimir Sementsov-Ogievskiy <vsementsov@yandex-team.ru>
63
S: Maintained
64
F: include/qemu/transactions.h
65
F: util/transactions.c
66
+T: git https://gitlab.com/vsementsov/qemu.git block
67
68
QAPI
69
M: Markus Armbruster <armbru@redhat.com>
70
@@ -XXX,XX +XXX,XX @@ F: block/iscsi-opts.c
71
72
Network Block Device (NBD)
73
M: Eric Blake <eblake@redhat.com>
74
-M: Vladimir Sementsov-Ogievskiy <v.sementsov-og@mail.ru>
75
+M: Vladimir Sementsov-Ogievskiy <vsementsov@yandex-team.ru>
76
L: qemu-block@nongnu.org
77
S: Maintained
78
F: block/nbd*
79
@@ -XXX,XX +XXX,XX @@ F: docs/interop/nbd.txt
80
F: docs/tools/qemu-nbd.rst
81
F: tests/qemu-iotests/tests/*nbd*
82
T: git https://repo.or.cz/qemu/ericb.git nbd
83
-T: git https://src.openvz.org/scm/~vsementsov/qemu.git nbd
84
+T: git https://gitlab.com/vsementsov/qemu.git block
85
86
NFS
87
M: Peter Lieven <pl@kamp.de>
88
@@ -XXX,XX +XXX,XX @@ F: block/dmg.c
89
parallels
90
M: Stefan Hajnoczi <stefanha@redhat.com>
91
M: Denis V. Lunev <den@openvz.org>
92
-M: Vladimir Sementsov-Ogievskiy <v.sementsov-og@mail.ru>
93
+M: Vladimir Sementsov-Ogievskiy <vsementsov@yandex-team.ru>
94
L: qemu-block@nongnu.org
95
S: Supported
96
F: block/parallels.c
97
F: block/parallels-ext.c
98
F: docs/interop/parallels.txt
99
-T: git https://src.openvz.org/scm/~vsementsov/qemu.git parallels
100
+T: git https://gitlab.com/vsementsov/qemu.git block
101
102
qed
103
M: Stefan Hajnoczi <stefanha@redhat.com>
31
--
104
--
32
2.13.5
105
2.36.1
33
106
34
107
diff view generated by jsdifflib
1
From: Eduardo Habkost <ehabkost@redhat.com>
1
From: Sam Li <faithilikerun@gmail.com>
2
2
3
If QEMU is running on a system that's out of memory and mmap()
3
Linux recently added a new io_uring(7) optimization API that QEMU
4
fails, QEMU aborts with no error message at all, making it hard
4
doesn't take advantage of yet. The liburing library that QEMU uses
5
to debug the reason for the failure.
5
has added a corresponding new API calling io_uring_register_ring_fd().
6
When this API is called after creating the ring, the io_uring_submit()
7
library function passes a flag to the io_uring_enter(2) syscall
8
allowing it to skip the ring file descriptor fdget()/fdput()
9
operations. This saves some CPU cycles.
6
10
7
Add perror() calls that will print error information before
11
Signed-off-by: Sam Li <faithilikerun@gmail.com>
8
aborting.
12
Message-id: 20220531105011.111082-1-faithilikerun@gmail.com
9
10
Signed-off-by: Eduardo Habkost <ehabkost@redhat.com>
11
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
12
Tested-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
13
Message-id: 20170829212053.6003-1-ehabkost@redhat.com
14
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
13
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
15
---
14
---
16
util/oslib-posix.c | 2 ++
15
meson.build | 1 +
17
1 file changed, 2 insertions(+)
16
block/io_uring.c | 12 +++++++++++-
17
2 files changed, 12 insertions(+), 1 deletion(-)
18
18
19
diff --git a/util/oslib-posix.c b/util/oslib-posix.c
19
diff --git a/meson.build b/meson.build
20
index XXXXXXX..XXXXXXX 100644
20
index XXXXXXX..XXXXXXX 100644
21
--- a/util/oslib-posix.c
21
--- a/meson.build
22
+++ b/util/oslib-posix.c
22
+++ b/meson.build
23
@@ -XXX,XX +XXX,XX @@ void *qemu_alloc_stack(size_t *sz)
23
@@ -XXX,XX +XXX,XX @@ config_host_data.set('CONFIG_LIBNFS', libnfs.found())
24
ptr = mmap(NULL, *sz, PROT_READ | PROT_WRITE,
24
config_host_data.set('CONFIG_LIBSSH', libssh.found())
25
MAP_PRIVATE | MAP_ANONYMOUS, -1, 0);
25
config_host_data.set('CONFIG_LINUX_AIO', libaio.found())
26
if (ptr == MAP_FAILED) {
26
config_host_data.set('CONFIG_LINUX_IO_URING', linux_io_uring.found())
27
+ perror("failed to allocate memory for stack");
27
+config_host_data.set('CONFIG_LIBURING_REGISTER_RING_FD', cc.has_function('io_uring_register_ring_fd', prefix: '#include <liburing.h>', dependencies:linux_io_uring))
28
abort();
28
config_host_data.set('CONFIG_LIBPMEM', libpmem.found())
29
config_host_data.set('CONFIG_NUMA', numa.found())
30
config_host_data.set('CONFIG_OPENGL', opengl.found())
31
diff --git a/block/io_uring.c b/block/io_uring.c
32
index XXXXXXX..XXXXXXX 100644
33
--- a/block/io_uring.c
34
+++ b/block/io_uring.c
35
@@ -XXX,XX +XXX,XX @@
36
#include "qapi/error.h"
37
#include "trace.h"
38
39
+
40
/* io_uring ring size */
41
#define MAX_ENTRIES 128
42
43
@@ -XXX,XX +XXX,XX @@ LuringState *luring_init(Error **errp)
29
}
44
}
30
45
31
@@ -XXX,XX +XXX,XX @@ void *qemu_alloc_stack(size_t *sz)
46
ioq_init(&s->io_q);
32
guardpage = ptr;
47
+#ifdef CONFIG_LIBURING_REGISTER_RING_FD
33
#endif
48
+ if (io_uring_register_ring_fd(&s->ring) < 0) {
34
if (mprotect(guardpage, pagesz, PROT_NONE) != 0) {
49
+ /*
35
+ perror("failed to set up stack guard page");
50
+ * Only warn about this error: we will fallback to the non-optimized
36
abort();
51
+ * io_uring operations.
37
}
52
+ */
38
53
+ warn_report("failed to register linux io_uring ring file descriptor");
54
+ }
55
+#endif
56
+
57
return s;
58
-
59
}
60
61
void luring_cleanup(LuringState *s)
39
--
62
--
40
2.13.5
63
2.36.1
41
42
diff view generated by jsdifflib
1
From: Alberto Garcia <berto@igalia.com>
1
From: Jagannathan Raman <jag.raman@oracle.com>
2
2
3
There's a few cases which we're passing an Error pointer to a function
3
Add blocker to prevent hot-unplug of devices
4
only to discard it immediately afterwards without checking it. In
5
these cases we can simply remove the variable and pass NULL instead.
6
4
7
Signed-off-by: Alberto Garcia <berto@igalia.com>
5
TYPE_VFIO_USER_SERVER, which is introduced shortly, attaches itself to a
8
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
6
PCIDevice on which it depends. If the attached PCIDevice gets removed
9
Reviewed-by: Eric Blake <eblake@redhat.com>
7
while the server in use, it could cause it crash. To prevent this,
10
Message-id: 20170829120836.16091-1-berto@igalia.com
8
TYPE_VFIO_USER_SERVER adds an unplug blocker for the PCIDevice.
9
10
Signed-off-by: Elena Ufimtseva <elena.ufimtseva@oracle.com>
11
Signed-off-by: John G Johnson <john.g.johnson@oracle.com>
12
Signed-off-by: Jagannathan Raman <jag.raman@oracle.com>
13
Reviewed-by: Stefan Hajnoczi <stefanha@redhat.com>
14
Message-id: c41ef80b7cc063314d629737bed2159e5713f2e0.1655151679.git.jag.raman@oracle.com
11
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
15
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
12
---
16
---
13
block/qcow.c | 12 +++---------
17
include/hw/qdev-core.h | 29 +++++++++++++++++++++++++++++
14
block/qcow2.c | 8 ++------
18
hw/core/qdev.c | 24 ++++++++++++++++++++++++
15
dump.c | 4 +---
19
softmmu/qdev-monitor.c | 4 ++++
16
3 files changed, 6 insertions(+), 18 deletions(-)
20
3 files changed, 57 insertions(+)
17
21
18
diff --git a/block/qcow.c b/block/qcow.c
22
diff --git a/include/hw/qdev-core.h b/include/hw/qdev-core.h
19
index XXXXXXX..XXXXXXX 100644
23
index XXXXXXX..XXXXXXX 100644
20
--- a/block/qcow.c
24
--- a/include/hw/qdev-core.h
21
+++ b/block/qcow.c
25
+++ b/include/hw/qdev-core.h
22
@@ -XXX,XX +XXX,XX @@ static uint64_t get_cluster_offset(BlockDriverState *bs,
26
@@ -XXX,XX +XXX,XX @@ struct DeviceState {
23
start_sect = (offset & ~(s->cluster_size - 1)) >> 9;
27
int instance_id_alias;
24
for(i = 0; i < s->cluster_sectors; i++) {
28
int alias_required_for_version;
25
if (i < n_start || i >= n_end) {
29
ResettableState reset;
26
- Error *err = NULL;
30
+ GSList *unplug_blockers;
27
memset(s->cluster_data, 0x00, 512);
31
};
28
if (qcrypto_block_encrypt(s->crypto, start_sect + i,
32
29
s->cluster_data,
33
struct DeviceListener {
30
BDRV_SECTOR_SIZE,
34
@@ -XXX,XX +XXX,XX @@ void qdev_simple_device_unplug_cb(HotplugHandler *hotplug_dev,
31
- &err) < 0) {
35
void qdev_machine_creation_done(void);
32
- error_free(err);
36
bool qdev_machine_modified(void);
33
+ NULL) < 0) {
37
34
errno = EIO;
38
+/**
35
return -1;
39
+ * qdev_add_unplug_blocker: Add an unplug blocker to a device
36
}
40
+ *
37
@@ -XXX,XX +XXX,XX @@ static coroutine_fn int qcow_co_readv(BlockDriverState *bs, int64_t sector_num,
41
+ * @dev: Device to be blocked from unplug
38
QEMUIOVector hd_qiov;
42
+ * @reason: Reason for blocking
39
uint8_t *buf;
43
+ */
40
void *orig_buf;
44
+void qdev_add_unplug_blocker(DeviceState *dev, Error *reason);
41
- Error *err = NULL;
45
+
42
46
+/**
43
if (qiov->niov > 1) {
47
+ * qdev_del_unplug_blocker: Remove an unplug blocker from a device
44
buf = orig_buf = qemu_try_blockalign(bs, qiov->size);
48
+ *
45
@@ -XXX,XX +XXX,XX @@ static coroutine_fn int qcow_co_readv(BlockDriverState *bs, int64_t sector_num,
49
+ * @dev: Device to be unblocked
46
if (bs->encrypted) {
50
+ * @reason: Pointer to the Error used with qdev_add_unplug_blocker.
47
assert(s->crypto);
51
+ * Used as a handle to lookup the blocker for deletion.
48
if (qcrypto_block_decrypt(s->crypto, sector_num, buf,
52
+ */
49
- n * BDRV_SECTOR_SIZE, &err) < 0) {
53
+void qdev_del_unplug_blocker(DeviceState *dev, Error *reason);
50
+ n * BDRV_SECTOR_SIZE, NULL) < 0) {
54
+
51
goto fail;
55
+/**
52
}
56
+ * qdev_unplug_blocked: Confirm if a device is blocked from unplug
53
}
57
+ *
54
@@ -XXX,XX +XXX,XX @@ done:
58
+ * @dev: Device to be tested
55
return ret;
59
+ * @reason: Returns one of the reasons why the device is blocked,
56
60
+ * if any
57
fail:
61
+ *
58
- error_free(err);
62
+ * Returns: true if device is blocked from unplug, false otherwise
59
ret = -EIO;
63
+ */
60
goto done;
64
+bool qdev_unplug_blocked(DeviceState *dev, Error **errp);
61
}
65
+
62
@@ -XXX,XX +XXX,XX @@ static coroutine_fn int qcow_co_writev(BlockDriverState *bs, int64_t sector_num,
66
/**
63
break;
67
* GpioPolarity: Polarity of a GPIO line
64
}
68
*
65
if (bs->encrypted) {
69
diff --git a/hw/core/qdev.c b/hw/core/qdev.c
66
- Error *err = NULL;
67
assert(s->crypto);
68
if (qcrypto_block_encrypt(s->crypto, sector_num, buf,
69
- n * BDRV_SECTOR_SIZE, &err) < 0) {
70
- error_free(err);
71
+ n * BDRV_SECTOR_SIZE, NULL) < 0) {
72
ret = -EIO;
73
break;
74
}
75
diff --git a/block/qcow2.c b/block/qcow2.c
76
index XXXXXXX..XXXXXXX 100644
70
index XXXXXXX..XXXXXXX 100644
77
--- a/block/qcow2.c
71
--- a/hw/core/qdev.c
78
+++ b/block/qcow2.c
72
+++ b/hw/core/qdev.c
79
@@ -XXX,XX +XXX,XX @@ static coroutine_fn int qcow2_co_preadv(BlockDriverState *bs, uint64_t offset,
73
@@ -XXX,XX +XXX,XX @@ char *qdev_get_dev_path(DeviceState *dev)
80
assert(s->crypto);
81
assert((offset & (BDRV_SECTOR_SIZE - 1)) == 0);
82
assert((cur_bytes & (BDRV_SECTOR_SIZE - 1)) == 0);
83
- Error *err = NULL;
84
if (qcrypto_block_decrypt(s->crypto,
85
(s->crypt_physical_offset ?
86
cluster_offset + offset_in_cluster :
87
offset) >> BDRV_SECTOR_BITS,
88
cluster_data,
89
cur_bytes,
90
- &err) < 0) {
91
- error_free(err);
92
+ NULL) < 0) {
93
ret = -EIO;
94
goto fail;
95
}
96
@@ -XXX,XX +XXX,XX @@ static coroutine_fn int qcow2_co_pwritev(BlockDriverState *bs, uint64_t offset,
97
qemu_iovec_concat(&hd_qiov, qiov, bytes_done, cur_bytes);
98
99
if (bs->encrypted) {
100
- Error *err = NULL;
101
assert(s->crypto);
102
if (!cluster_data) {
103
cluster_data = qemu_try_blockalign(bs->file->bs,
104
@@ -XXX,XX +XXX,XX @@ static coroutine_fn int qcow2_co_pwritev(BlockDriverState *bs, uint64_t offset,
105
cluster_offset + offset_in_cluster :
106
offset) >> BDRV_SECTOR_BITS,
107
cluster_data,
108
- cur_bytes, &err) < 0) {
109
- error_free(err);
110
+ cur_bytes, NULL) < 0) {
111
ret = -EIO;
112
goto fail;
113
}
114
diff --git a/dump.c b/dump.c
115
index XXXXXXX..XXXXXXX 100644
116
--- a/dump.c
117
+++ b/dump.c
118
@@ -XXX,XX +XXX,XX @@ static void dump_process(DumpState *s, Error **errp)
119
120
static void *dump_thread(void *data)
121
{
122
- Error *err = NULL;
123
DumpState *s = (DumpState *)data;
124
- dump_process(s, &err);
125
- error_free(err);
126
+ dump_process(s, NULL);
127
return NULL;
74
return NULL;
128
}
75
}
129
76
77
+void qdev_add_unplug_blocker(DeviceState *dev, Error *reason)
78
+{
79
+ dev->unplug_blockers = g_slist_prepend(dev->unplug_blockers, reason);
80
+}
81
+
82
+void qdev_del_unplug_blocker(DeviceState *dev, Error *reason)
83
+{
84
+ dev->unplug_blockers = g_slist_remove(dev->unplug_blockers, reason);
85
+}
86
+
87
+bool qdev_unplug_blocked(DeviceState *dev, Error **errp)
88
+{
89
+ ERRP_GUARD();
90
+
91
+ if (dev->unplug_blockers) {
92
+ error_propagate(errp, error_copy(dev->unplug_blockers->data));
93
+ return true;
94
+ }
95
+
96
+ return false;
97
+}
98
+
99
static bool device_get_realized(Object *obj, Error **errp)
100
{
101
DeviceState *dev = DEVICE(obj);
102
@@ -XXX,XX +XXX,XX @@ static void device_finalize(Object *obj)
103
104
DeviceState *dev = DEVICE(obj);
105
106
+ g_assert(!dev->unplug_blockers);
107
+
108
QLIST_FOREACH_SAFE(ngl, &dev->gpios, node, next) {
109
QLIST_REMOVE(ngl, node);
110
qemu_free_irqs(ngl->in, ngl->num_in);
111
diff --git a/softmmu/qdev-monitor.c b/softmmu/qdev-monitor.c
112
index XXXXXXX..XXXXXXX 100644
113
--- a/softmmu/qdev-monitor.c
114
+++ b/softmmu/qdev-monitor.c
115
@@ -XXX,XX +XXX,XX @@ void qdev_unplug(DeviceState *dev, Error **errp)
116
HotplugHandlerClass *hdc;
117
Error *local_err = NULL;
118
119
+ if (qdev_unplug_blocked(dev, errp)) {
120
+ return;
121
+ }
122
+
123
if (dev->parent_bus && !qbus_is_hotpluggable(dev->parent_bus)) {
124
error_setg(errp, QERR_BUS_NO_HOTPLUG, dev->parent_bus->name);
125
return;
130
--
126
--
131
2.13.5
127
2.36.1
132
133
diff view generated by jsdifflib
New patch
1
From: Jagannathan Raman <jag.raman@oracle.com>
1
2
3
Allow hotplugging of PCI(e) devices to remote machine
4
5
Signed-off-by: Elena Ufimtseva <elena.ufimtseva@oracle.com>
6
Signed-off-by: John G Johnson <john.g.johnson@oracle.com>
7
Signed-off-by: Jagannathan Raman <jag.raman@oracle.com>
8
Reviewed-by: Stefan Hajnoczi <stefanha@redhat.com>
9
Message-id: d1e6cfa0afb528ad343758f9b1d918be0175c5e5.1655151679.git.jag.raman@oracle.com
10
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
11
---
12
hw/remote/machine.c | 10 ++++++++++
13
1 file changed, 10 insertions(+)
14
15
diff --git a/hw/remote/machine.c b/hw/remote/machine.c
16
index XXXXXXX..XXXXXXX 100644
17
--- a/hw/remote/machine.c
18
+++ b/hw/remote/machine.c
19
@@ -XXX,XX +XXX,XX @@
20
#include "qapi/error.h"
21
#include "hw/pci/pci_host.h"
22
#include "hw/remote/iohub.h"
23
+#include "hw/qdev-core.h"
24
25
static void remote_machine_init(MachineState *machine)
26
{
27
@@ -XXX,XX +XXX,XX @@ static void remote_machine_init(MachineState *machine)
28
29
pci_bus_irqs(pci_host->bus, remote_iohub_set_irq, remote_iohub_map_irq,
30
&s->iohub, REMOTE_IOHUB_NB_PIRQS);
31
+
32
+ qbus_set_hotplug_handler(BUS(pci_host->bus), OBJECT(s));
33
}
34
35
static void remote_machine_class_init(ObjectClass *oc, void *data)
36
{
37
MachineClass *mc = MACHINE_CLASS(oc);
38
+ HotplugHandlerClass *hc = HOTPLUG_HANDLER_CLASS(oc);
39
40
mc->init = remote_machine_init;
41
mc->desc = "Experimental remote machine";
42
+
43
+ hc->unplug = qdev_simple_device_unplug_cb;
44
}
45
46
static const TypeInfo remote_machine = {
47
@@ -XXX,XX +XXX,XX @@ static const TypeInfo remote_machine = {
48
.parent = TYPE_MACHINE,
49
.instance_size = sizeof(RemoteMachineState),
50
.class_init = remote_machine_class_init,
51
+ .interfaces = (InterfaceInfo[]) {
52
+ { TYPE_HOTPLUG_HANDLER },
53
+ { }
54
+ }
55
};
56
57
static void remote_machine_register_types(void)
58
--
59
2.36.1
diff view generated by jsdifflib
New patch
1
From: Jagannathan Raman <jag.raman@oracle.com>
1
2
3
Add vfio-user to x-remote machine. It is a boolean, which indicates if
4
the machine supports vfio-user protocol. The machine configures the bus
5
differently vfio-user and multiprocess protocols, so this property
6
informs it on how to configure the bus.
7
8
This property should be short lived. Once vfio-user fully replaces
9
multiprocess, this property could be removed.
10
11
Signed-off-by: Elena Ufimtseva <elena.ufimtseva@oracle.com>
12
Signed-off-by: John G Johnson <john.g.johnson@oracle.com>
13
Signed-off-by: Jagannathan Raman <jag.raman@oracle.com>
14
Reviewed-by: Stefan Hajnoczi <stefanha@redhat.com>
15
Message-id: 5d51a152a419cbda35d070b8e49b772b60a7230a.1655151679.git.jag.raman@oracle.com
16
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
17
---
18
include/hw/remote/machine.h | 2 ++
19
hw/remote/machine.c | 23 +++++++++++++++++++++++
20
2 files changed, 25 insertions(+)
21
22
diff --git a/include/hw/remote/machine.h b/include/hw/remote/machine.h
23
index XXXXXXX..XXXXXXX 100644
24
--- a/include/hw/remote/machine.h
25
+++ b/include/hw/remote/machine.h
26
@@ -XXX,XX +XXX,XX @@ struct RemoteMachineState {
27
28
RemotePCIHost *host;
29
RemoteIOHubState iohub;
30
+
31
+ bool vfio_user;
32
};
33
34
/* Used to pass to co-routine device and ioc. */
35
diff --git a/hw/remote/machine.c b/hw/remote/machine.c
36
index XXXXXXX..XXXXXXX 100644
37
--- a/hw/remote/machine.c
38
+++ b/hw/remote/machine.c
39
@@ -XXX,XX +XXX,XX @@ static void remote_machine_init(MachineState *machine)
40
qbus_set_hotplug_handler(BUS(pci_host->bus), OBJECT(s));
41
}
42
43
+static bool remote_machine_get_vfio_user(Object *obj, Error **errp)
44
+{
45
+ RemoteMachineState *s = REMOTE_MACHINE(obj);
46
+
47
+ return s->vfio_user;
48
+}
49
+
50
+static void remote_machine_set_vfio_user(Object *obj, bool value, Error **errp)
51
+{
52
+ RemoteMachineState *s = REMOTE_MACHINE(obj);
53
+
54
+ if (phase_check(PHASE_MACHINE_CREATED)) {
55
+ error_setg(errp, "Error enabling vfio-user - machine already created");
56
+ return;
57
+ }
58
+
59
+ s->vfio_user = value;
60
+}
61
+
62
static void remote_machine_class_init(ObjectClass *oc, void *data)
63
{
64
MachineClass *mc = MACHINE_CLASS(oc);
65
@@ -XXX,XX +XXX,XX @@ static void remote_machine_class_init(ObjectClass *oc, void *data)
66
mc->desc = "Experimental remote machine";
67
68
hc->unplug = qdev_simple_device_unplug_cb;
69
+
70
+ object_class_property_add_bool(oc, "vfio-user",
71
+ remote_machine_get_vfio_user,
72
+ remote_machine_set_vfio_user);
73
}
74
75
static const TypeInfo remote_machine = {
76
--
77
2.36.1
diff view generated by jsdifflib
New patch
1
1
From: Jagannathan Raman <jag.raman@oracle.com>
2
3
add the libvfio-user library as a submodule. build it as a meson
4
subproject.
5
6
libvfio-user is distributed with BSD 3-Clause license and
7
json-c with MIT (Expat) license
8
9
Signed-off-by: Elena Ufimtseva <elena.ufimtseva@oracle.com>
10
Signed-off-by: John G Johnson <john.g.johnson@oracle.com>
11
Signed-off-by: Jagannathan Raman <jag.raman@oracle.com>
12
Reviewed-by: Stefan Hajnoczi <stefanha@redhat.com>
13
Message-id: c2adec87958b081d1dc8775d4aa05c897912f025.1655151679.git.jag.raman@oracle.com
14
15
[Changed submodule URL to QEMU's libvfio-user mirror on GitLab. The QEMU
16
project mirrors its dependencies so that it can provide full source code
17
even in the event that its dependencies become unavailable. Note that
18
the mirror repo is manually updated, so please contact me to make newer
19
libvfio-user commits available. If I become a bottleneck we can set up a
20
cronjob.
21
22
Updated scripts/meson-buildoptions.sh to match the meson_options.txt
23
change. Failure to do so can result in scripts/meson-buildoptions.sh
24
being modified by the build system later on and you end up with a dirty
25
working tree.
26
--Stefan]
27
28
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
29
---
30
MAINTAINERS | 1 +
31
meson_options.txt | 2 ++
32
configure | 17 +++++++++++++++++
33
meson.build | 23 ++++++++++++++++++++++-
34
.gitlab-ci.d/buildtest.yml | 1 +
35
.gitmodules | 3 +++
36
Kconfig.host | 4 ++++
37
hw/remote/Kconfig | 4 ++++
38
hw/remote/meson.build | 2 ++
39
scripts/meson-buildoptions.sh | 4 ++++
40
subprojects/libvfio-user | 1 +
41
tests/docker/dockerfiles/centos8.docker | 2 ++
42
12 files changed, 63 insertions(+), 1 deletion(-)
43
create mode 160000 subprojects/libvfio-user
44
45
diff --git a/MAINTAINERS b/MAINTAINERS
46
index XXXXXXX..XXXXXXX 100644
47
--- a/MAINTAINERS
48
+++ b/MAINTAINERS
49
@@ -XXX,XX +XXX,XX @@ F: hw/remote/proxy-memory-listener.c
50
F: include/hw/remote/proxy-memory-listener.h
51
F: hw/remote/iohub.c
52
F: include/hw/remote/iohub.h
53
+F: subprojects/libvfio-user
54
55
EBPF:
56
M: Jason Wang <jasowang@redhat.com>
57
diff --git a/meson_options.txt b/meson_options.txt
58
index XXXXXXX..XXXXXXX 100644
59
--- a/meson_options.txt
60
+++ b/meson_options.txt
61
@@ -XXX,XX +XXX,XX @@ option('cfi_debug', type: 'boolean', value: 'false',
62
description: 'Verbose errors in case of CFI violation')
63
option('multiprocess', type: 'feature', value: 'auto',
64
description: 'Out of process device emulation support')
65
+option('vfio_user_server', type: 'feature', value: 'disabled',
66
+ description: 'vfio-user server support')
67
option('dbus_display', type: 'feature', value: 'auto',
68
description: '-display dbus support')
69
option('tpm', type : 'feature', value : 'auto',
70
diff --git a/configure b/configure
71
index XXXXXXX..XXXXXXX 100755
72
--- a/configure
73
+++ b/configure
74
@@ -XXX,XX +XXX,XX @@ meson_args=""
75
ninja=""
76
bindir="bin"
77
skip_meson=no
78
+vfio_user_server="disabled"
79
80
# The following Meson options are handled manually (still they
81
# are included in the automatically generated help message)
82
@@ -XXX,XX +XXX,XX @@ for opt do
83
;;
84
--disable-blobs) meson_option_parse --disable-install-blobs ""
85
;;
86
+ --enable-vfio-user-server) vfio_user_server="enabled"
87
+ ;;
88
+ --disable-vfio-user-server) vfio_user_server="disabled"
89
+ ;;
90
--enable-tcmalloc) meson_option_parse --enable-malloc=tcmalloc tcmalloc
91
;;
92
--enable-jemalloc) meson_option_parse --enable-malloc=jemalloc jemalloc
93
@@ -XXX,XX +XXX,XX @@ write_container_target_makefile() {
94
95
96
97
+##########################################
98
+# check for vfio_user_server
99
+
100
+case "$vfio_user_server" in
101
+ enabled )
102
+ if test "$git_submodules_action" != "ignore"; then
103
+ git_submodules="${git_submodules} subprojects/libvfio-user"
104
+ fi
105
+ ;;
106
+esac
107
+
108
##########################################
109
# End of CC checks
110
# After here, no more $cc or $ld runs
111
@@ -XXX,XX +XXX,XX @@ if test "$skip_meson" = no; then
112
test "$slirp" != auto && meson_option_add "-Dslirp=$slirp"
113
test "$smbd" != '' && meson_option_add "-Dsmbd=$smbd"
114
test "$tcg" != enabled && meson_option_add "-Dtcg=$tcg"
115
+ test "$vfio_user_server" != auto && meson_option_add "-Dvfio_user_server=$vfio_user_server"
116
run_meson() {
117
NINJA=$ninja $meson setup --prefix "$prefix" "$@" $cross_arg "$PWD" "$source_path"
118
}
119
diff --git a/meson.build b/meson.build
120
index XXXXXXX..XXXXXXX 100644
121
--- a/meson.build
122
+++ b/meson.build
123
@@ -XXX,XX +XXX,XX @@ multiprocess_allowed = get_option('multiprocess') \
124
.require(targetos == 'linux', error_message: 'Multiprocess QEMU is supported only on Linux') \
125
.allowed()
126
127
+vfio_user_server_allowed = get_option('vfio_user_server') \
128
+ .require(targetos == 'linux', error_message: 'vfio-user server is supported only on Linux') \
129
+ .allowed()
130
+
131
have_tpm = get_option('tpm') \
132
.require(targetos != 'windows', error_message: 'TPM emulation only available on POSIX systems') \
133
.allowed()
134
@@ -XXX,XX +XXX,XX @@ host_kconfig = \
135
(have_virtfs ? ['CONFIG_VIRTFS=y'] : []) + \
136
('CONFIG_LINUX' in config_host ? ['CONFIG_LINUX=y'] : []) + \
137
(have_pvrdma ? ['CONFIG_PVRDMA=y'] : []) + \
138
- (multiprocess_allowed ? ['CONFIG_MULTIPROCESS_ALLOWED=y'] : [])
139
+ (multiprocess_allowed ? ['CONFIG_MULTIPROCESS_ALLOWED=y'] : []) + \
140
+ (vfio_user_server_allowed ? ['CONFIG_VFIO_USER_SERVER_ALLOWED=y'] : [])
141
142
ignored = [ 'TARGET_XML_FILES', 'TARGET_ABI_DIR', 'TARGET_ARCH' ]
143
144
@@ -XXX,XX +XXX,XX @@ if have_system
145
endif
146
endif
147
148
+libvfio_user_dep = not_found
149
+if have_system and vfio_user_server_allowed
150
+ have_internal = fs.exists(meson.current_source_dir() / 'subprojects/libvfio-user/meson.build')
151
+
152
+ if not have_internal
153
+ error('libvfio-user source not found - please pull git submodule')
154
+ endif
155
+
156
+ libvfio_user_proj = subproject('libvfio-user')
157
+
158
+ libvfio_user_lib = libvfio_user_proj.get_variable('libvfio_user_dep')
159
+
160
+ libvfio_user_dep = declare_dependency(dependencies: [libvfio_user_lib])
161
+endif
162
+
163
fdt = not_found
164
if have_system
165
fdt_opt = get_option('fdt')
166
@@ -XXX,XX +XXX,XX @@ summary_info += {'target list': ' '.join(target_dirs)}
167
if have_system
168
summary_info += {'default devices': get_option('default_devices')}
169
summary_info += {'out of process emulation': multiprocess_allowed}
170
+ summary_info += {'vfio-user server': vfio_user_server_allowed}
171
endif
172
summary(summary_info, bool_yn: true, section: 'Targets and accelerators')
173
174
diff --git a/.gitlab-ci.d/buildtest.yml b/.gitlab-ci.d/buildtest.yml
175
index XXXXXXX..XXXXXXX 100644
176
--- a/.gitlab-ci.d/buildtest.yml
177
+++ b/.gitlab-ci.d/buildtest.yml
178
@@ -XXX,XX +XXX,XX @@ build-system-centos:
179
IMAGE: centos8
180
CONFIGURE_ARGS: --disable-nettle --enable-gcrypt --enable-fdt=system
181
--enable-modules --enable-trace-backends=dtrace --enable-docs
182
+ --enable-vfio-user-server
183
TARGETS: ppc64-softmmu or1k-softmmu s390x-softmmu
184
x86_64-softmmu rx-softmmu sh4-softmmu nios2-softmmu
185
MAKE_CHECK_ARGS: check-build
186
diff --git a/.gitmodules b/.gitmodules
187
index XXXXXXX..XXXXXXX 100644
188
--- a/.gitmodules
189
+++ b/.gitmodules
190
@@ -XXX,XX +XXX,XX @@
191
[submodule "tests/lcitool/libvirt-ci"]
192
    path = tests/lcitool/libvirt-ci
193
    url = https://gitlab.com/libvirt/libvirt-ci.git
194
+[submodule "subprojects/libvfio-user"]
195
+    path = subprojects/libvfio-user
196
+    url = https://gitlab.com/qemu-project/libvfio-user.git
197
diff --git a/Kconfig.host b/Kconfig.host
198
index XXXXXXX..XXXXXXX 100644
199
--- a/Kconfig.host
200
+++ b/Kconfig.host
201
@@ -XXX,XX +XXX,XX @@ config MULTIPROCESS_ALLOWED
202
config FUZZ
203
bool
204
select SPARSE_MEM
205
+
206
+config VFIO_USER_SERVER_ALLOWED
207
+ bool
208
+ imply VFIO_USER_SERVER
209
diff --git a/hw/remote/Kconfig b/hw/remote/Kconfig
210
index XXXXXXX..XXXXXXX 100644
211
--- a/hw/remote/Kconfig
212
+++ b/hw/remote/Kconfig
213
@@ -XXX,XX +XXX,XX @@ config MULTIPROCESS
214
bool
215
depends on PCI && PCI_EXPRESS && KVM
216
select REMOTE_PCIHOST
217
+
218
+config VFIO_USER_SERVER
219
+ bool
220
+ depends on MULTIPROCESS
221
diff --git a/hw/remote/meson.build b/hw/remote/meson.build
222
index XXXXXXX..XXXXXXX 100644
223
--- a/hw/remote/meson.build
224
+++ b/hw/remote/meson.build
225
@@ -XXX,XX +XXX,XX @@ remote_ss.add(when: 'CONFIG_MULTIPROCESS', if_true: files('remote-obj.c'))
226
remote_ss.add(when: 'CONFIG_MULTIPROCESS', if_true: files('proxy.c'))
227
remote_ss.add(when: 'CONFIG_MULTIPROCESS', if_true: files('iohub.c'))
228
229
+remote_ss.add(when: 'CONFIG_VFIO_USER_SERVER', if_true: libvfio_user_dep)
230
+
231
specific_ss.add(when: 'CONFIG_MULTIPROCESS', if_true: files('memory.c'))
232
specific_ss.add(when: 'CONFIG_MULTIPROCESS', if_true: files('proxy-memory-listener.c'))
233
234
diff --git a/scripts/meson-buildoptions.sh b/scripts/meson-buildoptions.sh
235
index XXXXXXX..XXXXXXX 100644
236
--- a/scripts/meson-buildoptions.sh
237
+++ b/scripts/meson-buildoptions.sh
238
@@ -XXX,XX +XXX,XX @@ meson_options_help() {
239
printf "%s\n" ' usb-redir libusbredir support'
240
printf "%s\n" ' vde vde network backend support'
241
printf "%s\n" ' vdi vdi image format support'
242
+ printf "%s\n" ' vfio-user-server'
243
+ printf "%s\n" ' vfio-user server support'
244
printf "%s\n" ' vhost-crypto vhost-user crypto backend support'
245
printf "%s\n" ' vhost-kernel vhost kernel backend support'
246
printf "%s\n" ' vhost-net vhost-net kernel acceleration support'
247
@@ -XXX,XX +XXX,XX @@ _meson_option_parse() {
248
--disable-vde) printf "%s" -Dvde=disabled ;;
249
--enable-vdi) printf "%s" -Dvdi=enabled ;;
250
--disable-vdi) printf "%s" -Dvdi=disabled ;;
251
+ --enable-vfio-user-server) printf "%s" -Dvfio_user_server=enabled ;;
252
+ --disable-vfio-user-server) printf "%s" -Dvfio_user_server=disabled ;;
253
--enable-vhost-crypto) printf "%s" -Dvhost_crypto=enabled ;;
254
--disable-vhost-crypto) printf "%s" -Dvhost_crypto=disabled ;;
255
--enable-vhost-kernel) printf "%s" -Dvhost_kernel=enabled ;;
256
diff --git a/subprojects/libvfio-user b/subprojects/libvfio-user
257
new file mode 160000
258
index XXXXXXX..XXXXXXX
259
--- /dev/null
260
+++ b/subprojects/libvfio-user
261
@@ -0,0 +1 @@
262
+Subproject commit 0b28d205572c80b568a1003db2c8f37ca333e4d7
263
diff --git a/tests/docker/dockerfiles/centos8.docker b/tests/docker/dockerfiles/centos8.docker
264
index XXXXXXX..XXXXXXX 100644
265
--- a/tests/docker/dockerfiles/centos8.docker
266
+++ b/tests/docker/dockerfiles/centos8.docker
267
@@ -XXX,XX +XXX,XX @@ RUN dnf update -y && \
268
libbpf-devel \
269
libcacard-devel \
270
libcap-ng-devel \
271
+ libcmocka-devel \
272
libcurl-devel \
273
libdrm-devel \
274
libepoxy-devel \
275
@@ -XXX,XX +XXX,XX @@ RUN dnf update -y && \
276
libgcrypt-devel \
277
libiscsi-devel \
278
libjpeg-devel \
279
+ json-c-devel \
280
libnfs-devel \
281
libpmem-devel \
282
libpng-devel \
283
--
284
2.36.1
diff view generated by jsdifflib
1
Add the scripts/ directory to sys.path so Python 2.6 will be able to
1
From: Jagannathan Raman <jag.raman@oracle.com>
2
import argparse.
2
3
3
Define vfio-user object which is remote process server for QEMU. Setup
4
Cc: Daniel P. Berrange <berrange@redhat.com>
4
object initialization functions and properties necessary to instantiate
5
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
5
the object
6
Acked-by: John Snow <jsnow@redhat.com>
6
7
Acked-by: Fam Zheng <famz@redhat.com>
7
Signed-off-by: Elena Ufimtseva <elena.ufimtseva@oracle.com>
8
Message-id: 20170825155732.15665-4-stefanha@redhat.com
8
Signed-off-by: John G Johnson <john.g.johnson@oracle.com>
9
Signed-off-by: Jagannathan Raman <jag.raman@oracle.com>
10
Reviewed-by: Stefan Hajnoczi <stefanha@redhat.com>
11
Message-id: e45a17001e9b38f451543a664ababdf860e5f2f2.1655151679.git.jag.raman@oracle.com
9
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
12
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
10
---
13
---
11
tests/migration/guestperf/shell.py | 8 +++++---
14
MAINTAINERS | 1 +
12
1 file changed, 5 insertions(+), 3 deletions(-)
15
qapi/qom.json | 20 +++-
13
16
include/hw/remote/machine.h | 2 +
14
diff --git a/tests/migration/guestperf/shell.py b/tests/migration/guestperf/shell.py
17
hw/remote/machine.c | 27 +++++
15
index XXXXXXX..XXXXXXX 100644
18
hw/remote/vfio-user-obj.c | 210 ++++++++++++++++++++++++++++++++++++
16
--- a/tests/migration/guestperf/shell.py
19
hw/remote/meson.build | 1 +
17
+++ b/tests/migration/guestperf/shell.py
20
hw/remote/trace-events | 3 +
18
@@ -XXX,XX +XXX,XX @@
21
7 files changed, 262 insertions(+), 2 deletions(-)
22
create mode 100644 hw/remote/vfio-user-obj.c
23
24
diff --git a/MAINTAINERS b/MAINTAINERS
25
index XXXXXXX..XXXXXXX 100644
26
--- a/MAINTAINERS
27
+++ b/MAINTAINERS
28
@@ -XXX,XX +XXX,XX @@ F: include/hw/remote/proxy-memory-listener.h
29
F: hw/remote/iohub.c
30
F: include/hw/remote/iohub.h
31
F: subprojects/libvfio-user
32
+F: hw/remote/vfio-user-obj.c
33
34
EBPF:
35
M: Jason Wang <jasowang@redhat.com>
36
diff --git a/qapi/qom.json b/qapi/qom.json
37
index XXXXXXX..XXXXXXX 100644
38
--- a/qapi/qom.json
39
+++ b/qapi/qom.json
40
@@ -XXX,XX +XXX,XX @@
41
{ 'struct': 'RemoteObjectProperties',
42
'data': { 'fd': 'str', 'devid': 'str' } }
43
44
+##
45
+# @VfioUserServerProperties:
46
+#
47
+# Properties for x-vfio-user-server objects.
48
+#
49
+# @socket: socket to be used by the libvfio-user library
50
+#
51
+# @device: the ID of the device to be emulated at the server
52
+#
53
+# Since: 7.1
54
+##
55
+{ 'struct': 'VfioUserServerProperties',
56
+ 'data': { 'socket': 'SocketAddress', 'device': 'str' } }
57
+
58
##
59
# @RngProperties:
19
#
60
#
20
61
@@ -XXX,XX +XXX,XX @@
21
62
'tls-creds-psk',
22
-import argparse
63
'tls-creds-x509',
23
-import fnmatch
64
'tls-cipher-suites',
24
import os
65
- { 'name': 'x-remote-object', 'features': [ 'unstable' ] }
25
import os.path
66
+ { 'name': 'x-remote-object', 'features': [ 'unstable' ] },
26
-import platform
67
+ { 'name': 'x-vfio-user-server', 'features': [ 'unstable' ] }
27
import sys
68
] }
28
+sys.path.append(os.path.join(os.path.dirname(__file__),
69
29
+ '..', '..', '..', 'scripts'))
70
##
30
+import argparse
71
@@ -XXX,XX +XXX,XX @@
31
+import fnmatch
72
'tls-creds-psk': 'TlsCredsPskProperties',
32
+import platform
73
'tls-creds-x509': 'TlsCredsX509Properties',
33
74
'tls-cipher-suites': 'TlsCredsProperties',
34
from guestperf.hardware import Hardware
75
- 'x-remote-object': 'RemoteObjectProperties'
35
from guestperf.engine import Engine
76
+ 'x-remote-object': 'RemoteObjectProperties',
77
+ 'x-vfio-user-server': 'VfioUserServerProperties'
78
} }
79
80
##
81
diff --git a/include/hw/remote/machine.h b/include/hw/remote/machine.h
82
index XXXXXXX..XXXXXXX 100644
83
--- a/include/hw/remote/machine.h
84
+++ b/include/hw/remote/machine.h
85
@@ -XXX,XX +XXX,XX @@ struct RemoteMachineState {
86
RemoteIOHubState iohub;
87
88
bool vfio_user;
89
+
90
+ bool auto_shutdown;
91
};
92
93
/* Used to pass to co-routine device and ioc. */
94
diff --git a/hw/remote/machine.c b/hw/remote/machine.c
95
index XXXXXXX..XXXXXXX 100644
96
--- a/hw/remote/machine.c
97
+++ b/hw/remote/machine.c
98
@@ -XXX,XX +XXX,XX @@ static void remote_machine_set_vfio_user(Object *obj, bool value, Error **errp)
99
s->vfio_user = value;
100
}
101
102
+static bool remote_machine_get_auto_shutdown(Object *obj, Error **errp)
103
+{
104
+ RemoteMachineState *s = REMOTE_MACHINE(obj);
105
+
106
+ return s->auto_shutdown;
107
+}
108
+
109
+static void remote_machine_set_auto_shutdown(Object *obj, bool value,
110
+ Error **errp)
111
+{
112
+ RemoteMachineState *s = REMOTE_MACHINE(obj);
113
+
114
+ s->auto_shutdown = value;
115
+}
116
+
117
+static void remote_machine_instance_init(Object *obj)
118
+{
119
+ RemoteMachineState *s = REMOTE_MACHINE(obj);
120
+
121
+ s->auto_shutdown = true;
122
+}
123
+
124
static void remote_machine_class_init(ObjectClass *oc, void *data)
125
{
126
MachineClass *mc = MACHINE_CLASS(oc);
127
@@ -XXX,XX +XXX,XX @@ static void remote_machine_class_init(ObjectClass *oc, void *data)
128
object_class_property_add_bool(oc, "vfio-user",
129
remote_machine_get_vfio_user,
130
remote_machine_set_vfio_user);
131
+
132
+ object_class_property_add_bool(oc, "auto-shutdown",
133
+ remote_machine_get_auto_shutdown,
134
+ remote_machine_set_auto_shutdown);
135
}
136
137
static const TypeInfo remote_machine = {
138
.name = TYPE_REMOTE_MACHINE,
139
.parent = TYPE_MACHINE,
140
.instance_size = sizeof(RemoteMachineState),
141
+ .instance_init = remote_machine_instance_init,
142
.class_init = remote_machine_class_init,
143
.interfaces = (InterfaceInfo[]) {
144
{ TYPE_HOTPLUG_HANDLER },
145
diff --git a/hw/remote/vfio-user-obj.c b/hw/remote/vfio-user-obj.c
146
new file mode 100644
147
index XXXXXXX..XXXXXXX
148
--- /dev/null
149
+++ b/hw/remote/vfio-user-obj.c
150
@@ -XXX,XX +XXX,XX @@
151
+/**
152
+ * QEMU vfio-user-server server object
153
+ *
154
+ * Copyright © 2022 Oracle and/or its affiliates.
155
+ *
156
+ * This work is licensed under the terms of the GNU GPL-v2, version 2 or later.
157
+ *
158
+ * See the COPYING file in the top-level directory.
159
+ *
160
+ */
161
+
162
+/**
163
+ * Usage: add options:
164
+ * -machine x-remote,vfio-user=on,auto-shutdown=on
165
+ * -device <PCI-device>,id=<pci-dev-id>
166
+ * -object x-vfio-user-server,id=<id>,type=unix,path=<socket-path>,
167
+ * device=<pci-dev-id>
168
+ *
169
+ * Note that x-vfio-user-server object must be used with x-remote machine only.
170
+ * This server could only support PCI devices for now.
171
+ *
172
+ * type - SocketAddress type - presently "unix" alone is supported. Required
173
+ * option
174
+ *
175
+ * path - named unix socket, it will be created by the server. It is
176
+ * a required option
177
+ *
178
+ * device - id of a device on the server, a required option. PCI devices
179
+ * alone are supported presently.
180
+ */
181
+
182
+#include "qemu/osdep.h"
183
+
184
+#include "qom/object.h"
185
+#include "qom/object_interfaces.h"
186
+#include "qemu/error-report.h"
187
+#include "trace.h"
188
+#include "sysemu/runstate.h"
189
+#include "hw/boards.h"
190
+#include "hw/remote/machine.h"
191
+#include "qapi/error.h"
192
+#include "qapi/qapi-visit-sockets.h"
193
+
194
+#define TYPE_VFU_OBJECT "x-vfio-user-server"
195
+OBJECT_DECLARE_TYPE(VfuObject, VfuObjectClass, VFU_OBJECT)
196
+
197
+/**
198
+ * VFU_OBJECT_ERROR - reports an error message. If auto_shutdown
199
+ * is set, it aborts the machine on error. Otherwise, it logs an
200
+ * error message without aborting.
201
+ */
202
+#define VFU_OBJECT_ERROR(o, fmt, ...) \
203
+ { \
204
+ if (vfu_object_auto_shutdown()) { \
205
+ error_setg(&error_abort, (fmt), ## __VA_ARGS__); \
206
+ } else { \
207
+ error_report((fmt), ## __VA_ARGS__); \
208
+ } \
209
+ } \
210
+
211
+struct VfuObjectClass {
212
+ ObjectClass parent_class;
213
+
214
+ unsigned int nr_devs;
215
+};
216
+
217
+struct VfuObject {
218
+ /* private */
219
+ Object parent;
220
+
221
+ SocketAddress *socket;
222
+
223
+ char *device;
224
+
225
+ Error *err;
226
+};
227
+
228
+static bool vfu_object_auto_shutdown(void)
229
+{
230
+ bool auto_shutdown = true;
231
+ Error *local_err = NULL;
232
+
233
+ if (!current_machine) {
234
+ return auto_shutdown;
235
+ }
236
+
237
+ auto_shutdown = object_property_get_bool(OBJECT(current_machine),
238
+ "auto-shutdown",
239
+ &local_err);
240
+
241
+ /*
242
+ * local_err would be set if no such property exists - safe to ignore.
243
+ * Unlikely scenario as auto-shutdown is always defined for
244
+ * TYPE_REMOTE_MACHINE, and TYPE_VFU_OBJECT only works with
245
+ * TYPE_REMOTE_MACHINE
246
+ */
247
+ if (local_err) {
248
+ auto_shutdown = true;
249
+ error_free(local_err);
250
+ }
251
+
252
+ return auto_shutdown;
253
+}
254
+
255
+static void vfu_object_set_socket(Object *obj, Visitor *v, const char *name,
256
+ void *opaque, Error **errp)
257
+{
258
+ VfuObject *o = VFU_OBJECT(obj);
259
+
260
+ qapi_free_SocketAddress(o->socket);
261
+
262
+ o->socket = NULL;
263
+
264
+ visit_type_SocketAddress(v, name, &o->socket, errp);
265
+
266
+ if (o->socket->type != SOCKET_ADDRESS_TYPE_UNIX) {
267
+ error_setg(errp, "vfu: Unsupported socket type - %s",
268
+ SocketAddressType_str(o->socket->type));
269
+ qapi_free_SocketAddress(o->socket);
270
+ o->socket = NULL;
271
+ return;
272
+ }
273
+
274
+ trace_vfu_prop("socket", o->socket->u.q_unix.path);
275
+}
276
+
277
+static void vfu_object_set_device(Object *obj, const char *str, Error **errp)
278
+{
279
+ VfuObject *o = VFU_OBJECT(obj);
280
+
281
+ g_free(o->device);
282
+
283
+ o->device = g_strdup(str);
284
+
285
+ trace_vfu_prop("device", str);
286
+}
287
+
288
+static void vfu_object_init(Object *obj)
289
+{
290
+ VfuObjectClass *k = VFU_OBJECT_GET_CLASS(obj);
291
+ VfuObject *o = VFU_OBJECT(obj);
292
+
293
+ k->nr_devs++;
294
+
295
+ if (!object_dynamic_cast(OBJECT(current_machine), TYPE_REMOTE_MACHINE)) {
296
+ error_setg(&o->err, "vfu: %s only compatible with %s machine",
297
+ TYPE_VFU_OBJECT, TYPE_REMOTE_MACHINE);
298
+ return;
299
+ }
300
+}
301
+
302
+static void vfu_object_finalize(Object *obj)
303
+{
304
+ VfuObjectClass *k = VFU_OBJECT_GET_CLASS(obj);
305
+ VfuObject *o = VFU_OBJECT(obj);
306
+
307
+ k->nr_devs--;
308
+
309
+ qapi_free_SocketAddress(o->socket);
310
+
311
+ o->socket = NULL;
312
+
313
+ g_free(o->device);
314
+
315
+ o->device = NULL;
316
+
317
+ if (!k->nr_devs && vfu_object_auto_shutdown()) {
318
+ qemu_system_shutdown_request(SHUTDOWN_CAUSE_GUEST_SHUTDOWN);
319
+ }
320
+}
321
+
322
+static void vfu_object_class_init(ObjectClass *klass, void *data)
323
+{
324
+ VfuObjectClass *k = VFU_OBJECT_CLASS(klass);
325
+
326
+ k->nr_devs = 0;
327
+
328
+ object_class_property_add(klass, "socket", "SocketAddress", NULL,
329
+ vfu_object_set_socket, NULL, NULL);
330
+ object_class_property_set_description(klass, "socket",
331
+ "SocketAddress "
332
+ "(ex: type=unix,path=/tmp/sock). "
333
+ "Only UNIX is presently supported");
334
+ object_class_property_add_str(klass, "device", NULL,
335
+ vfu_object_set_device);
336
+ object_class_property_set_description(klass, "device",
337
+ "device ID - only PCI devices "
338
+ "are presently supported");
339
+}
340
+
341
+static const TypeInfo vfu_object_info = {
342
+ .name = TYPE_VFU_OBJECT,
343
+ .parent = TYPE_OBJECT,
344
+ .instance_size = sizeof(VfuObject),
345
+ .instance_init = vfu_object_init,
346
+ .instance_finalize = vfu_object_finalize,
347
+ .class_size = sizeof(VfuObjectClass),
348
+ .class_init = vfu_object_class_init,
349
+ .interfaces = (InterfaceInfo[]) {
350
+ { TYPE_USER_CREATABLE },
351
+ { }
352
+ }
353
+};
354
+
355
+static void vfu_register_types(void)
356
+{
357
+ type_register_static(&vfu_object_info);
358
+}
359
+
360
+type_init(vfu_register_types);
361
diff --git a/hw/remote/meson.build b/hw/remote/meson.build
362
index XXXXXXX..XXXXXXX 100644
363
--- a/hw/remote/meson.build
364
+++ b/hw/remote/meson.build
365
@@ -XXX,XX +XXX,XX @@ remote_ss.add(when: 'CONFIG_MULTIPROCESS', if_true: files('message.c'))
366
remote_ss.add(when: 'CONFIG_MULTIPROCESS', if_true: files('remote-obj.c'))
367
remote_ss.add(when: 'CONFIG_MULTIPROCESS', if_true: files('proxy.c'))
368
remote_ss.add(when: 'CONFIG_MULTIPROCESS', if_true: files('iohub.c'))
369
+remote_ss.add(when: 'CONFIG_VFIO_USER_SERVER', if_true: files('vfio-user-obj.c'))
370
371
remote_ss.add(when: 'CONFIG_VFIO_USER_SERVER', if_true: libvfio_user_dep)
372
373
diff --git a/hw/remote/trace-events b/hw/remote/trace-events
374
index XXXXXXX..XXXXXXX 100644
375
--- a/hw/remote/trace-events
376
+++ b/hw/remote/trace-events
377
@@ -XXX,XX +XXX,XX @@
378
379
mpqemu_send_io_error(int cmd, int size, int nfds) "send command %d size %d, %d file descriptors to remote process"
380
mpqemu_recv_io_error(int cmd, int size, int nfds) "failed to receive %d size %d, %d file descriptors to remote process"
381
+
382
+# vfio-user-obj.c
383
+vfu_prop(const char *prop, const char *val) "vfu: setting %s as %s"
36
--
384
--
37
2.13.5
385
2.36.1
38
386
39
387
diff view generated by jsdifflib
1
From: Alberto Garcia <berto@igalia.com>
1
From: Jagannathan Raman <jag.raman@oracle.com>
2
2
3
The throttling code can change internally the value of bkt->max if it
3
create a context with the vfio-user library to run a PCI device
4
hasn't been set by the user. The problem with this is that if we want
5
to retrieve the original value we have to undo this change first. This
6
is ugly and unnecessary: this patch removes the throttle_fix_bucket()
7
and throttle_unfix_bucket() functions completely and moves the logic
8
to throttle_compute_wait().
9
4
10
Signed-off-by: Alberto Garcia <berto@igalia.com>
5
Signed-off-by: Elena Ufimtseva <elena.ufimtseva@oracle.com>
11
Reviewed-by: Manos Pitsidianakis <el13635@mail.ntua.gr>
6
Signed-off-by: John G Johnson <john.g.johnson@oracle.com>
12
Message-id: 5b0b9e1ac6eb208d709eddc7b09e7669a523bff3.1503580370.git.berto@igalia.com
7
Signed-off-by: Jagannathan Raman <jag.raman@oracle.com>
8
Reviewed-by: Stefan Hajnoczi <stefanha@redhat.com>
9
Message-id: a452871ac8c812ff96fc4f0ce6037f4769953fab.1655151679.git.jag.raman@oracle.com
13
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
10
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
14
---
11
---
15
util/throttle.c | 62 +++++++++++++++++++++------------------------------------
12
hw/remote/vfio-user-obj.c | 82 +++++++++++++++++++++++++++++++++++++++
16
1 file changed, 23 insertions(+), 39 deletions(-)
13
1 file changed, 82 insertions(+)
17
14
18
diff --git a/util/throttle.c b/util/throttle.c
15
diff --git a/hw/remote/vfio-user-obj.c b/hw/remote/vfio-user-obj.c
19
index XXXXXXX..XXXXXXX 100644
16
index XXXXXXX..XXXXXXX 100644
20
--- a/util/throttle.c
17
--- a/hw/remote/vfio-user-obj.c
21
+++ b/util/throttle.c
18
+++ b/hw/remote/vfio-user-obj.c
22
@@ -XXX,XX +XXX,XX @@ static int64_t throttle_do_compute_wait(double limit, double extra)
19
@@ -XXX,XX +XXX,XX @@
23
int64_t throttle_compute_wait(LeakyBucket *bkt)
20
#include "hw/remote/machine.h"
21
#include "qapi/error.h"
22
#include "qapi/qapi-visit-sockets.h"
23
+#include "qemu/notify.h"
24
+#include "sysemu/sysemu.h"
25
+#include "libvfio-user.h"
26
27
#define TYPE_VFU_OBJECT "x-vfio-user-server"
28
OBJECT_DECLARE_TYPE(VfuObject, VfuObjectClass, VFU_OBJECT)
29
@@ -XXX,XX +XXX,XX @@ struct VfuObject {
30
char *device;
31
32
Error *err;
33
+
34
+ Notifier machine_done;
35
+
36
+ vfu_ctx_t *vfu_ctx;
37
};
38
39
+static void vfu_object_init_ctx(VfuObject *o, Error **errp);
40
+
41
static bool vfu_object_auto_shutdown(void)
24
{
42
{
25
double extra; /* the number of extra units blocking the io */
43
bool auto_shutdown = true;
26
+ double bucket_size; /* I/O before throttling to bkt->avg */
44
@@ -XXX,XX +XXX,XX @@ static void vfu_object_set_socket(Object *obj, Visitor *v, const char *name,
27
+ double burst_bucket_size; /* Before throttling to bkt->max */
45
{
28
46
VfuObject *o = VFU_OBJECT(obj);
29
if (!bkt->avg) {
47
30
return 0;
48
+ if (o->vfu_ctx) {
31
}
49
+ error_setg(errp, "vfu: Unable to set socket property - server busy");
32
50
+ return;
33
- /* If the bucket is full then we have to wait */
34
- extra = bkt->level - bkt->max * bkt->burst_length;
35
+ if (!bkt->max) {
36
+ /* If bkt->max is 0 we still want to allow short bursts of I/O
37
+ * from the guest, otherwise every other request will be throttled
38
+ * and performance will suffer considerably. */
39
+ bucket_size = bkt->avg / 10;
40
+ burst_bucket_size = 0;
41
+ } else {
42
+ /* If we have a burst limit then we have to wait until all I/O
43
+ * at burst rate has finished before throttling to bkt->avg */
44
+ bucket_size = bkt->max * bkt->burst_length;
45
+ burst_bucket_size = bkt->max / 10;
46
+ }
51
+ }
47
+
52
+
48
+ /* If the main bucket is full then we have to wait */
53
qapi_free_SocketAddress(o->socket);
49
+ extra = bkt->level - bucket_size;
54
50
if (extra > 0) {
55
o->socket = NULL;
51
return throttle_do_compute_wait(bkt->avg, extra);
56
@@ -XXX,XX +XXX,XX @@ static void vfu_object_set_socket(Object *obj, Visitor *v, const char *name,
52
}
57
}
53
58
54
- /* If the bucket is not full yet we have to make sure that we
59
trace_vfu_prop("socket", o->socket->u.q_unix.path);
55
- * fulfill the goal of bkt->max units per second. */
60
+
56
+ /* If the main bucket is not full yet we still have to check the
61
+ vfu_object_init_ctx(o, errp);
57
+ * burst bucket in order to enforce the burst limit */
58
if (bkt->burst_length > 1) {
59
- /* We use 1/10 of the max value to smooth the throttling.
60
- * See throttle_fix_bucket() for more details. */
61
- extra = bkt->burst_level - bkt->max / 10;
62
+ extra = bkt->burst_level - burst_bucket_size;
63
if (extra > 0) {
64
return throttle_do_compute_wait(bkt->max, extra);
65
}
66
@@ -XXX,XX +XXX,XX @@ bool throttle_is_valid(ThrottleConfig *cfg, Error **errp)
67
return true;
68
}
62
}
69
63
70
-/* fix bucket parameters */
64
static void vfu_object_set_device(Object *obj, const char *str, Error **errp)
71
-static void throttle_fix_bucket(LeakyBucket *bkt)
65
{
72
-{
66
VfuObject *o = VFU_OBJECT(obj);
73
- double min;
67
74
-
68
+ if (o->vfu_ctx) {
75
- /* zero bucket level */
69
+ error_setg(errp, "vfu: Unable to set device property - server busy");
76
- bkt->level = bkt->burst_level = 0;
70
+ return;
77
-
71
+ }
78
- /* If bkt->max is 0 we still want to allow short bursts of I/O
72
+
79
- * from the guest, otherwise every other request will be throttled
73
g_free(o->device);
80
- * and performance will suffer considerably. */
74
81
- min = bkt->avg / 10;
75
o->device = g_strdup(str);
82
- if (bkt->avg && !bkt->max) {
76
83
- bkt->max = min;
77
trace_vfu_prop("device", str);
84
- }
78
+
85
-}
79
+ vfu_object_init_ctx(o, errp);
86
-
80
+}
87
-/* undo internal bucket parameter changes (see throttle_fix_bucket()) */
81
+
88
-static void throttle_unfix_bucket(LeakyBucket *bkt)
82
+/*
89
-{
83
+ * TYPE_VFU_OBJECT depends on the availability of the 'socket' and 'device'
90
- if (bkt->max < bkt->avg) {
84
+ * properties. It also depends on devices instantiated in QEMU. These
91
- bkt->max = 0;
85
+ * dependencies are not available during the instance_init phase of this
92
- }
86
+ * object's life-cycle. As such, the server is initialized after the
93
-}
87
+ * machine is setup. machine_init_done_notifier notifies TYPE_VFU_OBJECT
94
-
88
+ * when the machine is setup, and the dependencies are available.
95
/* Used to configure the throttle
89
+ */
96
*
90
+static void vfu_object_machine_done(Notifier *notifier, void *data)
97
* @ts: the throttle state we are working on
91
+{
98
@@ -XXX,XX +XXX,XX @@ void throttle_config(ThrottleState *ts,
92
+ VfuObject *o = container_of(notifier, VfuObject, machine_done);
99
93
+ Error *err = NULL;
100
ts->cfg = *cfg;
94
+
101
95
+ vfu_object_init_ctx(o, &err);
102
+ /* Zero bucket level */
96
+
103
for (i = 0; i < BUCKETS_COUNT; i++) {
97
+ if (err) {
104
- throttle_fix_bucket(&ts->cfg.buckets[i]);
98
+ error_propagate(&error_abort, err);
105
+ ts->cfg.buckets[i].level = 0;
99
+ }
106
+ ts->cfg.buckets[i].burst_level = 0;
100
+}
101
+
102
+static void vfu_object_init_ctx(VfuObject *o, Error **errp)
103
+{
104
+ ERRP_GUARD();
105
+
106
+ if (o->vfu_ctx || !o->socket || !o->device ||
107
+ !phase_check(PHASE_MACHINE_READY)) {
108
+ return;
109
+ }
110
+
111
+ if (o->err) {
112
+ error_propagate(errp, o->err);
113
+ o->err = NULL;
114
+ return;
115
+ }
116
+
117
+ o->vfu_ctx = vfu_create_ctx(VFU_TRANS_SOCK, o->socket->u.q_unix.path, 0,
118
+ o, VFU_DEV_TYPE_PCI);
119
+ if (o->vfu_ctx == NULL) {
120
+ error_setg(errp, "vfu: Failed to create context - %s", strerror(errno));
121
+ return;
122
+ }
123
}
124
125
static void vfu_object_init(Object *obj)
126
@@ -XXX,XX +XXX,XX @@ static void vfu_object_init(Object *obj)
127
TYPE_VFU_OBJECT, TYPE_REMOTE_MACHINE);
128
return;
107
}
129
}
108
130
+
109
ts->previous_leak = qemu_clock_get_ns(clock_type);
131
+ if (!phase_check(PHASE_MACHINE_READY)) {
110
@@ -XXX,XX +XXX,XX @@ void throttle_config(ThrottleState *ts,
132
+ o->machine_done.notify = vfu_object_machine_done;
111
*/
133
+ qemu_add_machine_init_done_notifier(&o->machine_done);
112
void throttle_get_config(ThrottleState *ts, ThrottleConfig *cfg)
134
+ }
113
{
135
+
114
- int i;
115
-
116
*cfg = ts->cfg;
117
-
118
- for (i = 0; i < BUCKETS_COUNT; i++) {
119
- throttle_unfix_bucket(&cfg->buckets[i]);
120
- }
121
}
136
}
122
137
123
138
static void vfu_object_finalize(Object *obj)
139
@@ -XXX,XX +XXX,XX @@ static void vfu_object_finalize(Object *obj)
140
141
o->socket = NULL;
142
143
+ if (o->vfu_ctx) {
144
+ vfu_destroy_ctx(o->vfu_ctx);
145
+ o->vfu_ctx = NULL;
146
+ }
147
+
148
g_free(o->device);
149
150
o->device = NULL;
151
@@ -XXX,XX +XXX,XX @@ static void vfu_object_finalize(Object *obj)
152
if (!k->nr_devs && vfu_object_auto_shutdown()) {
153
qemu_system_shutdown_request(SHUTDOWN_CAUSE_GUEST_SHUTDOWN);
154
}
155
+
156
+ if (o->machine_done.notify) {
157
+ qemu_remove_machine_init_done_notifier(&o->machine_done);
158
+ o->machine_done.notify = NULL;
159
+ }
160
}
161
162
static void vfu_object_class_init(ObjectClass *klass, void *data)
124
--
163
--
125
2.13.5
164
2.36.1
126
127
diff view generated by jsdifflib
1
From: Alberto Garcia <berto@igalia.com>
1
From: Jagannathan Raman <jag.raman@oracle.com>
2
2
3
LeakyBucket.burst_length is defined as an unsigned integer but the
3
Find the PCI device with specified id. Initialize the device context
4
code never checks for overflows and it only makes sure that the value
4
with the QEMU PCI device
5
is not 0.
6
5
7
In practice this means that the user can set something like
6
Signed-off-by: Elena Ufimtseva <elena.ufimtseva@oracle.com>
8
throttling.iops-total-max-length=4294967300 despite being larger than
7
Signed-off-by: John G Johnson <john.g.johnson@oracle.com>
9
UINT_MAX and the final value after casting to unsigned int will be 4.
8
Signed-off-by: Jagannathan Raman <jag.raman@oracle.com>
10
9
Reviewed-by: Stefan Hajnoczi <stefanha@redhat.com>
11
This patch changes the data type to uint64_t. This does not increase
10
Message-id: 7798dbd730099b33fdd00c4c202cfe79e5c5c151.1655151679.git.jag.raman@oracle.com
12
the storage size of LeakyBucket, and allows us to assign the value
13
directly from qemu_opt_get_number() or BlockIOThrottle and then do the
14
checks directly in throttle_is_valid().
15
16
The value of burst_length does not have a specific upper limit,
17
but since the bucket size is defined by max * burst_length we have
18
to prevent overflows. Instead of going for UINT64_MAX or something
19
similar this patch reuses THROTTLE_VALUE_MAX, which allows I/O bursts
20
of 1 GiB/s for 10 days in a row.
21
22
Signed-off-by: Alberto Garcia <berto@igalia.com>
23
Message-id: 1b2e3049803f71cafb2e1fa1be4fb47147a0d398.1503580370.git.berto@igalia.com
24
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
11
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
25
---
12
---
26
include/qemu/throttle.h | 2 +-
13
hw/remote/vfio-user-obj.c | 67 +++++++++++++++++++++++++++++++++++++++
27
util/throttle.c | 5 +++++
14
1 file changed, 67 insertions(+)
28
2 files changed, 6 insertions(+), 1 deletion(-)
29
15
30
diff --git a/include/qemu/throttle.h b/include/qemu/throttle.h
16
diff --git a/hw/remote/vfio-user-obj.c b/hw/remote/vfio-user-obj.c
31
index XXXXXXX..XXXXXXX 100644
17
index XXXXXXX..XXXXXXX 100644
32
--- a/include/qemu/throttle.h
18
--- a/hw/remote/vfio-user-obj.c
33
+++ b/include/qemu/throttle.h
19
+++ b/hw/remote/vfio-user-obj.c
34
@@ -XXX,XX +XXX,XX @@ typedef struct LeakyBucket {
20
@@ -XXX,XX +XXX,XX @@
35
uint64_t max; /* leaky bucket max burst in units */
21
#include "qemu/notify.h"
36
double level; /* bucket level in units */
22
#include "sysemu/sysemu.h"
37
double burst_level; /* bucket level in units (for computing bursts) */
23
#include "libvfio-user.h"
38
- unsigned burst_length; /* max length of the burst period, in seconds */
24
+#include "hw/qdev-core.h"
39
+ uint64_t burst_length; /* max length of the burst period, in seconds */
25
+#include "hw/pci/pci.h"
40
} LeakyBucket;
26
41
27
#define TYPE_VFU_OBJECT "x-vfio-user-server"
42
/* The following structure is used to configure a ThrottleState
28
OBJECT_DECLARE_TYPE(VfuObject, VfuObjectClass, VFU_OBJECT)
43
diff --git a/util/throttle.c b/util/throttle.c
29
@@ -XXX,XX +XXX,XX @@ struct VfuObject {
44
index XXXXXXX..XXXXXXX 100644
30
Notifier machine_done;
45
--- a/util/throttle.c
31
46
+++ b/util/throttle.c
32
vfu_ctx_t *vfu_ctx;
47
@@ -XXX,XX +XXX,XX @@ bool throttle_is_valid(ThrottleConfig *cfg, Error **errp)
48
return false;
49
}
50
51
+ if (bkt->max && bkt->burst_length > THROTTLE_VALUE_MAX / bkt->max) {
52
+ error_setg(errp, "burst length too high for this burst rate");
53
+ return false;
54
+ }
55
+
33
+
56
if (bkt->max && !bkt->avg) {
34
+ PCIDevice *pci_dev;
57
error_setg(errp, "bps_max/iops_max require corresponding"
35
+
58
" bps/iops values");
36
+ Error *unplug_blocker;
37
};
38
39
static void vfu_object_init_ctx(VfuObject *o, Error **errp);
40
@@ -XXX,XX +XXX,XX @@ static void vfu_object_machine_done(Notifier *notifier, void *data)
41
static void vfu_object_init_ctx(VfuObject *o, Error **errp)
42
{
43
ERRP_GUARD();
44
+ DeviceState *dev = NULL;
45
+ vfu_pci_type_t pci_type = VFU_PCI_TYPE_CONVENTIONAL;
46
+ int ret;
47
48
if (o->vfu_ctx || !o->socket || !o->device ||
49
!phase_check(PHASE_MACHINE_READY)) {
50
@@ -XXX,XX +XXX,XX @@ static void vfu_object_init_ctx(VfuObject *o, Error **errp)
51
error_setg(errp, "vfu: Failed to create context - %s", strerror(errno));
52
return;
53
}
54
+
55
+ dev = qdev_find_recursive(sysbus_get_default(), o->device);
56
+ if (dev == NULL) {
57
+ error_setg(errp, "vfu: Device %s not found", o->device);
58
+ goto fail;
59
+ }
60
+
61
+ if (!object_dynamic_cast(OBJECT(dev), TYPE_PCI_DEVICE)) {
62
+ error_setg(errp, "vfu: %s not a PCI device", o->device);
63
+ goto fail;
64
+ }
65
+
66
+ o->pci_dev = PCI_DEVICE(dev);
67
+
68
+ object_ref(OBJECT(o->pci_dev));
69
+
70
+ if (pci_is_express(o->pci_dev)) {
71
+ pci_type = VFU_PCI_TYPE_EXPRESS;
72
+ }
73
+
74
+ ret = vfu_pci_init(o->vfu_ctx, pci_type, PCI_HEADER_TYPE_NORMAL, 0);
75
+ if (ret < 0) {
76
+ error_setg(errp,
77
+ "vfu: Failed to attach PCI device %s to context - %s",
78
+ o->device, strerror(errno));
79
+ goto fail;
80
+ }
81
+
82
+ error_setg(&o->unplug_blocker,
83
+ "vfu: %s for %s must be deleted before unplugging",
84
+ TYPE_VFU_OBJECT, o->device);
85
+ qdev_add_unplug_blocker(DEVICE(o->pci_dev), o->unplug_blocker);
86
+
87
+ return;
88
+
89
+fail:
90
+ vfu_destroy_ctx(o->vfu_ctx);
91
+ if (o->unplug_blocker && o->pci_dev) {
92
+ qdev_del_unplug_blocker(DEVICE(o->pci_dev), o->unplug_blocker);
93
+ error_free(o->unplug_blocker);
94
+ o->unplug_blocker = NULL;
95
+ }
96
+ if (o->pci_dev) {
97
+ object_unref(OBJECT(o->pci_dev));
98
+ o->pci_dev = NULL;
99
+ }
100
+ o->vfu_ctx = NULL;
101
}
102
103
static void vfu_object_init(Object *obj)
104
@@ -XXX,XX +XXX,XX @@ static void vfu_object_finalize(Object *obj)
105
106
o->device = NULL;
107
108
+ if (o->unplug_blocker && o->pci_dev) {
109
+ qdev_del_unplug_blocker(DEVICE(o->pci_dev), o->unplug_blocker);
110
+ error_free(o->unplug_blocker);
111
+ o->unplug_blocker = NULL;
112
+ }
113
+
114
+ if (o->pci_dev) {
115
+ object_unref(OBJECT(o->pci_dev));
116
+ o->pci_dev = NULL;
117
+ }
118
+
119
if (!k->nr_devs && vfu_object_auto_shutdown()) {
120
qemu_system_shutdown_request(SHUTDOWN_CAUSE_GUEST_SHUTDOWN);
121
}
59
--
122
--
60
2.13.5
123
2.36.1
61
62
diff view generated by jsdifflib
1
From: Alberto Garcia <berto@igalia.com>
1
From: Jagannathan Raman <jag.raman@oracle.com>
2
2
3
Both the throttling limits set with the throttling.iops-* and
3
Setup a handler to run vfio-user context. The context is driven by
4
throttling.bps-* options and their QMP equivalents defined in the
4
messages to the file descriptor associated with it - get the fd for
5
BlockIOThrottle struct are integer values.
5
the context and hook up the handler with it
6
6
7
Those limits are also reported in the BlockDeviceInfo struct and they
7
Signed-off-by: Elena Ufimtseva <elena.ufimtseva@oracle.com>
8
are integers there as well.
8
Signed-off-by: John G Johnson <john.g.johnson@oracle.com>
9
9
Signed-off-by: Jagannathan Raman <jag.raman@oracle.com>
10
Therefore there's no reason to store them internally as double and do
10
Reviewed-by: Stefan Hajnoczi <stefanha@redhat.com>
11
the conversion everytime we're setting or querying them, so this patch
11
Message-id: e934b0090529d448b6a7972b21dfc3d7421ce494.1655151679.git.jag.raman@oracle.com
12
uses uint64_t for those types. Let's also use an unsigned type because
13
we don't allow negative values anyway.
14
15
LeakyBucket.level and LeakyBucket.burst_level do however remain double
16
because their value changes depending on the fraction of time elapsed
17
since the previous I/O operation.
18
19
Signed-off-by: Alberto Garcia <berto@igalia.com>
20
Message-id: f29b840422767b5be2c41c2dfdbbbf6c5f8fedf8.1503580370.git.berto@igalia.com
21
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
12
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
22
---
13
---
23
include/qemu/throttle.h | 4 ++--
14
qapi/misc.json | 31 ++++++++++
24
tests/test-throttle.c | 3 ++-
15
hw/remote/vfio-user-obj.c | 118 +++++++++++++++++++++++++++++++++++++-
25
util/throttle.c | 7 +++----
16
2 files changed, 148 insertions(+), 1 deletion(-)
26
3 files changed, 7 insertions(+), 7 deletions(-)
27
17
28
diff --git a/include/qemu/throttle.h b/include/qemu/throttle.h
18
diff --git a/qapi/misc.json b/qapi/misc.json
29
index XXXXXXX..XXXXXXX 100644
19
index XXXXXXX..XXXXXXX 100644
30
--- a/include/qemu/throttle.h
20
--- a/qapi/misc.json
31
+++ b/include/qemu/throttle.h
21
+++ b/qapi/misc.json
32
@@ -XXX,XX +XXX,XX @@ typedef enum {
22
@@ -XXX,XX +XXX,XX @@
23
##
24
{ 'event': 'RTC_CHANGE',
25
'data': { 'offset': 'int', 'qom-path': 'str' } }
26
+
27
+##
28
+# @VFU_CLIENT_HANGUP:
29
+#
30
+# Emitted when the client of a TYPE_VFIO_USER_SERVER closes the
31
+# communication channel
32
+#
33
+# @vfu-id: ID of the TYPE_VFIO_USER_SERVER object. It is the last component
34
+# of @vfu-qom-path referenced below
35
+#
36
+# @vfu-qom-path: path to the TYPE_VFIO_USER_SERVER object in the QOM tree
37
+#
38
+# @dev-id: ID of attached PCI device
39
+#
40
+# @dev-qom-path: path to attached PCI device in the QOM tree
41
+#
42
+# Since: 7.1
43
+#
44
+# Example:
45
+#
46
+# <- { "event": "VFU_CLIENT_HANGUP",
47
+# "data": { "vfu-id": "vfu1",
48
+# "vfu-qom-path": "/objects/vfu1",
49
+# "dev-id": "sas1",
50
+# "dev-qom-path": "/machine/peripheral/sas1" },
51
+# "timestamp": { "seconds": 1265044230, "microseconds": 450486 } }
52
+#
53
+##
54
+{ 'event': 'VFU_CLIENT_HANGUP',
55
+ 'data': { 'vfu-id': 'str', 'vfu-qom-path': 'str',
56
+ 'dev-id': 'str', 'dev-qom-path': 'str' } }
57
diff --git a/hw/remote/vfio-user-obj.c b/hw/remote/vfio-user-obj.c
58
index XXXXXXX..XXXXXXX 100644
59
--- a/hw/remote/vfio-user-obj.c
60
+++ b/hw/remote/vfio-user-obj.c
61
@@ -XXX,XX +XXX,XX @@
62
*
63
* device - id of a device on the server, a required option. PCI devices
64
* alone are supported presently.
65
+ *
66
+ * notes - x-vfio-user-server could block IO and monitor during the
67
+ * initialization phase.
33
*/
68
*/
34
69
35
typedef struct LeakyBucket {
70
#include "qemu/osdep.h"
36
- double avg; /* average goal in units per second */
71
@@ -XXX,XX +XXX,XX @@
37
- double max; /* leaky bucket max burst in units */
72
#include "hw/remote/machine.h"
38
+ uint64_t avg; /* average goal in units per second */
73
#include "qapi/error.h"
39
+ uint64_t max; /* leaky bucket max burst in units */
74
#include "qapi/qapi-visit-sockets.h"
40
double level; /* bucket level in units */
75
+#include "qapi/qapi-events-misc.h"
41
double burst_level; /* bucket level in units (for computing bursts) */
76
#include "qemu/notify.h"
42
unsigned burst_length; /* max length of the burst period, in seconds */
77
+#include "qemu/thread.h"
43
diff --git a/tests/test-throttle.c b/tests/test-throttle.c
78
#include "sysemu/sysemu.h"
44
index XXXXXXX..XXXXXXX 100644
79
#include "libvfio-user.h"
45
--- a/tests/test-throttle.c
80
#include "hw/qdev-core.h"
46
+++ b/tests/test-throttle.c
81
#include "hw/pci/pci.h"
47
@@ -XXX,XX +XXX,XX @@ static void test_enabled(void)
82
+#include "qemu/timer.h"
48
for (i = 0; i < BUCKETS_COUNT; i++) {
83
49
throttle_config_init(&cfg);
84
#define TYPE_VFU_OBJECT "x-vfio-user-server"
50
set_cfg_value(false, i, 150);
85
OBJECT_DECLARE_TYPE(VfuObject, VfuObjectClass, VFU_OBJECT)
51
+ g_assert(throttle_is_valid(&cfg, NULL));
86
@@ -XXX,XX +XXX,XX @@ struct VfuObject {
52
g_assert(throttle_enabled(&cfg));
87
PCIDevice *pci_dev;
53
}
88
54
89
Error *unplug_blocker;
55
for (i = 0; i < BUCKETS_COUNT; i++) {
90
+
56
throttle_config_init(&cfg);
91
+ int vfu_poll_fd;
57
set_cfg_value(false, i, -150);
92
};
58
- g_assert(!throttle_enabled(&cfg));
93
59
+ g_assert(!throttle_is_valid(&cfg, NULL));
94
static void vfu_object_init_ctx(VfuObject *o, Error **errp);
95
@@ -XXX,XX +XXX,XX @@ static void vfu_object_set_device(Object *obj, const char *str, Error **errp)
96
vfu_object_init_ctx(o, errp);
97
}
98
99
+static void vfu_object_ctx_run(void *opaque)
100
+{
101
+ VfuObject *o = opaque;
102
+ const char *vfu_id;
103
+ char *vfu_path, *pci_dev_path;
104
+ int ret = -1;
105
+
106
+ while (ret != 0) {
107
+ ret = vfu_run_ctx(o->vfu_ctx);
108
+ if (ret < 0) {
109
+ if (errno == EINTR) {
110
+ continue;
111
+ } else if (errno == ENOTCONN) {
112
+ vfu_id = object_get_canonical_path_component(OBJECT(o));
113
+ vfu_path = object_get_canonical_path(OBJECT(o));
114
+ g_assert(o->pci_dev);
115
+ pci_dev_path = object_get_canonical_path(OBJECT(o->pci_dev));
116
+ /* o->device is a required property and is non-NULL here */
117
+ g_assert(o->device);
118
+ qapi_event_send_vfu_client_hangup(vfu_id, vfu_path,
119
+ o->device, pci_dev_path);
120
+ qemu_set_fd_handler(o->vfu_poll_fd, NULL, NULL, NULL);
121
+ o->vfu_poll_fd = -1;
122
+ object_unparent(OBJECT(o));
123
+ g_free(vfu_path);
124
+ g_free(pci_dev_path);
125
+ break;
126
+ } else {
127
+ VFU_OBJECT_ERROR(o, "vfu: Failed to run device %s - %s",
128
+ o->device, strerror(errno));
129
+ break;
130
+ }
131
+ }
132
+ }
133
+}
134
+
135
+static void vfu_object_attach_ctx(void *opaque)
136
+{
137
+ VfuObject *o = opaque;
138
+ GPollFD pfds[1];
139
+ int ret;
140
+
141
+ qemu_set_fd_handler(o->vfu_poll_fd, NULL, NULL, NULL);
142
+
143
+ pfds[0].fd = o->vfu_poll_fd;
144
+ pfds[0].events = G_IO_IN | G_IO_HUP | G_IO_ERR;
145
+
146
+retry_attach:
147
+ ret = vfu_attach_ctx(o->vfu_ctx);
148
+ if (ret < 0 && (errno == EAGAIN || errno == EWOULDBLOCK)) {
149
+ /**
150
+ * vfu_object_attach_ctx can block QEMU's main loop
151
+ * during attach - the monitor and other IO
152
+ * could be unresponsive during this time.
153
+ */
154
+ (void)qemu_poll_ns(pfds, 1, 500 * (int64_t)SCALE_MS);
155
+ goto retry_attach;
156
+ } else if (ret < 0) {
157
+ VFU_OBJECT_ERROR(o, "vfu: Failed to attach device %s to context - %s",
158
+ o->device, strerror(errno));
159
+ return;
160
+ }
161
+
162
+ o->vfu_poll_fd = vfu_get_poll_fd(o->vfu_ctx);
163
+ if (o->vfu_poll_fd < 0) {
164
+ VFU_OBJECT_ERROR(o, "vfu: Failed to get poll fd %s", o->device);
165
+ return;
166
+ }
167
+
168
+ qemu_set_fd_handler(o->vfu_poll_fd, vfu_object_ctx_run, NULL, o);
169
+}
170
+
171
/*
172
* TYPE_VFU_OBJECT depends on the availability of the 'socket' and 'device'
173
* properties. It also depends on devices instantiated in QEMU. These
174
@@ -XXX,XX +XXX,XX @@ static void vfu_object_machine_done(Notifier *notifier, void *data)
60
}
175
}
61
}
176
}
62
177
63
diff --git a/util/throttle.c b/util/throttle.c
178
+/**
64
index XXXXXXX..XXXXXXX 100644
179
+ * vfu_object_init_ctx: Create and initialize libvfio-user context. Add
65
--- a/util/throttle.c
180
+ * an unplug blocker for the associated PCI device. Setup a FD handler
66
+++ b/util/throttle.c
181
+ * to process incoming messages in the context's socket.
67
@@ -XXX,XX +XXX,XX @@ int64_t throttle_compute_wait(LeakyBucket *bkt)
182
+ *
68
/* If bkt->max is 0 we still want to allow short bursts of I/O
183
+ * The socket and device properties are mandatory, and this function
69
* from the guest, otherwise every other request will be throttled
184
+ * will not create the context without them - the setters for these
70
* and performance will suffer considerably. */
185
+ * properties should call this function when the property is set. The
71
- bucket_size = bkt->avg / 10;
186
+ * machine should also be ready when this function is invoked - it is
72
+ bucket_size = (double) bkt->avg / 10;
187
+ * because QEMU objects are initialized before devices, and the
73
burst_bucket_size = 0;
188
+ * associated PCI device wouldn't be available at the object
74
} else {
189
+ * initialization time. Until these conditions are satisfied, this
75
/* If we have a burst limit then we have to wait until all I/O
190
+ * function would return early without performing any task.
76
* at burst rate has finished before throttling to bkt->avg */
191
+ */
77
bucket_size = bkt->max * bkt->burst_length;
192
static void vfu_object_init_ctx(VfuObject *o, Error **errp)
78
- burst_bucket_size = bkt->max / 10;
193
{
79
+ burst_bucket_size = (double) bkt->max / 10;
194
ERRP_GUARD();
195
@@ -XXX,XX +XXX,XX @@ static void vfu_object_init_ctx(VfuObject *o, Error **errp)
196
return;
80
}
197
}
81
198
82
/* If the main bucket is full then we have to wait */
199
- o->vfu_ctx = vfu_create_ctx(VFU_TRANS_SOCK, o->socket->u.q_unix.path, 0,
83
@@ -XXX,XX +XXX,XX @@ bool throttle_is_valid(ThrottleConfig *cfg, Error **errp)
200
+ o->vfu_ctx = vfu_create_ctx(VFU_TRANS_SOCK, o->socket->u.q_unix.path,
84
201
+ LIBVFIO_USER_FLAG_ATTACH_NB,
85
for (i = 0; i < BUCKETS_COUNT; i++) {
202
o, VFU_DEV_TYPE_PCI);
86
LeakyBucket *bkt = &cfg->buckets[i];
203
if (o->vfu_ctx == NULL) {
87
- if (bkt->avg < 0 || bkt->max < 0 ||
204
error_setg(errp, "vfu: Failed to create context - %s", strerror(errno));
88
- bkt->avg > THROTTLE_VALUE_MAX || bkt->max > THROTTLE_VALUE_MAX) {
205
@@ -XXX,XX +XXX,XX @@ static void vfu_object_init_ctx(VfuObject *o, Error **errp)
89
+ if (bkt->avg > THROTTLE_VALUE_MAX || bkt->max > THROTTLE_VALUE_MAX) {
206
TYPE_VFU_OBJECT, o->device);
90
error_setg(errp, "bps/iops/max values must be within [0, %lld]",
207
qdev_add_unplug_blocker(DEVICE(o->pci_dev), o->unplug_blocker);
91
THROTTLE_VALUE_MAX);
208
92
return false;
209
+ ret = vfu_realize_ctx(o->vfu_ctx);
210
+ if (ret < 0) {
211
+ error_setg(errp, "vfu: Failed to realize device %s- %s",
212
+ o->device, strerror(errno));
213
+ goto fail;
214
+ }
215
+
216
+ o->vfu_poll_fd = vfu_get_poll_fd(o->vfu_ctx);
217
+ if (o->vfu_poll_fd < 0) {
218
+ error_setg(errp, "vfu: Failed to get poll fd %s", o->device);
219
+ goto fail;
220
+ }
221
+
222
+ qemu_set_fd_handler(o->vfu_poll_fd, vfu_object_attach_ctx, NULL, o);
223
+
224
return;
225
226
fail:
227
@@ -XXX,XX +XXX,XX @@ static void vfu_object_init(Object *obj)
228
qemu_add_machine_init_done_notifier(&o->machine_done);
229
}
230
231
+ o->vfu_poll_fd = -1;
232
}
233
234
static void vfu_object_finalize(Object *obj)
235
@@ -XXX,XX +XXX,XX @@ static void vfu_object_finalize(Object *obj)
236
237
o->socket = NULL;
238
239
+ if (o->vfu_poll_fd != -1) {
240
+ qemu_set_fd_handler(o->vfu_poll_fd, NULL, NULL, NULL);
241
+ o->vfu_poll_fd = -1;
242
+ }
243
+
244
if (o->vfu_ctx) {
245
vfu_destroy_ctx(o->vfu_ctx);
246
o->vfu_ctx = NULL;
93
--
247
--
94
2.13.5
248
2.36.1
95
96
diff view generated by jsdifflib
1
From: Alberto Garcia <berto@igalia.com>
1
From: Jagannathan Raman <jag.raman@oracle.com>
2
2
3
The way the throttling algorithm works is that requests start being
3
Define and register handlers for PCI config space accesses
4
throttled once the bucket level exceeds the burst limit. When we get
5
there the bucket leaks at the level set by the user (bkt->avg), and
6
that leak rate is what prevents guest I/O from exceeding the desired
7
limit.
8
4
9
If we don't allow bursts (i.e. bkt->max == 0) then we can start
5
Signed-off-by: Elena Ufimtseva <elena.ufimtseva@oracle.com>
10
throttling requests immediately. The problem with keeping the
6
Signed-off-by: John G Johnson <john.g.johnson@oracle.com>
11
threshold at 0 is that it only allows one request at a time, and as
7
Signed-off-by: Jagannathan Raman <jag.raman@oracle.com>
12
soon as there's a bit of I/O from the guest every other request will
8
Reviewed-by: Stefan Hajnoczi <stefanha@redhat.com>
13
be throttled and performance will suffer considerably. That can even
9
Message-id: be9d2ccf9b1d24e50dcd9c23404dbf284142cec7.1655151679.git.jag.raman@oracle.com
14
make the guest unable to reach the throttle limit if that limit is
15
high enough, and that happens regardless of the block scheduler used
16
by the guest.
17
18
Increasing that threshold gives flexibility to the guest, allowing it
19
to perform short bursts of I/O before being throttled. Increasing the
20
threshold too much does not make a difference in the long run (because
21
it's the leak rate what defines the actual throughput) but it does
22
allow the guest to perform longer initial bursts and exceed the
23
throttle limit for a short while.
24
25
A burst value of bkt->avg / 10 allows the guest to perform 100ms'
26
worth of I/O at the target rate without being throttled.
27
28
Signed-off-by: Alberto Garcia <berto@igalia.com>
29
Message-id: 31aae6645f0d1fbf3860fb2b528b757236f0c0a7.1503580370.git.berto@igalia.com
30
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
10
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
31
---
11
---
32
util/throttle.c | 11 +++--------
12
hw/remote/vfio-user-obj.c | 51 +++++++++++++++++++++++++++++++++++++++
33
1 file changed, 3 insertions(+), 8 deletions(-)
13
hw/remote/trace-events | 2 ++
14
2 files changed, 53 insertions(+)
34
15
35
diff --git a/util/throttle.c b/util/throttle.c
16
diff --git a/hw/remote/vfio-user-obj.c b/hw/remote/vfio-user-obj.c
36
index XXXXXXX..XXXXXXX 100644
17
index XXXXXXX..XXXXXXX 100644
37
--- a/util/throttle.c
18
--- a/hw/remote/vfio-user-obj.c
38
+++ b/util/throttle.c
19
+++ b/hw/remote/vfio-user-obj.c
39
@@ -XXX,XX +XXX,XX @@ static void throttle_fix_bucket(LeakyBucket *bkt)
20
@@ -XXX,XX +XXX,XX @@
40
/* zero bucket level */
21
#include "qapi/qapi-events-misc.h"
41
bkt->level = bkt->burst_level = 0;
22
#include "qemu/notify.h"
42
23
#include "qemu/thread.h"
43
- /* The following is done to cope with the Linux CFQ block scheduler
24
+#include "qemu/main-loop.h"
44
- * which regroup reads and writes by block of 100ms in the guest.
25
#include "sysemu/sysemu.h"
45
- * When they are two process one making reads and one making writes cfq
26
#include "libvfio-user.h"
46
- * make a pattern looking like the following:
27
#include "hw/qdev-core.h"
47
- * WWWWWWWWWWWRRRRRRRRRRRRRRWWWWWWWWWWWWWwRRRRRRRRRRRRRRRRR
28
@@ -XXX,XX +XXX,XX @@ retry_attach:
48
- * Having a max burst value of 100ms of the average will help smooth the
29
qemu_set_fd_handler(o->vfu_poll_fd, vfu_object_ctx_run, NULL, o);
49
- * throttling
30
}
50
- */
31
51
+ /* If bkt->max is 0 we still want to allow short bursts of I/O
32
+static ssize_t vfu_object_cfg_access(vfu_ctx_t *vfu_ctx, char * const buf,
52
+ * from the guest, otherwise every other request will be throttled
33
+ size_t count, loff_t offset,
53
+ * and performance will suffer considerably. */
34
+ const bool is_write)
54
min = bkt->avg / 10;
35
+{
55
if (bkt->avg && !bkt->max) {
36
+ VfuObject *o = vfu_get_private(vfu_ctx);
56
bkt->max = min;
37
+ uint32_t pci_access_width = sizeof(uint32_t);
38
+ size_t bytes = count;
39
+ uint32_t val = 0;
40
+ char *ptr = buf;
41
+ int len;
42
+
43
+ /*
44
+ * Writes to the BAR registers would trigger an update to the
45
+ * global Memory and IO AddressSpaces. But the remote device
46
+ * never uses the global AddressSpaces, therefore overlapping
47
+ * memory regions are not a problem
48
+ */
49
+ while (bytes > 0) {
50
+ len = (bytes > pci_access_width) ? pci_access_width : bytes;
51
+ if (is_write) {
52
+ memcpy(&val, ptr, len);
53
+ pci_host_config_write_common(o->pci_dev, offset,
54
+ pci_config_size(o->pci_dev),
55
+ val, len);
56
+ trace_vfu_cfg_write(offset, val);
57
+ } else {
58
+ val = pci_host_config_read_common(o->pci_dev, offset,
59
+ pci_config_size(o->pci_dev), len);
60
+ memcpy(ptr, &val, len);
61
+ trace_vfu_cfg_read(offset, val);
62
+ }
63
+ offset += len;
64
+ ptr += len;
65
+ bytes -= len;
66
+ }
67
+
68
+ return count;
69
+}
70
+
71
/*
72
* TYPE_VFU_OBJECT depends on the availability of the 'socket' and 'device'
73
* properties. It also depends on devices instantiated in QEMU. These
74
@@ -XXX,XX +XXX,XX @@ static void vfu_object_init_ctx(VfuObject *o, Error **errp)
75
TYPE_VFU_OBJECT, o->device);
76
qdev_add_unplug_blocker(DEVICE(o->pci_dev), o->unplug_blocker);
77
78
+ ret = vfu_setup_region(o->vfu_ctx, VFU_PCI_DEV_CFG_REGION_IDX,
79
+ pci_config_size(o->pci_dev), &vfu_object_cfg_access,
80
+ VFU_REGION_FLAG_RW | VFU_REGION_FLAG_ALWAYS_CB,
81
+ NULL, 0, -1, 0);
82
+ if (ret < 0) {
83
+ error_setg(errp,
84
+ "vfu: Failed to setup config space handlers for %s- %s",
85
+ o->device, strerror(errno));
86
+ goto fail;
87
+ }
88
+
89
ret = vfu_realize_ctx(o->vfu_ctx);
90
if (ret < 0) {
91
error_setg(errp, "vfu: Failed to realize device %s- %s",
92
diff --git a/hw/remote/trace-events b/hw/remote/trace-events
93
index XXXXXXX..XXXXXXX 100644
94
--- a/hw/remote/trace-events
95
+++ b/hw/remote/trace-events
96
@@ -XXX,XX +XXX,XX @@ mpqemu_recv_io_error(int cmd, int size, int nfds) "failed to receive %d size %d,
97
98
# vfio-user-obj.c
99
vfu_prop(const char *prop, const char *val) "vfu: setting %s as %s"
100
+vfu_cfg_read(uint32_t offset, uint32_t val) "vfu: cfg: 0x%u -> 0x%x"
101
+vfu_cfg_write(uint32_t offset, uint32_t val) "vfu: cfg: 0x%u <- 0x%x"
57
--
102
--
58
2.13.5
103
2.36.1
59
60
diff view generated by jsdifflib
1
From: Alberto Garcia <berto@igalia.com>
1
From: Jagannathan Raman <jag.raman@oracle.com>
2
2
3
The level of the burst bucket is stored in bkt.burst_level, not
3
Assign separate address space for each device in the remote processes.
4
bkt.burst_length.
4
5
5
Signed-off-by: Elena Ufimtseva <elena.ufimtseva@oracle.com>
6
Signed-off-by: Alberto Garcia <berto@igalia.com>
6
Signed-off-by: John G Johnson <john.g.johnson@oracle.com>
7
Reviewed-by: Manos Pitsidianakis <el13635@mail.ntua.gr>
7
Signed-off-by: Jagannathan Raman <jag.raman@oracle.com>
8
Message-id: 49aab2711d02f285567f3b3b13a113847af33812.1503580370.git.berto@igalia.com
8
Reviewed-by: Stefan Hajnoczi <stefanha@redhat.com>
9
Message-id: afe0b0a97582cdad42b5b25636a29c523265a10a.1655151679.git.jag.raman@oracle.com
9
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
10
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
10
---
11
---
11
include/qemu/throttle.h | 2 +-
12
MAINTAINERS | 2 +
12
1 file changed, 1 insertion(+), 1 deletion(-)
13
include/hw/remote/iommu.h | 40 ++++++++++++
13
14
hw/remote/iommu.c | 131 ++++++++++++++++++++++++++++++++++++++
14
diff --git a/include/qemu/throttle.h b/include/qemu/throttle.h
15
hw/remote/machine.c | 13 +++-
16
hw/remote/meson.build | 1 +
17
5 files changed, 186 insertions(+), 1 deletion(-)
18
create mode 100644 include/hw/remote/iommu.h
19
create mode 100644 hw/remote/iommu.c
20
21
diff --git a/MAINTAINERS b/MAINTAINERS
15
index XXXXXXX..XXXXXXX 100644
22
index XXXXXXX..XXXXXXX 100644
16
--- a/include/qemu/throttle.h
23
--- a/MAINTAINERS
17
+++ b/include/qemu/throttle.h
24
+++ b/MAINTAINERS
18
@@ -XXX,XX +XXX,XX @@ typedef enum {
25
@@ -XXX,XX +XXX,XX @@ F: hw/remote/iohub.c
19
* - The bkt.avg rate does not apply until the bucket is full,
26
F: include/hw/remote/iohub.h
20
* allowing the user to do bursts until then. The I/O limit during
27
F: subprojects/libvfio-user
21
* bursts is bkt.max. To enforce this limit we keep an additional
28
F: hw/remote/vfio-user-obj.c
22
- * bucket in bkt.burst_length that leaks at a rate of bkt.max units
29
+F: hw/remote/iommu.c
23
+ * bucket in bkt.burst_level that leaks at a rate of bkt.max units
30
+F: include/hw/remote/iommu.h
24
* per second.
31
25
*
32
EBPF:
26
* - Because of all of the above, the user can perform I/O at a
33
M: Jason Wang <jasowang@redhat.com>
34
diff --git a/include/hw/remote/iommu.h b/include/hw/remote/iommu.h
35
new file mode 100644
36
index XXXXXXX..XXXXXXX
37
--- /dev/null
38
+++ b/include/hw/remote/iommu.h
39
@@ -XXX,XX +XXX,XX @@
40
+/**
41
+ * Copyright © 2022 Oracle and/or its affiliates.
42
+ *
43
+ * This work is licensed under the terms of the GNU GPL, version 2 or later.
44
+ * See the COPYING file in the top-level directory.
45
+ *
46
+ */
47
+
48
+#ifndef REMOTE_IOMMU_H
49
+#define REMOTE_IOMMU_H
50
+
51
+#include "hw/pci/pci_bus.h"
52
+#include "hw/pci/pci.h"
53
+
54
+#ifndef INT2VOIDP
55
+#define INT2VOIDP(i) (void *)(uintptr_t)(i)
56
+#endif
57
+
58
+typedef struct RemoteIommuElem {
59
+ MemoryRegion *mr;
60
+
61
+ AddressSpace as;
62
+} RemoteIommuElem;
63
+
64
+#define TYPE_REMOTE_IOMMU "x-remote-iommu"
65
+OBJECT_DECLARE_SIMPLE_TYPE(RemoteIommu, REMOTE_IOMMU)
66
+
67
+struct RemoteIommu {
68
+ Object parent;
69
+
70
+ GHashTable *elem_by_devfn;
71
+
72
+ QemuMutex lock;
73
+};
74
+
75
+void remote_iommu_setup(PCIBus *pci_bus);
76
+
77
+void remote_iommu_unplug_dev(PCIDevice *pci_dev);
78
+
79
+#endif
80
diff --git a/hw/remote/iommu.c b/hw/remote/iommu.c
81
new file mode 100644
82
index XXXXXXX..XXXXXXX
83
--- /dev/null
84
+++ b/hw/remote/iommu.c
85
@@ -XXX,XX +XXX,XX @@
86
+/**
87
+ * IOMMU for remote device
88
+ *
89
+ * Copyright © 2022 Oracle and/or its affiliates.
90
+ *
91
+ * This work is licensed under the terms of the GNU GPL, version 2 or later.
92
+ * See the COPYING file in the top-level directory.
93
+ *
94
+ */
95
+
96
+#include "qemu/osdep.h"
97
+
98
+#include "hw/remote/iommu.h"
99
+#include "hw/pci/pci_bus.h"
100
+#include "hw/pci/pci.h"
101
+#include "exec/memory.h"
102
+#include "exec/address-spaces.h"
103
+#include "trace.h"
104
+
105
+/**
106
+ * IOMMU for TYPE_REMOTE_MACHINE - manages DMA address space isolation
107
+ * for remote machine. It is used by TYPE_VFIO_USER_SERVER.
108
+ *
109
+ * - Each TYPE_VFIO_USER_SERVER instance handles one PCIDevice on a PCIBus.
110
+ * There is one RemoteIommu per PCIBus, so the RemoteIommu tracks multiple
111
+ * PCIDevices by maintaining a ->elem_by_devfn mapping.
112
+ *
113
+ * - memory_region_init_iommu() is not used because vfio-user MemoryRegions
114
+ * will be added to the elem->mr container instead. This is more natural
115
+ * than implementing the IOMMUMemoryRegionClass APIs since vfio-user
116
+ * provides something that is close to a full-fledged MemoryRegion and
117
+ * not like an IOMMU mapping.
118
+ *
119
+ * - When a device is hot unplugged, the elem->mr reference is dropped so
120
+ * all vfio-user MemoryRegions associated with this vfio-user server are
121
+ * destroyed.
122
+ */
123
+
124
+static AddressSpace *remote_iommu_find_add_as(PCIBus *pci_bus,
125
+ void *opaque, int devfn)
126
+{
127
+ RemoteIommu *iommu = opaque;
128
+ RemoteIommuElem *elem = NULL;
129
+
130
+ qemu_mutex_lock(&iommu->lock);
131
+
132
+ elem = g_hash_table_lookup(iommu->elem_by_devfn, INT2VOIDP(devfn));
133
+
134
+ if (!elem) {
135
+ elem = g_malloc0(sizeof(RemoteIommuElem));
136
+ g_hash_table_insert(iommu->elem_by_devfn, INT2VOIDP(devfn), elem);
137
+ }
138
+
139
+ if (!elem->mr) {
140
+ elem->mr = MEMORY_REGION(object_new(TYPE_MEMORY_REGION));
141
+ memory_region_set_size(elem->mr, UINT64_MAX);
142
+ address_space_init(&elem->as, elem->mr, NULL);
143
+ }
144
+
145
+ qemu_mutex_unlock(&iommu->lock);
146
+
147
+ return &elem->as;
148
+}
149
+
150
+void remote_iommu_unplug_dev(PCIDevice *pci_dev)
151
+{
152
+ AddressSpace *as = pci_device_iommu_address_space(pci_dev);
153
+ RemoteIommuElem *elem = NULL;
154
+
155
+ if (as == &address_space_memory) {
156
+ return;
157
+ }
158
+
159
+ elem = container_of(as, RemoteIommuElem, as);
160
+
161
+ address_space_destroy(&elem->as);
162
+
163
+ object_unref(elem->mr);
164
+
165
+ elem->mr = NULL;
166
+}
167
+
168
+static void remote_iommu_init(Object *obj)
169
+{
170
+ RemoteIommu *iommu = REMOTE_IOMMU(obj);
171
+
172
+ iommu->elem_by_devfn = g_hash_table_new_full(NULL, NULL, NULL, g_free);
173
+
174
+ qemu_mutex_init(&iommu->lock);
175
+}
176
+
177
+static void remote_iommu_finalize(Object *obj)
178
+{
179
+ RemoteIommu *iommu = REMOTE_IOMMU(obj);
180
+
181
+ qemu_mutex_destroy(&iommu->lock);
182
+
183
+ g_hash_table_destroy(iommu->elem_by_devfn);
184
+
185
+ iommu->elem_by_devfn = NULL;
186
+}
187
+
188
+void remote_iommu_setup(PCIBus *pci_bus)
189
+{
190
+ RemoteIommu *iommu = NULL;
191
+
192
+ g_assert(pci_bus);
193
+
194
+ iommu = REMOTE_IOMMU(object_new(TYPE_REMOTE_IOMMU));
195
+
196
+ pci_setup_iommu(pci_bus, remote_iommu_find_add_as, iommu);
197
+
198
+ object_property_add_child(OBJECT(pci_bus), "remote-iommu", OBJECT(iommu));
199
+
200
+ object_unref(OBJECT(iommu));
201
+}
202
+
203
+static const TypeInfo remote_iommu_info = {
204
+ .name = TYPE_REMOTE_IOMMU,
205
+ .parent = TYPE_OBJECT,
206
+ .instance_size = sizeof(RemoteIommu),
207
+ .instance_init = remote_iommu_init,
208
+ .instance_finalize = remote_iommu_finalize,
209
+};
210
+
211
+static void remote_iommu_register_types(void)
212
+{
213
+ type_register_static(&remote_iommu_info);
214
+}
215
+
216
+type_init(remote_iommu_register_types)
217
diff --git a/hw/remote/machine.c b/hw/remote/machine.c
218
index XXXXXXX..XXXXXXX 100644
219
--- a/hw/remote/machine.c
220
+++ b/hw/remote/machine.c
221
@@ -XXX,XX +XXX,XX @@
222
#include "qapi/error.h"
223
#include "hw/pci/pci_host.h"
224
#include "hw/remote/iohub.h"
225
+#include "hw/remote/iommu.h"
226
#include "hw/qdev-core.h"
227
228
static void remote_machine_init(MachineState *machine)
229
@@ -XXX,XX +XXX,XX @@ static void remote_machine_instance_init(Object *obj)
230
s->auto_shutdown = true;
231
}
232
233
+static void remote_machine_dev_unplug_cb(HotplugHandler *hotplug_dev,
234
+ DeviceState *dev, Error **errp)
235
+{
236
+ qdev_unrealize(dev);
237
+
238
+ if (object_dynamic_cast(OBJECT(dev), TYPE_PCI_DEVICE)) {
239
+ remote_iommu_unplug_dev(PCI_DEVICE(dev));
240
+ }
241
+}
242
+
243
static void remote_machine_class_init(ObjectClass *oc, void *data)
244
{
245
MachineClass *mc = MACHINE_CLASS(oc);
246
@@ -XXX,XX +XXX,XX @@ static void remote_machine_class_init(ObjectClass *oc, void *data)
247
mc->init = remote_machine_init;
248
mc->desc = "Experimental remote machine";
249
250
- hc->unplug = qdev_simple_device_unplug_cb;
251
+ hc->unplug = remote_machine_dev_unplug_cb;
252
253
object_class_property_add_bool(oc, "vfio-user",
254
remote_machine_get_vfio_user,
255
diff --git a/hw/remote/meson.build b/hw/remote/meson.build
256
index XXXXXXX..XXXXXXX 100644
257
--- a/hw/remote/meson.build
258
+++ b/hw/remote/meson.build
259
@@ -XXX,XX +XXX,XX @@ remote_ss.add(when: 'CONFIG_MULTIPROCESS', if_true: files('message.c'))
260
remote_ss.add(when: 'CONFIG_MULTIPROCESS', if_true: files('remote-obj.c'))
261
remote_ss.add(when: 'CONFIG_MULTIPROCESS', if_true: files('proxy.c'))
262
remote_ss.add(when: 'CONFIG_MULTIPROCESS', if_true: files('iohub.c'))
263
+remote_ss.add(when: 'CONFIG_MULTIPROCESS', if_true: files('iommu.c'))
264
remote_ss.add(when: 'CONFIG_VFIO_USER_SERVER', if_true: files('vfio-user-obj.c'))
265
266
remote_ss.add(when: 'CONFIG_VFIO_USER_SERVER', if_true: libvfio_user_dep)
27
--
267
--
28
2.13.5
268
2.36.1
29
269
30
270
diff view generated by jsdifflib
1
From: Alberto Garcia <berto@igalia.com>
1
From: Jagannathan Raman <jag.raman@oracle.com>
2
2
3
Use a pointer to the bucket instead of repeating cfg->buckets[i] all
3
Define and register callbacks to manage the RAM regions used for
4
the time. This makes the code more concise and will help us expand the
4
device DMA
5
checks later and save a few line breaks.
6
5
7
Signed-off-by: Alberto Garcia <berto@igalia.com>
6
Signed-off-by: Elena Ufimtseva <elena.ufimtseva@oracle.com>
8
Message-id: 763ffc40a26b17d54cf93f5a999e4656049fcf0c.1503580370.git.berto@igalia.com
7
Signed-off-by: John G Johnson <john.g.johnson@oracle.com>
8
Signed-off-by: Jagannathan Raman <jag.raman@oracle.com>
9
Reviewed-by: Stefan Hajnoczi <stefanha@redhat.com>
10
Message-id: faacbcd45c4d02c591f0dbfdc19041fbb3eae7eb.1655151679.git.jag.raman@oracle.com
9
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
11
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
10
---
12
---
11
util/throttle.c | 15 +++++++--------
13
hw/remote/machine.c | 5 ++++
12
1 file changed, 7 insertions(+), 8 deletions(-)
14
hw/remote/vfio-user-obj.c | 55 +++++++++++++++++++++++++++++++++++++++
15
hw/remote/trace-events | 2 ++
16
3 files changed, 62 insertions(+)
13
17
14
diff --git a/util/throttle.c b/util/throttle.c
18
diff --git a/hw/remote/machine.c b/hw/remote/machine.c
15
index XXXXXXX..XXXXXXX 100644
19
index XXXXXXX..XXXXXXX 100644
16
--- a/util/throttle.c
20
--- a/hw/remote/machine.c
17
+++ b/util/throttle.c
21
+++ b/hw/remote/machine.c
18
@@ -XXX,XX +XXX,XX @@ bool throttle_is_valid(ThrottleConfig *cfg, Error **errp)
22
@@ -XXX,XX +XXX,XX @@
23
#include "hw/remote/iohub.h"
24
#include "hw/remote/iommu.h"
25
#include "hw/qdev-core.h"
26
+#include "hw/remote/iommu.h"
27
28
static void remote_machine_init(MachineState *machine)
29
{
30
@@ -XXX,XX +XXX,XX @@ static void remote_machine_init(MachineState *machine)
31
32
pci_host = PCI_HOST_BRIDGE(rem_host);
33
34
+ if (s->vfio_user) {
35
+ remote_iommu_setup(pci_host->bus);
36
+ }
37
+
38
remote_iohub_init(&s->iohub);
39
40
pci_bus_irqs(pci_host->bus, remote_iohub_set_irq, remote_iohub_map_irq,
41
diff --git a/hw/remote/vfio-user-obj.c b/hw/remote/vfio-user-obj.c
42
index XXXXXXX..XXXXXXX 100644
43
--- a/hw/remote/vfio-user-obj.c
44
+++ b/hw/remote/vfio-user-obj.c
45
@@ -XXX,XX +XXX,XX @@ static ssize_t vfu_object_cfg_access(vfu_ctx_t *vfu_ctx, char * const buf,
46
return count;
47
}
48
49
+static void dma_register(vfu_ctx_t *vfu_ctx, vfu_dma_info_t *info)
50
+{
51
+ VfuObject *o = vfu_get_private(vfu_ctx);
52
+ AddressSpace *dma_as = NULL;
53
+ MemoryRegion *subregion = NULL;
54
+ g_autofree char *name = NULL;
55
+ struct iovec *iov = &info->iova;
56
+
57
+ if (!info->vaddr) {
58
+ return;
59
+ }
60
+
61
+ name = g_strdup_printf("mem-%s-%"PRIx64"", o->device,
62
+ (uint64_t)info->vaddr);
63
+
64
+ subregion = g_new0(MemoryRegion, 1);
65
+
66
+ memory_region_init_ram_ptr(subregion, NULL, name,
67
+ iov->iov_len, info->vaddr);
68
+
69
+ dma_as = pci_device_iommu_address_space(o->pci_dev);
70
+
71
+ memory_region_add_subregion(dma_as->root, (hwaddr)iov->iov_base, subregion);
72
+
73
+ trace_vfu_dma_register((uint64_t)iov->iov_base, iov->iov_len);
74
+}
75
+
76
+static void dma_unregister(vfu_ctx_t *vfu_ctx, vfu_dma_info_t *info)
77
+{
78
+ VfuObject *o = vfu_get_private(vfu_ctx);
79
+ AddressSpace *dma_as = NULL;
80
+ MemoryRegion *mr = NULL;
81
+ ram_addr_t offset;
82
+
83
+ mr = memory_region_from_host(info->vaddr, &offset);
84
+ if (!mr) {
85
+ return;
86
+ }
87
+
88
+ dma_as = pci_device_iommu_address_space(o->pci_dev);
89
+
90
+ memory_region_del_subregion(dma_as->root, mr);
91
+
92
+ object_unparent((OBJECT(mr)));
93
+
94
+ trace_vfu_dma_unregister((uint64_t)info->iova.iov_base);
95
+}
96
+
97
/*
98
* TYPE_VFU_OBJECT depends on the availability of the 'socket' and 'device'
99
* properties. It also depends on devices instantiated in QEMU. These
100
@@ -XXX,XX +XXX,XX @@ static void vfu_object_init_ctx(VfuObject *o, Error **errp)
101
goto fail;
19
}
102
}
20
103
21
for (i = 0; i < BUCKETS_COUNT; i++) {
104
+ ret = vfu_setup_device_dma(o->vfu_ctx, &dma_register, &dma_unregister);
22
- if (cfg->buckets[i].avg < 0 ||
105
+ if (ret < 0) {
23
- cfg->buckets[i].max < 0 ||
106
+ error_setg(errp, "vfu: Failed to setup DMA handlers for %s",
24
- cfg->buckets[i].avg > THROTTLE_VALUE_MAX ||
107
+ o->device);
25
- cfg->buckets[i].max > THROTTLE_VALUE_MAX) {
108
+ goto fail;
26
+ LeakyBucket *bkt = &cfg->buckets[i];
109
+ }
27
+ if (bkt->avg < 0 || bkt->max < 0 ||
110
+
28
+ bkt->avg > THROTTLE_VALUE_MAX || bkt->max > THROTTLE_VALUE_MAX) {
111
ret = vfu_realize_ctx(o->vfu_ctx);
29
error_setg(errp, "bps/iops/max values must be within [0, %lld]",
112
if (ret < 0) {
30
THROTTLE_VALUE_MAX);
113
error_setg(errp, "vfu: Failed to realize device %s- %s",
31
return false;
114
diff --git a/hw/remote/trace-events b/hw/remote/trace-events
32
}
115
index XXXXXXX..XXXXXXX 100644
33
116
--- a/hw/remote/trace-events
34
- if (!cfg->buckets[i].burst_length) {
117
+++ b/hw/remote/trace-events
35
+ if (!bkt->burst_length) {
118
@@ -XXX,XX +XXX,XX @@ mpqemu_recv_io_error(int cmd, int size, int nfds) "failed to receive %d size %d,
36
error_setg(errp, "the burst length cannot be 0");
119
vfu_prop(const char *prop, const char *val) "vfu: setting %s as %s"
37
return false;
120
vfu_cfg_read(uint32_t offset, uint32_t val) "vfu: cfg: 0x%u -> 0x%x"
38
}
121
vfu_cfg_write(uint32_t offset, uint32_t val) "vfu: cfg: 0x%u <- 0x%x"
39
122
+vfu_dma_register(uint64_t gpa, size_t len) "vfu: registering GPA 0x%"PRIx64", %zu bytes"
40
- if (cfg->buckets[i].burst_length > 1 && !cfg->buckets[i].max) {
123
+vfu_dma_unregister(uint64_t gpa) "vfu: unregistering GPA 0x%"PRIx64""
41
+ if (bkt->burst_length > 1 && !bkt->max) {
42
error_setg(errp, "burst length set without burst rate");
43
return false;
44
}
45
46
- if (cfg->buckets[i].max && !cfg->buckets[i].avg) {
47
+ if (bkt->max && !bkt->avg) {
48
error_setg(errp, "bps_max/iops_max require corresponding"
49
" bps/iops values");
50
return false;
51
}
52
53
- if (cfg->buckets[i].max && cfg->buckets[i].max < cfg->buckets[i].avg) {
54
+ if (bkt->max && bkt->max < bkt->avg) {
55
error_setg(errp, "bps_max/iops_max cannot be lower than bps/iops");
56
return false;
57
}
58
--
124
--
59
2.13.5
125
2.36.1
60
61
diff view generated by jsdifflib
1
From: Alberto Garcia <berto@igalia.com>
1
From: Jagannathan Raman <jag.raman@oracle.com>
2
2
3
Signed-off-by: Alberto Garcia <berto@igalia.com>
3
Determine the BARs used by the PCI device and register handlers to
4
Message-id: a57dd6274e1b6dc9c28769fec4c7ea543be5c5e3.1503580370.git.berto@igalia.com
4
manage the access to the same.
5
6
Signed-off-by: Elena Ufimtseva <elena.ufimtseva@oracle.com>
7
Signed-off-by: John G Johnson <john.g.johnson@oracle.com>
8
Signed-off-by: Jagannathan Raman <jag.raman@oracle.com>
9
Reviewed-by: Stefan Hajnoczi <stefanha@redhat.com>
10
Message-id: 3373e10b5be5f42846f0632d4382466e1698c505.1655151679.git.jag.raman@oracle.com
5
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
11
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
6
---
12
---
7
tests/test-throttle.c | 77 +++++++++++++++++++++++++++++++++++++++++++++++++++
13
include/exec/memory.h | 3 +
8
1 file changed, 77 insertions(+)
14
hw/remote/vfio-user-obj.c | 190 ++++++++++++++++++++++++++++++++
15
softmmu/physmem.c | 4 +-
16
tests/qtest/fuzz/generic_fuzz.c | 9 +-
17
hw/remote/trace-events | 3 +
18
5 files changed, 203 insertions(+), 6 deletions(-)
9
19
10
diff --git a/tests/test-throttle.c b/tests/test-throttle.c
20
diff --git a/include/exec/memory.h b/include/exec/memory.h
11
index XXXXXXX..XXXXXXX 100644
21
index XXXXXXX..XXXXXXX 100644
12
--- a/tests/test-throttle.c
22
--- a/include/exec/memory.h
13
+++ b/tests/test-throttle.c
23
+++ b/include/exec/memory.h
14
@@ -XXX,XX +XXX,XX @@ static void test_is_valid(void)
24
@@ -XXX,XX +XXX,XX @@ MemTxResult address_space_write_cached_slow(MemoryRegionCache *cache,
15
test_is_valid_for_value(1, true);
25
hwaddr addr, const void *buf,
26
hwaddr len);
27
28
+int memory_access_size(MemoryRegion *mr, unsigned l, hwaddr addr);
29
+bool prepare_mmio_access(MemoryRegion *mr);
30
+
31
static inline bool memory_access_is_direct(MemoryRegion *mr, bool is_write)
32
{
33
if (is_write) {
34
diff --git a/hw/remote/vfio-user-obj.c b/hw/remote/vfio-user-obj.c
35
index XXXXXXX..XXXXXXX 100644
36
--- a/hw/remote/vfio-user-obj.c
37
+++ b/hw/remote/vfio-user-obj.c
38
@@ -XXX,XX +XXX,XX @@
39
#include "hw/qdev-core.h"
40
#include "hw/pci/pci.h"
41
#include "qemu/timer.h"
42
+#include "exec/memory.h"
43
44
#define TYPE_VFU_OBJECT "x-vfio-user-server"
45
OBJECT_DECLARE_TYPE(VfuObject, VfuObjectClass, VFU_OBJECT)
46
@@ -XXX,XX +XXX,XX @@ static void dma_unregister(vfu_ctx_t *vfu_ctx, vfu_dma_info_t *info)
47
trace_vfu_dma_unregister((uint64_t)info->iova.iov_base);
16
}
48
}
17
49
18
+static void test_ranges(void)
50
+static int vfu_object_mr_rw(MemoryRegion *mr, uint8_t *buf, hwaddr offset,
51
+ hwaddr size, const bool is_write)
19
+{
52
+{
53
+ uint8_t *ptr = buf;
54
+ bool release_lock = false;
55
+ uint8_t *ram_ptr = NULL;
56
+ MemTxResult result;
57
+ int access_size;
58
+ uint64_t val;
59
+
60
+ if (memory_access_is_direct(mr, is_write)) {
61
+ /**
62
+ * Some devices expose a PCI expansion ROM, which could be buffer
63
+ * based as compared to other regions which are primarily based on
64
+ * MemoryRegionOps. memory_region_find() would already check
65
+ * for buffer overflow, we don't need to repeat it here.
66
+ */
67
+ ram_ptr = memory_region_get_ram_ptr(mr);
68
+
69
+ if (is_write) {
70
+ memcpy((ram_ptr + offset), buf, size);
71
+ } else {
72
+ memcpy(buf, (ram_ptr + offset), size);
73
+ }
74
+
75
+ return 0;
76
+ }
77
+
78
+ while (size) {
79
+ /**
80
+ * The read/write logic used below is similar to the ones in
81
+ * flatview_read/write_continue()
82
+ */
83
+ release_lock = prepare_mmio_access(mr);
84
+
85
+ access_size = memory_access_size(mr, size, offset);
86
+
87
+ if (is_write) {
88
+ val = ldn_he_p(ptr, access_size);
89
+
90
+ result = memory_region_dispatch_write(mr, offset, val,
91
+ size_memop(access_size),
92
+ MEMTXATTRS_UNSPECIFIED);
93
+ } else {
94
+ result = memory_region_dispatch_read(mr, offset, &val,
95
+ size_memop(access_size),
96
+ MEMTXATTRS_UNSPECIFIED);
97
+
98
+ stn_he_p(ptr, access_size, val);
99
+ }
100
+
101
+ if (release_lock) {
102
+ qemu_mutex_unlock_iothread();
103
+ release_lock = false;
104
+ }
105
+
106
+ if (result != MEMTX_OK) {
107
+ return -1;
108
+ }
109
+
110
+ size -= access_size;
111
+ ptr += access_size;
112
+ offset += access_size;
113
+ }
114
+
115
+ return 0;
116
+}
117
+
118
+static size_t vfu_object_bar_rw(PCIDevice *pci_dev, int pci_bar,
119
+ hwaddr bar_offset, char * const buf,
120
+ hwaddr len, const bool is_write)
121
+{
122
+ MemoryRegionSection section = { 0 };
123
+ uint8_t *ptr = (uint8_t *)buf;
124
+ MemoryRegion *section_mr = NULL;
125
+ uint64_t section_size;
126
+ hwaddr section_offset;
127
+ hwaddr size = 0;
128
+
129
+ while (len) {
130
+ section = memory_region_find(pci_dev->io_regions[pci_bar].memory,
131
+ bar_offset, len);
132
+
133
+ if (!section.mr) {
134
+ warn_report("vfu: invalid address 0x%"PRIx64"", bar_offset);
135
+ return size;
136
+ }
137
+
138
+ section_mr = section.mr;
139
+ section_offset = section.offset_within_region;
140
+ section_size = int128_get64(section.size);
141
+
142
+ if (is_write && section_mr->readonly) {
143
+ warn_report("vfu: attempting to write to readonly region in "
144
+ "bar %d - [0x%"PRIx64" - 0x%"PRIx64"]",
145
+ pci_bar, bar_offset,
146
+ (bar_offset + section_size));
147
+ memory_region_unref(section_mr);
148
+ return size;
149
+ }
150
+
151
+ if (vfu_object_mr_rw(section_mr, ptr, section_offset,
152
+ section_size, is_write)) {
153
+ warn_report("vfu: failed to %s "
154
+ "[0x%"PRIx64" - 0x%"PRIx64"] in bar %d",
155
+ is_write ? "write to" : "read from", bar_offset,
156
+ (bar_offset + section_size), pci_bar);
157
+ memory_region_unref(section_mr);
158
+ return size;
159
+ }
160
+
161
+ size += section_size;
162
+ bar_offset += section_size;
163
+ ptr += section_size;
164
+ len -= section_size;
165
+
166
+ memory_region_unref(section_mr);
167
+ }
168
+
169
+ return size;
170
+}
171
+
172
+/**
173
+ * VFU_OBJECT_BAR_HANDLER - macro for defining handlers for PCI BARs.
174
+ *
175
+ * To create handler for BAR number 2, VFU_OBJECT_BAR_HANDLER(2) would
176
+ * define vfu_object_bar2_handler
177
+ */
178
+#define VFU_OBJECT_BAR_HANDLER(BAR_NO) \
179
+ static ssize_t vfu_object_bar##BAR_NO##_handler(vfu_ctx_t *vfu_ctx, \
180
+ char * const buf, size_t count, \
181
+ loff_t offset, const bool is_write) \
182
+ { \
183
+ VfuObject *o = vfu_get_private(vfu_ctx); \
184
+ PCIDevice *pci_dev = o->pci_dev; \
185
+ \
186
+ return vfu_object_bar_rw(pci_dev, BAR_NO, offset, \
187
+ buf, count, is_write); \
188
+ } \
189
+
190
+VFU_OBJECT_BAR_HANDLER(0)
191
+VFU_OBJECT_BAR_HANDLER(1)
192
+VFU_OBJECT_BAR_HANDLER(2)
193
+VFU_OBJECT_BAR_HANDLER(3)
194
+VFU_OBJECT_BAR_HANDLER(4)
195
+VFU_OBJECT_BAR_HANDLER(5)
196
+VFU_OBJECT_BAR_HANDLER(6)
197
+
198
+static vfu_region_access_cb_t *vfu_object_bar_handlers[PCI_NUM_REGIONS] = {
199
+ &vfu_object_bar0_handler,
200
+ &vfu_object_bar1_handler,
201
+ &vfu_object_bar2_handler,
202
+ &vfu_object_bar3_handler,
203
+ &vfu_object_bar4_handler,
204
+ &vfu_object_bar5_handler,
205
+ &vfu_object_bar6_handler,
206
+};
207
+
208
+/**
209
+ * vfu_object_register_bars - Identify active BAR regions of pdev and setup
210
+ * callbacks to handle read/write accesses
211
+ */
212
+static void vfu_object_register_bars(vfu_ctx_t *vfu_ctx, PCIDevice *pdev)
213
+{
214
+ int flags = VFU_REGION_FLAG_RW;
20
+ int i;
215
+ int i;
21
+
216
+
22
+ for (i = 0; i < BUCKETS_COUNT; i++) {
217
+ for (i = 0; i < PCI_NUM_REGIONS; i++) {
23
+ LeakyBucket *b = &cfg.buckets[i];
218
+ if (!pdev->io_regions[i].size) {
24
+ throttle_config_init(&cfg);
219
+ continue;
25
+
220
+ }
26
+ /* avg = 0 means throttling is disabled, but the config is valid */
221
+
27
+ b->avg = 0;
222
+ if ((i == VFU_PCI_DEV_ROM_REGION_IDX) ||
28
+ g_assert(throttle_is_valid(&cfg, NULL));
223
+ pdev->io_regions[i].memory->readonly) {
29
+ g_assert(!throttle_enabled(&cfg));
224
+ flags &= ~VFU_REGION_FLAG_WRITE;
30
+
225
+ }
31
+ /* These are valid configurations (values <= THROTTLE_VALUE_MAX) */
226
+
32
+ b->avg = 1;
227
+ vfu_setup_region(vfu_ctx, VFU_PCI_DEV_BAR0_REGION_IDX + i,
33
+ g_assert(throttle_is_valid(&cfg, NULL));
228
+ (size_t)pdev->io_regions[i].size,
34
+
229
+ vfu_object_bar_handlers[i],
35
+ b->avg = THROTTLE_VALUE_MAX;
230
+ flags, NULL, 0, -1, 0);
36
+ g_assert(throttle_is_valid(&cfg, NULL));
231
+
37
+
232
+ trace_vfu_bar_register(i, pdev->io_regions[i].addr,
38
+ b->avg = THROTTLE_VALUE_MAX;
233
+ pdev->io_regions[i].size);
39
+ b->max = THROTTLE_VALUE_MAX;
40
+ g_assert(throttle_is_valid(&cfg, NULL));
41
+
42
+ /* Values over THROTTLE_VALUE_MAX are not allowed */
43
+ b->avg = THROTTLE_VALUE_MAX + 1;
44
+ g_assert(!throttle_is_valid(&cfg, NULL));
45
+
46
+ b->avg = THROTTLE_VALUE_MAX;
47
+ b->max = THROTTLE_VALUE_MAX + 1;
48
+ g_assert(!throttle_is_valid(&cfg, NULL));
49
+
50
+ /* burst_length must be between 1 and THROTTLE_VALUE_MAX */
51
+ b->avg = 1;
52
+ b->max = 1;
53
+ b->burst_length = 0;
54
+ g_assert(!throttle_is_valid(&cfg, NULL));
55
+
56
+ b->avg = 1;
57
+ b->max = 1;
58
+ b->burst_length = 1;
59
+ g_assert(throttle_is_valid(&cfg, NULL));
60
+
61
+ b->avg = 1;
62
+ b->max = 1;
63
+ b->burst_length = THROTTLE_VALUE_MAX;
64
+ g_assert(throttle_is_valid(&cfg, NULL));
65
+
66
+ b->avg = 1;
67
+ b->max = 1;
68
+ b->burst_length = THROTTLE_VALUE_MAX + 1;
69
+ g_assert(!throttle_is_valid(&cfg, NULL));
70
+
71
+ /* burst_length * max cannot exceed THROTTLE_VALUE_MAX */
72
+ b->avg = 1;
73
+ b->max = 2;
74
+ b->burst_length = THROTTLE_VALUE_MAX / 2;
75
+ g_assert(throttle_is_valid(&cfg, NULL));
76
+
77
+ b->avg = 1;
78
+ b->max = 3;
79
+ b->burst_length = THROTTLE_VALUE_MAX / 2;
80
+ g_assert(!throttle_is_valid(&cfg, NULL));
81
+
82
+ b->avg = 1;
83
+ b->max = THROTTLE_VALUE_MAX;
84
+ b->burst_length = 1;
85
+ g_assert(throttle_is_valid(&cfg, NULL));
86
+
87
+ b->avg = 1;
88
+ b->max = THROTTLE_VALUE_MAX;
89
+ b->burst_length = 2;
90
+ g_assert(!throttle_is_valid(&cfg, NULL));
91
+ }
234
+ }
92
+}
235
+}
93
+
236
+
94
static void test_max_is_missing_limit(void)
237
/*
238
* TYPE_VFU_OBJECT depends on the availability of the 'socket' and 'device'
239
* properties. It also depends on devices instantiated in QEMU. These
240
@@ -XXX,XX +XXX,XX @@ static void vfu_object_init_ctx(VfuObject *o, Error **errp)
241
goto fail;
242
}
243
244
+ vfu_object_register_bars(o->vfu_ctx, o->pci_dev);
245
+
246
ret = vfu_realize_ctx(o->vfu_ctx);
247
if (ret < 0) {
248
error_setg(errp, "vfu: Failed to realize device %s- %s",
249
diff --git a/softmmu/physmem.c b/softmmu/physmem.c
250
index XXXXXXX..XXXXXXX 100644
251
--- a/softmmu/physmem.c
252
+++ b/softmmu/physmem.c
253
@@ -XXX,XX +XXX,XX @@ void memory_region_flush_rom_device(MemoryRegion *mr, hwaddr addr, hwaddr size)
254
invalidate_and_set_dirty(mr, addr, size);
255
}
256
257
-static int memory_access_size(MemoryRegion *mr, unsigned l, hwaddr addr)
258
+int memory_access_size(MemoryRegion *mr, unsigned l, hwaddr addr)
95
{
259
{
96
int i;
260
unsigned access_size_max = mr->ops->valid.max_access_size;
97
@@ -XXX,XX +XXX,XX @@ int main(int argc, char **argv)
261
98
g_test_add_func("/throttle/config/enabled", test_enabled);
262
@@ -XXX,XX +XXX,XX @@ static int memory_access_size(MemoryRegion *mr, unsigned l, hwaddr addr)
99
g_test_add_func("/throttle/config/conflicting", test_conflicting_config);
263
return l;
100
g_test_add_func("/throttle/config/is_valid", test_is_valid);
264
}
101
+ g_test_add_func("/throttle/config/ranges", test_ranges);
265
102
g_test_add_func("/throttle/config/max", test_max_is_missing_limit);
266
-static bool prepare_mmio_access(MemoryRegion *mr)
103
g_test_add_func("/throttle/config/iops_size",
267
+bool prepare_mmio_access(MemoryRegion *mr)
104
test_iops_size_is_missing_limit);
268
{
269
bool release_lock = false;
270
271
diff --git a/tests/qtest/fuzz/generic_fuzz.c b/tests/qtest/fuzz/generic_fuzz.c
272
index XXXXXXX..XXXXXXX 100644
273
--- a/tests/qtest/fuzz/generic_fuzz.c
274
+++ b/tests/qtest/fuzz/generic_fuzz.c
275
@@ -XXX,XX +XXX,XX @@ static void *pattern_alloc(pattern p, size_t len)
276
return buf;
277
}
278
279
-static int memory_access_size(MemoryRegion *mr, unsigned l, hwaddr addr)
280
+static int fuzz_memory_access_size(MemoryRegion *mr, unsigned l, hwaddr addr)
281
{
282
unsigned access_size_max = mr->ops->valid.max_access_size;
283
284
@@ -XXX,XX +XXX,XX @@ void fuzz_dma_read_cb(size_t addr, size_t len, MemoryRegion *mr)
285
286
/*
287
* If mr1 isn't RAM, address_space_translate doesn't update l. Use
288
- * memory_access_size to identify the number of bytes that it is safe
289
- * to write without accidentally writing to another MemoryRegion.
290
+ * fuzz_memory_access_size to identify the number of bytes that it
291
+ * is safe to write without accidentally writing to another
292
+ * MemoryRegion.
293
*/
294
if (!memory_region_is_ram(mr1)) {
295
- l = memory_access_size(mr1, l, addr1);
296
+ l = fuzz_memory_access_size(mr1, l, addr1);
297
}
298
if (memory_region_is_ram(mr1) ||
299
memory_region_is_romd(mr1) ||
300
diff --git a/hw/remote/trace-events b/hw/remote/trace-events
301
index XXXXXXX..XXXXXXX 100644
302
--- a/hw/remote/trace-events
303
+++ b/hw/remote/trace-events
304
@@ -XXX,XX +XXX,XX @@ vfu_cfg_read(uint32_t offset, uint32_t val) "vfu: cfg: 0x%u -> 0x%x"
305
vfu_cfg_write(uint32_t offset, uint32_t val) "vfu: cfg: 0x%u <- 0x%x"
306
vfu_dma_register(uint64_t gpa, size_t len) "vfu: registering GPA 0x%"PRIx64", %zu bytes"
307
vfu_dma_unregister(uint64_t gpa) "vfu: unregistering GPA 0x%"PRIx64""
308
+vfu_bar_register(int i, uint64_t addr, uint64_t size) "vfu: BAR %d: addr 0x%"PRIx64" size 0x%"PRIx64""
309
+vfu_bar_rw_enter(const char *op, uint64_t addr) "vfu: %s request for BAR address 0x%"PRIx64""
310
+vfu_bar_rw_exit(const char *op, uint64_t addr) "vfu: Finished %s of BAR address 0x%"PRIx64""
105
--
311
--
106
2.13.5
312
2.36.1
107
108
diff view generated by jsdifflib
1
The minimum Python version supported by QEMU is 2.6. The argparse
1
From: Jagannathan Raman <jag.raman@oracle.com>
2
standard library module was only added in Python 2.7. Many scripts
3
would like to use argparse because it supports command-line
4
sub-commands.
5
2
6
This patch adds argparse. See the top of argparse.py for details.
3
Forward remote device's interrupts to the guest
7
4
8
Suggested-by: Daniel P. Berrange <berrange@redhat.com>
5
Signed-off-by: Elena Ufimtseva <elena.ufimtseva@oracle.com>
9
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
6
Signed-off-by: John G Johnson <john.g.johnson@oracle.com>
10
Acked-by: John Snow <jsnow@redhat.com>
7
Signed-off-by: Jagannathan Raman <jag.raman@oracle.com>
11
Message-id: 20170825155732.15665-2-stefanha@redhat.com
8
Message-id: 9523479eaafe050677f4de2af5dd0df18c27cfd9.1655151679.git.jag.raman@oracle.com
12
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
9
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
13
---
10
---
14
COPYING.PYTHON | 270 ++++++
11
MAINTAINERS | 1 +
15
scripts/argparse.py | 2406 +++++++++++++++++++++++++++++++++++++++++++++++++++
12
include/hw/pci/msi.h | 1 +
16
2 files changed, 2676 insertions(+)
13
include/hw/pci/msix.h | 1 +
17
create mode 100644 COPYING.PYTHON
14
include/hw/pci/pci.h | 13 +++
18
create mode 100644 scripts/argparse.py
15
include/hw/remote/vfio-user-obj.h | 6 ++
16
hw/pci/msi.c | 49 +++++++--
17
hw/pci/msix.c | 35 ++++++-
18
hw/pci/pci.c | 13 +++
19
hw/remote/machine.c | 16 ++-
20
hw/remote/vfio-user-obj.c | 167 ++++++++++++++++++++++++++++++
21
stubs/vfio-user-obj.c | 6 ++
22
hw/remote/trace-events | 1 +
23
stubs/meson.build | 1 +
24
13 files changed, 298 insertions(+), 12 deletions(-)
25
create mode 100644 include/hw/remote/vfio-user-obj.h
26
create mode 100644 stubs/vfio-user-obj.c
19
27
20
diff --git a/COPYING.PYTHON b/COPYING.PYTHON
28
diff --git a/MAINTAINERS b/MAINTAINERS
29
index XXXXXXX..XXXXXXX 100644
30
--- a/MAINTAINERS
31
+++ b/MAINTAINERS
32
@@ -XXX,XX +XXX,XX @@ F: hw/remote/iohub.c
33
F: include/hw/remote/iohub.h
34
F: subprojects/libvfio-user
35
F: hw/remote/vfio-user-obj.c
36
+F: include/hw/remote/vfio-user-obj.h
37
F: hw/remote/iommu.c
38
F: include/hw/remote/iommu.h
39
40
diff --git a/include/hw/pci/msi.h b/include/hw/pci/msi.h
41
index XXXXXXX..XXXXXXX 100644
42
--- a/include/hw/pci/msi.h
43
+++ b/include/hw/pci/msi.h
44
@@ -XXX,XX +XXX,XX @@ void msi_notify(PCIDevice *dev, unsigned int vector);
45
void msi_send_message(PCIDevice *dev, MSIMessage msg);
46
void msi_write_config(PCIDevice *dev, uint32_t addr, uint32_t val, int len);
47
unsigned int msi_nr_vectors_allocated(const PCIDevice *dev);
48
+void msi_set_mask(PCIDevice *dev, int vector, bool mask, Error **errp);
49
50
static inline bool msi_present(const PCIDevice *dev)
51
{
52
diff --git a/include/hw/pci/msix.h b/include/hw/pci/msix.h
53
index XXXXXXX..XXXXXXX 100644
54
--- a/include/hw/pci/msix.h
55
+++ b/include/hw/pci/msix.h
56
@@ -XXX,XX +XXX,XX @@ void msix_clr_pending(PCIDevice *dev, int vector);
57
int msix_vector_use(PCIDevice *dev, unsigned vector);
58
void msix_vector_unuse(PCIDevice *dev, unsigned vector);
59
void msix_unuse_all_vectors(PCIDevice *dev);
60
+void msix_set_mask(PCIDevice *dev, int vector, bool mask, Error **errp);
61
62
void msix_notify(PCIDevice *dev, unsigned vector);
63
64
diff --git a/include/hw/pci/pci.h b/include/hw/pci/pci.h
65
index XXXXXXX..XXXXXXX 100644
66
--- a/include/hw/pci/pci.h
67
+++ b/include/hw/pci/pci.h
68
@@ -XXX,XX +XXX,XX @@ extern bool pci_available;
69
#define PCI_SLOT(devfn) (((devfn) >> 3) & 0x1f)
70
#define PCI_FUNC(devfn) ((devfn) & 0x07)
71
#define PCI_BUILD_BDF(bus, devfn) ((bus << 8) | (devfn))
72
+#define PCI_BDF_TO_DEVFN(x) ((x) & 0xff)
73
#define PCI_BUS_MAX 256
74
#define PCI_DEVFN_MAX 256
75
#define PCI_SLOT_MAX 32
76
@@ -XXX,XX +XXX,XX @@ typedef void PCIMapIORegionFunc(PCIDevice *pci_dev, int region_num,
77
pcibus_t addr, pcibus_t size, int type);
78
typedef void PCIUnregisterFunc(PCIDevice *pci_dev);
79
80
+typedef void MSITriggerFunc(PCIDevice *dev, MSIMessage msg);
81
+typedef MSIMessage MSIPrepareMessageFunc(PCIDevice *dev, unsigned vector);
82
+typedef MSIMessage MSIxPrepareMessageFunc(PCIDevice *dev, unsigned vector);
83
+
84
typedef struct PCIIORegion {
85
pcibus_t addr; /* current PCI mapping address. -1 means not mapped */
86
#define PCI_BAR_UNMAPPED (~(pcibus_t)0)
87
@@ -XXX,XX +XXX,XX @@ struct PCIDevice {
88
/* Space to store MSIX table & pending bit array */
89
uint8_t *msix_table;
90
uint8_t *msix_pba;
91
+
92
+ /* May be used by INTx or MSI during interrupt notification */
93
+ void *irq_opaque;
94
+
95
+ MSITriggerFunc *msi_trigger;
96
+ MSIPrepareMessageFunc *msi_prepare_message;
97
+ MSIxPrepareMessageFunc *msix_prepare_message;
98
+
99
/* MemoryRegion container for msix exclusive BAR setup */
100
MemoryRegion msix_exclusive_bar;
101
/* Memory Regions for MSIX table and pending bit entries. */
102
diff --git a/include/hw/remote/vfio-user-obj.h b/include/hw/remote/vfio-user-obj.h
21
new file mode 100644
103
new file mode 100644
22
index XXXXXXX..XXXXXXX
104
index XXXXXXX..XXXXXXX
23
--- /dev/null
105
--- /dev/null
24
+++ b/COPYING.PYTHON
106
+++ b/include/hw/remote/vfio-user-obj.h
25
@@ -XXX,XX +XXX,XX @@
107
@@ -XXX,XX +XXX,XX @@
26
+A. HISTORY OF THE SOFTWARE
108
+#ifndef VFIO_USER_OBJ_H
27
+==========================
109
+#define VFIO_USER_OBJ_H
28
+
110
+
29
+Python was created in the early 1990s by Guido van Rossum at Stichting
111
+void vfu_object_set_bus_irq(PCIBus *pci_bus);
30
+Mathematisch Centrum (CWI, see http://www.cwi.nl) in the Netherlands
112
+
31
+as a successor of a language called ABC. Guido remains Python's
113
+#endif
32
+principal author, although it includes many contributions from others.
114
diff --git a/hw/pci/msi.c b/hw/pci/msi.c
33
+
115
index XXXXXXX..XXXXXXX 100644
34
+In 1995, Guido continued his work on Python at the Corporation for
116
--- a/hw/pci/msi.c
35
+National Research Initiatives (CNRI, see http://www.cnri.reston.va.us)
117
+++ b/hw/pci/msi.c
36
+in Reston, Virginia where he released several versions of the
118
@@ -XXX,XX +XXX,XX @@ void msi_set_message(PCIDevice *dev, MSIMessage msg)
37
+software.
119
pci_set_word(dev->config + msi_data_off(dev, msi64bit), msg.data);
38
+
120
}
39
+In May 2000, Guido and the Python core development team moved to
121
40
+BeOpen.com to form the BeOpen PythonLabs team. In October of the same
122
-MSIMessage msi_get_message(PCIDevice *dev, unsigned int vector)
41
+year, the PythonLabs team moved to Digital Creations (now Zope
123
+static MSIMessage msi_prepare_message(PCIDevice *dev, unsigned int vector)
42
+Corporation, see http://www.zope.com). In 2001, the Python Software
124
{
43
+Foundation (PSF, see http://www.python.org/psf/) was formed, a
125
uint16_t flags = pci_get_word(dev->config + msi_flags_off(dev));
44
+non-profit organization created specifically to own Python-related
126
bool msi64bit = flags & PCI_MSI_FLAGS_64BIT;
45
+Intellectual Property. Zope Corporation is a sponsoring member of
127
@@ -XXX,XX +XXX,XX @@ MSIMessage msi_get_message(PCIDevice *dev, unsigned int vector)
46
+the PSF.
128
return msg;
47
+
129
}
48
+All Python releases are Open Source (see http://www.opensource.org for
130
49
+the Open Source Definition). Historically, most, but not all, Python
131
+MSIMessage msi_get_message(PCIDevice *dev, unsigned int vector)
50
+releases have also been GPL-compatible; the table below summarizes
132
+{
51
+the various releases.
133
+ return dev->msi_prepare_message(dev, vector);
52
+
134
+}
53
+ Release Derived Year Owner GPL-
135
+
54
+ from compatible? (1)
136
bool msi_enabled(const PCIDevice *dev)
55
+
137
{
56
+ 0.9.0 thru 1.2 1991-1995 CWI yes
138
return msi_present(dev) &&
57
+ 1.3 thru 1.5.2 1.2 1995-1999 CNRI yes
139
@@ -XXX,XX +XXX,XX @@ int msi_init(struct PCIDevice *dev, uint8_t offset,
58
+ 1.6 1.5.2 2000 CNRI no
140
0xffffffff >> (PCI_MSI_VECTORS_MAX - nr_vectors));
59
+ 2.0 1.6 2000 BeOpen.com no
141
}
60
+ 1.6.1 1.6 2001 CNRI yes (2)
142
61
+ 2.1 2.0+1.6.1 2001 PSF no
143
+ dev->msi_prepare_message = msi_prepare_message;
62
+ 2.0.1 2.0+1.6.1 2001 PSF yes
144
+
63
+ 2.1.1 2.1+2.0.1 2001 PSF yes
145
return 0;
64
+ 2.2 2.1.1 2001 PSF yes
146
}
65
+ 2.1.2 2.1.1 2002 PSF yes
147
66
+ 2.1.3 2.1.2 2002 PSF yes
148
@@ -XXX,XX +XXX,XX @@ void msi_uninit(struct PCIDevice *dev)
67
+ 2.2.1 2.2 2002 PSF yes
149
cap_size = msi_cap_sizeof(flags);
68
+ 2.2.2 2.2.1 2002 PSF yes
150
pci_del_capability(dev, PCI_CAP_ID_MSI, cap_size);
69
+ 2.2.3 2.2.2 2003 PSF yes
151
dev->cap_present &= ~QEMU_PCI_CAP_MSI;
70
+ 2.3 2.2.2 2002-2003 PSF yes
152
+ dev->msi_prepare_message = NULL;
71
+ 2.3.1 2.3 2002-2003 PSF yes
153
72
+ 2.3.2 2.3.1 2002-2003 PSF yes
154
MSI_DEV_PRINTF(dev, "uninit\n");
73
+ 2.3.3 2.3.2 2002-2003 PSF yes
155
}
74
+ 2.3.4 2.3.3 2004 PSF yes
156
@@ -XXX,XX +XXX,XX @@ bool msi_is_masked(const PCIDevice *dev, unsigned int vector)
75
+ 2.3.5 2.3.4 2005 PSF yes
157
return mask & (1U << vector);
76
+ 2.4 2.3 2004 PSF yes
158
}
77
+ 2.4.1 2.4 2005 PSF yes
159
78
+ 2.4.2 2.4.1 2005 PSF yes
160
+void msi_set_mask(PCIDevice *dev, int vector, bool mask, Error **errp)
79
+ 2.4.3 2.4.2 2006 PSF yes
161
+{
80
+ 2.5 2.4 2006 PSF yes
162
+ ERRP_GUARD();
81
+ 2.7 2.6 2010 PSF yes
163
+ uint16_t flags = pci_get_word(dev->config + msi_flags_off(dev));
82
+
164
+ bool msi64bit = flags & PCI_MSI_FLAGS_64BIT;
83
+Footnotes:
165
+ uint32_t irq_state, vector_mask, pending;
84
+
166
+
85
+(1) GPL-compatible doesn't mean that we're distributing Python under
167
+ if (vector > PCI_MSI_VECTORS_MAX) {
86
+ the GPL. All Python licenses, unlike the GPL, let you distribute
168
+ error_setg(errp, "msi: vector %d not allocated. max vector is %d",
87
+ a modified version without making your changes open source. The
169
+ vector, PCI_MSI_VECTORS_MAX);
88
+ GPL-compatible licenses make it possible to combine Python with
170
+ return;
89
+ other software that is released under the GPL; the others don't.
171
+ }
90
+
172
+
91
+(2) According to Richard Stallman, 1.6.1 is not GPL-compatible,
173
+ vector_mask = (1U << vector);
92
+ because its license has a choice of law clause. According to
174
+
93
+ CNRI, however, Stallman's lawyer has told CNRI's lawyer that 1.6.1
175
+ irq_state = pci_get_long(dev->config + msi_mask_off(dev, msi64bit));
94
+ is "not incompatible" with the GPL.
176
+
95
+
177
+ if (mask) {
96
+Thanks to the many outside volunteers who have worked under Guido's
178
+ irq_state |= vector_mask;
97
+direction to make these releases possible.
179
+ } else {
98
+
180
+ irq_state &= ~vector_mask;
99
+
181
+ }
100
+B. TERMS AND CONDITIONS FOR ACCESSING OR OTHERWISE USING PYTHON
182
+
101
+===============================================================
183
+ pci_set_long(dev->config + msi_mask_off(dev, msi64bit), irq_state);
102
+
184
+
103
+PYTHON SOFTWARE FOUNDATION LICENSE VERSION 2
185
+ pending = pci_get_long(dev->config + msi_pending_off(dev, msi64bit));
104
+--------------------------------------------
186
+ if (!mask && (pending & vector_mask)) {
105
+
187
+ pending &= ~vector_mask;
106
+1. This LICENSE AGREEMENT is between the Python Software Foundation
188
+ pci_set_long(dev->config + msi_pending_off(dev, msi64bit), pending);
107
+("PSF"), and the Individual or Organization ("Licensee") accessing and
189
+ msi_notify(dev, vector);
108
+otherwise using this software ("Python") in source or binary form and
190
+ }
109
+its associated documentation.
191
+}
110
+
192
+
111
+2. Subject to the terms and conditions of this License Agreement, PSF
193
void msi_notify(PCIDevice *dev, unsigned int vector)
112
+hereby grants Licensee a nonexclusive, royalty-free, world-wide
194
{
113
+license to reproduce, analyze, test, perform and/or display publicly,
195
uint16_t flags = pci_get_word(dev->config + msi_flags_off(dev));
114
+prepare derivative works, distribute, and otherwise use Python
196
@@ -XXX,XX +XXX,XX @@ void msi_notify(PCIDevice *dev, unsigned int vector)
115
+alone or in any derivative version, provided, however, that PSF's
197
116
+License Agreement and PSF's notice of copyright, i.e., "Copyright (c)
198
void msi_send_message(PCIDevice *dev, MSIMessage msg)
117
+2001, 2002, 2003, 2004, 2005, 2006 Python Software Foundation; All Rights
199
{
118
+Reserved" are retained in Python alone or in any derivative version
200
- MemTxAttrs attrs = {};
119
+prepared by Licensee.
201
-
120
+
202
- attrs.requester_id = pci_requester_id(dev);
121
+3. In the event Licensee prepares a derivative work that is based on
203
- address_space_stl_le(&dev->bus_master_as, msg.address, msg.data,
122
+or incorporates Python or any part thereof, and wants to make
204
- attrs, NULL);
123
+the derivative work available to others as provided herein, then
205
+ dev->msi_trigger(dev, msg);
124
+Licensee hereby agrees to include in any such work a brief summary of
206
}
125
+the changes made to Python.
207
126
+
208
/* Normally called by pci_default_write_config(). */
127
+4. PSF is making Python available to Licensee on an "AS IS"
209
diff --git a/hw/pci/msix.c b/hw/pci/msix.c
128
+basis. PSF MAKES NO REPRESENTATIONS OR WARRANTIES, EXPRESS OR
210
index XXXXXXX..XXXXXXX 100644
129
+IMPLIED. BY WAY OF EXAMPLE, BUT NOT LIMITATION, PSF MAKES NO AND
211
--- a/hw/pci/msix.c
130
+DISCLAIMS ANY REPRESENTATION OR WARRANTY OF MERCHANTABILITY OR FITNESS
212
+++ b/hw/pci/msix.c
131
+FOR ANY PARTICULAR PURPOSE OR THAT THE USE OF PYTHON WILL NOT
213
@@ -XXX,XX +XXX,XX @@
132
+INFRINGE ANY THIRD PARTY RIGHTS.
214
#define MSIX_ENABLE_MASK (PCI_MSIX_FLAGS_ENABLE >> 8)
133
+
215
#define MSIX_MASKALL_MASK (PCI_MSIX_FLAGS_MASKALL >> 8)
134
+5. PSF SHALL NOT BE LIABLE TO LICENSEE OR ANY OTHER USERS OF PYTHON
216
135
+FOR ANY INCIDENTAL, SPECIAL, OR CONSEQUENTIAL DAMAGES OR LOSS AS
217
-MSIMessage msix_get_message(PCIDevice *dev, unsigned vector)
136
+A RESULT OF MODIFYING, DISTRIBUTING, OR OTHERWISE USING PYTHON,
218
+static MSIMessage msix_prepare_message(PCIDevice *dev, unsigned vector)
137
+OR ANY DERIVATIVE THEREOF, EVEN IF ADVISED OF THE POSSIBILITY THEREOF.
219
{
138
+
220
uint8_t *table_entry = dev->msix_table + vector * PCI_MSIX_ENTRY_SIZE;
139
+6. This License Agreement will automatically terminate upon a material
221
MSIMessage msg;
140
+breach of its terms and conditions.
222
@@ -XXX,XX +XXX,XX @@ MSIMessage msix_get_message(PCIDevice *dev, unsigned vector)
141
+
223
return msg;
142
+7. Nothing in this License Agreement shall be deemed to create any
224
}
143
+relationship of agency, partnership, or joint venture between PSF and
225
144
+Licensee. This License Agreement does not grant permission to use PSF
226
+MSIMessage msix_get_message(PCIDevice *dev, unsigned vector)
145
+trademarks or trade name in a trademark sense to endorse or promote
227
+{
146
+products or services of Licensee, or any third party.
228
+ return dev->msix_prepare_message(dev, vector);
147
+
229
+}
148
+8. By copying, installing or otherwise using Python, Licensee
230
+
149
+agrees to be bound by the terms and conditions of this License
231
/*
150
+Agreement.
232
* Special API for POWER to configure the vectors through
151
+
233
* a side channel. Should never be used by devices.
152
+
234
@@ -XXX,XX +XXX,XX @@ static void msix_handle_mask_update(PCIDevice *dev, int vector, bool was_masked)
153
+BEOPEN.COM LICENSE AGREEMENT FOR PYTHON 2.0
235
}
154
+-------------------------------------------
236
}
155
+
237
156
+BEOPEN PYTHON OPEN SOURCE LICENSE AGREEMENT VERSION 1
238
+void msix_set_mask(PCIDevice *dev, int vector, bool mask, Error **errp)
157
+
239
+{
158
+1. This LICENSE AGREEMENT is between BeOpen.com ("BeOpen"), having an
240
+ ERRP_GUARD();
159
+office at 160 Saratoga Avenue, Santa Clara, CA 95051, and the
241
+ unsigned offset;
160
+Individual or Organization ("Licensee") accessing and otherwise using
242
+ bool was_masked;
161
+this software in source or binary form and its associated
243
+
162
+documentation ("the Software").
244
+ if (vector > dev->msix_entries_nr) {
163
+
245
+ error_setg(errp, "msix: vector %d not allocated. max vector is %d",
164
+2. Subject to the terms and conditions of this BeOpen Python License
246
+ vector, dev->msix_entries_nr);
165
+Agreement, BeOpen hereby grants Licensee a non-exclusive,
247
+ return;
166
+royalty-free, world-wide license to reproduce, analyze, test, perform
248
+ }
167
+and/or display publicly, prepare derivative works, distribute, and
249
+
168
+otherwise use the Software alone or in any derivative version,
250
+ offset = vector * PCI_MSIX_ENTRY_SIZE + PCI_MSIX_ENTRY_VECTOR_CTRL;
169
+provided, however, that the BeOpen Python License is retained in the
251
+
170
+Software, alone or in any derivative version prepared by Licensee.
252
+ was_masked = msix_is_masked(dev, vector);
171
+
253
+
172
+3. BeOpen is making the Software available to Licensee on an "AS IS"
254
+ if (mask) {
173
+basis. BEOPEN MAKES NO REPRESENTATIONS OR WARRANTIES, EXPRESS OR
255
+ dev->msix_table[offset] |= PCI_MSIX_ENTRY_CTRL_MASKBIT;
174
+IMPLIED. BY WAY OF EXAMPLE, BUT NOT LIMITATION, BEOPEN MAKES NO AND
256
+ } else {
175
+DISCLAIMS ANY REPRESENTATION OR WARRANTY OF MERCHANTABILITY OR FITNESS
257
+ dev->msix_table[offset] &= ~PCI_MSIX_ENTRY_CTRL_MASKBIT;
176
+FOR ANY PARTICULAR PURPOSE OR THAT THE USE OF THE SOFTWARE WILL NOT
258
+ }
177
+INFRINGE ANY THIRD PARTY RIGHTS.
259
+
178
+
260
+ msix_handle_mask_update(dev, vector, was_masked);
179
+4. BEOPEN SHALL NOT BE LIABLE TO LICENSEE OR ANY OTHER USERS OF THE
261
+}
180
+SOFTWARE FOR ANY INCIDENTAL, SPECIAL, OR CONSEQUENTIAL DAMAGES OR LOSS
262
+
181
+AS A RESULT OF USING, MODIFYING OR DISTRIBUTING THE SOFTWARE, OR ANY
263
static bool msix_masked(PCIDevice *dev)
182
+DERIVATIVE THEREOF, EVEN IF ADVISED OF THE POSSIBILITY THEREOF.
264
{
183
+
265
return dev->config[dev->msix_cap + MSIX_CONTROL_OFFSET] & MSIX_MASKALL_MASK;
184
+5. This License Agreement will automatically terminate upon a material
266
@@ -XXX,XX +XXX,XX @@ int msix_init(struct PCIDevice *dev, unsigned short nentries,
185
+breach of its terms and conditions.
267
"msix-pba", pba_size);
186
+
268
memory_region_add_subregion(pba_bar, pba_offset, &dev->msix_pba_mmio);
187
+6. This License Agreement shall be governed by and interpreted in all
269
188
+respects by the law of the State of California, excluding conflict of
270
+ dev->msix_prepare_message = msix_prepare_message;
189
+law provisions. Nothing in this License Agreement shall be deemed to
271
+
190
+create any relationship of agency, partnership, or joint venture
272
return 0;
191
+between BeOpen and Licensee. This License Agreement does not grant
273
}
192
+permission to use BeOpen trademarks or trade names in a trademark
274
193
+sense to endorse or promote products or services of Licensee, or any
275
@@ -XXX,XX +XXX,XX @@ void msix_uninit(PCIDevice *dev, MemoryRegion *table_bar, MemoryRegion *pba_bar)
194
+third party. As an exception, the "BeOpen Python" logos available at
276
g_free(dev->msix_entry_used);
195
+http://www.pythonlabs.com/logos.html may be used according to the
277
dev->msix_entry_used = NULL;
196
+permissions granted on that web page.
278
dev->cap_present &= ~QEMU_PCI_CAP_MSIX;
197
+
279
+ dev->msix_prepare_message = NULL;
198
+7. By copying, installing or otherwise using the software, Licensee
280
}
199
+agrees to be bound by the terms and conditions of this License
281
200
+Agreement.
282
void msix_uninit_exclusive_bar(PCIDevice *dev)
201
+
283
diff --git a/hw/pci/pci.c b/hw/pci/pci.c
202
+
284
index XXXXXXX..XXXXXXX 100644
203
+CNRI LICENSE AGREEMENT FOR PYTHON 1.6.1
285
--- a/hw/pci/pci.c
204
+---------------------------------------
286
+++ b/hw/pci/pci.c
205
+
287
@@ -XXX,XX +XXX,XX @@ void pci_device_deassert_intx(PCIDevice *dev)
206
+1. This LICENSE AGREEMENT is between the Corporation for National
288
}
207
+Research Initiatives, having an office at 1895 Preston White Drive,
289
}
208
+Reston, VA 20191 ("CNRI"), and the Individual or Organization
290
209
+("Licensee") accessing and otherwise using Python 1.6.1 software in
291
+static void pci_msi_trigger(PCIDevice *dev, MSIMessage msg)
210
+source or binary form and its associated documentation.
292
+{
211
+
293
+ MemTxAttrs attrs = {};
212
+2. Subject to the terms and conditions of this License Agreement, CNRI
294
+
213
+hereby grants Licensee a nonexclusive, royalty-free, world-wide
295
+ attrs.requester_id = pci_requester_id(dev);
214
+license to reproduce, analyze, test, perform and/or display publicly,
296
+ address_space_stl_le(&dev->bus_master_as, msg.address, msg.data,
215
+prepare derivative works, distribute, and otherwise use Python 1.6.1
297
+ attrs, NULL);
216
+alone or in any derivative version, provided, however, that CNRI's
298
+}
217
+License Agreement and CNRI's notice of copyright, i.e., "Copyright (c)
299
+
218
+1995-2001 Corporation for National Research Initiatives; All Rights
300
static void pci_reset_regions(PCIDevice *dev)
219
+Reserved" are retained in Python 1.6.1 alone or in any derivative
301
{
220
+version prepared by Licensee. Alternately, in lieu of CNRI's License
302
int r;
221
+Agreement, Licensee may substitute the following text (omitting the
303
@@ -XXX,XX +XXX,XX @@ static void pci_qdev_unrealize(DeviceState *dev)
222
+quotes): "Python 1.6.1 is made available subject to the terms and
304
223
+conditions in CNRI's License Agreement. This Agreement together with
305
pci_device_deassert_intx(pci_dev);
224
+Python 1.6.1 may be located on the Internet using the following
306
do_pci_unregister_device(pci_dev);
225
+unique, persistent identifier (known as a handle): 1895.22/1013. This
307
+
226
+Agreement may also be obtained from a proxy server on the Internet
308
+ pci_dev->msi_trigger = NULL;
227
+using the following URL: http://hdl.handle.net/1895.22/1013".
309
}
228
+
310
229
+3. In the event Licensee prepares a derivative work that is based on
311
void pci_register_bar(PCIDevice *pci_dev, int region_num,
230
+or incorporates Python 1.6.1 or any part thereof, and wants to make
312
@@ -XXX,XX +XXX,XX @@ static void pci_qdev_realize(DeviceState *qdev, Error **errp)
231
+the derivative work available to others as provided herein, then
313
}
232
+Licensee hereby agrees to include in any such work a brief summary of
314
233
+the changes made to Python 1.6.1.
315
pci_set_power(pci_dev, true);
234
+
316
+
235
+4. CNRI is making Python 1.6.1 available to Licensee on an "AS IS"
317
+ pci_dev->msi_trigger = pci_msi_trigger;
236
+basis. CNRI MAKES NO REPRESENTATIONS OR WARRANTIES, EXPRESS OR
318
}
237
+IMPLIED. BY WAY OF EXAMPLE, BUT NOT LIMITATION, CNRI MAKES NO AND
319
238
+DISCLAIMS ANY REPRESENTATION OR WARRANTY OF MERCHANTABILITY OR FITNESS
320
PCIDevice *pci_new_multifunction(int devfn, bool multifunction,
239
+FOR ANY PARTICULAR PURPOSE OR THAT THE USE OF PYTHON 1.6.1 WILL NOT
321
diff --git a/hw/remote/machine.c b/hw/remote/machine.c
240
+INFRINGE ANY THIRD PARTY RIGHTS.
322
index XXXXXXX..XXXXXXX 100644
241
+
323
--- a/hw/remote/machine.c
242
+5. CNRI SHALL NOT BE LIABLE TO LICENSEE OR ANY OTHER USERS OF PYTHON
324
+++ b/hw/remote/machine.c
243
+1.6.1 FOR ANY INCIDENTAL, SPECIAL, OR CONSEQUENTIAL DAMAGES OR LOSS AS
325
@@ -XXX,XX +XXX,XX @@
244
+A RESULT OF MODIFYING, DISTRIBUTING, OR OTHERWISE USING PYTHON 1.6.1,
326
#include "hw/remote/iommu.h"
245
+OR ANY DERIVATIVE THEREOF, EVEN IF ADVISED OF THE POSSIBILITY THEREOF.
327
#include "hw/qdev-core.h"
246
+
328
#include "hw/remote/iommu.h"
247
+6. This License Agreement will automatically terminate upon a material
329
+#include "hw/remote/vfio-user-obj.h"
248
+breach of its terms and conditions.
330
+#include "hw/pci/msi.h"
249
+
331
250
+7. This License Agreement shall be governed by the federal
332
static void remote_machine_init(MachineState *machine)
251
+intellectual property law of the United States, including without
333
{
252
+limitation the federal copyright law, and, to the extent such
334
@@ -XXX,XX +XXX,XX @@ static void remote_machine_init(MachineState *machine)
253
+U.S. federal law does not apply, by the law of the Commonwealth of
335
254
+Virginia, excluding Virginia's conflict of law provisions.
336
if (s->vfio_user) {
255
+Notwithstanding the foregoing, with regard to derivative works based
337
remote_iommu_setup(pci_host->bus);
256
+on Python 1.6.1 that incorporate non-separable material that was
338
+
257
+previously distributed under the GNU General Public License (GPL), the
339
+ msi_nonbroken = true;
258
+law of the Commonwealth of Virginia shall govern this License
340
+
259
+Agreement only as to issues arising under or with respect to
341
+ vfu_object_set_bus_irq(pci_host->bus);
260
+Paragraphs 4, 5, and 7 of this License Agreement. Nothing in this
342
+ } else {
261
+License Agreement shall be deemed to create any relationship of
343
+ remote_iohub_init(&s->iohub);
262
+agency, partnership, or joint venture between CNRI and Licensee. This
344
+
263
+License Agreement does not grant permission to use CNRI trademarks or
345
+ pci_bus_irqs(pci_host->bus, remote_iohub_set_irq, remote_iohub_map_irq,
264
+trade name in a trademark sense to endorse or promote products or
346
+ &s->iohub, REMOTE_IOHUB_NB_PIRQS);
265
+services of Licensee, or any third party.
347
}
266
+
348
267
+8. By clicking on the "ACCEPT" button where indicated, or by copying,
349
- remote_iohub_init(&s->iohub);
268
+installing or otherwise using Python 1.6.1, Licensee agrees to be
350
-
269
+bound by the terms and conditions of this License Agreement.
351
- pci_bus_irqs(pci_host->bus, remote_iohub_set_irq, remote_iohub_map_irq,
270
+
352
- &s->iohub, REMOTE_IOHUB_NB_PIRQS);
271
+ ACCEPT
353
-
272
+
354
qbus_set_hotplug_handler(BUS(pci_host->bus), OBJECT(s));
273
+
355
}
274
+CWI LICENSE AGREEMENT FOR PYTHON 0.9.0 THROUGH 1.2
356
275
+--------------------------------------------------
357
diff --git a/hw/remote/vfio-user-obj.c b/hw/remote/vfio-user-obj.c
276
+
358
index XXXXXXX..XXXXXXX 100644
277
+Copyright (c) 1991 - 1995, Stichting Mathematisch Centrum Amsterdam,
359
--- a/hw/remote/vfio-user-obj.c
278
+The Netherlands. All rights reserved.
360
+++ b/hw/remote/vfio-user-obj.c
279
+
361
@@ -XXX,XX +XXX,XX @@
280
+Permission to use, copy, modify, and distribute this software and its
362
#include "hw/pci/pci.h"
281
+documentation for any purpose and without fee is hereby granted,
363
#include "qemu/timer.h"
282
+provided that the above copyright notice appear in all copies and that
364
#include "exec/memory.h"
283
+both that copyright notice and this permission notice appear in
365
+#include "hw/pci/msi.h"
284
+supporting documentation, and that the name of Stichting Mathematisch
366
+#include "hw/pci/msix.h"
285
+Centrum or CWI not be used in advertising or publicity pertaining to
367
+#include "hw/remote/vfio-user-obj.h"
286
+distribution of the software without specific, written prior
368
287
+permission.
369
#define TYPE_VFU_OBJECT "x-vfio-user-server"
288
+
370
OBJECT_DECLARE_TYPE(VfuObject, VfuObjectClass, VFU_OBJECT)
289
+STICHTING MATHEMATISCH CENTRUM DISCLAIMS ALL WARRANTIES WITH REGARD TO
371
@@ -XXX,XX +XXX,XX @@ struct VfuObject {
290
+THIS SOFTWARE, INCLUDING ALL IMPLIED WARRANTIES OF MERCHANTABILITY AND
372
Error *unplug_blocker;
291
+FITNESS, IN NO EVENT SHALL STICHTING MATHEMATISCH CENTRUM BE LIABLE
373
292
+FOR ANY SPECIAL, INDIRECT OR CONSEQUENTIAL DAMAGES OR ANY DAMAGES
374
int vfu_poll_fd;
293
+WHATSOEVER RESULTING FROM LOSS OF USE, DATA OR PROFITS, WHETHER IN AN
375
+
294
+ACTION OF CONTRACT, NEGLIGENCE OR OTHER TORTIOUS ACTION, ARISING OUT
376
+ MSITriggerFunc *default_msi_trigger;
295
+OF OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.
377
+ MSIPrepareMessageFunc *default_msi_prepare_message;
296
diff --git a/scripts/argparse.py b/scripts/argparse.py
378
+ MSIxPrepareMessageFunc *default_msix_prepare_message;
379
};
380
381
static void vfu_object_init_ctx(VfuObject *o, Error **errp);
382
@@ -XXX,XX +XXX,XX @@ static void vfu_object_register_bars(vfu_ctx_t *vfu_ctx, PCIDevice *pdev)
383
}
384
}
385
386
+static int vfu_object_map_irq(PCIDevice *pci_dev, int intx)
387
+{
388
+ int pci_bdf = PCI_BUILD_BDF(pci_bus_num(pci_get_bus(pci_dev)),
389
+ pci_dev->devfn);
390
+
391
+ return pci_bdf;
392
+}
393
+
394
+static void vfu_object_set_irq(void *opaque, int pirq, int level)
395
+{
396
+ PCIBus *pci_bus = opaque;
397
+ PCIDevice *pci_dev = NULL;
398
+ vfu_ctx_t *vfu_ctx = NULL;
399
+ int pci_bus_num, devfn;
400
+
401
+ if (level) {
402
+ pci_bus_num = PCI_BUS_NUM(pirq);
403
+ devfn = PCI_BDF_TO_DEVFN(pirq);
404
+
405
+ /*
406
+ * pci_find_device() performs at O(1) if the device is attached
407
+ * to the root PCI bus. Whereas, if the device is attached to a
408
+ * secondary PCI bus (such as when a root port is involved),
409
+ * finding the parent PCI bus could take O(n)
410
+ */
411
+ pci_dev = pci_find_device(pci_bus, pci_bus_num, devfn);
412
+
413
+ vfu_ctx = pci_dev->irq_opaque;
414
+
415
+ g_assert(vfu_ctx);
416
+
417
+ vfu_irq_trigger(vfu_ctx, 0);
418
+ }
419
+}
420
+
421
+static MSIMessage vfu_object_msi_prepare_msg(PCIDevice *pci_dev,
422
+ unsigned int vector)
423
+{
424
+ MSIMessage msg;
425
+
426
+ msg.address = 0;
427
+ msg.data = vector;
428
+
429
+ return msg;
430
+}
431
+
432
+static void vfu_object_msi_trigger(PCIDevice *pci_dev, MSIMessage msg)
433
+{
434
+ vfu_ctx_t *vfu_ctx = pci_dev->irq_opaque;
435
+
436
+ vfu_irq_trigger(vfu_ctx, msg.data);
437
+}
438
+
439
+static void vfu_object_setup_msi_cbs(VfuObject *o)
440
+{
441
+ o->default_msi_trigger = o->pci_dev->msi_trigger;
442
+ o->default_msi_prepare_message = o->pci_dev->msi_prepare_message;
443
+ o->default_msix_prepare_message = o->pci_dev->msix_prepare_message;
444
+
445
+ o->pci_dev->msi_trigger = vfu_object_msi_trigger;
446
+ o->pci_dev->msi_prepare_message = vfu_object_msi_prepare_msg;
447
+ o->pci_dev->msix_prepare_message = vfu_object_msi_prepare_msg;
448
+}
449
+
450
+static void vfu_object_restore_msi_cbs(VfuObject *o)
451
+{
452
+ o->pci_dev->msi_trigger = o->default_msi_trigger;
453
+ o->pci_dev->msi_prepare_message = o->default_msi_prepare_message;
454
+ o->pci_dev->msix_prepare_message = o->default_msix_prepare_message;
455
+}
456
+
457
+static void vfu_msix_irq_state(vfu_ctx_t *vfu_ctx, uint32_t start,
458
+ uint32_t count, bool mask)
459
+{
460
+ VfuObject *o = vfu_get_private(vfu_ctx);
461
+ Error *err = NULL;
462
+ uint32_t vector;
463
+
464
+ for (vector = start; vector < count; vector++) {
465
+ msix_set_mask(o->pci_dev, vector, mask, &err);
466
+ if (err) {
467
+ VFU_OBJECT_ERROR(o, "vfu: %s: %s", o->device,
468
+ error_get_pretty(err));
469
+ error_free(err);
470
+ err = NULL;
471
+ }
472
+ }
473
+}
474
+
475
+static void vfu_msi_irq_state(vfu_ctx_t *vfu_ctx, uint32_t start,
476
+ uint32_t count, bool mask)
477
+{
478
+ VfuObject *o = vfu_get_private(vfu_ctx);
479
+ Error *err = NULL;
480
+ uint32_t vector;
481
+
482
+ for (vector = start; vector < count; vector++) {
483
+ msi_set_mask(o->pci_dev, vector, mask, &err);
484
+ if (err) {
485
+ VFU_OBJECT_ERROR(o, "vfu: %s: %s", o->device,
486
+ error_get_pretty(err));
487
+ error_free(err);
488
+ err = NULL;
489
+ }
490
+ }
491
+}
492
+
493
+static int vfu_object_setup_irqs(VfuObject *o, PCIDevice *pci_dev)
494
+{
495
+ vfu_ctx_t *vfu_ctx = o->vfu_ctx;
496
+ int ret;
497
+
498
+ ret = vfu_setup_device_nr_irqs(vfu_ctx, VFU_DEV_INTX_IRQ, 1);
499
+ if (ret < 0) {
500
+ return ret;
501
+ }
502
+
503
+ if (msix_nr_vectors_allocated(pci_dev)) {
504
+ ret = vfu_setup_device_nr_irqs(vfu_ctx, VFU_DEV_MSIX_IRQ,
505
+ msix_nr_vectors_allocated(pci_dev));
506
+ vfu_setup_irq_state_callback(vfu_ctx, VFU_DEV_MSIX_IRQ,
507
+ &vfu_msix_irq_state);
508
+ } else if (msi_nr_vectors_allocated(pci_dev)) {
509
+ ret = vfu_setup_device_nr_irqs(vfu_ctx, VFU_DEV_MSI_IRQ,
510
+ msi_nr_vectors_allocated(pci_dev));
511
+ vfu_setup_irq_state_callback(vfu_ctx, VFU_DEV_MSI_IRQ,
512
+ &vfu_msi_irq_state);
513
+ }
514
+
515
+ if (ret < 0) {
516
+ return ret;
517
+ }
518
+
519
+ vfu_object_setup_msi_cbs(o);
520
+
521
+ pci_dev->irq_opaque = vfu_ctx;
522
+
523
+ return 0;
524
+}
525
+
526
+void vfu_object_set_bus_irq(PCIBus *pci_bus)
527
+{
528
+ int bus_num = pci_bus_num(pci_bus);
529
+ int max_bdf = PCI_BUILD_BDF(bus_num, PCI_DEVFN_MAX - 1);
530
+
531
+ pci_bus_irqs(pci_bus, vfu_object_set_irq, vfu_object_map_irq, pci_bus,
532
+ max_bdf);
533
+}
534
+
535
/*
536
* TYPE_VFU_OBJECT depends on the availability of the 'socket' and 'device'
537
* properties. It also depends on devices instantiated in QEMU. These
538
@@ -XXX,XX +XXX,XX @@ static void vfu_object_init_ctx(VfuObject *o, Error **errp)
539
540
vfu_object_register_bars(o->vfu_ctx, o->pci_dev);
541
542
+ ret = vfu_object_setup_irqs(o, o->pci_dev);
543
+ if (ret < 0) {
544
+ error_setg(errp, "vfu: Failed to setup interrupts for %s",
545
+ o->device);
546
+ goto fail;
547
+ }
548
+
549
ret = vfu_realize_ctx(o->vfu_ctx);
550
if (ret < 0) {
551
error_setg(errp, "vfu: Failed to realize device %s- %s",
552
@@ -XXX,XX +XXX,XX @@ fail:
553
o->unplug_blocker = NULL;
554
}
555
if (o->pci_dev) {
556
+ vfu_object_restore_msi_cbs(o);
557
+ o->pci_dev->irq_opaque = NULL;
558
object_unref(OBJECT(o->pci_dev));
559
o->pci_dev = NULL;
560
}
561
@@ -XXX,XX +XXX,XX @@ static void vfu_object_finalize(Object *obj)
562
}
563
564
if (o->pci_dev) {
565
+ vfu_object_restore_msi_cbs(o);
566
+ o->pci_dev->irq_opaque = NULL;
567
object_unref(OBJECT(o->pci_dev));
568
o->pci_dev = NULL;
569
}
570
diff --git a/stubs/vfio-user-obj.c b/stubs/vfio-user-obj.c
297
new file mode 100644
571
new file mode 100644
298
index XXXXXXX..XXXXXXX
572
index XXXXXXX..XXXXXXX
299
--- /dev/null
573
--- /dev/null
300
+++ b/scripts/argparse.py
574
+++ b/stubs/vfio-user-obj.c
301
@@ -XXX,XX +XXX,XX @@
575
@@ -XXX,XX +XXX,XX @@
302
+# This is a local copy of the standard library argparse module taken from PyPI.
576
+#include "qemu/osdep.h"
303
+# It is licensed under the Python Software Foundation License. This is a
577
+#include "hw/remote/vfio-user-obj.h"
304
+# fallback for Python 2.6 which does not include this module. Python 2.7+ and
578
+
305
+# 3+ will never load this module because built-in modules are loaded before
579
+void vfu_object_set_bus_irq(PCIBus *pci_bus)
306
+# anything in sys.path.
580
+{
307
+#
581
+}
308
+# If your script is not located in the same directory as this file, import it
582
diff --git a/hw/remote/trace-events b/hw/remote/trace-events
309
+# like this:
583
index XXXXXXX..XXXXXXX 100644
310
+#
584
--- a/hw/remote/trace-events
311
+# import os
585
+++ b/hw/remote/trace-events
312
+# import sys
586
@@ -XXX,XX +XXX,XX @@ vfu_dma_unregister(uint64_t gpa) "vfu: unregistering GPA 0x%"PRIx64""
313
+# sys.path.append(os.path.join(os.path.dirname(__file__), ..., 'scripts'))
587
vfu_bar_register(int i, uint64_t addr, uint64_t size) "vfu: BAR %d: addr 0x%"PRIx64" size 0x%"PRIx64""
314
+# import argparse
588
vfu_bar_rw_enter(const char *op, uint64_t addr) "vfu: %s request for BAR address 0x%"PRIx64""
315
+
589
vfu_bar_rw_exit(const char *op, uint64_t addr) "vfu: Finished %s of BAR address 0x%"PRIx64""
316
+# Author: Steven J. Bethard <steven.bethard@gmail.com>.
590
+vfu_interrupt(int pirq) "vfu: sending interrupt to device - PIRQ %d"
317
+# Maintainer: Thomas Waldmann <tw@waldmann-edv.de>
591
diff --git a/stubs/meson.build b/stubs/meson.build
318
+
592
index XXXXXXX..XXXXXXX 100644
319
+"""Command-line parsing library
593
--- a/stubs/meson.build
320
+
594
+++ b/stubs/meson.build
321
+This module is an optparse-inspired command-line parsing library that:
595
@@ -XXX,XX +XXX,XX @@ if have_system
322
+
596
else
323
+ - handles both optional and positional arguments
597
stub_ss.add(files('qdev.c'))
324
+ - produces highly informative usage messages
598
endif
325
+ - supports parsers that dispatch to sub-parsers
599
+stub_ss.add(when: 'CONFIG_VFIO_USER_SERVER', if_false: files('vfio-user-obj.c'))
326
+
327
+The following is a simple usage example that sums integers from the
328
+command-line and writes the result to a file::
329
+
330
+ parser = argparse.ArgumentParser(
331
+ description='sum the integers at the command line')
332
+ parser.add_argument(
333
+ 'integers', metavar='int', nargs='+', type=int,
334
+ help='an integer to be summed')
335
+ parser.add_argument(
336
+ '--log', default=sys.stdout, type=argparse.FileType('w'),
337
+ help='the file where the sum should be written')
338
+ args = parser.parse_args()
339
+ args.log.write('%s' % sum(args.integers))
340
+ args.log.close()
341
+
342
+The module contains the following public classes:
343
+
344
+ - ArgumentParser -- The main entry point for command-line parsing. As the
345
+ example above shows, the add_argument() method is used to populate
346
+ the parser with actions for optional and positional arguments. Then
347
+ the parse_args() method is invoked to convert the args at the
348
+ command-line into an object with attributes.
349
+
350
+ - ArgumentError -- The exception raised by ArgumentParser objects when
351
+ there are errors with the parser's actions. Errors raised while
352
+ parsing the command-line are caught by ArgumentParser and emitted
353
+ as command-line messages.
354
+
355
+ - FileType -- A factory for defining types of files to be created. As the
356
+ example above shows, instances of FileType are typically passed as
357
+ the type= argument of add_argument() calls.
358
+
359
+ - Action -- The base class for parser actions. Typically actions are
360
+ selected by passing strings like 'store_true' or 'append_const' to
361
+ the action= argument of add_argument(). However, for greater
362
+ customization of ArgumentParser actions, subclasses of Action may
363
+ be defined and passed as the action= argument.
364
+
365
+ - HelpFormatter, RawDescriptionHelpFormatter, RawTextHelpFormatter,
366
+ ArgumentDefaultsHelpFormatter -- Formatter classes which
367
+ may be passed as the formatter_class= argument to the
368
+ ArgumentParser constructor. HelpFormatter is the default,
369
+ RawDescriptionHelpFormatter and RawTextHelpFormatter tell the parser
370
+ not to change the formatting for help text, and
371
+ ArgumentDefaultsHelpFormatter adds information about argument defaults
372
+ to the help.
373
+
374
+All other classes in this module are considered implementation details.
375
+(Also note that HelpFormatter and RawDescriptionHelpFormatter are only
376
+considered public as object names -- the API of the formatter objects is
377
+still considered an implementation detail.)
378
+"""
379
+
380
+__version__ = '1.4.0' # we use our own version number independant of the
381
+ # one in stdlib and we release this on pypi.
382
+
383
+__external_lib__ = True # to make sure the tests really test THIS lib,
384
+ # not the builtin one in Python stdlib
385
+
386
+__all__ = [
387
+ 'ArgumentParser',
388
+ 'ArgumentError',
389
+ 'ArgumentTypeError',
390
+ 'FileType',
391
+ 'HelpFormatter',
392
+ 'ArgumentDefaultsHelpFormatter',
393
+ 'RawDescriptionHelpFormatter',
394
+ 'RawTextHelpFormatter',
395
+ 'Namespace',
396
+ 'Action',
397
+ 'ONE_OR_MORE',
398
+ 'OPTIONAL',
399
+ 'PARSER',
400
+ 'REMAINDER',
401
+ 'SUPPRESS',
402
+ 'ZERO_OR_MORE',
403
+]
404
+
405
+
406
+import copy as _copy
407
+import os as _os
408
+import re as _re
409
+import sys as _sys
410
+import textwrap as _textwrap
411
+
412
+from gettext import gettext as _
413
+
414
+try:
415
+ set
416
+except NameError:
417
+ # for python < 2.4 compatibility (sets module is there since 2.3):
418
+ from sets import Set as set
419
+
420
+try:
421
+ basestring
422
+except NameError:
423
+ basestring = str
424
+
425
+try:
426
+ sorted
427
+except NameError:
428
+ # for python < 2.4 compatibility:
429
+ def sorted(iterable, reverse=False):
430
+ result = list(iterable)
431
+ result.sort()
432
+ if reverse:
433
+ result.reverse()
434
+ return result
435
+
436
+
437
+def _callable(obj):
438
+ return hasattr(obj, '__call__') or hasattr(obj, '__bases__')
439
+
440
+
441
+SUPPRESS = '==SUPPRESS=='
442
+
443
+OPTIONAL = '?'
444
+ZERO_OR_MORE = '*'
445
+ONE_OR_MORE = '+'
446
+PARSER = 'A...'
447
+REMAINDER = '...'
448
+_UNRECOGNIZED_ARGS_ATTR = '_unrecognized_args'
449
+
450
+# =============================
451
+# Utility functions and classes
452
+# =============================
453
+
454
+class _AttributeHolder(object):
455
+ """Abstract base class that provides __repr__.
456
+
457
+ The __repr__ method returns a string in the format::
458
+ ClassName(attr=name, attr=name, ...)
459
+ The attributes are determined either by a class-level attribute,
460
+ '_kwarg_names', or by inspecting the instance __dict__.
461
+ """
462
+
463
+ def __repr__(self):
464
+ type_name = type(self).__name__
465
+ arg_strings = []
466
+ for arg in self._get_args():
467
+ arg_strings.append(repr(arg))
468
+ for name, value in self._get_kwargs():
469
+ arg_strings.append('%s=%r' % (name, value))
470
+ return '%s(%s)' % (type_name, ', '.join(arg_strings))
471
+
472
+ def _get_kwargs(self):
473
+ return sorted(self.__dict__.items())
474
+
475
+ def _get_args(self):
476
+ return []
477
+
478
+
479
+def _ensure_value(namespace, name, value):
480
+ if getattr(namespace, name, None) is None:
481
+ setattr(namespace, name, value)
482
+ return getattr(namespace, name)
483
+
484
+
485
+# ===============
486
+# Formatting Help
487
+# ===============
488
+
489
+class HelpFormatter(object):
490
+ """Formatter for generating usage messages and argument help strings.
491
+
492
+ Only the name of this class is considered a public API. All the methods
493
+ provided by the class are considered an implementation detail.
494
+ """
495
+
496
+ def __init__(self,
497
+ prog,
498
+ indent_increment=2,
499
+ max_help_position=24,
500
+ width=None):
501
+
502
+ # default setting for width
503
+ if width is None:
504
+ try:
505
+ width = int(_os.environ['COLUMNS'])
506
+ except (KeyError, ValueError):
507
+ width = 80
508
+ width -= 2
509
+
510
+ self._prog = prog
511
+ self._indent_increment = indent_increment
512
+ self._max_help_position = max_help_position
513
+ self._width = width
514
+
515
+ self._current_indent = 0
516
+ self._level = 0
517
+ self._action_max_length = 0
518
+
519
+ self._root_section = self._Section(self, None)
520
+ self._current_section = self._root_section
521
+
522
+ self._whitespace_matcher = _re.compile(r'\s+')
523
+ self._long_break_matcher = _re.compile(r'\n\n\n+')
524
+
525
+ # ===============================
526
+ # Section and indentation methods
527
+ # ===============================
528
+ def _indent(self):
529
+ self._current_indent += self._indent_increment
530
+ self._level += 1
531
+
532
+ def _dedent(self):
533
+ self._current_indent -= self._indent_increment
534
+ assert self._current_indent >= 0, 'Indent decreased below 0.'
535
+ self._level -= 1
536
+
537
+ class _Section(object):
538
+
539
+ def __init__(self, formatter, parent, heading=None):
540
+ self.formatter = formatter
541
+ self.parent = parent
542
+ self.heading = heading
543
+ self.items = []
544
+
545
+ def format_help(self):
546
+ # format the indented section
547
+ if self.parent is not None:
548
+ self.formatter._indent()
549
+ join = self.formatter._join_parts
550
+ for func, args in self.items:
551
+ func(*args)
552
+ item_help = join([func(*args) for func, args in self.items])
553
+ if self.parent is not None:
554
+ self.formatter._dedent()
555
+
556
+ # return nothing if the section was empty
557
+ if not item_help:
558
+ return ''
559
+
560
+ # add the heading if the section was non-empty
561
+ if self.heading is not SUPPRESS and self.heading is not None:
562
+ current_indent = self.formatter._current_indent
563
+ heading = '%*s%s:\n' % (current_indent, '', self.heading)
564
+ else:
565
+ heading = ''
566
+
567
+ # join the section-initial newline, the heading and the help
568
+ return join(['\n', heading, item_help, '\n'])
569
+
570
+ def _add_item(self, func, args):
571
+ self._current_section.items.append((func, args))
572
+
573
+ # ========================
574
+ # Message building methods
575
+ # ========================
576
+ def start_section(self, heading):
577
+ self._indent()
578
+ section = self._Section(self, self._current_section, heading)
579
+ self._add_item(section.format_help, [])
580
+ self._current_section = section
581
+
582
+ def end_section(self):
583
+ self._current_section = self._current_section.parent
584
+ self._dedent()
585
+
586
+ def add_text(self, text):
587
+ if text is not SUPPRESS and text is not None:
588
+ self._add_item(self._format_text, [text])
589
+
590
+ def add_usage(self, usage, actions, groups, prefix=None):
591
+ if usage is not SUPPRESS:
592
+ args = usage, actions, groups, prefix
593
+ self._add_item(self._format_usage, args)
594
+
595
+ def add_argument(self, action):
596
+ if action.help is not SUPPRESS:
597
+
598
+ # find all invocations
599
+ get_invocation = self._format_action_invocation
600
+ invocations = [get_invocation(action)]
601
+ for subaction in self._iter_indented_subactions(action):
602
+ invocations.append(get_invocation(subaction))
603
+
604
+ # update the maximum item length
605
+ invocation_length = max([len(s) for s in invocations])
606
+ action_length = invocation_length + self._current_indent
607
+ self._action_max_length = max(self._action_max_length,
608
+ action_length)
609
+
610
+ # add the item to the list
611
+ self._add_item(self._format_action, [action])
612
+
613
+ def add_arguments(self, actions):
614
+ for action in actions:
615
+ self.add_argument(action)
616
+
617
+ # =======================
618
+ # Help-formatting methods
619
+ # =======================
620
+ def format_help(self):
621
+ help = self._root_section.format_help()
622
+ if help:
623
+ help = self._long_break_matcher.sub('\n\n', help)
624
+ help = help.strip('\n') + '\n'
625
+ return help
626
+
627
+ def _join_parts(self, part_strings):
628
+ return ''.join([part
629
+ for part in part_strings
630
+ if part and part is not SUPPRESS])
631
+
632
+ def _format_usage(self, usage, actions, groups, prefix):
633
+ if prefix is None:
634
+ prefix = _('usage: ')
635
+
636
+ # if usage is specified, use that
637
+ if usage is not None:
638
+ usage = usage % dict(prog=self._prog)
639
+
640
+ # if no optionals or positionals are available, usage is just prog
641
+ elif usage is None and not actions:
642
+ usage = '%(prog)s' % dict(prog=self._prog)
643
+
644
+ # if optionals and positionals are available, calculate usage
645
+ elif usage is None:
646
+ prog = '%(prog)s' % dict(prog=self._prog)
647
+
648
+ # split optionals from positionals
649
+ optionals = []
650
+ positionals = []
651
+ for action in actions:
652
+ if action.option_strings:
653
+ optionals.append(action)
654
+ else:
655
+ positionals.append(action)
656
+
657
+ # build full usage string
658
+ format = self._format_actions_usage
659
+ action_usage = format(optionals + positionals, groups)
660
+ usage = ' '.join([s for s in [prog, action_usage] if s])
661
+
662
+ # wrap the usage parts if it's too long
663
+ text_width = self._width - self._current_indent
664
+ if len(prefix) + len(usage) > text_width:
665
+
666
+ # break usage into wrappable parts
667
+ part_regexp = r'\(.*?\)+|\[.*?\]+|\S+'
668
+ opt_usage = format(optionals, groups)
669
+ pos_usage = format(positionals, groups)
670
+ opt_parts = _re.findall(part_regexp, opt_usage)
671
+ pos_parts = _re.findall(part_regexp, pos_usage)
672
+ assert ' '.join(opt_parts) == opt_usage
673
+ assert ' '.join(pos_parts) == pos_usage
674
+
675
+ # helper for wrapping lines
676
+ def get_lines(parts, indent, prefix=None):
677
+ lines = []
678
+ line = []
679
+ if prefix is not None:
680
+ line_len = len(prefix) - 1
681
+ else:
682
+ line_len = len(indent) - 1
683
+ for part in parts:
684
+ if line_len + 1 + len(part) > text_width:
685
+ lines.append(indent + ' '.join(line))
686
+ line = []
687
+ line_len = len(indent) - 1
688
+ line.append(part)
689
+ line_len += len(part) + 1
690
+ if line:
691
+ lines.append(indent + ' '.join(line))
692
+ if prefix is not None:
693
+ lines[0] = lines[0][len(indent):]
694
+ return lines
695
+
696
+ # if prog is short, follow it with optionals or positionals
697
+ if len(prefix) + len(prog) <= 0.75 * text_width:
698
+ indent = ' ' * (len(prefix) + len(prog) + 1)
699
+ if opt_parts:
700
+ lines = get_lines([prog] + opt_parts, indent, prefix)
701
+ lines.extend(get_lines(pos_parts, indent))
702
+ elif pos_parts:
703
+ lines = get_lines([prog] + pos_parts, indent, prefix)
704
+ else:
705
+ lines = [prog]
706
+
707
+ # if prog is long, put it on its own line
708
+ else:
709
+ indent = ' ' * len(prefix)
710
+ parts = opt_parts + pos_parts
711
+ lines = get_lines(parts, indent)
712
+ if len(lines) > 1:
713
+ lines = []
714
+ lines.extend(get_lines(opt_parts, indent))
715
+ lines.extend(get_lines(pos_parts, indent))
716
+ lines = [prog] + lines
717
+
718
+ # join lines into usage
719
+ usage = '\n'.join(lines)
720
+
721
+ # prefix with 'usage:'
722
+ return '%s%s\n\n' % (prefix, usage)
723
+
724
+ def _format_actions_usage(self, actions, groups):
725
+ # find group indices and identify actions in groups
726
+ group_actions = set()
727
+ inserts = {}
728
+ for group in groups:
729
+ try:
730
+ start = actions.index(group._group_actions[0])
731
+ except ValueError:
732
+ continue
733
+ else:
734
+ end = start + len(group._group_actions)
735
+ if actions[start:end] == group._group_actions:
736
+ for action in group._group_actions:
737
+ group_actions.add(action)
738
+ if not group.required:
739
+ if start in inserts:
740
+ inserts[start] += ' ['
741
+ else:
742
+ inserts[start] = '['
743
+ inserts[end] = ']'
744
+ else:
745
+ if start in inserts:
746
+ inserts[start] += ' ('
747
+ else:
748
+ inserts[start] = '('
749
+ inserts[end] = ')'
750
+ for i in range(start + 1, end):
751
+ inserts[i] = '|'
752
+
753
+ # collect all actions format strings
754
+ parts = []
755
+ for i, action in enumerate(actions):
756
+
757
+ # suppressed arguments are marked with None
758
+ # remove | separators for suppressed arguments
759
+ if action.help is SUPPRESS:
760
+ parts.append(None)
761
+ if inserts.get(i) == '|':
762
+ inserts.pop(i)
763
+ elif inserts.get(i + 1) == '|':
764
+ inserts.pop(i + 1)
765
+
766
+ # produce all arg strings
767
+ elif not action.option_strings:
768
+ part = self._format_args(action, action.dest)
769
+
770
+ # if it's in a group, strip the outer []
771
+ if action in group_actions:
772
+ if part[0] == '[' and part[-1] == ']':
773
+ part = part[1:-1]
774
+
775
+ # add the action string to the list
776
+ parts.append(part)
777
+
778
+ # produce the first way to invoke the option in brackets
779
+ else:
780
+ option_string = action.option_strings[0]
781
+
782
+ # if the Optional doesn't take a value, format is:
783
+ # -s or --long
784
+ if action.nargs == 0:
785
+ part = '%s' % option_string
786
+
787
+ # if the Optional takes a value, format is:
788
+ # -s ARGS or --long ARGS
789
+ else:
790
+ default = action.dest.upper()
791
+ args_string = self._format_args(action, default)
792
+ part = '%s %s' % (option_string, args_string)
793
+
794
+ # make it look optional if it's not required or in a group
795
+ if not action.required and action not in group_actions:
796
+ part = '[%s]' % part
797
+
798
+ # add the action string to the list
799
+ parts.append(part)
800
+
801
+ # insert things at the necessary indices
802
+ for i in sorted(inserts, reverse=True):
803
+ parts[i:i] = [inserts[i]]
804
+
805
+ # join all the action items with spaces
806
+ text = ' '.join([item for item in parts if item is not None])
807
+
808
+ # clean up separators for mutually exclusive groups
809
+ open = r'[\[(]'
810
+ close = r'[\])]'
811
+ text = _re.sub(r'(%s) ' % open, r'\1', text)
812
+ text = _re.sub(r' (%s)' % close, r'\1', text)
813
+ text = _re.sub(r'%s *%s' % (open, close), r'', text)
814
+ text = _re.sub(r'\(([^|]*)\)', r'\1', text)
815
+ text = text.strip()
816
+
817
+ # return the text
818
+ return text
819
+
820
+ def _format_text(self, text):
821
+ if '%(prog)' in text:
822
+ text = text % dict(prog=self._prog)
823
+ text_width = self._width - self._current_indent
824
+ indent = ' ' * self._current_indent
825
+ return self._fill_text(text, text_width, indent) + '\n\n'
826
+
827
+ def _format_action(self, action):
828
+ # determine the required width and the entry label
829
+ help_position = min(self._action_max_length + 2,
830
+ self._max_help_position)
831
+ help_width = self._width - help_position
832
+ action_width = help_position - self._current_indent - 2
833
+ action_header = self._format_action_invocation(action)
834
+
835
+ # ho nelp; start on same line and add a final newline
836
+ if not action.help:
837
+ tup = self._current_indent, '', action_header
838
+ action_header = '%*s%s\n' % tup
839
+
840
+ # short action name; start on the same line and pad two spaces
841
+ elif len(action_header) <= action_width:
842
+ tup = self._current_indent, '', action_width, action_header
843
+ action_header = '%*s%-*s ' % tup
844
+ indent_first = 0
845
+
846
+ # long action name; start on the next line
847
+ else:
848
+ tup = self._current_indent, '', action_header
849
+ action_header = '%*s%s\n' % tup
850
+ indent_first = help_position
851
+
852
+ # collect the pieces of the action help
853
+ parts = [action_header]
854
+
855
+ # if there was help for the action, add lines of help text
856
+ if action.help:
857
+ help_text = self._expand_help(action)
858
+ help_lines = self._split_lines(help_text, help_width)
859
+ parts.append('%*s%s\n' % (indent_first, '', help_lines[0]))
860
+ for line in help_lines[1:]:
861
+ parts.append('%*s%s\n' % (help_position, '', line))
862
+
863
+ # or add a newline if the description doesn't end with one
864
+ elif not action_header.endswith('\n'):
865
+ parts.append('\n')
866
+
867
+ # if there are any sub-actions, add their help as well
868
+ for subaction in self._iter_indented_subactions(action):
869
+ parts.append(self._format_action(subaction))
870
+
871
+ # return a single string
872
+ return self._join_parts(parts)
873
+
874
+ def _format_action_invocation(self, action):
875
+ if not action.option_strings:
876
+ metavar, = self._metavar_formatter(action, action.dest)(1)
877
+ return metavar
878
+
879
+ else:
880
+ parts = []
881
+
882
+ # if the Optional doesn't take a value, format is:
883
+ # -s, --long
884
+ if action.nargs == 0:
885
+ parts.extend(action.option_strings)
886
+
887
+ # if the Optional takes a value, format is:
888
+ # -s ARGS, --long ARGS
889
+ else:
890
+ default = action.dest.upper()
891
+ args_string = self._format_args(action, default)
892
+ for option_string in action.option_strings:
893
+ parts.append('%s %s' % (option_string, args_string))
894
+
895
+ return ', '.join(parts)
896
+
897
+ def _metavar_formatter(self, action, default_metavar):
898
+ if action.metavar is not None:
899
+ result = action.metavar
900
+ elif action.choices is not None:
901
+ choice_strs = [str(choice) for choice in action.choices]
902
+ result = '{%s}' % ','.join(choice_strs)
903
+ else:
904
+ result = default_metavar
905
+
906
+ def format(tuple_size):
907
+ if isinstance(result, tuple):
908
+ return result
909
+ else:
910
+ return (result, ) * tuple_size
911
+ return format
912
+
913
+ def _format_args(self, action, default_metavar):
914
+ get_metavar = self._metavar_formatter(action, default_metavar)
915
+ if action.nargs is None:
916
+ result = '%s' % get_metavar(1)
917
+ elif action.nargs == OPTIONAL:
918
+ result = '[%s]' % get_metavar(1)
919
+ elif action.nargs == ZERO_OR_MORE:
920
+ result = '[%s [%s ...]]' % get_metavar(2)
921
+ elif action.nargs == ONE_OR_MORE:
922
+ result = '%s [%s ...]' % get_metavar(2)
923
+ elif action.nargs == REMAINDER:
924
+ result = '...'
925
+ elif action.nargs == PARSER:
926
+ result = '%s ...' % get_metavar(1)
927
+ else:
928
+ formats = ['%s' for _ in range(action.nargs)]
929
+ result = ' '.join(formats) % get_metavar(action.nargs)
930
+ return result
931
+
932
+ def _expand_help(self, action):
933
+ params = dict(vars(action), prog=self._prog)
934
+ for name in list(params):
935
+ if params[name] is SUPPRESS:
936
+ del params[name]
937
+ for name in list(params):
938
+ if hasattr(params[name], '__name__'):
939
+ params[name] = params[name].__name__
940
+ if params.get('choices') is not None:
941
+ choices_str = ', '.join([str(c) for c in params['choices']])
942
+ params['choices'] = choices_str
943
+ return self._get_help_string(action) % params
944
+
945
+ def _iter_indented_subactions(self, action):
946
+ try:
947
+ get_subactions = action._get_subactions
948
+ except AttributeError:
949
+ pass
950
+ else:
951
+ self._indent()
952
+ for subaction in get_subactions():
953
+ yield subaction
954
+ self._dedent()
955
+
956
+ def _split_lines(self, text, width):
957
+ text = self._whitespace_matcher.sub(' ', text).strip()
958
+ return _textwrap.wrap(text, width)
959
+
960
+ def _fill_text(self, text, width, indent):
961
+ text = self._whitespace_matcher.sub(' ', text).strip()
962
+ return _textwrap.fill(text, width, initial_indent=indent,
963
+ subsequent_indent=indent)
964
+
965
+ def _get_help_string(self, action):
966
+ return action.help
967
+
968
+
969
+class RawDescriptionHelpFormatter(HelpFormatter):
970
+ """Help message formatter which retains any formatting in descriptions.
971
+
972
+ Only the name of this class is considered a public API. All the methods
973
+ provided by the class are considered an implementation detail.
974
+ """
975
+
976
+ def _fill_text(self, text, width, indent):
977
+ return ''.join([indent + line for line in text.splitlines(True)])
978
+
979
+
980
+class RawTextHelpFormatter(RawDescriptionHelpFormatter):
981
+ """Help message formatter which retains formatting of all help text.
982
+
983
+ Only the name of this class is considered a public API. All the methods
984
+ provided by the class are considered an implementation detail.
985
+ """
986
+
987
+ def _split_lines(self, text, width):
988
+ return text.splitlines()
989
+
990
+
991
+class ArgumentDefaultsHelpFormatter(HelpFormatter):
992
+ """Help message formatter which adds default values to argument help.
993
+
994
+ Only the name of this class is considered a public API. All the methods
995
+ provided by the class are considered an implementation detail.
996
+ """
997
+
998
+ def _get_help_string(self, action):
999
+ help = action.help
1000
+ if '%(default)' not in action.help:
1001
+ if action.default is not SUPPRESS:
1002
+ defaulting_nargs = [OPTIONAL, ZERO_OR_MORE]
1003
+ if action.option_strings or action.nargs in defaulting_nargs:
1004
+ help += ' (default: %(default)s)'
1005
+ return help
1006
+
1007
+
1008
+# =====================
1009
+# Options and Arguments
1010
+# =====================
1011
+
1012
+def _get_action_name(argument):
1013
+ if argument is None:
1014
+ return None
1015
+ elif argument.option_strings:
1016
+ return '/'.join(argument.option_strings)
1017
+ elif argument.metavar not in (None, SUPPRESS):
1018
+ return argument.metavar
1019
+ elif argument.dest not in (None, SUPPRESS):
1020
+ return argument.dest
1021
+ else:
1022
+ return None
1023
+
1024
+
1025
+class ArgumentError(Exception):
1026
+ """An error from creating or using an argument (optional or positional).
1027
+
1028
+ The string value of this exception is the message, augmented with
1029
+ information about the argument that caused it.
1030
+ """
1031
+
1032
+ def __init__(self, argument, message):
1033
+ self.argument_name = _get_action_name(argument)
1034
+ self.message = message
1035
+
1036
+ def __str__(self):
1037
+ if self.argument_name is None:
1038
+ format = '%(message)s'
1039
+ else:
1040
+ format = 'argument %(argument_name)s: %(message)s'
1041
+ return format % dict(message=self.message,
1042
+ argument_name=self.argument_name)
1043
+
1044
+
1045
+class ArgumentTypeError(Exception):
1046
+ """An error from trying to convert a command line string to a type."""
1047
+ pass
1048
+
1049
+
1050
+# ==============
1051
+# Action classes
1052
+# ==============
1053
+
1054
+class Action(_AttributeHolder):
1055
+ """Information about how to convert command line strings to Python objects.
1056
+
1057
+ Action objects are used by an ArgumentParser to represent the information
1058
+ needed to parse a single argument from one or more strings from the
1059
+ command line. The keyword arguments to the Action constructor are also
1060
+ all attributes of Action instances.
1061
+
1062
+ Keyword Arguments:
1063
+
1064
+ - option_strings -- A list of command-line option strings which
1065
+ should be associated with this action.
1066
+
1067
+ - dest -- The name of the attribute to hold the created object(s)
1068
+
1069
+ - nargs -- The number of command-line arguments that should be
1070
+ consumed. By default, one argument will be consumed and a single
1071
+ value will be produced. Other values include:
1072
+ - N (an integer) consumes N arguments (and produces a list)
1073
+ - '?' consumes zero or one arguments
1074
+ - '*' consumes zero or more arguments (and produces a list)
1075
+ - '+' consumes one or more arguments (and produces a list)
1076
+ Note that the difference between the default and nargs=1 is that
1077
+ with the default, a single value will be produced, while with
1078
+ nargs=1, a list containing a single value will be produced.
1079
+
1080
+ - const -- The value to be produced if the option is specified and the
1081
+ option uses an action that takes no values.
1082
+
1083
+ - default -- The value to be produced if the option is not specified.
1084
+
1085
+ - type -- The type which the command-line arguments should be converted
1086
+ to, should be one of 'string', 'int', 'float', 'complex' or a
1087
+ callable object that accepts a single string argument. If None,
1088
+ 'string' is assumed.
1089
+
1090
+ - choices -- A container of values that should be allowed. If not None,
1091
+ after a command-line argument has been converted to the appropriate
1092
+ type, an exception will be raised if it is not a member of this
1093
+ collection.
1094
+
1095
+ - required -- True if the action must always be specified at the
1096
+ command line. This is only meaningful for optional command-line
1097
+ arguments.
1098
+
1099
+ - help -- The help string describing the argument.
1100
+
1101
+ - metavar -- The name to be used for the option's argument with the
1102
+ help string. If None, the 'dest' value will be used as the name.
1103
+ """
1104
+
1105
+ def __init__(self,
1106
+ option_strings,
1107
+ dest,
1108
+ nargs=None,
1109
+ const=None,
1110
+ default=None,
1111
+ type=None,
1112
+ choices=None,
1113
+ required=False,
1114
+ help=None,
1115
+ metavar=None):
1116
+ self.option_strings = option_strings
1117
+ self.dest = dest
1118
+ self.nargs = nargs
1119
+ self.const = const
1120
+ self.default = default
1121
+ self.type = type
1122
+ self.choices = choices
1123
+ self.required = required
1124
+ self.help = help
1125
+ self.metavar = metavar
1126
+
1127
+ def _get_kwargs(self):
1128
+ names = [
1129
+ 'option_strings',
1130
+ 'dest',
1131
+ 'nargs',
1132
+ 'const',
1133
+ 'default',
1134
+ 'type',
1135
+ 'choices',
1136
+ 'help',
1137
+ 'metavar',
1138
+ ]
1139
+ return [(name, getattr(self, name)) for name in names]
1140
+
1141
+ def __call__(self, parser, namespace, values, option_string=None):
1142
+ raise NotImplementedError(_('.__call__() not defined'))
1143
+
1144
+
1145
+class _StoreAction(Action):
1146
+
1147
+ def __init__(self,
1148
+ option_strings,
1149
+ dest,
1150
+ nargs=None,
1151
+ const=None,
1152
+ default=None,
1153
+ type=None,
1154
+ choices=None,
1155
+ required=False,
1156
+ help=None,
1157
+ metavar=None):
1158
+ if nargs == 0:
1159
+ raise ValueError('nargs for store actions must be > 0; if you '
1160
+ 'have nothing to store, actions such as store '
1161
+ 'true or store const may be more appropriate')
1162
+ if const is not None and nargs != OPTIONAL:
1163
+ raise ValueError('nargs must be %r to supply const' % OPTIONAL)
1164
+ super(_StoreAction, self).__init__(
1165
+ option_strings=option_strings,
1166
+ dest=dest,
1167
+ nargs=nargs,
1168
+ const=const,
1169
+ default=default,
1170
+ type=type,
1171
+ choices=choices,
1172
+ required=required,
1173
+ help=help,
1174
+ metavar=metavar)
1175
+
1176
+ def __call__(self, parser, namespace, values, option_string=None):
1177
+ setattr(namespace, self.dest, values)
1178
+
1179
+
1180
+class _StoreConstAction(Action):
1181
+
1182
+ def __init__(self,
1183
+ option_strings,
1184
+ dest,
1185
+ const,
1186
+ default=None,
1187
+ required=False,
1188
+ help=None,
1189
+ metavar=None):
1190
+ super(_StoreConstAction, self).__init__(
1191
+ option_strings=option_strings,
1192
+ dest=dest,
1193
+ nargs=0,
1194
+ const=const,
1195
+ default=default,
1196
+ required=required,
1197
+ help=help)
1198
+
1199
+ def __call__(self, parser, namespace, values, option_string=None):
1200
+ setattr(namespace, self.dest, self.const)
1201
+
1202
+
1203
+class _StoreTrueAction(_StoreConstAction):
1204
+
1205
+ def __init__(self,
1206
+ option_strings,
1207
+ dest,
1208
+ default=False,
1209
+ required=False,
1210
+ help=None):
1211
+ super(_StoreTrueAction, self).__init__(
1212
+ option_strings=option_strings,
1213
+ dest=dest,
1214
+ const=True,
1215
+ default=default,
1216
+ required=required,
1217
+ help=help)
1218
+
1219
+
1220
+class _StoreFalseAction(_StoreConstAction):
1221
+
1222
+ def __init__(self,
1223
+ option_strings,
1224
+ dest,
1225
+ default=True,
1226
+ required=False,
1227
+ help=None):
1228
+ super(_StoreFalseAction, self).__init__(
1229
+ option_strings=option_strings,
1230
+ dest=dest,
1231
+ const=False,
1232
+ default=default,
1233
+ required=required,
1234
+ help=help)
1235
+
1236
+
1237
+class _AppendAction(Action):
1238
+
1239
+ def __init__(self,
1240
+ option_strings,
1241
+ dest,
1242
+ nargs=None,
1243
+ const=None,
1244
+ default=None,
1245
+ type=None,
1246
+ choices=None,
1247
+ required=False,
1248
+ help=None,
1249
+ metavar=None):
1250
+ if nargs == 0:
1251
+ raise ValueError('nargs for append actions must be > 0; if arg '
1252
+ 'strings are not supplying the value to append, '
1253
+ 'the append const action may be more appropriate')
1254
+ if const is not None and nargs != OPTIONAL:
1255
+ raise ValueError('nargs must be %r to supply const' % OPTIONAL)
1256
+ super(_AppendAction, self).__init__(
1257
+ option_strings=option_strings,
1258
+ dest=dest,
1259
+ nargs=nargs,
1260
+ const=const,
1261
+ default=default,
1262
+ type=type,
1263
+ choices=choices,
1264
+ required=required,
1265
+ help=help,
1266
+ metavar=metavar)
1267
+
1268
+ def __call__(self, parser, namespace, values, option_string=None):
1269
+ items = _copy.copy(_ensure_value(namespace, self.dest, []))
1270
+ items.append(values)
1271
+ setattr(namespace, self.dest, items)
1272
+
1273
+
1274
+class _AppendConstAction(Action):
1275
+
1276
+ def __init__(self,
1277
+ option_strings,
1278
+ dest,
1279
+ const,
1280
+ default=None,
1281
+ required=False,
1282
+ help=None,
1283
+ metavar=None):
1284
+ super(_AppendConstAction, self).__init__(
1285
+ option_strings=option_strings,
1286
+ dest=dest,
1287
+ nargs=0,
1288
+ const=const,
1289
+ default=default,
1290
+ required=required,
1291
+ help=help,
1292
+ metavar=metavar)
1293
+
1294
+ def __call__(self, parser, namespace, values, option_string=None):
1295
+ items = _copy.copy(_ensure_value(namespace, self.dest, []))
1296
+ items.append(self.const)
1297
+ setattr(namespace, self.dest, items)
1298
+
1299
+
1300
+class _CountAction(Action):
1301
+
1302
+ def __init__(self,
1303
+ option_strings,
1304
+ dest,
1305
+ default=None,
1306
+ required=False,
1307
+ help=None):
1308
+ super(_CountAction, self).__init__(
1309
+ option_strings=option_strings,
1310
+ dest=dest,
1311
+ nargs=0,
1312
+ default=default,
1313
+ required=required,
1314
+ help=help)
1315
+
1316
+ def __call__(self, parser, namespace, values, option_string=None):
1317
+ new_count = _ensure_value(namespace, self.dest, 0) + 1
1318
+ setattr(namespace, self.dest, new_count)
1319
+
1320
+
1321
+class _HelpAction(Action):
1322
+
1323
+ def __init__(self,
1324
+ option_strings,
1325
+ dest=SUPPRESS,
1326
+ default=SUPPRESS,
1327
+ help=None):
1328
+ super(_HelpAction, self).__init__(
1329
+ option_strings=option_strings,
1330
+ dest=dest,
1331
+ default=default,
1332
+ nargs=0,
1333
+ help=help)
1334
+
1335
+ def __call__(self, parser, namespace, values, option_string=None):
1336
+ parser.print_help()
1337
+ parser.exit()
1338
+
1339
+
1340
+class _VersionAction(Action):
1341
+
1342
+ def __init__(self,
1343
+ option_strings,
1344
+ version=None,
1345
+ dest=SUPPRESS,
1346
+ default=SUPPRESS,
1347
+ help="show program's version number and exit"):
1348
+ super(_VersionAction, self).__init__(
1349
+ option_strings=option_strings,
1350
+ dest=dest,
1351
+ default=default,
1352
+ nargs=0,
1353
+ help=help)
1354
+ self.version = version
1355
+
1356
+ def __call__(self, parser, namespace, values, option_string=None):
1357
+ version = self.version
1358
+ if version is None:
1359
+ version = parser.version
1360
+ formatter = parser._get_formatter()
1361
+ formatter.add_text(version)
1362
+ parser.exit(message=formatter.format_help())
1363
+
1364
+
1365
+class _SubParsersAction(Action):
1366
+
1367
+ class _ChoicesPseudoAction(Action):
1368
+
1369
+ def __init__(self, name, aliases, help):
1370
+ metavar = dest = name
1371
+ if aliases:
1372
+ metavar += ' (%s)' % ', '.join(aliases)
1373
+ sup = super(_SubParsersAction._ChoicesPseudoAction, self)
1374
+ sup.__init__(option_strings=[], dest=dest, help=help,
1375
+ metavar=metavar)
1376
+
1377
+ def __init__(self,
1378
+ option_strings,
1379
+ prog,
1380
+ parser_class,
1381
+ dest=SUPPRESS,
1382
+ help=None,
1383
+ metavar=None):
1384
+
1385
+ self._prog_prefix = prog
1386
+ self._parser_class = parser_class
1387
+ self._name_parser_map = {}
1388
+ self._choices_actions = []
1389
+
1390
+ super(_SubParsersAction, self).__init__(
1391
+ option_strings=option_strings,
1392
+ dest=dest,
1393
+ nargs=PARSER,
1394
+ choices=self._name_parser_map,
1395
+ help=help,
1396
+ metavar=metavar)
1397
+
1398
+ def add_parser(self, name, **kwargs):
1399
+ # set prog from the existing prefix
1400
+ if kwargs.get('prog') is None:
1401
+ kwargs['prog'] = '%s %s' % (self._prog_prefix, name)
1402
+
1403
+ aliases = kwargs.pop('aliases', ())
1404
+
1405
+ # create a pseudo-action to hold the choice help
1406
+ if 'help' in kwargs:
1407
+ help = kwargs.pop('help')
1408
+ choice_action = self._ChoicesPseudoAction(name, aliases, help)
1409
+ self._choices_actions.append(choice_action)
1410
+
1411
+ # create the parser and add it to the map
1412
+ parser = self._parser_class(**kwargs)
1413
+ self._name_parser_map[name] = parser
1414
+
1415
+ # make parser available under aliases also
1416
+ for alias in aliases:
1417
+ self._name_parser_map[alias] = parser
1418
+
1419
+ return parser
1420
+
1421
+ def _get_subactions(self):
1422
+ return self._choices_actions
1423
+
1424
+ def __call__(self, parser, namespace, values, option_string=None):
1425
+ parser_name = values[0]
1426
+ arg_strings = values[1:]
1427
+
1428
+ # set the parser name if requested
1429
+ if self.dest is not SUPPRESS:
1430
+ setattr(namespace, self.dest, parser_name)
1431
+
1432
+ # select the parser
1433
+ try:
1434
+ parser = self._name_parser_map[parser_name]
1435
+ except KeyError:
1436
+ tup = parser_name, ', '.join(self._name_parser_map)
1437
+ msg = _('unknown parser %r (choices: %s)' % tup)
1438
+ raise ArgumentError(self, msg)
1439
+
1440
+ # parse all the remaining options into the namespace
1441
+ # store any unrecognized options on the object, so that the top
1442
+ # level parser can decide what to do with them
1443
+ namespace, arg_strings = parser.parse_known_args(arg_strings, namespace)
1444
+ if arg_strings:
1445
+ vars(namespace).setdefault(_UNRECOGNIZED_ARGS_ATTR, [])
1446
+ getattr(namespace, _UNRECOGNIZED_ARGS_ATTR).extend(arg_strings)
1447
+
1448
+
1449
+# ==============
1450
+# Type classes
1451
+# ==============
1452
+
1453
+class FileType(object):
1454
+ """Factory for creating file object types
1455
+
1456
+ Instances of FileType are typically passed as type= arguments to the
1457
+ ArgumentParser add_argument() method.
1458
+
1459
+ Keyword Arguments:
1460
+ - mode -- A string indicating how the file is to be opened. Accepts the
1461
+ same values as the builtin open() function.
1462
+ - bufsize -- The file's desired buffer size. Accepts the same values as
1463
+ the builtin open() function.
1464
+ """
1465
+
1466
+ def __init__(self, mode='r', bufsize=None):
1467
+ self._mode = mode
1468
+ self._bufsize = bufsize
1469
+
1470
+ def __call__(self, string):
1471
+ # the special argument "-" means sys.std{in,out}
1472
+ if string == '-':
1473
+ if 'r' in self._mode:
1474
+ return _sys.stdin
1475
+ elif 'w' in self._mode:
1476
+ return _sys.stdout
1477
+ else:
1478
+ msg = _('argument "-" with mode %r' % self._mode)
1479
+ raise ValueError(msg)
1480
+
1481
+ try:
1482
+ # all other arguments are used as file names
1483
+ if self._bufsize:
1484
+ return open(string, self._mode, self._bufsize)
1485
+ else:
1486
+ return open(string, self._mode)
1487
+ except IOError:
1488
+ err = _sys.exc_info()[1]
1489
+ message = _("can't open '%s': %s")
1490
+ raise ArgumentTypeError(message % (string, err))
1491
+
1492
+ def __repr__(self):
1493
+ args = [self._mode, self._bufsize]
1494
+ args_str = ', '.join([repr(arg) for arg in args if arg is not None])
1495
+ return '%s(%s)' % (type(self).__name__, args_str)
1496
+
1497
+# ===========================
1498
+# Optional and Positional Parsing
1499
+# ===========================
1500
+
1501
+class Namespace(_AttributeHolder):
1502
+ """Simple object for storing attributes.
1503
+
1504
+ Implements equality by attribute names and values, and provides a simple
1505
+ string representation.
1506
+ """
1507
+
1508
+ def __init__(self, **kwargs):
1509
+ for name in kwargs:
1510
+ setattr(self, name, kwargs[name])
1511
+
1512
+ __hash__ = None
1513
+
1514
+ def __eq__(self, other):
1515
+ return vars(self) == vars(other)
1516
+
1517
+ def __ne__(self, other):
1518
+ return not (self == other)
1519
+
1520
+ def __contains__(self, key):
1521
+ return key in self.__dict__
1522
+
1523
+
1524
+class _ActionsContainer(object):
1525
+
1526
+ def __init__(self,
1527
+ description,
1528
+ prefix_chars,
1529
+ argument_default,
1530
+ conflict_handler):
1531
+ super(_ActionsContainer, self).__init__()
1532
+
1533
+ self.description = description
1534
+ self.argument_default = argument_default
1535
+ self.prefix_chars = prefix_chars
1536
+ self.conflict_handler = conflict_handler
1537
+
1538
+ # set up registries
1539
+ self._registries = {}
1540
+
1541
+ # register actions
1542
+ self.register('action', None, _StoreAction)
1543
+ self.register('action', 'store', _StoreAction)
1544
+ self.register('action', 'store_const', _StoreConstAction)
1545
+ self.register('action', 'store_true', _StoreTrueAction)
1546
+ self.register('action', 'store_false', _StoreFalseAction)
1547
+ self.register('action', 'append', _AppendAction)
1548
+ self.register('action', 'append_const', _AppendConstAction)
1549
+ self.register('action', 'count', _CountAction)
1550
+ self.register('action', 'help', _HelpAction)
1551
+ self.register('action', 'version', _VersionAction)
1552
+ self.register('action', 'parsers', _SubParsersAction)
1553
+
1554
+ # raise an exception if the conflict handler is invalid
1555
+ self._get_handler()
1556
+
1557
+ # action storage
1558
+ self._actions = []
1559
+ self._option_string_actions = {}
1560
+
1561
+ # groups
1562
+ self._action_groups = []
1563
+ self._mutually_exclusive_groups = []
1564
+
1565
+ # defaults storage
1566
+ self._defaults = {}
1567
+
1568
+ # determines whether an "option" looks like a negative number
1569
+ self._negative_number_matcher = _re.compile(r'^-\d+$|^-\d*\.\d+$')
1570
+
1571
+ # whether or not there are any optionals that look like negative
1572
+ # numbers -- uses a list so it can be shared and edited
1573
+ self._has_negative_number_optionals = []
1574
+
1575
+ # ====================
1576
+ # Registration methods
1577
+ # ====================
1578
+ def register(self, registry_name, value, object):
1579
+ registry = self._registries.setdefault(registry_name, {})
1580
+ registry[value] = object
1581
+
1582
+ def _registry_get(self, registry_name, value, default=None):
1583
+ return self._registries[registry_name].get(value, default)
1584
+
1585
+ # ==================================
1586
+ # Namespace default accessor methods
1587
+ # ==================================
1588
+ def set_defaults(self, **kwargs):
1589
+ self._defaults.update(kwargs)
1590
+
1591
+ # if these defaults match any existing arguments, replace
1592
+ # the previous default on the object with the new one
1593
+ for action in self._actions:
1594
+ if action.dest in kwargs:
1595
+ action.default = kwargs[action.dest]
1596
+
1597
+ def get_default(self, dest):
1598
+ for action in self._actions:
1599
+ if action.dest == dest and action.default is not None:
1600
+ return action.default
1601
+ return self._defaults.get(dest, None)
1602
+
1603
+
1604
+ # =======================
1605
+ # Adding argument actions
1606
+ # =======================
1607
+ def add_argument(self, *args, **kwargs):
1608
+ """
1609
+ add_argument(dest, ..., name=value, ...)
1610
+ add_argument(option_string, option_string, ..., name=value, ...)
1611
+ """
1612
+
1613
+ # if no positional args are supplied or only one is supplied and
1614
+ # it doesn't look like an option string, parse a positional
1615
+ # argument
1616
+ chars = self.prefix_chars
1617
+ if not args or len(args) == 1 and args[0][0] not in chars:
1618
+ if args and 'dest' in kwargs:
1619
+ raise ValueError('dest supplied twice for positional argument')
1620
+ kwargs = self._get_positional_kwargs(*args, **kwargs)
1621
+
1622
+ # otherwise, we're adding an optional argument
1623
+ else:
1624
+ kwargs = self._get_optional_kwargs(*args, **kwargs)
1625
+
1626
+ # if no default was supplied, use the parser-level default
1627
+ if 'default' not in kwargs:
1628
+ dest = kwargs['dest']
1629
+ if dest in self._defaults:
1630
+ kwargs['default'] = self._defaults[dest]
1631
+ elif self.argument_default is not None:
1632
+ kwargs['default'] = self.argument_default
1633
+
1634
+ # create the action object, and add it to the parser
1635
+ action_class = self._pop_action_class(kwargs)
1636
+ if not _callable(action_class):
1637
+ raise ValueError('unknown action "%s"' % action_class)
1638
+ action = action_class(**kwargs)
1639
+
1640
+ # raise an error if the action type is not callable
1641
+ type_func = self._registry_get('type', action.type, action.type)
1642
+ if not _callable(type_func):
1643
+ raise ValueError('%r is not callable' % type_func)
1644
+
1645
+ return self._add_action(action)
1646
+
1647
+ def add_argument_group(self, *args, **kwargs):
1648
+ group = _ArgumentGroup(self, *args, **kwargs)
1649
+ self._action_groups.append(group)
1650
+ return group
1651
+
1652
+ def add_mutually_exclusive_group(self, **kwargs):
1653
+ group = _MutuallyExclusiveGroup(self, **kwargs)
1654
+ self._mutually_exclusive_groups.append(group)
1655
+ return group
1656
+
1657
+ def _add_action(self, action):
1658
+ # resolve any conflicts
1659
+ self._check_conflict(action)
1660
+
1661
+ # add to actions list
1662
+ self._actions.append(action)
1663
+ action.container = self
1664
+
1665
+ # index the action by any option strings it has
1666
+ for option_string in action.option_strings:
1667
+ self._option_string_actions[option_string] = action
1668
+
1669
+ # set the flag if any option strings look like negative numbers
1670
+ for option_string in action.option_strings:
1671
+ if self._negative_number_matcher.match(option_string):
1672
+ if not self._has_negative_number_optionals:
1673
+ self._has_negative_number_optionals.append(True)
1674
+
1675
+ # return the created action
1676
+ return action
1677
+
1678
+ def _remove_action(self, action):
1679
+ self._actions.remove(action)
1680
+
1681
+ def _add_container_actions(self, container):
1682
+ # collect groups by titles
1683
+ title_group_map = {}
1684
+ for group in self._action_groups:
1685
+ if group.title in title_group_map:
1686
+ msg = _('cannot merge actions - two groups are named %r')
1687
+ raise ValueError(msg % (group.title))
1688
+ title_group_map[group.title] = group
1689
+
1690
+ # map each action to its group
1691
+ group_map = {}
1692
+ for group in container._action_groups:
1693
+
1694
+ # if a group with the title exists, use that, otherwise
1695
+ # create a new group matching the container's group
1696
+ if group.title not in title_group_map:
1697
+ title_group_map[group.title] = self.add_argument_group(
1698
+ title=group.title,
1699
+ description=group.description,
1700
+ conflict_handler=group.conflict_handler)
1701
+
1702
+ # map the actions to their new group
1703
+ for action in group._group_actions:
1704
+ group_map[action] = title_group_map[group.title]
1705
+
1706
+ # add container's mutually exclusive groups
1707
+ # NOTE: if add_mutually_exclusive_group ever gains title= and
1708
+ # description= then this code will need to be expanded as above
1709
+ for group in container._mutually_exclusive_groups:
1710
+ mutex_group = self.add_mutually_exclusive_group(
1711
+ required=group.required)
1712
+
1713
+ # map the actions to their new mutex group
1714
+ for action in group._group_actions:
1715
+ group_map[action] = mutex_group
1716
+
1717
+ # add all actions to this container or their group
1718
+ for action in container._actions:
1719
+ group_map.get(action, self)._add_action(action)
1720
+
1721
+ def _get_positional_kwargs(self, dest, **kwargs):
1722
+ # make sure required is not specified
1723
+ if 'required' in kwargs:
1724
+ msg = _("'required' is an invalid argument for positionals")
1725
+ raise TypeError(msg)
1726
+
1727
+ # mark positional arguments as required if at least one is
1728
+ # always required
1729
+ if kwargs.get('nargs') not in [OPTIONAL, ZERO_OR_MORE]:
1730
+ kwargs['required'] = True
1731
+ if kwargs.get('nargs') == ZERO_OR_MORE and 'default' not in kwargs:
1732
+ kwargs['required'] = True
1733
+
1734
+ # return the keyword arguments with no option strings
1735
+ return dict(kwargs, dest=dest, option_strings=[])
1736
+
1737
+ def _get_optional_kwargs(self, *args, **kwargs):
1738
+ # determine short and long option strings
1739
+ option_strings = []
1740
+ long_option_strings = []
1741
+ for option_string in args:
1742
+ # error on strings that don't start with an appropriate prefix
1743
+ if not option_string[0] in self.prefix_chars:
1744
+ msg = _('invalid option string %r: '
1745
+ 'must start with a character %r')
1746
+ tup = option_string, self.prefix_chars
1747
+ raise ValueError(msg % tup)
1748
+
1749
+ # strings starting with two prefix characters are long options
1750
+ option_strings.append(option_string)
1751
+ if option_string[0] in self.prefix_chars:
1752
+ if len(option_string) > 1:
1753
+ if option_string[1] in self.prefix_chars:
1754
+ long_option_strings.append(option_string)
1755
+
1756
+ # infer destination, '--foo-bar' -> 'foo_bar' and '-x' -> 'x'
1757
+ dest = kwargs.pop('dest', None)
1758
+ if dest is None:
1759
+ if long_option_strings:
1760
+ dest_option_string = long_option_strings[0]
1761
+ else:
1762
+ dest_option_string = option_strings[0]
1763
+ dest = dest_option_string.lstrip(self.prefix_chars)
1764
+ if not dest:
1765
+ msg = _('dest= is required for options like %r')
1766
+ raise ValueError(msg % option_string)
1767
+ dest = dest.replace('-', '_')
1768
+
1769
+ # return the updated keyword arguments
1770
+ return dict(kwargs, dest=dest, option_strings=option_strings)
1771
+
1772
+ def _pop_action_class(self, kwargs, default=None):
1773
+ action = kwargs.pop('action', default)
1774
+ return self._registry_get('action', action, action)
1775
+
1776
+ def _get_handler(self):
1777
+ # determine function from conflict handler string
1778
+ handler_func_name = '_handle_conflict_%s' % self.conflict_handler
1779
+ try:
1780
+ return getattr(self, handler_func_name)
1781
+ except AttributeError:
1782
+ msg = _('invalid conflict_resolution value: %r')
1783
+ raise ValueError(msg % self.conflict_handler)
1784
+
1785
+ def _check_conflict(self, action):
1786
+
1787
+ # find all options that conflict with this option
1788
+ confl_optionals = []
1789
+ for option_string in action.option_strings:
1790
+ if option_string in self._option_string_actions:
1791
+ confl_optional = self._option_string_actions[option_string]
1792
+ confl_optionals.append((option_string, confl_optional))
1793
+
1794
+ # resolve any conflicts
1795
+ if confl_optionals:
1796
+ conflict_handler = self._get_handler()
1797
+ conflict_handler(action, confl_optionals)
1798
+
1799
+ def _handle_conflict_error(self, action, conflicting_actions):
1800
+ message = _('conflicting option string(s): %s')
1801
+ conflict_string = ', '.join([option_string
1802
+ for option_string, action
1803
+ in conflicting_actions])
1804
+ raise ArgumentError(action, message % conflict_string)
1805
+
1806
+ def _handle_conflict_resolve(self, action, conflicting_actions):
1807
+
1808
+ # remove all conflicting options
1809
+ for option_string, action in conflicting_actions:
1810
+
1811
+ # remove the conflicting option
1812
+ action.option_strings.remove(option_string)
1813
+ self._option_string_actions.pop(option_string, None)
1814
+
1815
+ # if the option now has no option string, remove it from the
1816
+ # container holding it
1817
+ if not action.option_strings:
1818
+ action.container._remove_action(action)
1819
+
1820
+
1821
+class _ArgumentGroup(_ActionsContainer):
1822
+
1823
+ def __init__(self, container, title=None, description=None, **kwargs):
1824
+ # add any missing keyword arguments by checking the container
1825
+ update = kwargs.setdefault
1826
+ update('conflict_handler', container.conflict_handler)
1827
+ update('prefix_chars', container.prefix_chars)
1828
+ update('argument_default', container.argument_default)
1829
+ super_init = super(_ArgumentGroup, self).__init__
1830
+ super_init(description=description, **kwargs)
1831
+
1832
+ # group attributes
1833
+ self.title = title
1834
+ self._group_actions = []
1835
+
1836
+ # share most attributes with the container
1837
+ self._registries = container._registries
1838
+ self._actions = container._actions
1839
+ self._option_string_actions = container._option_string_actions
1840
+ self._defaults = container._defaults
1841
+ self._has_negative_number_optionals = \
1842
+ container._has_negative_number_optionals
1843
+
1844
+ def _add_action(self, action):
1845
+ action = super(_ArgumentGroup, self)._add_action(action)
1846
+ self._group_actions.append(action)
1847
+ return action
1848
+
1849
+ def _remove_action(self, action):
1850
+ super(_ArgumentGroup, self)._remove_action(action)
1851
+ self._group_actions.remove(action)
1852
+
1853
+
1854
+class _MutuallyExclusiveGroup(_ArgumentGroup):
1855
+
1856
+ def __init__(self, container, required=False):
1857
+ super(_MutuallyExclusiveGroup, self).__init__(container)
1858
+ self.required = required
1859
+ self._container = container
1860
+
1861
+ def _add_action(self, action):
1862
+ if action.required:
1863
+ msg = _('mutually exclusive arguments must be optional')
1864
+ raise ValueError(msg)
1865
+ action = self._container._add_action(action)
1866
+ self._group_actions.append(action)
1867
+ return action
1868
+
1869
+ def _remove_action(self, action):
1870
+ self._container._remove_action(action)
1871
+ self._group_actions.remove(action)
1872
+
1873
+
1874
+class ArgumentParser(_AttributeHolder, _ActionsContainer):
1875
+ """Object for parsing command line strings into Python objects.
1876
+
1877
+ Keyword Arguments:
1878
+ - prog -- The name of the program (default: sys.argv[0])
1879
+ - usage -- A usage message (default: auto-generated from arguments)
1880
+ - description -- A description of what the program does
1881
+ - epilog -- Text following the argument descriptions
1882
+ - parents -- Parsers whose arguments should be copied into this one
1883
+ - formatter_class -- HelpFormatter class for printing help messages
1884
+ - prefix_chars -- Characters that prefix optional arguments
1885
+ - fromfile_prefix_chars -- Characters that prefix files containing
1886
+ additional arguments
1887
+ - argument_default -- The default value for all arguments
1888
+ - conflict_handler -- String indicating how to handle conflicts
1889
+ - add_help -- Add a -h/-help option
1890
+ """
1891
+
1892
+ def __init__(self,
1893
+ prog=None,
1894
+ usage=None,
1895
+ description=None,
1896
+ epilog=None,
1897
+ version=None,
1898
+ parents=[],
1899
+ formatter_class=HelpFormatter,
1900
+ prefix_chars='-',
1901
+ fromfile_prefix_chars=None,
1902
+ argument_default=None,
1903
+ conflict_handler='error',
1904
+ add_help=True):
1905
+
1906
+ if version is not None:
1907
+ import warnings
1908
+ warnings.warn(
1909
+ """The "version" argument to ArgumentParser is deprecated. """
1910
+ """Please use """
1911
+ """"add_argument(..., action='version', version="N", ...)" """
1912
+ """instead""", DeprecationWarning)
1913
+
1914
+ superinit = super(ArgumentParser, self).__init__
1915
+ superinit(description=description,
1916
+ prefix_chars=prefix_chars,
1917
+ argument_default=argument_default,
1918
+ conflict_handler=conflict_handler)
1919
+
1920
+ # default setting for prog
1921
+ if prog is None:
1922
+ prog = _os.path.basename(_sys.argv[0])
1923
+
1924
+ self.prog = prog
1925
+ self.usage = usage
1926
+ self.epilog = epilog
1927
+ self.version = version
1928
+ self.formatter_class = formatter_class
1929
+ self.fromfile_prefix_chars = fromfile_prefix_chars
1930
+ self.add_help = add_help
1931
+
1932
+ add_group = self.add_argument_group
1933
+ self._positionals = add_group(_('positional arguments'))
1934
+ self._optionals = add_group(_('optional arguments'))
1935
+ self._subparsers = None
1936
+
1937
+ # register types
1938
+ def identity(string):
1939
+ return string
1940
+ self.register('type', None, identity)
1941
+
1942
+ # add help and version arguments if necessary
1943
+ # (using explicit default to override global argument_default)
1944
+ if '-' in prefix_chars:
1945
+ default_prefix = '-'
1946
+ else:
1947
+ default_prefix = prefix_chars[0]
1948
+ if self.add_help:
1949
+ self.add_argument(
1950
+ default_prefix+'h', default_prefix*2+'help',
1951
+ action='help', default=SUPPRESS,
1952
+ help=_('show this help message and exit'))
1953
+ if self.version:
1954
+ self.add_argument(
1955
+ default_prefix+'v', default_prefix*2+'version',
1956
+ action='version', default=SUPPRESS,
1957
+ version=self.version,
1958
+ help=_("show program's version number and exit"))
1959
+
1960
+ # add parent arguments and defaults
1961
+ for parent in parents:
1962
+ self._add_container_actions(parent)
1963
+ try:
1964
+ defaults = parent._defaults
1965
+ except AttributeError:
1966
+ pass
1967
+ else:
1968
+ self._defaults.update(defaults)
1969
+
1970
+ # =======================
1971
+ # Pretty __repr__ methods
1972
+ # =======================
1973
+ def _get_kwargs(self):
1974
+ names = [
1975
+ 'prog',
1976
+ 'usage',
1977
+ 'description',
1978
+ 'version',
1979
+ 'formatter_class',
1980
+ 'conflict_handler',
1981
+ 'add_help',
1982
+ ]
1983
+ return [(name, getattr(self, name)) for name in names]
1984
+
1985
+ # ==================================
1986
+ # Optional/Positional adding methods
1987
+ # ==================================
1988
+ def add_subparsers(self, **kwargs):
1989
+ if self._subparsers is not None:
1990
+ self.error(_('cannot have multiple subparser arguments'))
1991
+
1992
+ # add the parser class to the arguments if it's not present
1993
+ kwargs.setdefault('parser_class', type(self))
1994
+
1995
+ if 'title' in kwargs or 'description' in kwargs:
1996
+ title = _(kwargs.pop('title', 'subcommands'))
1997
+ description = _(kwargs.pop('description', None))
1998
+ self._subparsers = self.add_argument_group(title, description)
1999
+ else:
2000
+ self._subparsers = self._positionals
2001
+
2002
+ # prog defaults to the usage message of this parser, skipping
2003
+ # optional arguments and with no "usage:" prefix
2004
+ if kwargs.get('prog') is None:
2005
+ formatter = self._get_formatter()
2006
+ positionals = self._get_positional_actions()
2007
+ groups = self._mutually_exclusive_groups
2008
+ formatter.add_usage(self.usage, positionals, groups, '')
2009
+ kwargs['prog'] = formatter.format_help().strip()
2010
+
2011
+ # create the parsers action and add it to the positionals list
2012
+ parsers_class = self._pop_action_class(kwargs, 'parsers')
2013
+ action = parsers_class(option_strings=[], **kwargs)
2014
+ self._subparsers._add_action(action)
2015
+
2016
+ # return the created parsers action
2017
+ return action
2018
+
2019
+ def _add_action(self, action):
2020
+ if action.option_strings:
2021
+ self._optionals._add_action(action)
2022
+ else:
2023
+ self._positionals._add_action(action)
2024
+ return action
2025
+
2026
+ def _get_optional_actions(self):
2027
+ return [action
2028
+ for action in self._actions
2029
+ if action.option_strings]
2030
+
2031
+ def _get_positional_actions(self):
2032
+ return [action
2033
+ for action in self._actions
2034
+ if not action.option_strings]
2035
+
2036
+ # =====================================
2037
+ # Command line argument parsing methods
2038
+ # =====================================
2039
+ def parse_args(self, args=None, namespace=None):
2040
+ args, argv = self.parse_known_args(args, namespace)
2041
+ if argv:
2042
+ msg = _('unrecognized arguments: %s')
2043
+ self.error(msg % ' '.join(argv))
2044
+ return args
2045
+
2046
+ def parse_known_args(self, args=None, namespace=None):
2047
+ # args default to the system args
2048
+ if args is None:
2049
+ args = _sys.argv[1:]
2050
+
2051
+ # default Namespace built from parser defaults
2052
+ if namespace is None:
2053
+ namespace = Namespace()
2054
+
2055
+ # add any action defaults that aren't present
2056
+ for action in self._actions:
2057
+ if action.dest is not SUPPRESS:
2058
+ if not hasattr(namespace, action.dest):
2059
+ if action.default is not SUPPRESS:
2060
+ setattr(namespace, action.dest, action.default)
2061
+
2062
+ # add any parser defaults that aren't present
2063
+ for dest in self._defaults:
2064
+ if not hasattr(namespace, dest):
2065
+ setattr(namespace, dest, self._defaults[dest])
2066
+
2067
+ # parse the arguments and exit if there are any errors
2068
+ try:
2069
+ namespace, args = self._parse_known_args(args, namespace)
2070
+ if hasattr(namespace, _UNRECOGNIZED_ARGS_ATTR):
2071
+ args.extend(getattr(namespace, _UNRECOGNIZED_ARGS_ATTR))
2072
+ delattr(namespace, _UNRECOGNIZED_ARGS_ATTR)
2073
+ return namespace, args
2074
+ except ArgumentError:
2075
+ err = _sys.exc_info()[1]
2076
+ self.error(str(err))
2077
+
2078
+ def _parse_known_args(self, arg_strings, namespace):
2079
+ # replace arg strings that are file references
2080
+ if self.fromfile_prefix_chars is not None:
2081
+ arg_strings = self._read_args_from_files(arg_strings)
2082
+
2083
+ # map all mutually exclusive arguments to the other arguments
2084
+ # they can't occur with
2085
+ action_conflicts = {}
2086
+ for mutex_group in self._mutually_exclusive_groups:
2087
+ group_actions = mutex_group._group_actions
2088
+ for i, mutex_action in enumerate(mutex_group._group_actions):
2089
+ conflicts = action_conflicts.setdefault(mutex_action, [])
2090
+ conflicts.extend(group_actions[:i])
2091
+ conflicts.extend(group_actions[i + 1:])
2092
+
2093
+ # find all option indices, and determine the arg_string_pattern
2094
+ # which has an 'O' if there is an option at an index,
2095
+ # an 'A' if there is an argument, or a '-' if there is a '--'
2096
+ option_string_indices = {}
2097
+ arg_string_pattern_parts = []
2098
+ arg_strings_iter = iter(arg_strings)
2099
+ for i, arg_string in enumerate(arg_strings_iter):
2100
+
2101
+ # all args after -- are non-options
2102
+ if arg_string == '--':
2103
+ arg_string_pattern_parts.append('-')
2104
+ for arg_string in arg_strings_iter:
2105
+ arg_string_pattern_parts.append('A')
2106
+
2107
+ # otherwise, add the arg to the arg strings
2108
+ # and note the index if it was an option
2109
+ else:
2110
+ option_tuple = self._parse_optional(arg_string)
2111
+ if option_tuple is None:
2112
+ pattern = 'A'
2113
+ else:
2114
+ option_string_indices[i] = option_tuple
2115
+ pattern = 'O'
2116
+ arg_string_pattern_parts.append(pattern)
2117
+
2118
+ # join the pieces together to form the pattern
2119
+ arg_strings_pattern = ''.join(arg_string_pattern_parts)
2120
+
2121
+ # converts arg strings to the appropriate and then takes the action
2122
+ seen_actions = set()
2123
+ seen_non_default_actions = set()
2124
+
2125
+ def take_action(action, argument_strings, option_string=None):
2126
+ seen_actions.add(action)
2127
+ argument_values = self._get_values(action, argument_strings)
2128
+
2129
+ # error if this argument is not allowed with other previously
2130
+ # seen arguments, assuming that actions that use the default
2131
+ # value don't really count as "present"
2132
+ if argument_values is not action.default:
2133
+ seen_non_default_actions.add(action)
2134
+ for conflict_action in action_conflicts.get(action, []):
2135
+ if conflict_action in seen_non_default_actions:
2136
+ msg = _('not allowed with argument %s')
2137
+ action_name = _get_action_name(conflict_action)
2138
+ raise ArgumentError(action, msg % action_name)
2139
+
2140
+ # take the action if we didn't receive a SUPPRESS value
2141
+ # (e.g. from a default)
2142
+ if argument_values is not SUPPRESS:
2143
+ action(self, namespace, argument_values, option_string)
2144
+
2145
+ # function to convert arg_strings into an optional action
2146
+ def consume_optional(start_index):
2147
+
2148
+ # get the optional identified at this index
2149
+ option_tuple = option_string_indices[start_index]
2150
+ action, option_string, explicit_arg = option_tuple
2151
+
2152
+ # identify additional optionals in the same arg string
2153
+ # (e.g. -xyz is the same as -x -y -z if no args are required)
2154
+ match_argument = self._match_argument
2155
+ action_tuples = []
2156
+ while True:
2157
+
2158
+ # if we found no optional action, skip it
2159
+ if action is None:
2160
+ extras.append(arg_strings[start_index])
2161
+ return start_index + 1
2162
+
2163
+ # if there is an explicit argument, try to match the
2164
+ # optional's string arguments to only this
2165
+ if explicit_arg is not None:
2166
+ arg_count = match_argument(action, 'A')
2167
+
2168
+ # if the action is a single-dash option and takes no
2169
+ # arguments, try to parse more single-dash options out
2170
+ # of the tail of the option string
2171
+ chars = self.prefix_chars
2172
+ if arg_count == 0 and option_string[1] not in chars:
2173
+ action_tuples.append((action, [], option_string))
2174
+ char = option_string[0]
2175
+ option_string = char + explicit_arg[0]
2176
+ new_explicit_arg = explicit_arg[1:] or None
2177
+ optionals_map = self._option_string_actions
2178
+ if option_string in optionals_map:
2179
+ action = optionals_map[option_string]
2180
+ explicit_arg = new_explicit_arg
2181
+ else:
2182
+ msg = _('ignored explicit argument %r')
2183
+ raise ArgumentError(action, msg % explicit_arg)
2184
+
2185
+ # if the action expect exactly one argument, we've
2186
+ # successfully matched the option; exit the loop
2187
+ elif arg_count == 1:
2188
+ stop = start_index + 1
2189
+ args = [explicit_arg]
2190
+ action_tuples.append((action, args, option_string))
2191
+ break
2192
+
2193
+ # error if a double-dash option did not use the
2194
+ # explicit argument
2195
+ else:
2196
+ msg = _('ignored explicit argument %r')
2197
+ raise ArgumentError(action, msg % explicit_arg)
2198
+
2199
+ # if there is no explicit argument, try to match the
2200
+ # optional's string arguments with the following strings
2201
+ # if successful, exit the loop
2202
+ else:
2203
+ start = start_index + 1
2204
+ selected_patterns = arg_strings_pattern[start:]
2205
+ arg_count = match_argument(action, selected_patterns)
2206
+ stop = start + arg_count
2207
+ args = arg_strings[start:stop]
2208
+ action_tuples.append((action, args, option_string))
2209
+ break
2210
+
2211
+ # add the Optional to the list and return the index at which
2212
+ # the Optional's string args stopped
2213
+ assert action_tuples
2214
+ for action, args, option_string in action_tuples:
2215
+ take_action(action, args, option_string)
2216
+ return stop
2217
+
2218
+ # the list of Positionals left to be parsed; this is modified
2219
+ # by consume_positionals()
2220
+ positionals = self._get_positional_actions()
2221
+
2222
+ # function to convert arg_strings into positional actions
2223
+ def consume_positionals(start_index):
2224
+ # match as many Positionals as possible
2225
+ match_partial = self._match_arguments_partial
2226
+ selected_pattern = arg_strings_pattern[start_index:]
2227
+ arg_counts = match_partial(positionals, selected_pattern)
2228
+
2229
+ # slice off the appropriate arg strings for each Positional
2230
+ # and add the Positional and its args to the list
2231
+ for action, arg_count in zip(positionals, arg_counts):
2232
+ args = arg_strings[start_index: start_index + arg_count]
2233
+ start_index += arg_count
2234
+ take_action(action, args)
2235
+
2236
+ # slice off the Positionals that we just parsed and return the
2237
+ # index at which the Positionals' string args stopped
2238
+ positionals[:] = positionals[len(arg_counts):]
2239
+ return start_index
2240
+
2241
+ # consume Positionals and Optionals alternately, until we have
2242
+ # passed the last option string
2243
+ extras = []
2244
+ start_index = 0
2245
+ if option_string_indices:
2246
+ max_option_string_index = max(option_string_indices)
2247
+ else:
2248
+ max_option_string_index = -1
2249
+ while start_index <= max_option_string_index:
2250
+
2251
+ # consume any Positionals preceding the next option
2252
+ next_option_string_index = min([
2253
+ index
2254
+ for index in option_string_indices
2255
+ if index >= start_index])
2256
+ if start_index != next_option_string_index:
2257
+ positionals_end_index = consume_positionals(start_index)
2258
+
2259
+ # only try to parse the next optional if we didn't consume
2260
+ # the option string during the positionals parsing
2261
+ if positionals_end_index > start_index:
2262
+ start_index = positionals_end_index
2263
+ continue
2264
+ else:
2265
+ start_index = positionals_end_index
2266
+
2267
+ # if we consumed all the positionals we could and we're not
2268
+ # at the index of an option string, there were extra arguments
2269
+ if start_index not in option_string_indices:
2270
+ strings = arg_strings[start_index:next_option_string_index]
2271
+ extras.extend(strings)
2272
+ start_index = next_option_string_index
2273
+
2274
+ # consume the next optional and any arguments for it
2275
+ start_index = consume_optional(start_index)
2276
+
2277
+ # consume any positionals following the last Optional
2278
+ stop_index = consume_positionals(start_index)
2279
+
2280
+ # if we didn't consume all the argument strings, there were extras
2281
+ extras.extend(arg_strings[stop_index:])
2282
+
2283
+ # if we didn't use all the Positional objects, there were too few
2284
+ # arg strings supplied.
2285
+ if positionals:
2286
+ self.error(_('too few arguments'))
2287
+
2288
+ # make sure all required actions were present, and convert defaults.
2289
+ for action in self._actions:
2290
+ if action not in seen_actions:
2291
+ if action.required:
2292
+ name = _get_action_name(action)
2293
+ self.error(_('argument %s is required') % name)
2294
+ else:
2295
+ # Convert action default now instead of doing it before
2296
+ # parsing arguments to avoid calling convert functions
2297
+ # twice (which may fail) if the argument was given, but
2298
+ # only if it was defined already in the namespace
2299
+ if (action.default is not None and
2300
+ isinstance(action.default, basestring) and
2301
+ hasattr(namespace, action.dest) and
2302
+ action.default is getattr(namespace, action.dest)):
2303
+ setattr(namespace, action.dest,
2304
+ self._get_value(action, action.default))
2305
+
2306
+ # make sure all required groups had one option present
2307
+ for group in self._mutually_exclusive_groups:
2308
+ if group.required:
2309
+ for action in group._group_actions:
2310
+ if action in seen_non_default_actions:
2311
+ break
2312
+
2313
+ # if no actions were used, report the error
2314
+ else:
2315
+ names = [_get_action_name(action)
2316
+ for action in group._group_actions
2317
+ if action.help is not SUPPRESS]
2318
+ msg = _('one of the arguments %s is required')
2319
+ self.error(msg % ' '.join(names))
2320
+
2321
+ # return the updated namespace and the extra arguments
2322
+ return namespace, extras
2323
+
2324
+ def _read_args_from_files(self, arg_strings):
2325
+ # expand arguments referencing files
2326
+ new_arg_strings = []
2327
+ for arg_string in arg_strings:
2328
+
2329
+ # for regular arguments, just add them back into the list
2330
+ if arg_string[0] not in self.fromfile_prefix_chars:
2331
+ new_arg_strings.append(arg_string)
2332
+
2333
+ # replace arguments referencing files with the file content
2334
+ else:
2335
+ try:
2336
+ args_file = open(arg_string[1:])
2337
+ try:
2338
+ arg_strings = []
2339
+ for arg_line in args_file.read().splitlines():
2340
+ for arg in self.convert_arg_line_to_args(arg_line):
2341
+ arg_strings.append(arg)
2342
+ arg_strings = self._read_args_from_files(arg_strings)
2343
+ new_arg_strings.extend(arg_strings)
2344
+ finally:
2345
+ args_file.close()
2346
+ except IOError:
2347
+ err = _sys.exc_info()[1]
2348
+ self.error(str(err))
2349
+
2350
+ # return the modified argument list
2351
+ return new_arg_strings
2352
+
2353
+ def convert_arg_line_to_args(self, arg_line):
2354
+ return [arg_line]
2355
+
2356
+ def _match_argument(self, action, arg_strings_pattern):
2357
+ # match the pattern for this action to the arg strings
2358
+ nargs_pattern = self._get_nargs_pattern(action)
2359
+ match = _re.match(nargs_pattern, arg_strings_pattern)
2360
+
2361
+ # raise an exception if we weren't able to find a match
2362
+ if match is None:
2363
+ nargs_errors = {
2364
+ None: _('expected one argument'),
2365
+ OPTIONAL: _('expected at most one argument'),
2366
+ ONE_OR_MORE: _('expected at least one argument'),
2367
+ }
2368
+ default = _('expected %s argument(s)') % action.nargs
2369
+ msg = nargs_errors.get(action.nargs, default)
2370
+ raise ArgumentError(action, msg)
2371
+
2372
+ # return the number of arguments matched
2373
+ return len(match.group(1))
2374
+
2375
+ def _match_arguments_partial(self, actions, arg_strings_pattern):
2376
+ # progressively shorten the actions list by slicing off the
2377
+ # final actions until we find a match
2378
+ result = []
2379
+ for i in range(len(actions), 0, -1):
2380
+ actions_slice = actions[:i]
2381
+ pattern = ''.join([self._get_nargs_pattern(action)
2382
+ for action in actions_slice])
2383
+ match = _re.match(pattern, arg_strings_pattern)
2384
+ if match is not None:
2385
+ result.extend([len(string) for string in match.groups()])
2386
+ break
2387
+
2388
+ # return the list of arg string counts
2389
+ return result
2390
+
2391
+ def _parse_optional(self, arg_string):
2392
+ # if it's an empty string, it was meant to be a positional
2393
+ if not arg_string:
2394
+ return None
2395
+
2396
+ # if it doesn't start with a prefix, it was meant to be positional
2397
+ if not arg_string[0] in self.prefix_chars:
2398
+ return None
2399
+
2400
+ # if the option string is present in the parser, return the action
2401
+ if arg_string in self._option_string_actions:
2402
+ action = self._option_string_actions[arg_string]
2403
+ return action, arg_string, None
2404
+
2405
+ # if it's just a single character, it was meant to be positional
2406
+ if len(arg_string) == 1:
2407
+ return None
2408
+
2409
+ # if the option string before the "=" is present, return the action
2410
+ if '=' in arg_string:
2411
+ option_string, explicit_arg = arg_string.split('=', 1)
2412
+ if option_string in self._option_string_actions:
2413
+ action = self._option_string_actions[option_string]
2414
+ return action, option_string, explicit_arg
2415
+
2416
+ # search through all possible prefixes of the option string
2417
+ # and all actions in the parser for possible interpretations
2418
+ option_tuples = self._get_option_tuples(arg_string)
2419
+
2420
+ # if multiple actions match, the option string was ambiguous
2421
+ if len(option_tuples) > 1:
2422
+ options = ', '.join([option_string
2423
+ for action, option_string, explicit_arg in option_tuples])
2424
+ tup = arg_string, options
2425
+ self.error(_('ambiguous option: %s could match %s') % tup)
2426
+
2427
+ # if exactly one action matched, this segmentation is good,
2428
+ # so return the parsed action
2429
+ elif len(option_tuples) == 1:
2430
+ option_tuple, = option_tuples
2431
+ return option_tuple
2432
+
2433
+ # if it was not found as an option, but it looks like a negative
2434
+ # number, it was meant to be positional
2435
+ # unless there are negative-number-like options
2436
+ if self._negative_number_matcher.match(arg_string):
2437
+ if not self._has_negative_number_optionals:
2438
+ return None
2439
+
2440
+ # if it contains a space, it was meant to be a positional
2441
+ if ' ' in arg_string:
2442
+ return None
2443
+
2444
+ # it was meant to be an optional but there is no such option
2445
+ # in this parser (though it might be a valid option in a subparser)
2446
+ return None, arg_string, None
2447
+
2448
+ def _get_option_tuples(self, option_string):
2449
+ result = []
2450
+
2451
+ # option strings starting with two prefix characters are only
2452
+ # split at the '='
2453
+ chars = self.prefix_chars
2454
+ if option_string[0] in chars and option_string[1] in chars:
2455
+ if '=' in option_string:
2456
+ option_prefix, explicit_arg = option_string.split('=', 1)
2457
+ else:
2458
+ option_prefix = option_string
2459
+ explicit_arg = None
2460
+ for option_string in self._option_string_actions:
2461
+ if option_string.startswith(option_prefix):
2462
+ action = self._option_string_actions[option_string]
2463
+ tup = action, option_string, explicit_arg
2464
+ result.append(tup)
2465
+
2466
+ # single character options can be concatenated with their arguments
2467
+ # but multiple character options always have to have their argument
2468
+ # separate
2469
+ elif option_string[0] in chars and option_string[1] not in chars:
2470
+ option_prefix = option_string
2471
+ explicit_arg = None
2472
+ short_option_prefix = option_string[:2]
2473
+ short_explicit_arg = option_string[2:]
2474
+
2475
+ for option_string in self._option_string_actions:
2476
+ if option_string == short_option_prefix:
2477
+ action = self._option_string_actions[option_string]
2478
+ tup = action, option_string, short_explicit_arg
2479
+ result.append(tup)
2480
+ elif option_string.startswith(option_prefix):
2481
+ action = self._option_string_actions[option_string]
2482
+ tup = action, option_string, explicit_arg
2483
+ result.append(tup)
2484
+
2485
+ # shouldn't ever get here
2486
+ else:
2487
+ self.error(_('unexpected option string: %s') % option_string)
2488
+
2489
+ # return the collected option tuples
2490
+ return result
2491
+
2492
+ def _get_nargs_pattern(self, action):
2493
+ # in all examples below, we have to allow for '--' args
2494
+ # which are represented as '-' in the pattern
2495
+ nargs = action.nargs
2496
+
2497
+ # the default (None) is assumed to be a single argument
2498
+ if nargs is None:
2499
+ nargs_pattern = '(-*A-*)'
2500
+
2501
+ # allow zero or one arguments
2502
+ elif nargs == OPTIONAL:
2503
+ nargs_pattern = '(-*A?-*)'
2504
+
2505
+ # allow zero or more arguments
2506
+ elif nargs == ZERO_OR_MORE:
2507
+ nargs_pattern = '(-*[A-]*)'
2508
+
2509
+ # allow one or more arguments
2510
+ elif nargs == ONE_OR_MORE:
2511
+ nargs_pattern = '(-*A[A-]*)'
2512
+
2513
+ # allow any number of options or arguments
2514
+ elif nargs == REMAINDER:
2515
+ nargs_pattern = '([-AO]*)'
2516
+
2517
+ # allow one argument followed by any number of options or arguments
2518
+ elif nargs == PARSER:
2519
+ nargs_pattern = '(-*A[-AO]*)'
2520
+
2521
+ # all others should be integers
2522
+ else:
2523
+ nargs_pattern = '(-*%s-*)' % '-*'.join('A' * nargs)
2524
+
2525
+ # if this is an optional action, -- is not allowed
2526
+ if action.option_strings:
2527
+ nargs_pattern = nargs_pattern.replace('-*', '')
2528
+ nargs_pattern = nargs_pattern.replace('-', '')
2529
+
2530
+ # return the pattern
2531
+ return nargs_pattern
2532
+
2533
+ # ========================
2534
+ # Value conversion methods
2535
+ # ========================
2536
+ def _get_values(self, action, arg_strings):
2537
+ # for everything but PARSER args, strip out '--'
2538
+ if action.nargs not in [PARSER, REMAINDER]:
2539
+ arg_strings = [s for s in arg_strings if s != '--']
2540
+
2541
+ # optional argument produces a default when not present
2542
+ if not arg_strings and action.nargs == OPTIONAL:
2543
+ if action.option_strings:
2544
+ value = action.const
2545
+ else:
2546
+ value = action.default
2547
+ if isinstance(value, basestring):
2548
+ value = self._get_value(action, value)
2549
+ self._check_value(action, value)
2550
+
2551
+ # when nargs='*' on a positional, if there were no command-line
2552
+ # args, use the default if it is anything other than None
2553
+ elif (not arg_strings and action.nargs == ZERO_OR_MORE and
2554
+ not action.option_strings):
2555
+ if action.default is not None:
2556
+ value = action.default
2557
+ else:
2558
+ value = arg_strings
2559
+ self._check_value(action, value)
2560
+
2561
+ # single argument or optional argument produces a single value
2562
+ elif len(arg_strings) == 1 and action.nargs in [None, OPTIONAL]:
2563
+ arg_string, = arg_strings
2564
+ value = self._get_value(action, arg_string)
2565
+ self._check_value(action, value)
2566
+
2567
+ # REMAINDER arguments convert all values, checking none
2568
+ elif action.nargs == REMAINDER:
2569
+ value = [self._get_value(action, v) for v in arg_strings]
2570
+
2571
+ # PARSER arguments convert all values, but check only the first
2572
+ elif action.nargs == PARSER:
2573
+ value = [self._get_value(action, v) for v in arg_strings]
2574
+ self._check_value(action, value[0])
2575
+
2576
+ # all other types of nargs produce a list
2577
+ else:
2578
+ value = [self._get_value(action, v) for v in arg_strings]
2579
+ for v in value:
2580
+ self._check_value(action, v)
2581
+
2582
+ # return the converted value
2583
+ return value
2584
+
2585
+ def _get_value(self, action, arg_string):
2586
+ type_func = self._registry_get('type', action.type, action.type)
2587
+ if not _callable(type_func):
2588
+ msg = _('%r is not callable')
2589
+ raise ArgumentError(action, msg % type_func)
2590
+
2591
+ # convert the value to the appropriate type
2592
+ try:
2593
+ result = type_func(arg_string)
2594
+
2595
+ # ArgumentTypeErrors indicate errors
2596
+ except ArgumentTypeError:
2597
+ name = getattr(action.type, '__name__', repr(action.type))
2598
+ msg = str(_sys.exc_info()[1])
2599
+ raise ArgumentError(action, msg)
2600
+
2601
+ # TypeErrors or ValueErrors also indicate errors
2602
+ except (TypeError, ValueError):
2603
+ name = getattr(action.type, '__name__', repr(action.type))
2604
+ msg = _('invalid %s value: %r')
2605
+ raise ArgumentError(action, msg % (name, arg_string))
2606
+
2607
+ # return the converted value
2608
+ return result
2609
+
2610
+ def _check_value(self, action, value):
2611
+ # converted value must be one of the choices (if specified)
2612
+ if action.choices is not None and value not in action.choices:
2613
+ tup = value, ', '.join(map(repr, action.choices))
2614
+ msg = _('invalid choice: %r (choose from %s)') % tup
2615
+ raise ArgumentError(action, msg)
2616
+
2617
+ # =======================
2618
+ # Help-formatting methods
2619
+ # =======================
2620
+ def format_usage(self):
2621
+ formatter = self._get_formatter()
2622
+ formatter.add_usage(self.usage, self._actions,
2623
+ self._mutually_exclusive_groups)
2624
+ return formatter.format_help()
2625
+
2626
+ def format_help(self):
2627
+ formatter = self._get_formatter()
2628
+
2629
+ # usage
2630
+ formatter.add_usage(self.usage, self._actions,
2631
+ self._mutually_exclusive_groups)
2632
+
2633
+ # description
2634
+ formatter.add_text(self.description)
2635
+
2636
+ # positionals, optionals and user-defined groups
2637
+ for action_group in self._action_groups:
2638
+ formatter.start_section(action_group.title)
2639
+ formatter.add_text(action_group.description)
2640
+ formatter.add_arguments(action_group._group_actions)
2641
+ formatter.end_section()
2642
+
2643
+ # epilog
2644
+ formatter.add_text(self.epilog)
2645
+
2646
+ # determine help from format above
2647
+ return formatter.format_help()
2648
+
2649
+ def format_version(self):
2650
+ import warnings
2651
+ warnings.warn(
2652
+ 'The format_version method is deprecated -- the "version" '
2653
+ 'argument to ArgumentParser is no longer supported.',
2654
+ DeprecationWarning)
2655
+ formatter = self._get_formatter()
2656
+ formatter.add_text(self.version)
2657
+ return formatter.format_help()
2658
+
2659
+ def _get_formatter(self):
2660
+ return self.formatter_class(prog=self.prog)
2661
+
2662
+ # =====================
2663
+ # Help-printing methods
2664
+ # =====================
2665
+ def print_usage(self, file=None):
2666
+ if file is None:
2667
+ file = _sys.stdout
2668
+ self._print_message(self.format_usage(), file)
2669
+
2670
+ def print_help(self, file=None):
2671
+ if file is None:
2672
+ file = _sys.stdout
2673
+ self._print_message(self.format_help(), file)
2674
+
2675
+ def print_version(self, file=None):
2676
+ import warnings
2677
+ warnings.warn(
2678
+ 'The print_version method is deprecated -- the "version" '
2679
+ 'argument to ArgumentParser is no longer supported.',
2680
+ DeprecationWarning)
2681
+ self._print_message(self.format_version(), file)
2682
+
2683
+ def _print_message(self, message, file=None):
2684
+ if message:
2685
+ if file is None:
2686
+ file = _sys.stderr
2687
+ file.write(message)
2688
+
2689
+ # ===============
2690
+ # Exiting methods
2691
+ # ===============
2692
+ def exit(self, status=0, message=None):
2693
+ if message:
2694
+ self._print_message(message, _sys.stderr)
2695
+ _sys.exit(status)
2696
+
2697
+ def error(self, message):
2698
+ """error(message: string)
2699
+
2700
+ Prints a usage message incorporating the message to stderr and
2701
+ exits.
2702
+
2703
+ If you override this in a subclass, it should not return -- it
2704
+ should either exit or raise an exception.
2705
+ """
2706
+ self.print_usage(_sys.stderr)
2707
+ self.exit(2, _('%s: error: %s\n') % (self.prog, message))
2708
--
600
--
2709
2.13.5
601
2.36.1
2710
2711
diff view generated by jsdifflib
1
Most qcow2 files are uncompressed so it is wasteful to allocate (32 + 1)
1
From: Jagannathan Raman <jag.raman@oracle.com>
2
* cluster_size + 512 bytes upfront. Allocate s->cluster_cache and
3
s->cluster_data when the first read operation is performance on a
4
compressed cluster.
5
2
6
The buffers are freed in .bdrv_close(). .bdrv_open() no longer has any
3
Adds handler to reset a remote device
7
code paths that can allocate these buffers, so remove the free functions
8
in the error code path.
9
4
10
This patch can result in significant memory savings when many qcow2
5
Signed-off-by: Elena Ufimtseva <elena.ufimtseva@oracle.com>
11
disks are attached or backing file chains are long:
6
Signed-off-by: John G Johnson <john.g.johnson@oracle.com>
12
7
Signed-off-by: Jagannathan Raman <jag.raman@oracle.com>
13
Before 12.81% (1,023,193,088B)
8
Reviewed-by: Stefan Hajnoczi <stefanha@redhat.com>
14
After 5.36% (393,893,888B)
9
Message-id: 112eeadf3bc4c6cdb100bc3f9a6fcfc20b467c1b.1655151679.git.jag.raman@oracle.com
15
16
Reported-by: Alexey Kardashevskiy <aik@ozlabs.ru>
17
Tested-by: Alexey Kardashevskiy <aik@ozlabs.ru>
18
Reviewed-by: Eric Blake <eblake@redhat.com>
19
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
20
Message-id: 20170821135530.32344-1-stefanha@redhat.com
21
Cc: Kevin Wolf <kwolf@redhat.com>
22
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
10
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
23
---
11
---
24
block/qcow2-cluster.c | 17 +++++++++++++++++
12
hw/remote/vfio-user-obj.c | 20 ++++++++++++++++++++
25
block/qcow2.c | 12 ------------
13
1 file changed, 20 insertions(+)
26
2 files changed, 17 insertions(+), 12 deletions(-)
27
14
28
diff --git a/block/qcow2-cluster.c b/block/qcow2-cluster.c
15
diff --git a/hw/remote/vfio-user-obj.c b/hw/remote/vfio-user-obj.c
29
index XXXXXXX..XXXXXXX 100644
16
index XXXXXXX..XXXXXXX 100644
30
--- a/block/qcow2-cluster.c
17
--- a/hw/remote/vfio-user-obj.c
31
+++ b/block/qcow2-cluster.c
18
+++ b/hw/remote/vfio-user-obj.c
32
@@ -XXX,XX +XXX,XX @@ int qcow2_decompress_cluster(BlockDriverState *bs, uint64_t cluster_offset)
19
@@ -XXX,XX +XXX,XX @@ void vfu_object_set_bus_irq(PCIBus *pci_bus)
33
nb_csectors = ((cluster_offset >> s->csize_shift) & s->csize_mask) + 1;
20
max_bdf);
34
sector_offset = coffset & 511;
21
}
35
csize = nb_csectors * 512 - sector_offset;
22
23
+static int vfu_object_device_reset(vfu_ctx_t *vfu_ctx, vfu_reset_type_t type)
24
+{
25
+ VfuObject *o = vfu_get_private(vfu_ctx);
36
+
26
+
37
+ /* Allocate buffers on first decompress operation, most images are
27
+ /* vfu_object_ctx_run() handles lost connection */
38
+ * uncompressed and the memory overhead can be avoided. The buffers
28
+ if (type == VFU_RESET_LOST_CONN) {
39
+ * are freed in .bdrv_close().
29
+ return 0;
40
+ */
30
+ }
41
+ if (!s->cluster_data) {
42
+ /* one more sector for decompressed data alignment */
43
+ s->cluster_data = qemu_try_blockalign(bs->file->bs,
44
+ QCOW_MAX_CRYPT_CLUSTERS * s->cluster_size + 512);
45
+ if (!s->cluster_data) {
46
+ return -ENOMEM;
47
+ }
48
+ }
49
+ if (!s->cluster_cache) {
50
+ s->cluster_cache = g_malloc(s->cluster_size);
51
+ }
52
+
31
+
53
BLKDBG_EVENT(bs->file, BLKDBG_READ_COMPRESSED);
32
+ qdev_reset_all(DEVICE(o->pci_dev));
54
ret = bdrv_read(bs->file, coffset >> 9, s->cluster_data,
33
+
55
nb_csectors);
34
+ return 0;
56
diff --git a/block/qcow2.c b/block/qcow2.c
35
+}
57
index XXXXXXX..XXXXXXX 100644
36
+
58
--- a/block/qcow2.c
37
/*
59
+++ b/block/qcow2.c
38
* TYPE_VFU_OBJECT depends on the availability of the 'socket' and 'device'
60
@@ -XXX,XX +XXX,XX @@ static int qcow2_do_open(BlockDriverState *bs, QDict *options, int flags,
39
* properties. It also depends on devices instantiated in QEMU. These
40
@@ -XXX,XX +XXX,XX @@ static void vfu_object_init_ctx(VfuObject *o, Error **errp)
61
goto fail;
41
goto fail;
62
}
42
}
63
43
64
- s->cluster_cache = g_malloc(s->cluster_size);
44
+ ret = vfu_setup_device_reset_cb(o->vfu_ctx, &vfu_object_device_reset);
65
- /* one more sector for decompressed data alignment */
45
+ if (ret < 0) {
66
- s->cluster_data = qemu_try_blockalign(bs->file->bs, QCOW_MAX_CRYPT_CLUSTERS
46
+ error_setg(errp, "vfu: Failed to setup reset callback");
67
- * s->cluster_size + 512);
47
+ goto fail;
68
- if (s->cluster_data == NULL) {
48
+ }
69
- error_setg(errp, "Could not allocate temporary cluster buffer");
49
+
70
- ret = -ENOMEM;
50
ret = vfu_realize_ctx(o->vfu_ctx);
71
- goto fail;
51
if (ret < 0) {
72
- }
52
error_setg(errp, "vfu: Failed to realize device %s- %s",
73
-
74
s->cluster_cache_offset = -1;
75
s->flags = flags;
76
77
@@ -XXX,XX +XXX,XX @@ static int qcow2_do_open(BlockDriverState *bs, QDict *options, int flags,
78
if (s->refcount_block_cache) {
79
qcow2_cache_destroy(bs, s->refcount_block_cache);
80
}
81
- g_free(s->cluster_cache);
82
- qemu_vfree(s->cluster_data);
83
qcrypto_block_free(s->crypto);
84
qapi_free_QCryptoBlockOpenOptions(s->crypto_opts);
85
return ret;
86
--
53
--
87
2.13.5
54
2.36.1
88
89
diff view generated by jsdifflib
1
Add the scripts/ directory to sys.path so Python 2.6 will be able to
1
Every laio_io_plug() call has a matching laio_io_unplug() call. There is
2
import argparse.
2
a plugged counter that tracks the number of levels of plugging and
3
allows for nesting.
3
4
4
Cc: Fam Zheng <famz@redhat.com>
5
The plugged counter must reflect the balance between laio_io_plug() and
6
laio_io_unplug() calls accurately. Otherwise I/O stalls occur since
7
io_submit(2) calls are skipped while plugged.
8
9
Reported-by: Nikolay Tenev <nt@storpool.com>
5
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
10
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
6
Acked-by: John Snow <jsnow@redhat.com>
11
Reviewed-by: Stefano Garzarella <sgarzare@redhat.com>
7
Acked-by: Fam Zheng <famz@redhat.com>
12
Message-id: 20220609164712.1539045-2-stefanha@redhat.com
8
Message-id: 20170825155732.15665-3-stefanha@redhat.com
13
Cc: Stefano Garzarella <sgarzare@redhat.com>
14
Fixes: 68d7946648 ("linux-aio: add `dev_max_batch` parameter to laio_io_unplug()")
15
[Stefano Garzarella suggested adding a Fixes tag.
16
--Stefan]
9
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
17
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
10
---
18
---
11
tests/docker/docker.py | 4 +++-
19
block/linux-aio.c | 4 +++-
12
1 file changed, 3 insertions(+), 1 deletion(-)
20
1 file changed, 3 insertions(+), 1 deletion(-)
13
21
14
diff --git a/tests/docker/docker.py b/tests/docker/docker.py
22
diff --git a/block/linux-aio.c b/block/linux-aio.c
15
index XXXXXXX..XXXXXXX 100755
23
index XXXXXXX..XXXXXXX 100644
16
--- a/tests/docker/docker.py
24
--- a/block/linux-aio.c
17
+++ b/tests/docker/docker.py
25
+++ b/block/linux-aio.c
18
@@ -XXX,XX +XXX,XX @@
26
@@ -XXX,XX +XXX,XX @@ void laio_io_unplug(BlockDriverState *bs, LinuxAioState *s,
19
27
uint64_t dev_max_batch)
20
import os
28
{
21
import sys
29
assert(s->io_q.plugged);
22
+sys.path.append(os.path.join(os.path.dirname(__file__),
30
+ s->io_q.plugged--;
23
+ '..', '..', 'scripts'))
31
+
24
+import argparse
32
if (s->io_q.in_queue >= laio_max_batch(s, dev_max_batch) ||
25
import subprocess
33
- (--s->io_q.plugged == 0 &&
26
import json
34
+ (!s->io_q.plugged &&
27
import hashlib
35
!s->io_q.blocked && !QSIMPLEQ_EMPTY(&s->io_q.pending))) {
28
import atexit
36
ioq_submit(s);
29
import uuid
37
}
30
-import argparse
31
import tempfile
32
import re
33
import signal
34
--
38
--
35
2.13.5
39
2.36.1
36
37
diff view generated by jsdifflib
1
From: Dan Aloni <dan@kernelim.com>
1
It may not be obvious why laio_io_unplug() checks max batch. I discussed
2
this with Stefano and have added a comment summarizing the reason.
2
3
3
The number of queues that should be return by the admin command should:
4
Cc: Stefano Garzarella <sgarzare@redhat.com>
4
5
Cc: Kevin Wolf <kwolf@redhat.com>
5
1) Only mention the number of non-admin queues.
6
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
6
2) It is zero-based, meaning that '0 == one non-admin queue',
7
Reviewed-by: Stefano Garzarella <sgarzare@redhat.com>
7
'1 == two non-admin queues', and so forth.
8
Message-id: 20220609164712.1539045-3-stefanha@redhat.com
8
9
Because our `num_queues` means the number of queues _plus_ the admin
10
queue, then the right calculation for the number returned from the admin
11
command is `num_queues - 2`, combining the two requirements mentioned.
12
13
The issue was discovered by reducing num_queues from 64 to 8 and running
14
a Linux VM with an SMP parameter larger than that (e.g. 22). It tries to
15
utilize all queues, and therefore fails with an invalid queue number
16
when trying to queue I/Os on the last queue.
17
18
Signed-off-by: Dan Aloni <dan@kernelim.com>
19
CC: Alex Friedman <alex@e8storage.com>
20
CC: Keith Busch <keith.busch@intel.com>
21
CC: Stefan Hajnoczi <stefanha@redhat.com>
22
Reviewed-by: Keith Busch <keith.busch@intel.com>
23
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
9
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
24
---
10
---
25
hw/block/nvme.c | 4 ++--
11
block/linux-aio.c | 6 ++++++
26
1 file changed, 2 insertions(+), 2 deletions(-)
12
1 file changed, 6 insertions(+)
27
13
28
diff --git a/hw/block/nvme.c b/hw/block/nvme.c
14
diff --git a/block/linux-aio.c b/block/linux-aio.c
29
index XXXXXXX..XXXXXXX 100644
15
index XXXXXXX..XXXXXXX 100644
30
--- a/hw/block/nvme.c
16
--- a/block/linux-aio.c
31
+++ b/hw/block/nvme.c
17
+++ b/block/linux-aio.c
32
@@ -XXX,XX +XXX,XX @@ static uint16_t nvme_get_feature(NvmeCtrl *n, NvmeCmd *cmd, NvmeRequest *req)
18
@@ -XXX,XX +XXX,XX @@ void laio_io_unplug(BlockDriverState *bs, LinuxAioState *s,
33
result = blk_enable_write_cache(n->conf.blk);
19
assert(s->io_q.plugged);
34
break;
20
s->io_q.plugged--;
35
case NVME_NUMBER_OF_QUEUES:
21
36
- result = cpu_to_le32((n->num_queues - 1) | ((n->num_queues - 1) << 16));
22
+ /*
37
+ result = cpu_to_le32((n->num_queues - 2) | ((n->num_queues - 2) << 16));
23
+ * Why max batch checking is performed here:
38
break;
24
+ * Another BDS may have queued requests with a higher dev_max_batch and
39
default:
25
+ * therefore in_queue could now exceed our dev_max_batch. Re-check the max
40
return NVME_INVALID_FIELD | NVME_DNR;
26
+ * batch so we can honor our device's dev_max_batch.
41
@@ -XXX,XX +XXX,XX @@ static uint16_t nvme_set_feature(NvmeCtrl *n, NvmeCmd *cmd, NvmeRequest *req)
27
+ */
42
break;
28
if (s->io_q.in_queue >= laio_max_batch(s, dev_max_batch) ||
43
case NVME_NUMBER_OF_QUEUES:
29
(!s->io_q.plugged &&
44
req->cqe.result =
30
!s->io_q.blocked && !QSIMPLEQ_EMPTY(&s->io_q.pending))) {
45
- cpu_to_le32((n->num_queues - 1) | ((n->num_queues - 1) << 16));
46
+ cpu_to_le32((n->num_queues - 2) | ((n->num_queues - 2) << 16));
47
break;
48
default:
49
return NVME_INVALID_FIELD | NVME_DNR;
50
--
31
--
51
2.13.5
32
2.36.1
52
53
diff view generated by jsdifflib