Re: PR/52068 CVS commit: src/sys/dev/pci

[Frank Kardel <kardel%netbsd.org@localhost>]

The following reply was made to PR kern/52068; it has been noted by GNATS.

From: Frank Kardel <kardel%netbsd.org@localhost>
To: Kengo NAKAHARA <k-nakahara%iij.ad.jp@localhost>
Cc: gnats-bugs%NetBSD.org@localhost
Subject: Re: PR/52068 CVS commit: src/sys/dev/pci
Date: Wed, 22 Mar 2017 11:31:02 +0100

 New ping is also fine. Leaves the ICMP check error counters alone.
 
 So we can consider this PR closed.
 
 Thanks
    Frank
 
 On 03/22/17 03:20, Kengo NAKAHARA wrote:
 > The following reply was made to PR kern/52068; it has been noted by GNATS.
 >
 > From: Kengo NAKAHARA <k-nakahara%iij.ad.jp@localhost>
 > To: kardel%netbsd.org@localhost, gnats-bugs%NetBSD.org@localhost, kern-bug-people%netbsd.org@localhost,
 >          gnats-admin%netbsd.org@localhost, netbsd-bugs%netbsd.org@localhost
 > Cc:
 > Subject: Re: PR/52068 CVS commit: src/sys/dev/pci
 > Date: Wed, 22 Mar 2017 11:15:14 +0900
 >
 >   Hi kardel@n.o,
 >   
 >   On 2017/03/22 6:21, Frank Kardel wrote:
 >   > I checked the new version and no more checksum errors for UDP and TCP.
 >   > Thanks for the fix.
 >   
 >   Thank you for your testing. I am glad it is fixed. :)
 >   
 >   > I do notice errors on ICMP though:
 >   snip
 >   > That seems a bit surprising and may have been there before the fix - I
 >   > just noticed it now.
 >   
 >   Hmm, I have met this problem last week. After consulting ryo@n.o,
 >   he fixed this problem by ping.c:r1.116 at May 17.
 >   Do you use a little old ping? If so, could you update and try again?
 >   
 >   
 >   Thanks,
 >   
 >   --
 >   //////////////////////////////////////////////////////////////////////
 >   Internet Initiative Japan Inc.
 >   
 >   Device Engineering Section,
 >   IoT Platform Development Department,
 >   Network Division,
 >   Technology Unit
 >   
 >   Kengo NAKAHARA <k-nakahara%iij.ad.jp@localhost>
 >