Add sizes for free() in vio(4)

2019-12-18 Thread Frederic Cambus
Hi tech@, Here is a diff to add sizes for free() in vio(4). There is an existing allocsize variable tracking size of allocations, turns out we can pass it to free() in the error path. Comments? OK? Index: sys/dev/pv/if_vio.c === RC

Add sizes for free() in clct(4)

2019-12-20 Thread Frederic Cambus
Hi tech@, Here is a diff to add sizes for free() in clct(4). Similar diff to the ones previously sent for other audio drivers. Comments? OK? Index: sys/dev/pci/cs4281.c === RCS file: /cvs/src/sys/dev/pci/cs4281.c,v retrieving revis

[PATCH] octeon.html

2016-01-13 Thread Frederic Cambus
Hi tech@, Small diff to fix the link to the Portwell CAM-0100. Index: octeon.html === RCS file: /cvs/www/octeon.html,v retrieving revision 1.26 diff -u -p -u -p -r1.26 octeon.html --- octeon.html 17 Dec 2015 11:51:26 - 1.26

[PATCH] loongson.html

2016-01-27 Thread Frederic Cambus
Hi tech@, All links are currently dead on this page. This diff fixes them all. PMON pages aren't hosted on their own server anymore but only on Opsycon site now. Lemote recently updated their site, there is no English version anymore but thankfully they kept the old one on a subdomain. Link to

Some cleanups and tweaks for wc.1

2016-05-08 Thread Frederic Cambus
Hi tech@, Some cleanups and tweaks for wc.1: - Removed unnecessary string.h include - Changed 'format_and_print' argument type to int64_t and casting inside the function - Declaring 'print_counts', 'format_and_print', and 'cnt' as static - Remove unnecessary cast for NULL, and (void) casts from

wsconsctl.8: mention the display.font variable in EXAMPLES

2017-05-26 Thread Frederic Cambus
Hi tech@, As mentioned by miod@ here [1], wsconsctl(8) has a currently undocumented 'display.font' variable allowing to change the current font on framebuffer consoles. [1] http://undeadly.org/cgi?action=article&sid=20131023125815 Comments? OK? Index: sbin/wsconsctl/wsconsctl.8

Re: wsconsctl.8: mention the display.font variable in EXAMPLES

2017-05-27 Thread Frederic Cambus
On Fri, May 26, 2017 at 05:37:25PM +0100, Jason McIntyre wrote: > the examples are expected to be typed as shown. so i think "name" should > be changed to an actual font name, and the description adjusted to show that. > > otherwise it may be neccessary to document it more fully if it's not so >

wscons: add missing comments in ascii.h

2017-05-27 Thread Frederic Cambus
Hi tech@, Add missing comments for CAN and SUB, for consistency. Comments? OK? Index: sys/dev/wscons/ascii.h === RCS file: /cvs/src/sys/dev/wscons/ascii.h,v retrieving revision 1.5 diff -u -p -r1.5 ascii.h --- sys/dev/wscons/ascii.h

wsfont: remove lucida16x29.h and omron12x20?

2017-05-29 Thread Frederic Cambus
Hi, Those fonts are not compiled in by default, and I see no reason to keep them: lucida16x29 is a bad conversion from a truetype font, it is too bold which makes it unreadable and unusable. omron12x20 is a serif font, and is very difficult to read due to inconsistencies in character spacing, al

Re: Remove unused function from compress(1)

2017-05-29 Thread Frederic Cambus
On Tue, May 23, 2017 at 12:12:11PM -0400, Brian Callahan wrote: > clang says zclose is an unused function. Seems to check out. Commited, thanks!

pcdisplay(4): remove unused pcdisplay_mapchar_simple function

2017-05-29 Thread Frederic Cambus
Hi tech@, Here is a diff to remove the unused pcdisplay_mapchar_simple function in pcdisplay(4). It has been unused since this code was imported from NetBSD. Comments? OK? Index: sys/dev/ic/pcdisplay_subr.c === RCS file: /cvs/src/s

fortune(6): use proper bool types

2017-05-29 Thread Frederic Cambus
Hi tech@, Here is a diff to use proper bool types for fortune(6). Comments? OK? Index: games/fortune/fortune/fortune.c === RCS file: /cvs/src/games/fortune/fortune/fortune.c,v retrieving revision 1.55 diff -u -p -r1.55 fortune.c ---

pcdisplay(4): properly disable hardware cursor

2017-05-30 Thread Frederic Cambus
Hi tech@, Use the CD bit (Cursor Disable) in the cursor start register to properly disable hardware cursor. >From NetBSD. See commit message [1] and diff [2] for more information. Comments? OK? [1] http://cvsweb.netbsd.org/bsdweb.cgi/src/sys/dev/ic/pcdisplay_subr.c?rev=1.35&content-type=text/x

