From nobody Sun Nov 03 17:45:12 2024 X-Original-To: dev-commits-src-all@mlmmj.nyi.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mlmmj.nyi.freebsd.org (Postfix) with ESMTP id 4XhMSS4d8Gz5cWdC; Sun, 03 Nov 2024 17:45:12 +0000 (UTC) (envelope-from git@FreeBSD.org) Received: from mxrelay.nyi.freebsd.org (mxrelay.nyi.freebsd.org [IPv6:2610:1c1:1:606c::19:3]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256 client-signature RSA-PSS (4096 bits) client-digest SHA256) (Client CN "mxrelay.nyi.freebsd.org", Issuer "R10" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id 4XhMSS3z6lz4hgR; Sun, 3 Nov 2024 17:45:12 +0000 (UTC) (envelope-from git@FreeBSD.org) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=freebsd.org; s=dkim; t=1730655912; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding; bh=x6Q0GuBMlmOLQ4DqBJas2wyDuy7cqzB3qwZDyzmm+Og=; b=NOxqRXJ4YBXH5e2sNU0gMXU4PRQzZiqotX8aMS/fB/Q7QWF14U/swFzKqrO0C2iIJn4v5I PDL7v+RSJKPOGIrliUUuQFOK9mMrJ9iFRYCyR07Zng/2ciAw66U2I0NflTKEfSgI8Ap/jS S7FD7WQQNiPHG1gDWMMXJwTORzdjwVvDHHKbwIDryScwuSxLWfTSa69S4adzT0bFgWXnVK QizSiv+9982zxEFpmmru3sViZYZHDokBspwRa3BxnNi9W/M+hRNvq5l1fRM1g282IZ8l/q 7HpPrZnGHHCtq/80rPrE7aFa2CvLhtz7rQ0Vp/cRGWi2mOAZAw/7E4sit2u61g== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=freebsd.org; s=dkim; t=1730655912; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding; bh=x6Q0GuBMlmOLQ4DqBJas2wyDuy7cqzB3qwZDyzmm+Og=; b=ONWfkGJozY8lJGgCo69FHr1U85vd/aHq/Uqha+Ife+WfpY00KIyuHv2/DVS3WcyjTmqTPZ 9Rw4X4a6Nh/LZdNt+UUvKVVjE3WSK7i3Nk0/JfRzYbF4OudgJiEwM2OML2N1ROZr3wIvoh ijmgw7w8/gaY80hNwJQaLfFDczdxZ4uGP+WvV2mwqxTaM7PFcv8HF0+nItM3Dg6A3vIELT tp+otX2U/Nhp5PwiOGMNM+Tf1hmteBPCGRNXe1GiFk5YBD8bkLdwL47HtmMJ/jizJoy69J sCIVJb4lhqsEQlS+jEH/BtQBjkImDKq51N1TOjvhZX91NgW16jLdQxxzszdq7Q== ARC-Authentication-Results: i=1; mx1.freebsd.org; none ARC-Seal: i=1; s=dkim; d=freebsd.org; t=1730655912; a=rsa-sha256; cv=none; b=h94z5Xhp7zfL6ERY5nyLnn8VD3pw/0ltnyA91xBPaNzfbatlXYDf+nHwD6YCYOPf3/GHv8 4JB7Mo1t0KBAiD+icATPdG2/R2pPcqBtK9Dfo7s2ICS5vQcfXtiq5Jo+CBWIUrb4CL2ZkX Bm6P+smdQTc8px3Z1s0E3gw4P8VKkx89d1T6GGhxxGdG8aUVb1aBzPIJ5ColEYjzV+54lN diuCSi5UkjqtOMKv5XObEQ5bMs3tWmNZ27g9pzg+Crz4nngrn2vMn983PAoui9YCjBJFGO k8fDB9LBttWwO0Jai0lz9873poQRKcQQ1wGKFlvVTM/Ao151hoUgKKoqClQPDg== Received: from gitrepo.freebsd.org (gitrepo.freebsd.org [IPv6:2610:1c1:1:6068::e6a:5]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256) (Client did not present a certificate) by mxrelay.nyi.freebsd.org (Postfix) with ESMTPS id 4XhMSS3ZLhzyst; Sun, 3 Nov 2024 17:45:12 +0000 (UTC) (envelope-from git@FreeBSD.org) Received: from gitrepo.freebsd.org ([127.0.1.44]) by gitrepo.freebsd.org (8.18.1/8.18.1) with ESMTP id 4A3HjCQW004152; Sun, 3 Nov 2024 17:45:12 GMT (envelope-from git@gitrepo.freebsd.org) Received: (from git@localhost) by gitrepo.freebsd.org (8.18.1/8.18.1/Submit) id 4A3HjC4S004149; Sun, 3 Nov 2024 17:45:12 GMT (envelope-from git) Date: Sun, 3 Nov 2024 17:45:12 GMT Message-Id: <202411031745.4A3HjC4S004149@gitrepo.freebsd.org> To: src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-branches@FreeBSD.org From: Ed Maste Subject: git: da797702b0fd - stable/13 - vt: Fix frequency calcuation for bell List-Id: Commit messages for all branches of the src repository List-Archive: https://lists.freebsd.org/archives/dev-commits-src-all List-Help: List-Post: List-Subscribe: List-Unsubscribe: X-BeenThere: dev-commits-src-all@freebsd.org Sender: owner-dev-commits-src-all@FreeBSD.org MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit X-Git-Committer: emaste X-Git-Repository: src X-Git-Refname: refs/heads/stable/13 X-Git-Reftype: branch X-Git-Commit: da797702b0fdd255efc4a87934e0b9df5d2b07ed Auto-Submitted: auto-generated The branch stable/13 has been updated by emaste: URL: https://cgit.FreeBSD.org/src/commit/?id=da797702b0fdd255efc4a87934e0b9df5d2b07ed commit da797702b0fdd255efc4a87934e0b9df5d2b07ed Author: Warner Losh AuthorDate: 2021-11-03 21:55:55 +0000 Commit: Ed Maste CommitDate: 2024-11-03 17:44:50 +0000 vt: Fix frequency calcuation for bell 386BSD provided a MD function sysbeep. This took two arguments (pitch and period). Pitch was jammed into the PIT's divisor directly (which means the argument was expected to sound a tone at '1193182 / pitch' Hz). FreeBSD inherited this interface. In commit e46598588587 (svn 177642, Mar 26 2008), phk changed this function to take a tone to sound in hz. He converted all in-tree instances of 1193182 / hz to just hz (and kept the few misguided folks that passed hz directly unchanged -- this was part of what motivated the change). He converted the places where we pre-computed the 8254 divisor from being pitch to 1193182 / pitch (since that converts the divisor to the frequency and the interfaces that were exposed to userland exposed it in these units in places, continuing the tradition inherited from SCO System V/386 Unix in spots). In 2009, Ed Shouten was contracted by the FreeBSD Foundation to write / finish newcons. This work was done in perforce and was imported into subversion in user/ed/newcons in revision 199072 (https://svnweb.freebsd.org/base?view=revision&revision=199072) which was later imported into FreeBSD by ray@ (Aleksandr Rybalko). From that earliest import into svn import to this date, we ring the bell with: sysbeep(1193182 / VT_BELLPITCH, VT_BELLDURATION); where VT_BELLPITCH was defined to be 800. This results in a bell frequency of 1491Hz, more or less today. This is similar to the frequency that syscons and pcvt used (1493Hz and 1500Hz respectively). This in turn was inherited from 386BSD, it seems, which used the hard coded value 0x31b which is 795 -> 1500Hz. This '800' was intended to be the bell tone (eg 800Hz) and this interface was one that wasn't converted. The most common terminal prior to the rise of PCs was the VT100, which had an approximately 800Hz bell. Ed Shouten has confirmed that the original intent was 800Hz and changing this was overlooked after the change to -current was made. This restors that original intent and makes the bell less obnoxious in the process. Reviewed by: des, adrian Differential Revision: https://reviews.freebsd.org/D32594 Sponsored by: Netflix (cherry picked from commit ba48d52ca6c867559156dd916631f9ac47abe80f) This change was accidentally reverted in 80f21bb039ce. (cherry picked from commit 2416be588ea113cc06b924ed85861ed3bc391fe0) (cherry picked from commit 1c9f1cb4f0a71bef37796a8ba139b86cc716ee88) --- sys/dev/vt/vt_core.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/sys/dev/vt/vt_core.c b/sys/dev/vt/vt_core.c index b27628d17732..80d9ae1f22c2 100644 --- a/sys/dev/vt/vt_core.c +++ b/sys/dev/vt/vt_core.c @@ -119,7 +119,7 @@ static const struct terminal_class vt_termclass = { /* Bell pitch/duration. */ #define VT_BELLDURATION (SBT_1S / 20) -#define VT_BELLPITCH (1193182 / 800) /* Approx 1491Hz */ +#define VT_BELLPITCH 800 #define VT_UNIT(vw) ((vw)->vw_device->vd_unit * VT_MAXWINDOWS + \ (vw)->vw_number)