Currently the maximum number of bus and port for xHCI are both set to 255, it is theoretically possible but in fact not neccessary. This patch changes those two values to be more proper: 4 buses and

The variable native_assign_ports in struct pci_xhci_vdev is used to record wether certain root hub port in SOS is assigned to UOS. The logic uses zero to express 'not assigned' and nonzero to

The xHCI emulation greatly depends on the user space library libusb which is based on the usbfs module in Linux kernel. The libusb will bind usbfs to physical USB device which makes hardware control

Replace 'native_assign_ports' with 'port_map_tbl' to be more accurate for the role of this variable plays. Signed-off-by: Xiaoguang Wu <xiaoguang.wu@...> Reviewed-by: Liang Yang

Current design cannot get physical USB device information without the creation of pci_xhci_dev_emu. This brings some difficulties in certain situations, hence struct usb_native_devinfo is

Current DM design use two variables to do the indexing of xHCI Event Ring: er_enq_idx and er_events_cnt. They are members of the struct pci_xhci_rtsregs. In UOS, during the process of xHCI resuming,

PORTSC (Port Status and Control Register) register play a very important role in USB sub-system. This patch is used to refine related manipulation functions. Signed-off-by: Xiaoguang Wu

This patchset enable the USB xHCI S3 feature. Xiaoguang Wu (8): DM USB: xHCI: fix an xHCI issue to enable UOS s3 feature DM USB: xHCI: refine xHCI PORTSC Register related functions DM USB:

That is good.

This is for correctness. Eddie raised the question about the sequence of read MMIO. Some MMIO is mapped to register and multiple read could have side effect. When we emulate the MMIO read operation,

Let me add log to capture this case. As I know, trusty also trigger EPT violation. Not sure what kind of instruction it will use. Regards Yin, Fengwei

Acked-by: Yan, Like <like.yan@...>

It looks like not easy. This is part of switch..case. And there are other specific operations for each... Regards Yin, Fengwei

Could anyone help to review this patch series?

Could anyone help to review this patch series?

xgwu: ok, will change to: VPORT_NUM VPORT_STATE VPORT_NUM_STATE thanks.

According to SDM 25.1.1 Relative priority of Faults and VM Exits, The faults with higher priority than VM exit doesn't include lock prefix. Also there is no other place talking about the lock prefix

V1: In order to better comply with MISRA C, add const qualifiers whereeven required. In the patch, these are being added to pointers which are normally used in "get" functions. V2: Corrected the

Acked-by: Anthony Xu <anthony.xu@...>