Hi Michael,

> > +                   pr_warn("TimeSync IC pkt recv failed (Err: %d)\n",
> > +                           ret);
> 
> Let's use pr_warn_once().
> 
> If there's a packet at the head of the ring buffer that specifies a bogus 
> length,
> we could take the error path.  But the bad packet stays at the head of the 
> ring buffer,
> so if we end up back here again, we'll spit out the same error message.  We
> actually should not end up here again because Hyper-V shouldn't interrupt
> when adding a packet to a non-empty ring buffer, but who knows what might
> happen.
> 
Valid point, will fix this in the next iteration.

Thanks,
Vineeth

Reply via email to