wsfont: remove qvss8x15, vt220l8x8, and vt220l8x10?

2017-05-30 Thread Frederic Cambus
Hi, This would be the last round of console fonts removal. Those fonts are not compiled in by default, and are smaller than the smallest font (bold8x16_iso1) we currently include. Comments? OK to remove them?

fortune(6): fix an infinite loop

2017-06-02 Thread Frederic Cambus
Hi tech@, When invoking fortune with the -l option (to get long dictums only), the program gets stuck in an infinite loop because fortlen() doesn't return the fortune length correctly. Comments? OK? Index: games/fortune/fortune/fortune.c ==

monop(6): correct number of players in the man page

2017-06-08 Thread Frederic Cambus
Hi tech@, The man page says monop monitors a game between 1 to 9 users, but the program enforces a range from 2 to 9. Comments? OK? Index: games/monop/monop.6 === RCS file: /cvs/src/games/monop/monop.6,v retrieving revision 1.15 dif

monop(6): drop the conditional shrt macro definitions

2017-06-10 Thread Frederic Cambus
Hi tech@, Drop the conditional shrt macro definitions and use short everywhere. Comments? OK? Index: games/monop/houses.c === RCS file: /cvs/src/games/monop/houses.c,v retrieving revision 1.10 diff -u -p -r1.10 houses.c --- games/mo

wsfont: remove iso7/pcvt encoding macros and entries

2017-06-13 Thread Frederic Cambus
Hi tech@, We do not support iso7 nor pcvt encoding, so remove macro definitions and commented entries. Comments? OK? Index: sys/dev/wscons/wsconsio.h === RCS file: /cvs/src/sys/dev/wscons/wsconsio.h,v retrieving revision 1.82 diff -

Re: wsfont: remove iso7/pcvt encoding macros and entries

2017-06-15 Thread Frederic Cambus
On Tue, Jun 13, 2017 at 11:29:59AM +, Miod Vallat wrote: > > > Hi tech@, > > > > We do not support iso7 nor pcvt encoding, so remove macro definitions > > and commented entries. > > If you do that, then you can probably optimize away > vga_valid_primary_font() in sys/dev/ic/vga.c as well. Ma

Re: wsfont: remove iso7/pcvt encoding macros and entries

2017-06-16 Thread Frederic Cambus
On Thu, Jun 15, 2017 at 01:52:07PM +0200, Frederic Cambus wrote: > On Tue, Jun 13, 2017 at 11:29:59AM +, Miod Vallat wrote: > > > > > Hi tech@, > > > > > > We do not support iso7 nor pcvt encoding, so remove macro definitions > > > and commented e

wsfontload(8): account for the WSFONT_NAME_SIZE change

2017-06-17 Thread Frederic Cambus
Hi tech@, Since WSFONT_NAME_SIZE was switched from 16 to 32, account for the change in wsfontload as well: tweak spacing and printf format string. Comments? OK? Index: usr.sbin/wsfontload/wsfontload.c === RCS file: /cvs/src/usr.sbin

inteldrm: add a handler for the WSDISPLAYIO_GINFO ioctl

2017-06-17 Thread Frederic Cambus
Hi tech@, Add a handler for the WSDISPLAYIO_GINFO ioctl in inteldrm, allowing to retrieve basic information about a framebuffer display. When running wsconsctl on a machine with inteldrm support, this now returns those additional fields: display.width=1600 display.height=900 display.depth=32 Co

Re: inteldrm: add a handler for the WSDISPLAYIO_GINFO ioctl

2017-06-18 Thread Frederic Cambus
On Sat, Jun 17, 2017 at 05:27:38PM -0400, Ted Unangst wrote: > > Add a handler for the WSDISPLAYIO_GINFO ioctl in inteldrm, allowing > > to retrieve basic information about a framebuffer display. [...] > why? what program will use this information? The reason I would like to add this handler is

wsfontload(8): close file descriptors on all exit paths

2017-06-18 Thread Frederic Cambus
Hi tech@, Close file descriptors on all exit paths in wsfontload(8). Comments? OK? Index: usr.sbin/wsfontload/wsfontload.c === RCS file: /cvs/src/usr.sbin/wsfontload/wsfontload.c,v retrieving revision 1.18 diff -u -p -r1.18 wsfontlo

radeondrm: add a handler for the WSDISPLAYIO_GINFO ioctl

2017-06-18 Thread Frederic Cambus
Hi tech@, Add a handler for the WSDISPLAYIO_GINFO ioctl in radeondrm, allowing to retrieve basic information about a framebuffer display. Same rationale as the diff I sent for inteldrm, and it can also be tested with wsconsctl which will return new display values for width, height and depth. Com

Re: monop(6): drop the conditional shrt macro definitions

