>Number:         6226
>Category:       port-i386
>Synopsis:       misleading panic message on 1.3.2/i386 makes debugging hard
>Confidential:   no
>Severity:       non-critical
>Priority:       medium
>Responsible:    gnats-admin (GNATS administrator)
>State:          open
>Class:          sw-bug
>Submitter-Id:   net
>Arrival-Date:   Thu Oct  1 16:05:00 1998
>Last-Modified:
>Originator:     John Darrow
>Organization:
	Computing Services
	Wheaton College, Wheaton, IL
>Release:        NetBSD 1.3.2
>Environment:
System: NetBSD scilab15.wheaton.edu 1.3.2 NetBSD 1.3.2 (SCINFS) #0: Mon Aug 24 04:19:53 CDT 1998 jdarrow@scilab15.wheaton.edu:/var/src/sys/arch/i386/compile/SCINFS i386


>Description:
	(Found while back-porting Jason Thorpe's epic driver to 1.3.2)

	The 1.3.2 sys/arch/i386/i386/machdep.c contains a stub for
	_bus_dmamap_load_mbuf which panics with a "not implemented"
	message.  However, the message reports that the panic is in
	"_bus_dmamap_load" (which _is_ implemented) instead of
	"_bus_dmamap_load_mbuf".

	This made it difficult to track down exactly what I had to
	pull up from current in order to implement the missing
	functionality.
>How-To-Repeat:
	Backport driver.  Panic machine.  Try to find source of panic.
	Pull hair out.
>Fix:

*** /sys/arch/i386/i386/machdep.c	Fri May  8 11:55:32 1998
--- /sys/arch/i386/i386/machdep.c_fixed	Thu Oct  1 17:43:34 1998
***************
*** 2379,2385 ****
  	int flags;
  {
  
! 	panic("_bus_dmamap_load: not implemented");
  }
  
  /*
--- 2379,2385 ----
  	int flags;
  {
  
! 	panic("_bus_dmamap_load_mbuf: not implemented");
  }
  
  /*

Please pull up this fix into 1.3.3 (unless a _bus_dmamap_load_mbuf
implementation is pulled up instead).
>Audit-Trail:
>Unformatted: