SetFilePointer для устройства PCIe

[chaika_sv]

Добрый день.

У меня есть kmdf-драйвер для PCIe-устройства. Сейчас стоит задача разработать cpp-приложение верхнего уровня для работы с этим драйвером.
Драйвер принимает от cpp-приложения IRP-запросы и обрабатывает их в режиме PIO (взаимодействие через окно BAR0). В частности, запрос на запись обрабатывается так:

Код:

  ...
  WdfMemoryCopyToBuffer(
                  regMemory,
                  0,
                  (INT *)devExt->RegsBase,    //   Указатель на начало окна BAR0
                  Length);
                
  WdfRequestCompleteWithInformation(Request, STATUS_SUCCESS, Length);
  ...

Запрос на запись посылается из cpp-приложения командой:

Код:

  WriteFile(
                 hDevice,
                 WriteBuffer,
                 WriteBufferSize,
                 &bytes,
                 NULL);

После выполнения этой команды содержимое буфера WriteBuffer действительно помещается в начало окна BAR0, а bytes становится равным значению WriteBufferSize.
ВОПРОС. Как сделать произвольный доступ к окну BAR0?

Попробовал перед WriteFile выполнить функцию SetFilePointer:

Код:

  SetFilePointer(
                 hDevice, 
                 offset,     //  Младший адрес смещения относительно начала файла (пробовал разные значения)
                 NULL,       //  Окно BAR0 всего 2КБ, так что длинный указатель тут не нужен (делаем старший адрес - NULL)
                 FILE_BEGIN);

но содержимое буфера всё равно пишется в начало окна BAR0.

Более того, если я два раза подряд выполняю функцию WriteFile, то второй вызов функции приводит к тому, что перезаписываются данные, записанные первым вызовом. Т.е. указатель после WriteFile почему-то не перемещается 

[resource]

А как вы обрабатываете IRP_MJ_SET_INFORMATION ? Ведь именно этот IRP, если не ошибаюсь, приходит при вызове SetFilePointer.

Попробую даже сам же и ответить на свой вопрос  Вы его не обрабатываете вообще никак. А надо.

[chaika_sv]

Вы его не обрабатываете вообще никак.

Действительно не обрабатываю.

А надо.

А надо ли? Опыта написания драйверов у меня много нет, но я всегда думал, что в WDF на этот случай должен быть дефолтный обработчик (мне ж там ничего особенного реализовывать не надо).

Тем более, что для команд чтения SetFilePointer работает!

[resource]

Нда. Про Wdf я не вкурсе, поэтому врать не буду.

[Ochkarik]

попробуйте в драйвере сразу при записи смещение задать - будет работать?
железка поддерживает? выравнивание?

[chaika_sv]

попробуйте в драйвере сразу при записи смещение задать - будет работать?

Пробовал. Работает.

железка поддерживает? выравнивание?

Да.

[Ochkarik]

а где в этом коде  :

Код:

  WdfMemoryCopyToBuffer(
                  regMemory,
                  0,
                  (INT *)devExt->RegsBase,    //   Указатель на начало окна BAR0
                  Length);

вообще задаваемое пользователем смещение? у вас destination = devExt->RegsBase всегда один и тот же?

[chaika_sv]

Ochkarik, да, теперь я понял, что мне нужно как-то из IRP-запроса получить смещение. Что-то вроде такого:

Код:

 WdfMemoryCopyToBuffer(
                  regMemory,
                  0,
                  (INT *)devExt->RegsBase + offset,    //   Указатель на начало окна BAR0 + смещение
                  Length);

На RSDN подсказывают, что мне нужно это:
WdfRequestGetParameters — WDF_REQUEST_PARAMETERS::Write::DeviceOffset

Буду пробовать. Что получится - отпишусь.

[chaika_sv]

Проблема решена. Сделал так:

Код:

	
    INT offset;
    WDF_REQUEST_PARAMETERS  params;

    ...

    WDF_REQUEST_PARAMETERS_INIT(&params);

    WdfRequestGetParameters(
                            Request,
                            &params
                            );	

    switch (params.Type) {
        case WdfRequestTypeWrite:
            offset = params.Parameters.Write.DeviceOffset;
            break;
        default:
            return;
    }
	
	WdfMemoryCopyToBuffer(
						regMemory,
						0,
						(INT *)devExt->RegsBase + offset,
						Length);

Всё верно Ochkarik сказал: про смещение я сначала совсем забыл

[Ochkarik]

 

[sys_dev]

не совсем по теме, но не понимаю таких строк:

Код:

    switch (params.Type) {
        case WdfRequestTypeWrite:
            offset = params.Parameters.Write.DeviceOffset;
            break;
        default:
            return;
    }

Не думаю, что надо надеяться на то что компиллер догадается вместо таблички развернуть в обыкновенные проверки cmp reg,num. По крайней мере GCC для ARM один раз оператор из двух switch у меня изготовил табличку!

Думаю код вида:

Код:

    if (params.Type == WdfRequestTypeWrite)
            offset = params.Parameters.Write.DeviceOffset;

будет куда лучше, конечно, мои слова не касаются ситуации когда в будущем предполагается задавать идентификаторы типа больше 3-4

[Ochkarik]

оно конечно да... но...
такое оформление все таки немного читабельность кода повышает(ИМХО). сразу понятно что параметр может принимать не одно и не два значения. а что до компилятора... если не ошибаюсь майкрософтовский еще лет пять-десять назад вполне адекватно оценивал что ему делать) по крайней мере на x86)