2017-06-22 Thread Frederic Cambus
On Sat, Jun 10, 2017 at 05:11:31PM -0700, Philip Guenther wrote: > >> Drop the conditional shrt macro definitions and use short everywhere. > > > > Why not use uint8_t then? > > Isn't the point of the code to *not* use an unsigned type? Yes, those changes [1] were commited with the following com

adventure(6): remove unused functions

2017-06-23 Thread Frederic Cambus
Hi tech@, Remove unused confirm() and datime() functions. Those functions are unused and have been compiled out since 1998, it's time to let them go. Comments? OK? Index: games/adventure/io.c === RCS file: /cvs/src/games/adventure/

telnet(1): remove unnecessary #ifdefs

2017-07-05 Thread Frederic Cambus
Hi tech@, Remove unnecessary #ifdefs in telnet. No binary change. Comments? OK? Index: usr.bin/telnet/externs.h === RCS file: /cvs/src/usr.bin/telnet/externs.h,v retrieving revision 1.30 diff -u -p -r1.30 externs.h --- usr.bin/telne

lpt.4: make configuration lines match GENERIC files

2017-07-05 Thread Frederic Cambus
Hi tech@, Make configuration lines match GENERIC files. This adds amd64 and splits up alpha and i386. Comments? OK? Index: share/man/man4/lpt.4 === RCS file: /cvs/src/share/man/man4/lpt.4,v retrieving revision 1.7 diff -u -p -r1.7 l

backgammon(6): remove compiled out user count checks

2017-07-11 Thread Frederic Cambus
Hi tech@, The user count checks have been compiled out since 1998, let's get rid of them. Comments? OK? Index: games/backgammon/backgammon/main.c === RCS file: /cvs/src/games/backgammon/backgammon/main.c,v retrieving revision 1.23 d

amd(8): plog format string should be literal

2017-07-28 Thread Frederic Cambus
Hi tech@, plog format string should be literal. Comments? OK? Index: usr.sbin/amd/amd/ifs_ops.c === RCS file: /cvs/src/usr.sbin/amd/amd/ifs_ops.c,v retrieving revision 1.5 diff -u -p -r1.5 ifs_ops.c --- usr.sbin/amd/amd/ifs_ops.c 2

fsdb(8): remove erroneous h length modifier

2017-07-29 Thread Frederic Cambus
Hi tech@, Remove erroneous h length modifier, the argument has type 'int'. Comments? OK? Index: sbin/fsdb/fsdbutil.c === RCS file: /cvs/src/sbin/fsdb/fsdbutil.c,v retrieving revision 1.17 diff -u -p -r1.17 fsdbutil.c --- sbin/fsdb/f

Remove duplicated commented out include in files.i386

2017-08-02 Thread Frederic Cambus
Hi tech@, Remove duplicated commented out "dev/rasops/files.rasops" include. Comments? OK? Index: sys/arch/i386/conf/files.i386 === RCS file: /cvs/src/sys/arch/i386/conf/files.i386,v retrieving revision 1.233 diff -u -p -r1.233 file

Update inaccurate comment in rasops(9)

2017-08-04 Thread Frederic Cambus
Hi tech@, Update inaccurate comment: rasops_copycols() doesn't use bcopy() anymore, but either memmove() or slow_bcopy(). Comments? OK? Index: sys/dev/rasops/rasops.c === RCS file: /cvs/src/sys/dev/rasops/rasops.c,v retrieving revis

wscons(4): remove compiled out case statements

2017-08-09 Thread Frederic Cambus
Hi tech@, Remove some case statements which have been compiled out since 2000. Comments? OK? Index: sys/dev/wscons/wsemul_vt100.c === RCS file: /cvs/src/sys/dev/wscons/wsemul_vt100.c,v retrieving revision 1.35 diff -u -p -r1.35 wsem

wscons(4): set WS_KERNEL_COLATTR to WSATTR_HILIT in vt100 emulation

2017-08-12 Thread Frederic Cambus
Hi tech@, Currently, output originating from the kernel is displayed using WSCOL_WHITE on WSCOL_BLUE, which translates to NORMAL_WHITE on NORMAL_BLUE. When booting in BIOS mode, NORMAL_WHITE is 0xaa in text mode (which is very hard to read on a blue backgroud), and when framebuffer attaches,

Document the RI_ROTATE_CCW flag in rasops.9

2017-08-21 Thread Frederic Cambus
Hi tech@, Document the newly introduced RI_ROTATE_CCW flag. Comments? OK? Index: share/man/man9/rasops.9 === RCS file: /cvs/src/share/man/man9/rasops.9,v retrieving revision 1.17 diff -u -p -r1.17 rasops.9 --- share/man/man9/rasops.

Re: Update inaccurate comment in rasops(9)

