[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
RE: [PATCH] WHPX: refactor load library
From: |
Sunil Muthuswamy |
Subject: |
RE: [PATCH] WHPX: refactor load library |
Date: |
Wed, 13 Nov 2019 16:59:57 +0000 |
> -----Original Message-----
> From: Paolo Bonzini <address@hidden>
> Sent: Wednesday, November 13, 2019 7:00 AM
> To: Sunil Muthuswamy <address@hidden>; Richard Henderson <address@hidden>;
> Eduardo Habkost <address@hidden>;
> Stefan Weil <address@hidden>
> Cc: address@hidden; Justin Terry (VM) <address@hidden>
> Subject: Re: [PATCH] WHPX: refactor load library
>
> On 08/11/19 21:31, Sunil Muthuswamy wrote:
> >
> > +typedef enum WHPFunctionList {
> > + WINHV_PLATFORM_FNS_DEFAULT,
> > + WINHV_EMULATION_FNS_DEFAULT,
> > +} WHPFunctionList;
> >
>
> What does "default" stand for? I assume you have more changes to this
> function in the future.
>
Yes, there are more functions coming, such as for XSAVE. I used "default" to
represent
whatever is there currently, for lack of a better term.
> > + * Load the functions from the given library, using the given handle. If a
> > + * handle is provided, it is used, otherwise the library is opened. The
> > + * handle will be updated on return with the opened one.
> > + */
> > +static bool load_whp_dipatch_fns(HMODULE *handle, WHPFunctionList
> > function_list)
> > +{
>
> Typo, "dipatch" instead of "dispatch".
> >
> > + if (hLib) {
> > + FreeLibrary(hWinHvEmulation);
> > + }
>
> The argument to FreeLibrary should be hLib.
>
Thanks, will fix these in the next version.
Re: [PATCH] WHPX: refactor load library, Paolo Bonzini, 2019/11/13
- RE: [PATCH] WHPX: refactor load library,
Sunil Muthuswamy <=