NetBSD Problem Report #56191

From martin@duskware.de  Wed May 19 17:12:21 2021
Return-Path: <martin@duskware.de>
Received: from mail.netbsd.org (mail.netbsd.org [199.233.217.200])
	(using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits))
	(Client CN "mail.NetBSD.org", Issuer "mail.NetBSD.org CA" (not verified))
	by mollari.NetBSD.org (Postfix) with ESMTPS id CD9991A924E
	for <gnats-bugs@gnats.NetBSD.org>; Wed, 19 May 2021 17:12:20 +0000 (UTC)
From: martin@NetBSD.org
Reply-To: martin@NetBSD.org
To: gnats-bugs@NetBSD.org
Subject: ZFS tests fail on sparc64
X-Send-Pr-Version: 3.95

>Number:         56191
>Category:       kern
>Synopsis:       ZFS tests fail on sparc64
>Confidential:   no
>Severity:       serious
>Priority:       high
>Responsible:    kern-bug-people
>State:          closed
>Class:          sw-bug
>Submitter-Id:   net
>Arrival-Date:   Wed May 19 17:15:00 +0000 2021
>Closed-Date:    Fri Jun 04 19:22:32 +0000 2021
>Last-Modified:  Fri Jun 04 19:22:32 +0000 2021
>Originator:     Martin Husemann
>Release:        NetBSD 9.99.82
>Organization:
The NetBSD Foundation, Inc.
>Environment:
System: NetBSD thirdstage.duskware.de 9.99.82 NetBSD 9.99.82 (MODULAR) #447: Wed May 19 17:37:49 CEST 2021 martin@thirdstage.duskware.de:/usr/src/sys/arch/sparc64/compile/MODULAR sparc64
Architecture: sparc64
Machine: sparc64
>Description:

Roughly about the time we switched to gcc 10, many ZFS tests started
failing on sparc64 - with some not very clear error message:

tc-start: 1621443124.796297, zfs_read_after_unlink
tc-se:cannot create 'mnt': operation not supported on this type of pool
tc-so:[   1.0000000] entropy: ready
tc-so:[   1.0200050] ZFS filesystem version: 5
tc-end: 1621443125.178562, zfs_read_after_unlink, failed, mount failed: Unknown error: 256


>How-To-Repeat:
Run the vfs tests on sparc64

>Fix:
n/a

>Release-Note:

>Audit-Trail:
From: "J. Hannken-Illjes" <hannken@eis.cs.tu-bs.de>
To: NetBSD GNATS <gnats-bugs@netbsd.org>
Cc: 
Subject: Re: kern/56191: ZFS tests fail on sparc64
Date: Fri, 21 May 2021 14:23:44 +0200

 --Apple-Mail=_72868C3E-4847-453B-9143-D8E557F28218
 Content-Type: multipart/mixed;
 	boundary="Apple-Mail=_A3C04EE1-85E8-402C-89CB-ED1DE0233B22"


 --Apple-Mail=_A3C04EE1-85E8-402C-89CB-ED1DE0233B22
 Content-Transfer-Encoding: 7bit
 Content-Type: text/plain;
 	charset=us-ascii

 Please try the attached diff.
 --
 J. Hannken-Illjes - hannken@eis.cs.tu-bs.de - TU Braunschweig


 --Apple-Mail=_A3C04EE1-85E8-402C-89CB-ED1DE0233B22
 Content-Disposition: attachment;
 	filename=002_nvpair_endian.diff
 Content-Type: application/octet-stream;
 	x-unix-mode=0644;
 	name="002_nvpair_endian.diff"
 Content-Transfer-Encoding: 7bit

 nvpair_endian

 Replace the Solaris style endian check (exactly one of
 _LITTLE_ENDIAN or _BIG_ENDIAN is defined) with standard check
 "#if BYTE_ORDER == BIG_ENDIAN" like we do it elseware.

 Should fix PR 56191 (ZFS tests fail on sparc64)

 diff -r 628b20895fb3 -r 7b8c2578f1d8 external/cddl/osnet/dist/common/nvpair/nvpair.c
 --- external/cddl/osnet/dist/common/nvpair/nvpair.c
 +++ external/cddl/osnet/dist/common/nvpair/nvpair.c
 @@ -2255,11 +2255,11 @@ nvlist_common(nvlist_t *nvl, char *buf, 
  	int err = 0;
  	nvstream_t nvs;
  	int nvl_endian;
 -#ifdef	_LITTLE_ENDIAN
 -	int host_endian = 1;
 +#if BYTE_ORDER == BIG_ENDIAN
 +	int host_endian = 0;
  #else
 -	int host_endian = 0;
 -#endif	/* _LITTLE_ENDIAN */
 +	int host_endian = 1;
 +#endif	/* BYTE_ORDER */
  	nvs_header_t *nvh = (void *)buf;

  	if (buflen == NULL || nvl == NULL ||

 --Apple-Mail=_A3C04EE1-85E8-402C-89CB-ED1DE0233B22--

 --Apple-Mail=_72868C3E-4847-453B-9143-D8E557F28218
 Content-Transfer-Encoding: 7bit
 Content-Disposition: attachment;
 	filename=signature.asc
 Content-Type: application/pgp-signature;
 	name=signature.asc
 Content-Description: Message signed with OpenPGP

 -----BEGIN PGP SIGNATURE-----

 iQEzBAEBCAAdFiEE2BL3ha7Xao4WUZVYKoaVJdNr+uEFAmCnplAACgkQKoaVJdNr
 +uFjSAf/RPqUzfEMFhBKVB9B9omAndao8oh1QOyPnB6f6JyQLNg3jbK04PZihKbR
 PM83SnDZ9CNKT4E72o25AyzIlh9G0ezwEWMpqh/QWIWt3o6QYB3wL13xy/CGShwh
 nTu5VHJ/s02F0r79xJqnYmvNX3vva54S2PBhFuGNNQV4/WvFJoxPqYqoVtbwPwYK
 8EBIA0sMXl9Hh1YaYasTxX/Xo83NlZt+try0ximHLpQfUXJuiAult+oJkoxBqh3t
 +g031zTmnWCBNC8d/yzJcMeT/2J4kFs01F4g63qbC7AMN4t3LMxzx0Mw4nz07UnH
 T+/n3mR98FhV9Xd3afqAZEBwNccfDA==
 =nvSV
 -----END PGP SIGNATURE-----

 --Apple-Mail=_72868C3E-4847-453B-9143-D8E557F28218--

From: Martin Husemann <martin@duskware.de>
To: gnats-bugs@netbsd.org
Cc: 
Subject: Re: kern/56191: ZFS tests fail on sparc64
Date: Fri, 21 May 2021 18:48:29 +0200

 With that patch:


 Failed test cases:
     t_io:rumpfs_wrrd_after_unlink, t_renamerace:puffs_renamerace_cycle

 Summary for 12 test programs:
     751 passed test cases.
     2 failed test cases.
     8 expected failed test cases.
     137 skipped test cases.


 Looks good, thanks!

 Martin

From: "Juergen Hannken-Illjes" <hannken@netbsd.org>
To: gnats-bugs@gnats.NetBSD.org
Cc: 
Subject: PR/56191 CVS commit: src/external/cddl/osnet/dist/common/nvpair
Date: Fri, 21 May 2021 17:32:49 +0000

 Module Name:	src
 Committed By:	hannken
 Date:		Fri May 21 17:32:49 UTC 2021

 Modified Files:
 	src/external/cddl/osnet/dist/common/nvpair: nvpair.c

 Log Message:
 Replace the Solaris style endian check (exactly one of
 _LITTLE_ENDIAN or _BIG_ENDIAN is defined) with standard check
 "#if BYTE_ORDER == BIG_ENDIAN" like we do it elseware.

 Should fix PR 56191 (ZFS tests fail on sparc64)


 To generate a diff of this commit:
 cvs rdiff -u -r1.5 -r1.6 src/external/cddl/osnet/dist/common/nvpair/nvpair.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: dholland@NetBSD.org
State-Changed-When: Fri, 04 Jun 2021 19:22:32 +0000
State-Changed-Why:
fixed, thanks


>Unformatted:

Home	PR Database Search