ReadFile（GPSとの通信）Win7での挙動

Question

はじまして。
GPS（Garmmin　GPSmap 60Cx)からUSB通信でデータをインンポートするツールをViualStudio2008　C++/CLIで作っているものです。
DeviceIoControlを実行後、読み取ったバイトサイズが0になるまでReadFileを複数回実行してデータ（ウエイポイント）を取得しています。GPSには、9つのウエイポイントを登録しています。WindowsXPでは、9つすべて問題なく取得できます。しかし、Windows７（32bit,64bit両方とも)では、はじめの3つしか取得できません。デバッグ中にReadFile関数のあとにブレークポイントを置くか、または、Sleep（１）の処理を入れて実行すると9つ全て取得できます。
Sleep（１）の処理を入れる事でなぜうまく動くのかがわからず困っています。Sleep（１）の処理を入れる事で、うまく動く理由、あるいは、別の解決策がわかる方がいらっしゃいましたら、教えていただけないでしょうか。

該当箇所のソースは、以下の通りです。
typedef struct
    {
　　unsigned char  mPacketType;
    unsigned char  mReserved1;
    unsigned short mReserved2;
    unsigned short mPacketId;
    unsigned short mReserved3;
    unsigned long  mDataSize;
BYTE           mData[MAX_BUFFER_SIZE-12];
    } GetPacket_t;

static std::vector<GetPacket_t> vecPacket;
#define ASYNC_DATA_SIZE 64
#define IOCTL_ASYNC_IN        CTL_CODE (FILE_DEVICE_UNKNOWN, 0x850, METHOD_BUFFERED, FILE_ANY_ACCESS)

GetPacket_t* GarComm::GComm::GetPacket2()
{
GetPacket_t* thePacket = NULL;
GetPacket_t* tempthePacket = NULL;

DWORD theBufferSize = 0;
BYTE* theBuffer = NULL;

for( ; ; 
{
// Read async data until the driver returns less than the max async data size, which signifies the end of a packet
BYTE theTempBuffer[ASYNC_DATA_SIZE];
BYTE* theNewBuffer = NULL;
DWORD theBytesReturned = 0;

DeviceIoControl( gHandle,
 IOCTL_ASYNC_IN,
 0,
 0,
 theTempBuffer,
 sizeof( theTempBuffer ),
 &theBytesReturned,
 NULL );

theBufferSize += ASYNC_DATA_SIZE;
theNewBuffer = (BYTE*) malloc( theBufferSize );
    memcpy( theNewBuffer, theBuffer, theBufferSize - ASYNC_DATA_SIZE );
memcpy( theNewBuffer + theBufferSize - ASYNC_DATA_SIZE,theTempBuffer,ASYNC_DATA_SIZE );

if(theBuffer != NULL ){
free( theBuffer );
}

theBuffer = theNewBuffer;

if( theBytesReturned != ASYNC_DATA_SIZE )
{
thePacket = (GetPacket_t*) theBuffer;
break;
 }

}
// If this was a small "signal" packet, read a real packet using ReadFile
if( thePacket->mPacketType == 0 &&
thePacket->mPacketId == 2 )
    {
free( thePacket );

int recCnt=0;
int^ ndbg;
BYTE* theNewBuffer =NULL;
DWORD theBytesReturned = 0;
int ret=0;

for( ; ; )
{
if(theNewBuffer!=NULL)free(theNewBuffer);
theNewBuffer = (BYTE*) malloc( MAX_BUFFER_SIZE );
theBytesReturned = 0;

// A full implementation would keep reading (and queueing) packets until the driver returns a 0 size buffer.
ret=ReadFile( gHandle, theNewBuffer, MAX_BUFFER_SIZE, &theBytesReturned, NULL );
recCnt++;
ndbg=recCnt;

tempthePacket=(GetPacket_t*) theNewBuffer;
// ベクターにパケットを保持。
vecPacket.push_back(*tempthePacket);
if (theBytesReturned==0){
free(theNewBuffer);
break;
}

// Win７
Sleep(1);

}
return &(vecPacket[0]);
}
else
    {
return thePacket;
    }
}

ちなみに、ハンドルをCreateFileでFILE_FLAG_OVERLAPPEDを指定し、DeviceIoControl、ReadFile関数にOverlapped構造体を指定して、非同期で動くかす方法も試したのですが、結果は、同じで、ReadFile関数の処理が完了したあとに、次のReadFile関数を実行する前にSleep（１）を入れないとWindows７では、全てのデータを取得することができませんでした。
また、動作を確認したPCは、
Win7のPCは、Core i5-2500  3.3Ghz 、
WinXPのPCは、Pentium4　３Gｈｚ（くらい）
とマシンスペックには、差があります・・・。
USBは両方とも2.0です。

どうぞよろしくお願いいたします。

AsanoNagi · Accepted Answer

それは、たんに、XP のマシンより、7 のマシンの方が「速い」というだけのことかもしれません。

シリアルポートの読み込みで、「まだデータが届いていない（or そう判断された場合）」読み込みバッファの中身は、「なし」になります。

なので、基本は、
・読み込まれるべきデータが全部そろうまで読み込みを行い、
・途中でデータがなくなる可能性に対応するために、タイムアウトの処理を行う
ということになります。

実際には、USB を使っていたとしても、GPS側の処理スピードで、各データブロックの間には、そこそこの時間が空くことはよくあります。

処理時間が間に合うのなら、
・データが全部そろった → 終了
・読み込みバッファが空 → Sleep(1) で再試行
・Sleep(1) で再試行が、何回も続く → abort 
というのも、処理方法としては、間違ってはいないと思います。

hidebun · Answer

> theDevDetailData->DevicePath
このパスは間違えていないとは思いますが、正しいものが入っていますか？

> SetupComm（）、GetCommTimeouts 、SetCommTimeouts のいずれの関数も失敗してしまい（GetLastErrorの戻り値：ファンクションが間違っています。）

これはそれぞれの関数の戻り値が０なのでしょうか？
その上での「GetLastError戻り値：ファクションが間違っています」なのでしょうか？
GerminのGPSを使ったプログラムサンプルなどは、検索したら色々見つかりそうなものですが、
RS232Cのプログラミングは地味に難しいので、１つずつ調べていくしかないです。

hidebun · Answer

初期設定はちゃんと行っておられますか？
以下、MSDNより抜粋
>通信デバイスからデータを読み取っているとき、ReadFile 関数の動作は、現在の通信タイムアウト値の影響を受けます。SetCommTimeouts と GetCommTimeouts の各関数は、この通信タイムアウト値の設定と取得を行います。タイムアウト値の設定を怠ると、予測できない結果が生じることがあります。

初期設定として
・ReadFileのタイムアウト値はどの程度になっているのでしょう。
・SetupComm（）関数で送受信バッファーの容量を設定します。十分な値が設定されてますか？ご確認下さい。

ここからは個人的な感想なんですが、Sleep(1)で動くなら、それで良いと思います。
Sleepがないと、for(;;)ではCPUコア１つを100%使ってしまうことになるので。
GPSの通信を受信する程度では、ビジーループにする必要はないでしょう。

ReadFile（GPSとの通信）Win7での挙動

それは、たんに、XP のマシンより、7 のマシンの方が「速い」というだけのことかもしれません。

> theDevDetailData->DevicePath

初期設定はちゃんと行っておられますか？

この回答への補足

似たような質問が見つかりました

関連するカテゴリからQ&Aを探す

デイリーランキングこのカテゴリの人気デイリーQ&Aランキング

マンスリーランキングこのカテゴリの人気マンスリーQ&Aランキング