From nobody@FreeBSD.org  Tue Mar 28 04:30:30 2000
Return-Path: <nobody@FreeBSD.org>
Received: from freefall.freebsd.org (freefall.FreeBSD.ORG [204.216.27.21])
	by hub.freebsd.org (Postfix) with ESMTP id 04D7137B6BA
	for <freebsd-gnats-submit@FreeBSD.org>; Tue, 28 Mar 2000 04:30:30 -0800 (PST)
	(envelope-from nobody@FreeBSD.org)
Received: (from nobody@localhost)
	by freefall.freebsd.org (8.9.3/8.9.2) id EAA20843;
	Tue, 28 Mar 2000 04:30:29 -0800 (PST)
	(envelope-from nobody@FreeBSD.org)
Message-Id: <200003281230.EAA20843@freefall.freebsd.org>
Date: Tue, 28 Mar 2000 04:30:29 -0800 (PST)
From: lindroos@nls.fi
Sender: nobody@FreeBSD.org
To: freebsd-gnats-submit@FreeBSD.org
Subject: misconfigured syscons bell causes panic on alpha
X-Send-Pr-Version: www-1.0

>Number:         17637
>Category:       alpha
>Synopsis:       misconfigured syscons bell causes panic on alpha
>Confidential:   no
>Severity:       serious
>Priority:       medium
>Responsible:    billf
>State:          closed
>Quarter:        
>Keywords:       
>Date-Required:  
>Class:          sw-bug
>Submitter-Id:   current-users
>Arrival-Date:   Tue Mar 28 04:40:00 PST 2000
>Closed-Date:    Wed Mar 31 14:55:56 PST 2004
>Last-Modified:  Wed Mar 31 14:55:56 PST 2004
>Originator:     Dennis Lindroos
>Release:        FreeBSD 4.0-RELEASE
>Organization:
>Environment:
FreeBSD bag.nls.fi 4.0-RELEASE FreeBSD 4.0-RELEASE #0: Thu Mar 23
21:36:50 EET 2000     root@bag.nls.fi:/usr/src/sys/compile/NLS  alpha   
>Description:
If the syscons bell pitch is set to zero, then any terminal bell will
cause a system panic. 
>How-To-Repeat:
kbdcontrol -b 0.0
perl -e 'print chr(7)'

>Fix:


>Release-Note:
>Audit-Trail:

From: Bosko Milekic <bmilekic@dsuper.net>
To: lindroos@nls.fi
Cc: FreeBSD-gnats-submit@FreeBSD.ORG
Subject: Re: alpha/17637: misconfigured syscons bell causes panic on alpha
Date: Tue, 28 Mar 2000 12:19:23 -0500 (EST)

  Hi,
 
  	It appears that the alpha's sysbeep() does a TIMER_DIV() on the given
   pitch, which could cause problems if the pitch is 0.
 
   Please try the following patch and let me know if it works:
 
   http://pages.infinit.net/bmilekic/code/diffs/alpha.beep.patch
 
 
  ..........................................................................
   Bosko Milekic * bmilekic@dsuper.net * http://pages.infinit.net/bmilekic/
   Montreal, Quebec, Canada. *  Technokratis:  http://www.technokratis.com/
 
 
 

From: Dennis Lindroos <lindroos@can.nls.fi>
To: Bosko Milekic <bmilekic@dsuper.net>
Cc: FreeBSD-gnats-submit@FreeBSD.ORG
Subject: Re: alpha/17637: misconfigured syscons bell causes panic on alpha
Date: Wed, 29 Mar 2000 10:15:35 +0300 (EET DST)

 Hi
 
 On Tue, 28 Mar 2000, Bosko Milekic wrote:
 >   It appears that the alpha's sysbeep() does a TIMER_DIV() on the given
 >   pitch, which could cause problems if the pitch is 0.
 >   Please try the following patch and let me know if it works:
 > 
 >   http://pages.infinit.net/bmilekic/code/diffs/alpha.beep.patch
 > 
 > 
 >  ..........................................................................
 >   Bosko Milekic * bmilekic@dsuper.net * http://pages.infinit.net/bmilekic/
 >   Montreal, Quebec, Canada. *  Technokratis:  http://www.technokratis.com/
 > 
  
 Yes. It works, but to turn the bell off completely (without a short tick) i 
 added "if (pitch)" before the "outb(IO_PPI, inb(IO_PPI) | 3);" line.
 (I know i should use the visual bell instead.. :)
 
 Btw. The hardware is an Avanti (AlphaServer 400 4/166).
 
 Dennis Lindroos
 lindroos@nls.fi
 
 
