[PATCH] taskmgr: Implement process page sorting.
Haoyang Chen
chenhaoyang at uniontech.com
Tue Apr 6 03:09:41 CDT 2021
在 2021/4/6 下午4:01, Haoyang Chen 写道:
> From: Nikolay Sivov <nsivov at codeweavers.com>
> Subject: Re: [PATCH] taskmgr: Implement process page sorting.
> Message-Id: <7dd964b5-ed70-f504-ebc6-53739780a8be at codeweavers.com>
> Date: Tue, 6 Apr 2021 08:42:36 +0300
> In-Reply-To: <20210406014834.20910-1-chenhaoyang at uniontech.com>
> References: <20210406014834.20910-1-chenhaoyang at uniontech.com>
>
> On 4/6/21 4:48 AM, Haoyang Chen wrote:
>> Signed-off-by: Haoyang Chen <chenhaoyang at uniontech.com>
>> ---
>> programs/taskmgr/perfdata.c | 76 +++++++++++++++++++++++++++++++++++++
>> programs/taskmgr/perfdata.h | 1 +
>> programs/taskmgr/procpage.c | 10 ++---
>> 3 files changed, 81 insertions(+), 6 deletions(-)
>>
>> diff --git a/programs/taskmgr/perfdata.c b/programs/taskmgr/perfdata.c
>> index 2cbd6b01669..52a08da4d88 100644
>> --- a/programs/taskmgr/perfdata.c
>> +++ b/programs/taskmgr/perfdata.c
>> @@ -833,3 +833,79 @@ ULONG PerfDataGetTotalThreadCount(void)
>>
>> return ThreadCount;
>> }
>> +/* sort code copied from DPA */
>
> Why do you need to copy it?
>
The original DPA_QuickSort sort is for LPVOID, now it is used for
PPERFDATA.
>> +static BOOL bSortAscending = TRUE;
>
> This does not have to be global.
>
Because both processpagecomparefunction and perdatasort use it.
>> +static INT CALLBACK ProcessPageCompareFunc(LPVOID first, LPVOID
>> second, LPARAM lParam)
>> +{
>> + PPERFDATA pPerfData1 = NULL;
>> + PPERFDATA pPerfData2 = NULL;
>> + INT iSubItem = lParam;
>> +
>> + if (bSortAscending)
>> + {
>> + pPerfData1 = (PPERFDATA) first;
>> + pPerfData2 = (PPERFDATA) second;
>> + }
>> + else
>> + {
>> + pPerfData2 = (PPERFDATA) first;
>> + pPerfData1 = (PPERFDATA) second;
>> + }
>> +
>> + switch (iSubItem)
>> + {
>> + case 0:
>> + return lstrcmpW(pPerfData1->ImageName,
>> pPerfData2->ImageName);
>> + case 1:
>> + return pPerfData1->ProcessId > pPerfData2->ProcessId;
>> + case 2:
>> + return pPerfData1->CPUUsage > pPerfData2->CPUUsage;
>> + case 3:
>> + return pPerfData1->CPUTime.QuadPart >
>> pPerfData2->CPUTime.QuadPart;
>> + case 4:
>> + return pPerfData1->vmCounters.VirtualSize >
>> pPerfData2->vmCounters.VirtualSize;
>> + }
>> + return 0;
>> +}
>
> We have 25 columns in total, with symbolic names defined for them.
ok, I will send v2, thanks.
>> +
>> +static VOID DPA_QuickSort (PPERFDATA lpPtrs, INT l, INT r,
>> + PFNDPACOMPARE pfnCompare, LPARAM lParam)
>> +{
>> + INT m;
>> + PERFDATA t;
>> +
>> + if (l==r) /* one element is always sorted */
>> + return;
>> + if (r<l) /* oops, got it in the wrong order */
>> + {
>> + DPA_QuickSort(lpPtrs, r, l, pfnCompare, lParam);
>> + return;
>> + }
>> + m = (l+r)/2; /* divide by two */
>> + DPA_QuickSort(lpPtrs, l, m, pfnCompare, lParam);
>> + DPA_QuickSort(lpPtrs, m+1, r, pfnCompare, lParam);
>> +
>> + /* join the two sides */
>> + while( (l<=m) && (m<r) )
>> + {
>> + if(pfnCompare(&lpPtrs[l],&lpPtrs[m+1],lParam)>0)
>> + {
>> + t = lpPtrs[m+1];
>> + memmove(&lpPtrs[l+1],&lpPtrs[l],(m-l+1)*sizeof(lpPtrs[l]));
>> + lpPtrs[l] = t;
>> +
>> + m++;
>> + }
>> + l++;
>> + }
>> +}
>> +
>> +VOID PerDataSort(UINT iSubItem)
>> +{
>> + EnterCriticalSection(&PerfDataCriticalSection);
>> +
>> + bSortAscending = !bSortAscending;
>> + DPA_QuickSort(pPerfData, 0, ProcessCount - 1,
>> ProcessPageCompareFunc, iSubItem);
>> +
>> + LeaveCriticalSection(&PerfDataCriticalSection);
>> +}
>> \ No newline at end of file
>> diff --git a/programs/taskmgr/perfdata.h b/programs/taskmgr/perfdata.h
>> index 615e7db2a64..f39b03c503d 100644
>> --- a/programs/taskmgr/perfdata.h
>> +++ b/programs/taskmgr/perfdata.h
>> @@ -110,4 +110,5 @@ ULONG PerfDataGetSystemHandleCount(void);
>>
>> ULONG PerfDataGetTotalThreadCount(void);
>>
>> +VOID PerDataSort(UINT iSubItem);
>> #endif /* __PERFDATA_H */
>> diff --git a/programs/taskmgr/procpage.c b/programs/taskmgr/procpage.c
>> index 5ea2e6b044f..f8dc0a0e89d 100644
>> --- a/programs/taskmgr/procpage.c
>> +++ b/programs/taskmgr/procpage.c
>> @@ -155,12 +155,14 @@ static void ProcessPageOnNotify(LPARAM lParam)
>> ULONG ColumnIndex;
>> IO_COUNTERS iocounters;
>> TIME time;
>> + NMLISTVIEW* pnmListView;
>> static const WCHAR wszFmtD[] = {'%','d',0};
>> static const WCHAR wszFmt02D[] = {'%','0','2','d',0};
>> static const WCHAR wszUnitK[] = {' ','K',0};
>>
>> pnmh = (LPNMHDR) lParam;
>> pnmdi = (NMLVDISPINFOW*) lParam;
>> + pnmListView = (NM_LISTVIEW*)lParam;
>>
>> if (pnmh->hwndFrom == hProcessPageListCtrl)
>> {
>> @@ -343,12 +345,8 @@ static void ProcessPageOnNotify(LPARAM lParam)
>> {
>> case HDN_ITEMCLICKW:
>>
>> - /*
>> - * FIXME: Fix the column sorting
>> - *
>> - *ListView_SortItems(hApplicationPageListCtrl,
>> ApplicationPageCompareFunc, NULL);
>> - *bSortAscending = !bSortAscending;
>> - */
>> + PerDataSort(pnmListView->iItem);
>> + InvalidateRect(hProcessPageListCtrl, NULL, TRUE);
>>
>> break;
>>
> HDN_ITEMCLICK does not use NMLISTVIEW structure.
ok, I will send v2, thanks.
>
>
>
>
>
More information about the wine-devel
mailing list