Bug 17637

Summary:	misconfigured syscons bell causes panic on alpha
Product:	Base System	Reporter:	lindroos <lindroos>
Component:	alpha	Assignee:	bill fumerola <billf>
Status:	Closed FIXED
Severity:	Affects Only Me
Priority:	Normal
Version:	4.0-RELEASE
Hardware:	Any
OS:	Any

Description lindroos 2000-03-28 13:40:00 UTC

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)'

Comment 1 bmilekic@dsuper.net 2000-03-28 18:19:23 UTC

 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/

Comment 2 lindroos 2000-03-29 08:15:35 UTC

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

Comment 3 bill fumerola freebsd_committer

2000-03-29 23:35:43 UTC

Responsible Changed
From-To: freebsd-alpha->billf

I'm working on this and as soon as I can get a kernel to compile on beast, I'll commit it.

Comment 4 bill fumerola freebsd_committer

2000-03-31 00:05:11 UTC

State Changed
From-To: open->suspended

Committed to HEAD, I'll wait a few days and merge this down.

Comment 5 pfeifer 2001-01-03 16:07:11 UTC

> 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/

Comment 6 wilko freebsd_committer

2001-06-17 21:14:19 UTC

State Changed
From-To: suspended->closed

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>

Comment 7 wilko freebsd_committer

2001-06-19 19:35:12 UTC

State Changed
From-To: closed->open

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).

Comment 8 bill fumerola freebsd_committer

2004-03-31 11:07:53 UTC

Responsible Changed
From-To: billf->bmilekic

maybe bosko wants to look at this.

Comment 9 bill fumerola freebsd_committer

2004-03-31 23:55:34 UTC

State Changed
From-To: open->closed

reopen if the issue comes up again

Comment 10 bill fumerola freebsd_committer

2004-03-31 23:55:34 UTC

Responsible Changed
From-To: bmilekic->billf

reopen if the issue comes up again