Re: [PATCH 0/2] MR5501: ntdll: Add NtQueryInformationProcess(ProcessQuotaLimits) stub.

[Vijay Kiran Kamuju (＠infyquest)]

On Mon Apr 15 12:40:06 2024 +0000, Jinoh Kang wrote:
> Unless an app relies on *these* specific values, I think it makes more
> sense to maintain consistency across APIs.
> You have two options:
> **Option 1: Make GetProcessWorkingSetSizeEx() a wrapper that calls
> NtQueryInformationProcess(ProcessQuotaLimits).**  I prefer this option,
> since we don't have to break any tests.
> **Option 2: Make ProcessQuotaLimits consistent with
> GetProcessWorkingSetSizeEx().**  This will need some `todo_wine` in the
> newly introduced tests, but needs no further work other than:
>                     /* 32 MB working set size (see
> GetProcessWorkingSetSizeEx) */
>                     qlimits.MinimumWorkingSetSize = 32*1024*1024;
>                     qlimits.MaximumWorkingSetSize = 32*1024*1024;

I would go with option 1, but this needs QUOTA_LIMITS_EX implementation. That might need to wait after this patch.