Responsible-Changed-From-To: freebsd-alpha->billf 
Responsible-Changed-By: billf 
Responsible-Changed-When: Wed Mar 29 14:35:43 PST 2000 
Responsible-Changed-Why:  
I'm working on this and as soon as I can get a kernel to compile on beast, I'll commit it. 
State-Changed-From-To: open->suspended 
State-Changed-By: billf 
State-Changed-When: Thu Mar 30 15:05:11 PST 2000 
State-Changed-Why:  
Committed to HEAD, I'll wait a few days and merge this down. 

From: Gerald Pfeifer <pfeifer@dbai.tuwien.ac.at>
To: <freebsd-gnats-submit@freebsd.org>, <billf@freebsd.org>
Cc: <lindroos@nls.fi>
Subject: Re: alpha/17637: misconfigured syscons bell causes panic on alpha
Date: Wed, 3 Jan 2001 17:07:11 +0100 (CET)

 > State-Changed-From-To: open->suspended
 > State-Changed-By: billf
 > State-Changed-When: Thu Mar 30 15:05:11 PST 2000
                       ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
 > State-Changed-Why:
 > Committed to HEAD, I'll wait a few days and merge this down.
                                ^^^^^^^^^^
 
 Hmm, I think a few days have passed... :-)
 
 How about closing this PR know?
 
 Gerald
 -- 
 Gerald "Jerry" pfeifer@dbai.tuwien.ac.at http://www.dbai.tuwien.ac.at/~pfeifer/
 
 
State-Changed-From-To: suspended->closed 
State-Changed-By: wilko 
State-Changed-When: Sun Jun 17 13:14:19 PDT 2001 
State-Changed-Why:  
Problem was fixed long ago, so let us also close the PR.. 

---------------------------- 
revision 1.15 
date: 2000/03/30 22:39:48;  author: billf;  state: Exp;  lines: +5 -3 
Avoid dividing by zero when beeping with a zero pitch. This was bad. 

PR:             alpha/17637 
Submitted by:   Bosko Milekic <bmilekic@dsuper.net> 
Reported by:    Dennis Lindroos <lindroos@nls.fi> 



http://www.FreeBSD.org/cgi/query-pr.cgi?pr=17637 
State-Changed-From-To: closed->open 
State-Changed-By: wilko 
State-Changed-When: Tue Jun 19 11:35:12 PDT 2001 
State-Changed-Why:  
Re-open PR. The patch was indeed applied to the source but there 
are several people (billf, bde) that don't like the patch as such 
(not clean, etc).  


http://www.FreeBSD.org/cgi/query-pr.cgi?pr=17637 
Responsible-Changed-From-To: billf->bmilekic 
Responsible-Changed-By: billf 
Responsible-Changed-When: Wed Mar 31 02:07:53 PST 2004 
Responsible-Changed-Why:  
maybe bosko wants to look at this. 

http://www.freebsd.org/cgi/query-pr.cgi?pr=17637 
State-Changed-From-To: open->closed 
State-Changed-By: billf 
State-Changed-When: Wed Mar 31 14:55:34 PST 2004 
State-Changed-Why:  
reopen if the issue comes up again 


Responsible-Changed-From-To: bmilekic->billf 
Responsible-Changed-By: billf 
Responsible-Changed-When: Wed Mar 31 14:55:34 PST 2004 
Responsible-Changed-Why:  
reopen if the issue comes up again 

http://www.freebsd.org/cgi/query-pr.cgi?pr=17637 
>Unformatted:
