Sure. I will change to (p->wildcard || p->data == data) Thanks > > >

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

The loc of code is 1731. (most of ~5900 loc are comments) ------------------------------------------------------------------------------- Language files blank comment

Can we generate the OFFSET MACRO at build time? Anthony

15 files changed, 5975 insertions(+), 70 deletions(-) This patch adds ~ 5900 LOC. Way too big, it increases the ACRN hypervisor code size by > 20%. that's my concern. Any other solutions? SOS is a

We don't want to complicate the debug tools too much. I remember we can already set the size of the log file. That is enough. Thx Eddie

Acked-by: Eddie Dong <eddie.dong@...> This seems to be better to be "d--".

The vcpuid base on these assumption: (1) if we don't plan to support this feature, we could filter it. (2) if the leaf greater than the limit (base or enternal), we should return the value in the

I think it should be re-connection instead of disconnection, right? Currently, the native S3 will disable the vbus for all xHCI ports and re-drive during S3 resume. This behavior cause native USB

I don't see a PR for this in GitHub. Can it be submitted there for review? -- david

For "= 0" and "< 0", please define some macros...

dificulties? Then usb_dev_info is not useful now? Should we clear it? This new structure is confused to me. There has already one structure struct usb_dev which included native usb device

When running longevity test and capturing acrntrace, generated acrntrace files sizes are too big. Sometimes we don't care very old trace. This patch adds ring buffer mode, fixes acrntrace file size

Can we remove intr variable? Is hasn't changed in this function and pass to pci_xhci_port_disconnect directly.. Ditto. What is mean of intr? From this patch, it almost useless.. Let's keep

Thanks! @David, can you take a look at this when you get a chance? Thanks, Geoffroy

Got it.

junjunshan1 <junjun.shan@...> writes: Some general conventions about commit logs: 1. Each line should be no more than 80 characters. Break the paragraph if it is longer than that.

MISARC has requirements about Marco redefinition, usage of ++ or -- and assignment operator in boolean expression. This patch is used to solve these violations. The modifications are summarized as