CVS commit: src/usr.sbin/envstat

2024-10-06 Thread Roland Illig
Module Name:src Committed By: rillig Date: Sun Oct 6 19:08:34 UTC 2024 Modified Files: src/usr.sbin/envstat: config.c Log Message: envstat: remove redundant local variables This fixes the lint warnings about "effectively discards 'const'". No binary change. To generat

CVS commit: src/usr.sbin/envstat

2024-10-06 Thread Roland Illig
Module Name:src Committed By: rillig Date: Sun Oct 6 19:08:34 UTC 2024 Modified Files: src/usr.sbin/envstat: config.c Log Message: envstat: remove redundant local variables This fixes the lint warnings about "effectively discards 'const'". No binary change. To generat

CVS commit: src/usr.sbin/envstat

2023-06-18 Thread Rin Okuyama
Module Name:src Committed By: rin Date: Mon Jun 19 03:03:11 UTC 2023 Modified Files: src/usr.sbin/envstat: envstat.c Log Message: Silence wrong maybe-uninitialized raised by GCC/x86_64 10.4.0 -Os. To generate a diff of this commit: cvs rdiff -u -r1.103 -r1.104 src/usr.sb

CVS commit: src/usr.sbin/envstat

2023-06-18 Thread Rin Okuyama
Module Name:src Committed By: rin Date: Mon Jun 19 03:03:11 UTC 2023 Modified Files: src/usr.sbin/envstat: envstat.c Log Message: Silence wrong maybe-uninitialized raised by GCC/x86_64 10.4.0 -Os. To generate a diff of this commit: cvs rdiff -u -r1.103 -r1.104 src/usr.sb

Re: CVS commit: src/usr.sbin/envstat

2012-12-17 Thread Christos Zoulas
In article , Iain Hibbert wrote: > >I didn't understand it, in any case.. can you explain why, in C, when >exit() explicitly closes all open file descriptors and releases any memory >etc (see a detailed list in exit(3) and _exit(3)), there might be a >benefit in doing so formally before calling e

Re: CVS commit: src/usr.sbin/envstat

2012-12-17 Thread Iain Hibbert
On Sun, 16 Dec 2012, David Holland wrote: > On Fri, Dec 14, 2012 at 06:18:57PM +0100, Marc Balmer wrote: > > proper resource management is a good thing - if the code continues > > to run. In this case, where the program exits, there no benefit > > from freeing up memory etc. > > Buncombe. is t

Re: CVS commit: src/usr.sbin/envstat

2012-12-16 Thread David Holland
On Fri, Dec 14, 2012 at 06:18:57PM +0100, Marc Balmer wrote: > Similarly it just isn't worth trying to free resources prior > to program exit. Have you ever waited while a big C++ program > runs all its destructors, paging in code and data just to exit! > >>> > >>> Depends, it

Re: CVS commit: src/usr.sbin/envstat

2012-12-14 Thread Jukka Ruohonen
On Fri, Dec 14, 2012 at 06:18:57PM +0100, Marc Balmer wrote: > > Ironically, proper ressource managements tends to make it much easier to > > fulfill at least two of the three items... > > proper resource management is a good thing - if the code continues to run. > In this case, where the program

Re: CVS commit: src/usr.sbin/envstat

2012-12-14 Thread Marc Balmer
Am 14.12.2012 um 18:05 schrieb Joerg Sonnenberger : > On Fri, Dec 14, 2012 at 05:51:44PM +0100, Marc Balmer wrote: >> >> >> Am 14.12.2012 um 17:07 schrieb Joerg Sonnenberger : >> >>> On Fri, Dec 14, 2012 at 08:59:37AM +, David Laight wrote: Similarly it just isn't worth trying to free

Re: CVS commit: src/usr.sbin/envstat

2012-12-14 Thread Joerg Sonnenberger
On Fri, Dec 14, 2012 at 05:51:44PM +0100, Marc Balmer wrote: > > > Am 14.12.2012 um 17:07 schrieb Joerg Sonnenberger : > > > On Fri, Dec 14, 2012 at 08:59:37AM +, David Laight wrote: > >> Similarly it just isn't worth trying to free resources prior > >> to program exit. Have you ever waited

Re: CVS commit: src/usr.sbin/envstat

2012-12-14 Thread Marc Balmer
Am 14.12.2012 um 17:07 schrieb Joerg Sonnenberger : > On Fri, Dec 14, 2012 at 08:59:37AM +, David Laight wrote: >> Similarly it just isn't worth trying to free resources prior >> to program exit. Have you ever waited while a big C++ program >> runs all its destructors, paging in code and dat

Re: CVS commit: src/usr.sbin/envstat

2012-12-14 Thread Joerg Sonnenberger
On Fri, Dec 14, 2012 at 08:59:37AM +, David Laight wrote: > Similarly it just isn't worth trying to free resources prior > to program exit. Have you ever waited while a big C++ program > runs all its destructors, paging in code and data just to exit! Depends, it makes tools like valgrind a lot

Re: CVS commit: src/usr.sbin/envstat

2012-12-14 Thread David Laight
On Thu, Dec 13, 2012 at 08:45:02PM -0800, Paul Goyette wrote: > It seems that the following commit has introduced a regression for the > dev/sysmon/t_swsensor atf tests (for details, see test results at > http://screamer.whooppee.com/amd64-results/4722_1_atf.html) > > >Module Name:src > >Com

Re: CVS commit: src/usr.sbin/envstat

2012-12-13 Thread Paul Goyette
It seems that the following commit has introduced a regression for the dev/sysmon/t_swsensor atf tests (for details, see test results at http://screamer.whooppee.com/amd64-results/4722_1_atf.html) Module Name:src Committed By: christos Date: Thu Dec 13 20:06:42 UTC 2012 Modifi

Re: CVS commit: src/usr.sbin/envstat

2012-12-13 Thread Paul Goyette
On Fri, 14 Dec 2012, Jukka Ruohonen wrote: On Thu, Dec 13, 2012 at 11:53:24AM -0800, Paul Goyette wrote: While we're making sure to free() things, should we not also defend against memory leaks in the case where an option is used more than once? While we are here, I wonder why sysmon(9) does

Re: CVS commit: src/usr.sbin/envstat

2012-12-13 Thread Warner Losh
On Dec 13, 2012, at 4:30 PM, John Nemeth wrote: > On Mar 31, 12:00am, Paul Goyette wrote: > } > } > Module Name:src > } > Committed By: christos > } > Date: Thu Dec 13 19:31:25 UTC 2012 > } > > } > Modified Files: > } > src/usr.sbin/envstat: envstat.c > } > } > Log Messag

Re: CVS commit: src/usr.sbin/envstat

2012-12-13 Thread John Nemeth
On Mar 31, 12:00am, Paul Goyette wrote: } } > Module Name:src } > Committed By: christos } > Date: Thu Dec 13 19:31:25 UTC 2012 } > } > Modified Files: } > src/usr.sbin/envstat: envstat.c } } > Log Message: } > PR/47316: Henning Petersen: Memory leak in envstat with config

Re: CVS commit: src/usr.sbin/envstat

2012-12-13 Thread Jukka Ruohonen
On Thu, Dec 13, 2012 at 11:53:24AM -0800, Paul Goyette wrote: > While we're making sure to free() things, should we not also defend > against memory leaks in the case where an option is used more than once? While we are here, I wonder why sysmon(9) does not follow the common guidelines w.r.t. for

Re: CVS commit: src/usr.sbin/envstat

2012-12-13 Thread Paul Goyette
Module Name:src Committed By: christos Date: Thu Dec 13 19:31:25 UTC 2012 Modified Files: src/usr.sbin/envstat: envstat.c Log Message: PR/47316: Henning Petersen: Memory leak in envstat with config file. While we're making sure to free() things, should we not also def

re: CVS commit: src/usr.sbin/envstat

2011-06-04 Thread Paul Goyette
On Sun, 5 Jun 2011, matthew green wrote: Module Name:src Committed By: pgoyette Date: Sat Jun 4 13:29:03 UTC 2011 Modified Files: src/usr.sbin/envstat: envstat.c Log Message: Since there is no longer a value-avg property, remove the code that extracts it. what's th

re: CVS commit: src/usr.sbin/envstat

2011-06-04 Thread matthew green
> Module Name: src > Committed By: pgoyette > Date: Sat Jun 4 13:29:03 UTC 2011 > > Modified Files: > src/usr.sbin/envstat: envstat.c > > Log Message: > Since there is no longer a value-avg property, remove the code that > extracts it. what's the behaviour of old envstat and new

CVS commit: src/usr.sbin/envstat

2010-02-15 Thread Paul Goyette
Module Name:src Committed By: pgoyette Date: Mon Feb 15 23:04:11 UTC 2010 Modified Files: src/usr.sbin/envstat: envsys.conf.5 Log Message: Document the new {high, maximum}-capacity keywords, and some general clean-up. To generate a diff of this commit: cvs rdiff -u -r1.1

CVS commit: src/usr.sbin/envstat

2010-02-15 Thread Paul Goyette
Module Name:src Committed By: pgoyette Date: Mon Feb 15 22:38:28 UTC 2010 Modified Files: src/usr.sbin/envstat: envstat.8 Log Message: Update cross-reference to refer to new aibs(4) instead of aiboost(4) To generate a diff of this commit: cvs rdiff -u -r1.54 -r1.55 src/u

CVS commit: src/usr.sbin/envstat

2010-02-15 Thread Paul Goyette
Module Name:src Committed By: pgoyette Date: Mon Feb 15 22:37:14 UTC 2010 Modified Files: src/usr.sbin/envstat: config.c config_lex.l envstat.c Log Message: Update userland envstat(8) to handle new {high,maximum}-capacity limits. To generate a diff of this commit: cvs rd

CVS commit: src/usr.sbin/envstat

2010-02-12 Thread Nicolas Joly
Module Name:src Committed By: njoly Date: Fri Feb 12 14:26:27 UTC 2010 Modified Files: src/usr.sbin/envstat: envstat.c Log Message: Fix double free, when requesting an invalid sensor. To generate a diff of this commit: cvs rdiff -u -r1.75 -r1.76 src/usr.sbin/envstat/envs

CVS commit: src/usr.sbin/envstat

2010-02-11 Thread Constantine A. Murenin
Module Name:src Committed By: cnst Date: Fri Feb 12 05:02:40 UTC 2010 Modified Files: src/usr.sbin/envstat: envstat.c Log Message: remove the fourth (empty) column from the -T printouts; ok pgoyette To generate a diff of this commit: cvs rdiff -u -r1.74 -r1.75 src/usr.sb

CVS commit: src/usr.sbin/envstat

2010-02-08 Thread Constantine A. Murenin
Module Name:src Committed By: cnst Date: Tue Feb 9 05:32:51 UTC 2010 Modified Files: src/usr.sbin/envstat: envstat.c Log Message: fix a typo resulting in warn-max being shown in two columns (one time instead of crit-min); ok pgoyette To generate a diff of this commit:

CVS commit: src/usr.sbin/envstat

2010-01-30 Thread Thomas Klausner
Module Name:src Committed By: wiz Date: Sat Jan 30 08:57:49 UTC 2010 Modified Files: src/usr.sbin/envstat: envstat.8 Log Message: New sentence, new line. To generate a diff of this commit: cvs rdiff -u -r1.52 -r1.53 src/usr.sbin/envstat/envstat.8 Please note that diffs

CVS commit: src/usr.sbin/envstat

2010-01-29 Thread Paul Goyette
Module Name:src Committed By: pgoyette Date: Sat Jan 30 02:56:39 UTC 2010 Modified Files: src/usr.sbin/envstat: envstat.8 envstat.c Log Message: Since we never have both a limit value and a limit %capacity value, remove the Capacity column. Reduce inter-column spacing, an