Hidlibrary: ReadReport() 메모리 누수

카차르트,

불행히도 그 코드는 꽤 오래되었고 그것을 줄 시간이 없었습니다
모든 TLC. 이 시점에서 나는 단지 그것을 참조로 가지고 있습니다. 최선
내기는 포크/개선하는 것입니다.

미디엄

2011년 8월 14일 일요일 오전 10:37 kaczart <
답장@reply.github.com>작성:

훌륭한 도서관 Mike, 이 엄청난 작업에 감사드립니다.

문제는 InputReport를 데이터로 채우고 싶을 때입니다.

InputReport = device.ReadReport();

여기서 InputReport는 다음과 같습니다.

HidLibrary.HidReport InputReport = 신규
HidLibrary.HidReport(ukp.Capabilities.InputReportByteLength);

특히 보고서를 비동기식으로 읽을 때 누출 메모리가 매우 큽니다.
장치에서 모드(예: 1초에 100회). 몇 분 안에 할 수 있습니다
수백 메가 바이트가 걸립니다.

저는 C# 프로그래밍의 초보자이며 문제에 대한 해결책을 찾지 못했습니다.
위에 설명 ;(

인사말!

이 이메일에 직접 답장하거나 GitHub에서 확인하세요.
https://github.com/mikeobrien/HidLibrary/issues/11

카차르트,

해결책을 찾으셨습니까? 나는 단지 같은 것을 눈치 챘다.
훌륭한 도서관 Mike, 정말 감사합니다! 저는 Endicia USB 스케일로 작업하고 있는데 이것은 꽤 재미있습니다.

jrockfl,

불행히도 이 문제를 해결할 시간과 기술이 없었습니다.

인사말

문제는 HidReport 및 Devicedata가 해제되지 않아 가비지 수집을 기다리는 것입니다.
처리 후 해제하는 것은 쉬운 수정이지만 ...

마지막으로 이 문제를 해결해야 했으며 찾은 내용을 공유합니다.

1) 우선, 라이브러리의 모든 Read/Write 메서드는 장치가 연결되어 있는지 확인합니다. IsConnected 메서드가 EnumerateHidDevices()를 호출할 때마다 호출하기 때문에 성능이 많이 소모됩니다. 실제로 장치에서 대용량 데이터 패키지를 보낼 때 PC 호스트는 HID 장치보다 느리고 제한된 링 버퍼(기본적으로 32개 보고서, 512개로 확장될 수 있지만 최종 솔루션은 아님)로 인해 일부 패킷이 손실됩니다.
추신. 다음을 호출하여 링 버퍼를 32개에서 최대 512개 보고서로 확장할 수 있습니다.
NativeMethods .HidD_SetNumInputBuffers((int)device.ReadHandle, 512); // XP 이상에서 작동합니다.
2) 두 번째로 EnumerateHidDevices()에서 "yield return devicePath"로 인해 엄청난 메모리 누수가 발생합니다. 69행 및 NativeMethods.SetupDiDestroyDeviceInfoList(deviceInfoSet); 절대 호출되지 않습니다.

내 솔루션:
1) 읽기/쓰기마다 IsConnected를 호출하지 마십시오. HidDevice에서 추가 isConnected 변수를 만듭니다.
public bool isConnected;
연결이 성공한 후 변수를 true로 설정합니다(예: Connect() 함수에서):
device.isConnected = true;
모든 DeviceEventMonitorInserted() 및 DeviceEventMonitorRemoved() 이벤트에서 업데이트하십시오. 예:
if (삽입 != null ) Inserted();
다음으로 대체:
if (삽입 != null )
{
isConnected = 참;
삽입();
}
그리고:
if (제거됨 != null ) Removed();
다음으로 대체:
if (제거 != null )
{
isConnected = 거짓 ;
제거됨();
}
모든 읽기/쓰기 호출 기능에서 이 변수를 확인하는 것보다:
if(IsConnected) 대신 if(isConnected)
장치가 연결되어 있는지 확인해야 할 때마다 공용 변수 isConnected의 상태를 확인하십시오. 그것은 나를 위해 작동합니다 - 읽기 스레드의 경우 10-20% 대신 0-1% 프로세서 사용.

2) HidDevice.cs 줄 #69에서 다음을 삭제합니다.
// 반환 devicePath를 반환합니다.
NativeMethods .SetupDiDestroyDeviceInfoList(deviceInfoSet); 적절하게 호출됩니다.
다음으로 NativeMethods .SetupDiDestroyDeviceInfoList(deviceInfoSet) 다음에 다음을 추가합니다.
foreach(장치의 문자열 devicePath)
반환 장치 경로를 반환합니다.
엄청난 메모리 누수가 수정됩니다(제 경우에는 수신 데이터 1MB당 약 256MB).

바랍니다.

굉장합니다, 힌트를 주셔서 감사합니다, kaczart!!!

다음은 kaczart(그리고 나)가 발견한 성능 문제에 대한 제안된 해결 방법입니다.
https://github.com/macaba/HidLibrary/commit/8f1864e1351ccca4772d5bbc18a0f94d7ba07fe7

사용법은 다음과 같습니다.

device.ConnectionCheckOverride = device.IsConnected;
methodRequiringFasterUSBReadWrite();
device.ConnectionCheckOverride = false;

이로 인해 대량 USB 전송 테스트 케이스 실행 시간이 159초에서 47초로 단축되었습니다.

IsConnected 속성이 HidDeviceEventMonitor에서 사용되므로 현재 연결된 USB 장치를 열거하는 원래 기능을 유지해야 했기 때문에 나는 kaczart의 솔루션에 관심이 없었습니다. 간단히 말해서; 나는 Inserted 및 Removed 이벤트가 kaczart의 코드에서 호출될 것이라고 생각하지 않습니다(간단한 살펴보기에서).

모든 읽기 및 쓰기에서 장치 연결을 확인하는 것은 고주파수 애플리케이션에서 주요 성능 병목 현상입니다. 제 경우에는 장치 연결을 다시 확인하는 것보다 장치가 연결되었다고 가정하는 것이 30배 더 빨랐습니다. 왕복 시간은 약 70ms에서 2ms로 떨어졌습니다.

그래서 빠른 풀링 사용을 위해 내 자신의 IO를 처리하는 몇 가지 확장 메서드를 작성했습니다.

        [DllImport("hid.dll", SetLastError = true)]
        static internal extern bool HidD_SetOutputReport(IntPtr hidDeviceObject, byte[] lpReportBuffer, int reportBufferLength);

        [DllImport("kernel32.dll", SetLastError = true)]
        static internal extern bool ReadFile(IntPtr hFile, [Out] byte[] lpBuffer, uint nNumberOfBytesToRead, out uint lpNumberOfBytesRead, IntPtr lpOverlapped);
        public static bool FastWrite(this HidLibrary.HidDevice device, byte[] outputBuffer)
        {
            try
            {
                if (NativeMethods.HidD_SetOutputReport(device.Handle, outputBuffer, outputBuffer.Length))
                    return true;
                else
                    return false;
            }
            catch
            {
                return false;
            }
        }
        public static ReadStatus FastRead(this HidLibrary.HidDevice device, byte[] inputBuffer)
        {
            try
            {
                uint bytesRead;
                if (NativeMethods.ReadFile(device.Handle, inputBuffer, (uint)inputBuffer.Length, out bytesRead, IntPtr.Zero))
                {
                    return ReadStatus.Success;
                }
                else
                {
                    return ReadStatus.NoDataRead;
                }
            }
            catch (Exception)
            {
                return ReadStatus.ReadError;
            }
        }