No subject

packet arrives at this point. Even so the packet is correct but
still incomplete the check is run and finds that the packet
data are not consistend (small wonder).
If nobody can give better arguments I would opt for
this version:

assert(this->packet_offset <= this->packet_size);   // debug 
#if 1
  // check if we have a complete packet (unfortunatelly packet_size is bad
  // for some disks)
if (spu->packet_offset == spu->packet_size)   <------- line in question
  { unsigned int x=0,y;
    while(x+4<=spu->packet_offset){

On my examples I could not trigger the assertion. So, a large scale test on 
pre2 might be a good test.

-Arne