Commit 85e73968 authored by Johannes Berg's avatar Johannes Berg Committed by Richard Weinberger

um: virtio_uml: Fix time-travel external time propagation

When creating an external event, the current time needs to
be propagated to other participants of a simulation. This
is done in the places here where we kick a virtq etc.

However, it must be done for _all_ external events, and
that includes making the initial socket connection and
later closing it. Call time_travel_propagate_time() to do
this before making or closing the socket connection.

Apparently, at least for the initial connection creation,
due to the remote side in my use cases using microseconds
(rather than nanoseconds), this wasn't a problem yet; only
started failing between 5.14-rc1 and 5.15-rc1 (didn't test
others much), or possibly depending on the configuration,
where more delays happen before the virtio devices are
initialized.

Fixes: 88ce6424 ("um: Implement time-travel=ext")
Signed-off-by: default avatarJohannes Berg <johannes.berg@intel.com>
Signed-off-by: default avatarRichard Weinberger <richard@nod.at>
parent 4e8a5eda
...@@ -1090,6 +1090,8 @@ static void virtio_uml_release_dev(struct device *d) ...@@ -1090,6 +1090,8 @@ static void virtio_uml_release_dev(struct device *d)
container_of(d, struct virtio_device, dev); container_of(d, struct virtio_device, dev);
struct virtio_uml_device *vu_dev = to_virtio_uml_device(vdev); struct virtio_uml_device *vu_dev = to_virtio_uml_device(vdev);
time_travel_propagate_time();
/* might not have been opened due to not negotiating the feature */ /* might not have been opened due to not negotiating the feature */
if (vu_dev->req_fd >= 0) { if (vu_dev->req_fd >= 0) {
um_free_irq(vu_dev->irq, vu_dev); um_free_irq(vu_dev->irq, vu_dev);
...@@ -1136,6 +1138,8 @@ static int virtio_uml_probe(struct platform_device *pdev) ...@@ -1136,6 +1138,8 @@ static int virtio_uml_probe(struct platform_device *pdev)
vu_dev->pdev = pdev; vu_dev->pdev = pdev;
vu_dev->req_fd = -1; vu_dev->req_fd = -1;
time_travel_propagate_time();
do { do {
rc = os_connect_socket(pdata->socket_path); rc = os_connect_socket(pdata->socket_path);
} while (rc == -EINTR); } while (rc == -EINTR);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment