NetBSD Problem Report #51096

From gson@gson.org  Wed Apr 27 07:28:25 2016
Return-Path: <gson@gson.org>
Received: from mail.netbsd.org (mail.netbsd.org [199.233.217.200])
	(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits))
	(Client CN "mail.netbsd.org", Issuer "Postmaster NetBSD.org" (verified OK))
	by mollari.NetBSD.org (Postfix) with ESMTPS id EC61A7A467
	for <gnats-bugs@gnats.NetBSD.org>; Wed, 27 Apr 2016 07:28:24 +0000 (UTC)
Message-Id: <20160427072819.E9413744641@guava.gson.org>
Date: Wed, 27 Apr 2016 10:28:19 +0300 (EEST)
From: gson@gson.org (Andreas Gustafsson)
Reply-To: gson@gson.org (Andreas Gustafsson)
To: gnats-bugs@NetBSD.org
Subject: hid tests failing on i386, sparc since nick-nhusb merge
X-Send-Pr-Version: 3.95

>Number:         51096
>Category:       kern
>Synopsis:       hid tests failing on i386, sparc since nick-nhusb merge
>Confidential:   no
>Severity:       serious
>Priority:       medium
>Responsible:    kern-bug-people
>State:          closed
>Class:          sw-bug
>Submitter-Id:   net
>Arrival-Date:   Wed Apr 27 07:30:00 +0000 2016
>Closed-Date:    Sun Oct 01 11:47:11 +0000 2017
>Last-Modified:  Sun Oct 01 11:47:11 +0000 2017
>Originator:     Andreas Gustafsson
>Release:        NetBSD-current, source date >= 2016.04.23.12.50.35
>Organization:
>Environment:
System: NetBSD
Architecture: x86_64
Machine: amd64
>Description:

The dev/usb/t_hid/khid and dev/usb/t_hid/khid_parse_just_pop
test cases are failing on i386 and sparc:

  http://releng.netbsd.org/b5reports/i386/build/2016.04.23.12.53.52/test.html#dev_usb_t_hid_khid
  http://releng.netbsd.org/b5reports/sparc/build/2016.04.23.12.50.35/test.html#dev_usb_t_hid_khid

>How-To-Repeat:

Run the ATF tests on i386 or sparc.

>Fix:

>Release-Note:

>Audit-Trail:
From: Martin Husemann <martin@duskware.de>
To: gnats-bugs@NetBSD.org
Cc: 
Subject: Re: kern/51096: hid tests failing on i386, sparc since nick-nhusb merge
Date: Wed, 27 Apr 2016 09:38:05 +0200

 They also fail on arm:

 *** Check failed: /ssd/src/tests/lib/libusbhid/t_usbhid.c:254: data != -128: == 128
 *** Check failed: /ssd/src/tests/lib/libusbhid/t_usbhid.c:256: data != -1: == 255
 *** Check failed: /ssd/src/tests/lib/libusbhid/t_usbhid.c:265: data != -32768: == 32768
 *** Check failed: /ssd/src/tests/lib/libusbhid/t_usbhid.c:267: data != -1: == 65535

 Martin

From: Joerg Sonnenberger <joerg@bec.de>
To: gnats-bugs@NetBSD.org
Cc: 
Subject: Re: kern/51096: hid tests failing on i386, sparc since nick-nhusb
 merge
Date: Wed, 27 Apr 2016 11:08:51 +0200

 On Wed, Apr 27, 2016 at 07:30:00AM +0000, Andreas Gustafsson wrote:
 > The dev/usb/t_hid/khid and dev/usb/t_hid/khid_parse_just_pop
 > test cases are failing on i386 and sparc:
 > 
 >   http://releng.netbsd.org/b5reports/i386/build/2016.04.23.12.53.52/test.html#dev_usb_t_hid_khid
 >   http://releng.netbsd.org/b5reports/sparc/build/2016.04.23.12.50.35/test.html#dev_usb_t_hid_khid

 Given that those test cases have been failing in March on Sparc64
 already, it looks more like bogus test cases. I certainly don't think
 this is a regression.

 Joerg

From: Nick Hudson <skrll@netbsd.org>
To: gnats-bugs@NetBSD.org, kern-bug-people@netbsd.org,
 gnats-admin@netbsd.org, netbsd-bugs@netbsd.org
Cc: 
Subject: Re: kern/51096: hid tests failing on i386, sparc since nick-nhusb
 merge
Date: Thu, 28 Apr 2016 09:19:13 +0100

 I have no idea why, but cpu_softintr_p is returning true and this 
 KASSERT is firing

      405     KASSERTMSG((!cpu_intr_p() && !cpu_softintr_p()),
      406         "kmem(9) should not be used from the interrupt context");

 maybe rump_init is needed...

 Nick

From: "Jonathan A. Kollasch" <jakllsch@netbsd.org>
To: gnats-bugs@gnats.NetBSD.org
Cc: 
Subject: PR/51096 CVS commit: src/tests/dev/usb
Date: Mon, 2 May 2016 17:24:06 +0000

 Module Name:	src
 Committed By:	jakllsch
 Date:		Mon May  2 17:24:06 UTC 2016

 Modified Files:
 	src/tests/dev/usb: t_hid.c

 Log Message:
 Call rump_init() before using rump functionality.

 This change should make these test cases consistently fail on all ports.

 Related to PR kern/51096.


 To generate a diff of this commit:
 cvs rdiff -u -r1.5 -r1.6 src/tests/dev/usb/t_hid.c

 Please note that diffs are not public domain; they are subject to the
 copyright notices on the relevant files.

State-Changed-From-To: open->closed
State-Changed-By: gson@NetBSD.org
State-Changed-When: Sun, 01 Oct 2017 11:47:11 +0000
State-Changed-Why:
Fixed by jakllsch on 2016.05.05.17.40.26 in src/tests/dev/usb/t_hid.c 1.8 - thanks.


>Unformatted:

Home	PR Database Search