2017-08-21 Thread Frederic Cambus
On Fri, Aug 04, 2017 at 11:31:00PM +0200, Frederic Cambus wrote: > Update inaccurate comment: rasops_copycols() doesn't use bcopy() > anymore, but either memmove() or slow_bcopy(). > > Comments? OK? Ping. Anyone? > Index: sys

Remove the no longer true CAVEATS section in rasops.9

2017-08-29 Thread Frederic Cambus
Hi tech@, Remove the CAVEATS section in rasops.9, which no longer holds truth. Comments? OK? Index: share/man/man9/rasops.9 === RCS file: /cvs/src/share/man/man9/rasops.9,v retrieving revision 1.18 diff -u -p -r1.18 rasops.9 --- sha

Headers cleanup + use getprogname() for test(1)

2016-05-14 Thread Frederic Cambus
Hi tech@, Headers cleanup + use getprogname() for test(1) Index: bin/test/test.c === RCS file: /cvs/src/bin/test/test.c,v retrieving revision 1.16 diff -u -p -r1.16 test.c --- bin/test/test.c 13 Jan 2016 13:13:04 - 1.16

Use proper bool types in /usr/games

2016-05-14 Thread Frederic Cambus
Hi tech@, Proper bool types for fortune(6) and monop(6). Index: games/fortune/fortune/fortune.c === RCS file: /cvs/src/games/fortune/fortune/fortune.c,v retrieving revision 1.55 diff -u -p -r1.55 fortune.c --- games/fortune/fortune/f

Some cleanups and tweaks for wc(1)

2016-09-03 Thread Frederic Cambus
Hi tech@, Some cleanups and tweaks for wc(1): - Removed unnecessary string.h include - Changed 'format_and_print' argument type to int64_t and casting inside the function - Declaring 'print_counts', 'format_and_print', and 'cnt' as static - Remove unnecessary cast for NULL, and (void) casts fro

Remove duplicated includes in stdlib.h and termios.h

2016-09-19 Thread Frederic Cambus
Hi tech@, This diff removes duplicated includes in stdlib.h and termios.h Comments? OK? Index: include/stdlib.h === RCS file: /cvs/src/include/stdlib.h,v retrieving revision 1.66 diff -u -p -r1.66 stdlib.h --- include/stdlib.h9

Remove reference to zaurus

2016-09-21 Thread Frederic Cambus
Hi tech@, This seems to be a leftover from the zaurus port removal. OK? Index: sbin/disklabel/Makefile === RCS file: /cvs/src/sbin/disklabel/Makefile,v retrieving revision 1.63 diff -u -p -r1.63 Makefile --- sbin/disklabel/Makefile

Remove more duplicated includes

2016-09-23 Thread Frederic Cambus
Hi tech@, A few remaining duplicated includes to remove. Comments? OK? Index: sys/arch/sparc64/include/asm.h === RCS file: /cvs/src/sys/arch/sparc64/include/asm.h,v retrieving revision 1.10 diff -u -p -r1.10 asm.h --- sys/arch/sparc

Re: Remove more duplicated includes

2016-09-25 Thread Frederic Cambus
On Fri, Sep 23, 2016 at 08:02:03PM +0200, Jeremie Courreges-Anglas wrote: > > > > A few remaining duplicated includes to remove. > > > > Comments? OK? > > ok except for ixgbe.h > > Better remove the unguarded #include, I think. Even if a kernel > configured without BPFILTER will not build... Th

cdce(4): Move include within the #if NBPFILTER directive

2016-09-25 Thread Frederic Cambus
Hi tech@, Move the net/bpf.h include within the #if NBPFILTER directive. Index: sys/dev/usb/if_cdce.c === RCS file: /cvs/src/sys/dev/usb/if_cdce.c,v retrieving revision 1.70 diff -u -p -r1.70 if_cdce.c --- sys/dev/usb/if_cdce.c

rasops.c: avoid calculating offset several times

2016-09-26 Thread Frederic Cambus
Hi tech@, Here is a diff to avoid calculating offset several times in rasops.c. This was done for a few functions already, but not all of them. Comments? OK? Index: sys/dev/rasops/rasops.c === RCS file: /cvs/src/sys/dev/rasops/rasop

Remove empty #ifdef and #ifndef blocks

2016-09-27 Thread Frederic Cambus
Hi tech@, It seems some #ifdef and #ifndef blocks are no longer necessary. Comments? OK? Index: sys/netinet/ip_spd.c === RCS file: /cvs/src/sys/netinet/ip_spd.c,v retrieving revision 1.90 diff -u -p -r1.90 ip_spd.c --- sys/netinet/i

share/misc: Complete 'airport' and 'inter.phone' for Poland

2016-10-01 Thread Frederic Cambus
Hi tech@, This diff adds missing airports and area codes for Poland, and fixes some erroneous entries. OK? Index: share/misc/airport === RCS file: /cvs/src/share/misc/airport,v retrieving revision 1.56 diff -u -p -r1.56 airport ---

