github/acrn-hypervisor
1
0
Fork 0
mirror of https://github.com/projectacrn/acrn-hypervisor.git synced 2025-06-23 22:18:17 +00:00
Code Issues Projects Releases Wiki Activity

DM USB: xHCI: delay 2s for enumeration in resuming process

Browse Source 
During UOS S3 resume, the UOS will handle USB disconnect flow then
re-enumeration flow due to vbus drop be emulted for UOS S3 case. The
re-enumeration flow will impact UOS S3 resume latency, this patch
delayed the re-enumeration 2s as the workaround for this issue. Need to
do performance optimization in future.

This 2s can be tune which is depend on how fast UOS S3 resume, it should
be as short as possible to make virtual USB device ready immediately
after UOS screen turn on.

Tracked-On: #2576
Signed-off-by: Conghui Chen <conghui.chen@intel.com>
Signed-off-by: Xiaoguang Wu <xiaoguang.wu@intel.com>
Acked-by: Yu Wang <yu1.wang@intel.com>
This commit is contained in:
Xiaoguang Wu 2019-02-20 16:42:52 +08:00 committed by wenlingz
parent 9844ff0266
commit ecedf31bd1
1 changed files with 18 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

18
devicemodel/hw/pci/xhci.c
View File

@ -408,6 +408,7 @@ struct pci_xhci_vdev {
sem_t vbdp_sem;
bool vbdp_polling;
int vbdp_dev_num;
int vbdp_delay;
struct pci_xhci_vbdp_dev_state vbdp_devs[XHCI_MAX_VIRT_PORTS];
/*
@ -782,6 +783,21 @@ xhci_vbdp_thread(void *data)
continue;
}
if (xdev->vbdp_delay == 1) {
xdev->vbdp_delay = 0;
/* FIXME: Delay 2s for virtual USB re-enumeration stage
* during UOS S3 resume. This is one workaround for
* improving UOS S3 resume latency, it need to be
* optimization instead of delay.
*
* And the delay time can be tune which is depend on how
* fast UOS S3 resume, it should be as short as possible
* to make virtual USB device ready immediately after
* UOS screen turn.
*/
usleep(2000000);
}
j = pci_xhci_get_native_port_index_by_path(xdev,
&xdev->vbdp_devs[i].path);
if (j < 0) {
@ -1327,6 +1343,7 @@ pci_xhci_usbcmd_write(struct pci_xhci_vdev *xdev, uint32_t cmd)
* and under that situation, the vbdp_devs and se_dev_num
* should also need to be cleared
*/
xdev->vbdp_delay = 1;
xdev->vbdp_dev_num = 0;
memset(xdev->vbdp_devs, 0, sizeof(xdev->vbdp_devs));
@ -4273,6 +4290,7 @@ pci_xhci_init(struct vmctx *ctx, struct pci_vdev *dev, char *opts)
pthread_mutex_init(&xdev->mtx, NULL);
/* create vbdp_thread */
xdev->vbdp_delay = 0;
xdev->vbdp_polling = true;
sem_init(&xdev->vbdp_sem, 0, 0);
error = pthread_create(&xdev->vbdp_thread, NULL, xhci_vbdp_thread,