Re: hidclass.sys: Properly quit hid_device_thread when bothcompletion event and halt event are signaled (try 2)

[Codeweavers]

> On Jan 7, 2016, at 8:14 PM, Changhui LIU  wrote:
>
> Hi Aric:
>
> The driver is a winelib hidusb.sys work in progress, not a real world native driver.
> China Merchant Bank's on-line bank program use HidD_SetFeature/HidD_GetFeature to
> access hid token.
> I write this dirver based on patches developed by Alexander Morozov .
> (http://wiki.winehq.org/USB, ftp://ftp.etersoft.ru/pub/people/amorozov/usb )

Ok I understand. I believe the GetFeature and SetFeature parts of his are not implemented at all. They are not hard but where not my first priority.

I have read Alexander's code but it is not clear to me how you are hooking that into hid unless you have written a custom mini driver?

-aric
> Now the on-line bank program can detect hid token on wine, but has some little bug.
> I will send this driver source code to wine upstream end of this month.
>
> Thank you for your great work on hidclass.sys .
>
> Regards,
> Changhui Liu
>
> ------------------ Original ------------------
> From:  "Aric Stewart";
> Date:  Thu, Jan 7, 2016 10:49 PM
> To:  "fracting"; "Sebastian Lackner";
> Cc:  "wine-devel"; "刘昌辉";
> Subject:  Re: hidclass.sys: Properly quit hid_device_thread when bothcompletion event and halt event are signaled (try 2)
>
> On 1/7/16 12:47 AM, Qian Hong wrote:
> > On Thu, Jan 7, 2016 at 1:23 PM, Sebastian Lackner  wrote:
> > > Wouldn't it be easier to just swap events[0] <-> events[1], to handle the
> > > exit event with higher priority? I do not see any real need for two calls
> > > to Wait*() functions.
> >
> > After discussing on irc, we agree that swapping  events[0] <->
> > events[1] is not enough.
> > The original patch fix not only one bug, but two separate problems.
> > In the existent implementation, If ntrc is never set to
> > STATUS_PENDING, then there is no chance to turn exit_now to true,
> > Changhui's patch fix this bug, which is discovered by a real world
> > online bank driver.
>
> My ears perk up here.  Real world online bank driver?  Can I see and play with it. I am really looking for real world cases that use HID.
>
> -aric