wsfont.c: remove reference to non existing font

2016-10-05 Thread Frederic Cambus
Hi tech@, Include file with font data (courier11x18.h) was removed from NetBSD due to licensing concerns [1] before wsfont was imported into OpenBSD. I renumbered the cookie values for consistency, and verified that no code in tree used hardcoded cookie values. Comments? OK? [1] http://cvsweb.n

cdce(4): Remove zaurus specific code

2016-10-07 Thread Frederic Cambus
Hi tech@, It seems there are still some leftovers from the zaurus port removal. Comments? OK? Index: sys/dev/usb/if_cdce.c === RCS file: /cvs/src/sys/dev/usb/if_cdce.c,v retrieving revision 1.71 diff -u -p -r1.71 if_cdce.c --- sys/d

Re: opencvs - correct type of p in annotate

2016-10-11 Thread Frederic Cambus
On Wed, Jun 22, 2016 at 12:24:03PM +0200, Joris Vink wrote: > Use the correct type for p. > > Index: annotate.c > === > RCS file: /cvs/src/usr.bin/cvs/annotate.c,v > retrieving revision 1.65 > diff -u -p -r1.65 annotate.c > --- annot

Re: cdce(4): Remove zaurus specific code

2016-10-11 Thread Frederic Cambus
On Fri, Oct 07, 2016 at 06:02:35PM +0200, Mark Kettenis wrote: > > It seems there are still some leftovers from the zaurus port removal. > > > > Comments? OK? > > Not ok. This is support for the zaurus as a usb device attached to an My mistake, thanks for the clarification. > OpenBSD machine.

Re: opencvs - use correct size when creating h_table

2016-10-12 Thread Frederic Cambus
On Wed, Jun 22, 2016 at 12:23:38PM +0200, Joris Vink wrote: > Don't allocate the length of a pointer but rather the > entire size of the struct hash_head data structure > when creating the h_table array. > RCS file: /cvs/src/usr.bin/cvs/hash.c,v > > - htable->h_table = xcalloc(hsize, sizeof(s

Re: [patch] opencvs rcsnum_free()

2016-10-13 Thread Frederic Cambus
On Fri, Jun 24, 2016 at 11:22:06AM +0200, Joris Vink wrote: > On Fri, Jun 24, 2016 at 01:58:20PM +0800, Michael W. Bombardieri wrote: > > If people are interested in opencvs diffs again, sharing a > > rcsnum_free()->free() > > clean-up item. Note that rcs(1) also has a version of rcsnum_free() wh

Re: opencvs - fix update -r and -A

2016-10-14 Thread Frederic Cambus
On Fri, Oct 14, 2016 at 09:24:03AM -0600, Todd C. Miller wrote: > On Fri, 14 Oct 2016 16:17:45 +0200, Joris Vink wrote: > > > In certain cases update -r and update -A would not properly set or reset > > the sticky tag for file(s). This patch fixes that problem. > > After reading through the appli

ykbec(4): Add support for lid state detection

2016-11-12 Thread Frederic Cambus
Hi tech@, This is an attempt at adding lid state detection in ykbec(4). Tested on a Lemote Yeeloong 8101B. Comments? OK? Index: share/man/man4/man4.loongson/ykbec.4 === RCS file: /cvs/src/share/man/man4/man4.loongson/ykbec.4,v retri

Remove useless #ifndef in atc(6) and pom(6)

2016-12-01 Thread Frederic Cambus
Hi tech@, M_PI is always defined, so we can drop those directives. Index: games/atc/def.h === RCS file: /cvs/src/games/atc/def.h,v retrieving revision 1.5 diff -u -p -r1.5 def.h --- games/atc/def.h 3 Jun 2003 03:01:38 -

Harmonize battery state thresholds accross platforms

2016-12-01 Thread Frederic Cambus
Hi tech@, Use the same values hardcoded in sys/dev/acpi/acpi.c where possible. Comments? OK? Index: sys/arch/loongson/dev/kb3310.c === RCS file: /cvs/src/sys/arch/loongson/dev/kb3310.c,v retrieving revision 1.21 diff -u -p -r1.21 kb

rasops(9): Remove rasops2 functions?

2016-12-11 Thread Frederic Cambus
Hi tech@, Unless I overlooked something, nothing seems to use the rasops2 (2-bit color depth) functions. It seems unlikely that any future new platform will ever need them. Should we remove them? Comments? OK? Index: sys/conf/files ==

Provide the "machdep.lidsuspend" sysctl on Loongson

2016-12-14 Thread Frederic Cambus
Hi tech@, Here is a diff to add the "machdep.lidsuspend" sysctl on Loongson. This is another required step to activate suspend on lid close on the Lemote Yeeloong. Comments? OK? Index: etc/etc.loongson/sysctl.conf === RCS file: /cv

Some tweaks for smfb(4)

2016-12-19 Thread Frederic Cambus
Hi tech@, Here is a diff with some tweaks for smfb(4): - Display resolution and color depth when attaching - Do not hardcode 'wantrows' and 'wantcols' when calling rasops_init Comments? OK? Index: sys/arch/loongson/dev/smfb.c === R

Re: Some tweaks for smfb(4)

2016-12-27 Thread Frederic Cambus
On Mon, Dec 19, 2016 at 12:21:54PM +0100, Mark Kettenis wrote: > > Here is a diff with some tweaks for smfb(4): > > > > - Display resolution and color depth when attaching > > We should try to do so in a uniform way though. None of our drivers > print the words "frame buffer". Most of them onl

Add sizes for free() in the VIA PadLock driver

2017-11-12 Thread Frederic Cambus
Hi tech@, Add sizes for free() in the VIA PadLock driver. Comments? OK? Index: sys/arch/amd64/amd64/via.c === RCS file: /cvs/src/sys/arch/amd64/amd64/via.c,v retrieving revision 1.24 diff -u -p -r1.24 via.c --- sys/arch/amd64/amd64/

Remove unnecessary includes (i386 VIA PadLock driver)

2017-12-08 Thread Frederic Cambus
Hi tech@, This diff remove unnecessary includes in the i386 version of the VIA PadLock driver. Comments? OK? Index: sys/arch/i386/i386/via.c === RCS file: /cvs/src/sys/arch/i386/i386/via.c,v retrieving revision 1.37 diff -u -p -r1.3

Add sizes for free() in the AMD PowerNow! K8 driver

2017-12-08 Thread Frederic Cambus
Hi tech@, Add sizes for free() in the AMD PowerNow! K8 driver. Comments? OK? Index: sys/arch/amd64/amd64/powernow-k8.c === RCS file: /cvs/src/sys/arch/amd64/amd64/powernow-k8.c,v retrieving revision 1.26 diff -u -p -r1.26 powernow-k

Add sizes for free() in libsa for amd64 and i386

2017-12-10 Thread Frederic Cambus
Hi tech@, Add sizes for free() in libsa for amd64 and i386. Comments? OK? Index: sys/arch/amd64/stand/libsa/diskprobe.c === RCS file: /cvs/src/sys/arch/amd64/stand/libsa/diskprobe.c,v retrieving revision 1.21 diff -u -p -r1.21 diskp

Change bcopy to memcpy (i386 VIA PadLock driver)

2017-12-13 Thread Frederic Cambus
Hi tech@, This diff changes bcopy to memcpy in the i386 version of the VIA PadLock driver. It was done a while ago in the amd64 version of the driver. Comments? OK? Index: sys/arch/i386/i386/via.c === RCS file: /cvs/src/sys/arch/i38

rasops(9): Remove dead assignment

2017-12-22 Thread Frederic Cambus
Hi tech@, Now that we call rasops_putchar_rotated(), we don't need ri anymore. Comments? OK? Index: sys/dev/rasops/rasops.c === RCS file: /cvs/src/sys/dev/rasops/rasops.c,v retrieving revision 1.48 diff -u -p -r1.48 rasops.c --- sys

Add sizes for free() in glxsb(4)

2017-12-24 Thread Frederic Cambus
Hi tech@, Add sizes for free() for the Geode LX Security Block crypto accelerator. Comments? OK? Index: sys/arch/i386/pci/glxsb.c === RCS file: /cvs/src/sys/arch/i386/pci/glxsb.c,v retrieving revision 1.33 diff -u -p -r1.33 glxsb.c

cpu.4: VIA amd64 compatible CPUs support Enhanced SpeedStep

2017-12-28 Thread Frederic Cambus
Hi tech@, VIA amd64 compatible CPUs support Enhanced SpeedStep, so reflect that in cpu.4. On my VIA Nano U3500: cpu0: VIA Nano U3500@1000MHz, 997.66 MHz cpu0: FPU,VME,DE,PSE,TSC,MSR,PAE,MCE,CX8,APIC,SEP,MTRR,PGE,MCA,CMOV,PAT, CFLUSH,ACPI,MMX,FXSR,SSE,SSE2,SS,TM,PBE,SSE3,MWAIT,VMX,EST,TM2,SSSE3,

Re: Remove unnecessary includes (i386 VIA PadLock driver)

2017-12-28 Thread Frederic Cambus
On Fri, Dec 08, 2017 at 12:39:36PM +0100, Frederic Cambus wrote: > This diff remove unnecessary includes in the i386 version of the > VIA PadLock driver. > > Comments? OK? Ping. Anyone? This is the last step in syncing amd64 and i386 versions of the VIA PadLock driver.

