vt: add comments for KDMKTONE ioctl implementation
ClosedPublic
Actions

Authored by emaste on Nov 1 2024, 4:29 PM.

Tags

None

Referenced Files

	F133447629: D47395.id145869.diff
	Sat, Oct 25, 9:12 PM

	Unknown Object (File)
	Thu, Oct 16, 10:12 AM

	Unknown Object (File)
	Tue, Oct 7, 3:08 AM

	Unknown Object (File)
	Sat, Oct 4, 1:02 AM

	Unknown Object (File)
	Sep 21 2025, 8:28 AM

	Unknown Object (File)
	Sep 9 2025, 10:49 PM

	Unknown Object (File)
	Sep 5 2025, 3:20 AM

	Unknown Object (File)
	Aug 13 2025, 7:11 AM

View All 49 Files

Subscribers

None

Details

Reviewers

imp

Commits

rG752c25c01c17: vt: add comments for KDMKTONE ioctl implementation
rGadba3c74209e: vt: add comments for KDMKTONE ioctl implementation

Summary

It appears we implemented the same KDMKTONE ioctl as Linux, which specifies the frequency as an 8254 PIT divisor for a 1.19MHz clock. Add a comment to vtterm_beep to avoid someone wanting to "fix" this in the future.

Also add an XXX comment that the period unit is supposed to be "timer ticks." Nothing in the base system uses this ioctl.

Diff Detail

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

emaste requested review of this revision.Nov 1 2024, 4:29 PM

emaste created this revision.

imp accepted this revision.Nov 2 2024, 4:53 AM

This revision is now accepted and ready to land.Nov 2 2024, 4:53 AM

Closed by commit rGadba3c74209e: vt: add comments for KDMKTONE ioctl implementation (authored by emaste). · Explain WhyNov 2 2024, 2:39 PM

This revision was automatically updated to reflect the committed changes.

emaste added a commit: rGadba3c74209e: vt: add comments for KDMKTONE ioctl implementation.

emaste added a commit: rG752c25c01c17: vt: add comments for KDMKTONE ioctl implementation.Nov 4 2024, 1:55 PM

Revision Contents
Changeset List

Path

Size

sys/

dev/

vt/

vt_core.c

8 lines

Diff 145869

View Options

vt: add comments for KDMKTONE ioctl implementationClosedPublicActions