Add sizes for free() in wscons(4)

2018-01-16 Thread Frederic Cambus
Hi tech@, Add sizes for free() in wscons(4). Comments? OK? Index: sys/dev/wscons/wsdisplay.c === RCS file: /cvs/src/sys/dev/wscons/wsdisplay.c,v retrieving revision 1.129 diff -u -p -r1.129 wsdisplay.c --- sys/dev/wscons/wsdisplay.c

rasops(9): Fix underline rotation on CCW rotated screens

2018-01-19 Thread Frederic Cambus
Hi tech@, Fix underline rotation on CCW (quarter counter-clockwise) rotated screens. Currently, the "underline" is actually drawn above text. Comments? OK? Index: sys/dev/rasops/rasops.c === RCS file: /cvs/src/sys/dev/rasops/rasops

Ansify cpu_sysctl() on mips64 platforms

2017-01-05 Thread Frederic Cambus
Hi tech@, Here is a diff to ansify cpu_sysctl() on mips64 platforms. Comments? OK? Index: sys/arch/loongson/loongson/machdep.c === RCS file: /cvs/src/sys/arch/loongson/loongson/machdep.c,v retrieving revision 1.70 diff -u -p -r1.70

Display color depth when attaching inteldrm and radeondrm

2017-01-06 Thread Frederic Cambus
Hi tech@, Here's a diff to display color depth alongside resolution when attaching inteldrm and radeondrm, using the same scheme as efifb(4). This is the first step in trying to have all frame buffer drivers display resolution and depth the same way. Tested only with inteldrm. On this machine, i

rasops(9): Use proper RGB values for the ANSI color palette

2017-01-06 Thread Frederic Cambus
Hi tech@, Here is a diff to use proper RGB values for the ANSI color palette in rasops(9). Comments? OK? Index: sys/dev/rasops/rasops.c === RCS file: /cvs/src/sys/dev/rasops/rasops.c,v retrieving revision 1.44 diff -u -p -r1.44 raso

Re: Some tweaks for smfb(4)

2017-01-09 Thread Frederic Cambus
On Tue, Dec 27, 2016 at 11:39:26PM +0100, Frederic Cambus wrote: > > > > Here is a diff with some tweaks for smfb(4): > > > > > > - Display resolution and color depth when attaching > > > > We should try to do so in a uniform way though. None of our

Use explicit_bzero() to wipe copybuffer at console logout

2017-01-10 Thread Frederic Cambus
Hi tech@, Use explicit_bzero() to wipe copybuffer when logging out of the console. Comments? OK? Index: sys/dev/wscons/wsdisplay.c === RCS file: /cvs/src/sys/dev/wscons/wsdisplay.c,v retrieving revision 1.125 diff -u -p -r1.125 wsdi

Ansify cpu_sysctl() on alpha, arm, arm64, luna88k and sparc64

2017-01-12 Thread Frederic Cambus
Hi tech@, Here is a diff to ansify cpu_sysctl() where not previously done. Comments? OK? Index: sys/arch/alpha/alpha/machdep.c === RCS file: /cvs/src/sys/arch/alpha/alpha/machdep.c,v retrieving revision 1.176 diff -u -p -r1.176 mach

wsfont.c: remove sony8x16 and sony12x24?

2017-01-12 Thread Frederic Cambus
Hi tech@, Most fonts in wsfont are not compiled in by default, and while some are worth keeping and using, I would like to propose deleting some of them in order to make space for larger sized fonts. The Sony fonts are Serif, which doesn't work very well for monospaced bitmap fonts intended for c

Harmonize frame buffer drivers information display

2017-01-14 Thread Frederic Cambus
Hi tech@, Harmonize the way frame buffer drivers display resolution and color depth when attaching. Comments? OK? Index: sys/arch/alpha/tc/cfb.c === RCS file: /cvs/src/sys/arch/alpha/tc/cfb.c,v retrieving revision 1.23 diff -u -p -r

Re: wsfont.c: remove sony8x16 and sony12x24?

2017-01-17 Thread Frederic Cambus
On Thu, Jan 12, 2017 at 01:39:47PM +0100, Frederic Cambus wrote: > Most fonts in wsfont are not compiled in by default, and while some are > worth keeping and using, I would like to propose deleting some of them > in order to make space for larger sized fonts. > > The Sony fonts a

Document the RI_CLEARMARGINS flag in rasops.9

2017-01-17 Thread Frederic Cambus
Hi tech@, The RI_CLEARMARGINS flag is used on sparc64 but isn't documented in the rasops man page. Comments? OK? Index: share/man/man9/rasops.9 === RCS file: /cvs/src/share/man/man9/rasops.9,v retrieving revision 1.16 diff -u -p -r1

Enable building wsfontload on Loongson

2017-01-18 Thread Frederic Cambus
Hi tech@, Here is a diff to enable building wsfontload on Loongson. Builds and works correctly, allowing loading and using custom fonts. Comments? OK? Index: distrib/sets/lists/base/md.loongson === RCS file: /cvs/src/distrib/sets/l

Fix broken example link in packages.7

2017-02-28 Thread Frederic Cambus
Hi tech@, In packages.7, PKG_PATH points to a broken link. This diff fixes it. Comments? OK? Index: share/man/man7/packages.7 === RCS file: /cvs/src/share/man/man7/packages.7,v retrieving revision 1.40 diff -u -p -r1.40 packages.7 -

Re: Fix broken example link in packages.7

2017-03-01 Thread Frederic Cambus
On Tue, Feb 28, 2017 at 09:14:49PM +, Stuart Henderson wrote: > On 2017/02/28 20:28, Frederic Cambus wrote: > > Hi tech@, > > > > In packages.7, PKG_PATH points to a broken link. This diff fixes it. > > https is going to be horrible for pkg_add -u... Following

Fix style.9 offenders in includes

2017-03-07 Thread Frederic Cambus
Hi tech@, Here is a diff to fix style.9 offenders in includes. Prototypes should not have variable names associated with the types. Comments? OK? Index: bsd_auth.h === RCS file: /cvs/src/include/bsd_auth.h,v retrieving revision 1.1

Mention maximum number of allowed fonts in wsfontload.8

2017-03-07 Thread Frederic Cambus
Hi tech@, WSDISPLAY_MAXFONTCOUNT macro was introduced in sys/dev/wscons/wsconsio.h (revision 1.75) to limit the number of fonts that can be loaded. Reflect that in the man page as well. Comments? OK? Index: usr.sbin/wsfontload/wsfontload.8 ===

Re: Fix broken example link in packages.7

2017-03-27 Thread Frederic Cambus
On Wed, Mar 01, 2017 at 11:16:57PM +, Stuart Henderson wrote: > > -# export PKG_PATH=ftp://ftp.openbsd.org/pub/OpenBSD/5.2/packages/i386/ > > +# export PKG_PATH=http://ftp.openbsd.org/pub/OpenBSD/%c/packages/%a/ > > Hmm - if you're going to trust autodetection of snapshots vs release > anyway

shutdown(8): warn format string should be literal

2017-04-03 Thread Frederic Cambus
Hi tech@, warn format string should be literal. Comments? OK? Index: sbin/shutdown/shutdown.c === RCS file: /cvs/src/sbin/shutdown/shutdown.c,v retrieving revision 1.45 diff -u -p -r1.45 shutdown.c --- sbin/shutdown/shutdown.c1

fsck_ffs(8): remove always false comparison

2017-04-09 Thread Frederic Cambus
Hi tech@, Remove always false comparison: inosused type is ino_t, which is unsigned. While there, fix a whitespace issue. Comments? OK? Index: sbin/fsck_ffs/pass1.c === RCS file: /cvs/src/sbin/fsck_ffs/pass1.c,v retrieving revision

adventure(6): clean up disabled declaration

2017-04-10 Thread Frederic Cambus
Hi tech@, Clean up disabled declaration in the text struct. Comments? OK? Index: games/adventure/hdr.h === RCS file: /cvs/src/games/adventure/hdr.h,v retrieving revision 1.15 diff -u -p -r1.15 hdr.h --- games/adventure/hdr.h 8

relayd(8): convert explicit_bzero() + free() to freezero()

2017-04-10 Thread Frederic Cambus
Hi tech@, Convert explicit_bzero() + free() to freezero(). Comments? OK? Index: usr.sbin/relayd/relayd.c === RCS file: /cvs/src/usr.sbin/relayd/relayd.c,v retrieving revision 1.165 diff -u -p -r1.165 relayd.c --- usr.sbin/relayd/rel

Re: relayd(8): convert explicit_bzero() + free() to freezero()

2017-04-11 Thread Frederic Cambus
On Mon, Apr 10, 2017 at 05:43:33PM +0200, Claudio Jeker wrote: > > --- usr.sbin/relayd/relayd.c24 Jan 2017 10:49:14 - 1.165 > > +++ usr.sbin/relayd/relayd.c10 Apr 2017 15:06:38 - > > @@ -550,8 +550,7 @@ purge_key(char **ptr, off_t len) > > if (key == NULL || len ==

fdisk: Add partition type GUIDs for NetBSD

2023-02-11 Thread Frederic Cambus
Hi tech@, Here is a diff to add partition type GUIDs for NetBSD in fdisk. The values come from NetBSD's sys/sys/disklabel_gpt.h header. Before the change: Disk: sd1 Usable LBA: 34 to 937703054 [937703088 Sectors] #: type [ start: size ] ---

<    1   2