#
6565c0bf |
| 12-Nov-2019 |
msaitoh <msaitoh@NetBSD.org> |
Add missing initialization of sc_dev.
|
#
d1579b2d |
| 03-Sep-2018 |
riastradh <riastradh@NetBSD.org> |
Rename min/max -> uimin/uimax for better honesty.
These functions are defined on unsigned int. The generic name min/max should not silently truncate to 32 bits on 64-bit systems. This is purely a n
Rename min/max -> uimin/uimax for better honesty.
These functions are defined on unsigned int. The generic name min/max should not silently truncate to 32 bits on 64-bit systems. This is purely a name change -- no functional change intended.
HOWEVER! Some subsystems have
#define min(a, b) ((a) < (b) ? (a) : (b)) #define max(a, b) ((a) > (b) ? (a) : (b))
even though our standard name for that is MIN/MAX. Although these may invite multiple evaluation bugs, these do _not_ cause integer truncation.
To avoid `fixing' these cases, I first changed the name in libkern, and then compile-tested every file where min/max occurred in order to confirm that it failed -- and thus confirm that nothing shadowed min/max -- before changing it.
I have left a handful of bootloaders that are too annoying to compile-test, and some dead code:
cobalt ews4800mips hp300 hppa ia64 luna68k vax acorn32/if_ie.c (not included in any kernels) macppc/if_gm.c (superseded by gem(4))
It should be easy to fix the fallout once identified -- this way of doing things fails safe, and the goal here, after all, is to _avoid_ silent integer truncations, not introduce them.
Maybe one day we can reintroduce min/max as type-generic things that never silently truncate. But we should avoid doing that for a while, so that existing code has a chance to be detected by the compiler for conversion to uimin/uimax without changing the semantics until we can properly audit it all. (Who knows, maybe in some cases integer truncation is actually intended!)
show more ...
|
#
82b8caba |
| 28-Oct-2017 |
riastradh <riastradh@NetBSD.org> |
Kill some more extern struct cfdriver declarations.
Down with externs in .c!
|
#
f9228f42 |
| 25-Jul-2014 |
dholland <dholland@NetBSD.org> |
Add d_discard to all struct cdevsw instances I could find.
All have been set to "nodiscard"; some should get a real implementation.
|
#
a68f9396 |
| 16-Mar-2014 |
dholland <dholland@NetBSD.org> |
Change (mostly mechanically) every cdevsw/bdevsw I can find to use designated initializers.
I have not built every extant kernel so I have probably broken at least one build; however I've also found
Change (mostly mechanically) every cdevsw/bdevsw I can find to use designated initializers.
I have not built every extant kernel so I have probably broken at least one build; however I've also found and fixed some wrong cdevsw/bdevsw entries so even if so I think we come out ahead.
show more ...
|
#
cbab9cad |
| 27-Oct-2012 |
chs <chs@NetBSD.org> |
split device_t/softc for all remaining drivers. replace "struct device *" with "device_t". use device_xname(), device_unit(), etc.
|
#
0762f691 |
| 12-Sep-2009 |
tsutsui <tsutsui@NetBSD.org> |
Fix warnings of printf(9) format.
|
#
055ae9c8 |
| 12-Sep-2009 |
tsutsui <tsutsui@NetBSD.org> |
Add a missing parentheses. PR kern/41859
|
#
529e91fc |
| 12-May-2009 |
cegger <cegger@NetBSD.org> |
struct device * -> device_t, no functional changes intended.
|
#
32c49409 |
| 12-May-2009 |
cegger <cegger@NetBSD.org> |
struct cfdata * -> cfdata_t, no functional changes intended.
|
#
82357f6d |
| 14-Mar-2009 |
dsl <dsl@NetBSD.org> |
ANSIfy another 1261 function definitions. The only ones left in sys are beyond by sed script! (or in sys/dist or sys/external) Mostly they have function pointer parameters.
|
#
454af1c0 |
| 14-Mar-2009 |
dsl <dsl@NetBSD.org> |
Change about 4500 of the K&R function definitions to ANSI ones. There are still about 1600 left, but they have ',' or /* ... */ in the actual variable definitions - which my awk script doesn't handle
Change about 4500 of the K&R function definitions to ANSI ones. There are still about 1600 left, but they have ',' or /* ... */ in the actual variable definitions - which my awk script doesn't handle. There are also many that need () -> (void). (The script does handle misordered arguments.)
show more ...
|
#
b0e5b3dc |
| 12-Jun-2008 |
cegger <cegger@NetBSD.org> |
- use device_lookup to get device_t - use device_lookup_private to get softc - ansify
|
#
ce099b40 |
| 28-Apr-2008 |
martin <martin@NetBSD.org> |
Remove clause 3 and 4 from TNF licenses
|
#
1b044f41 |
| 08-Apr-2008 |
cegger <cegger@NetBSD.org> |
use aprint_*_dev and device_xname
|
#
88ab7da9 |
| 09-Jul-2007 |
ad <ad@NetBSD.org> |
Merge some of the less invasive changes from the vmlocking branch:
- kthread, callout, devsw API changes - select()/poll() improvements - miscellaneous MT safety improvements
|
#
53524e44 |
| 04-Mar-2007 |
christos <christos@NetBSD.org> |
Kill caddr_t; there will be some MI fallout, but it will be fixed shortly.
|
#
d14a7576 |
| 16-Feb-2007 |
ad <ad@NetBSD.org> |
Fix spllowersoftclock() fallout.
|
#
22974bb6 |
| 13-Jan-2007 |
cube <cube@NetBSD.org> |
Complete initializers, teach drivers that we moved from struct proc to struct lwp, sprinkle some needed const qualifiers.
|
#
92c7bba3 |
| 29-Mar-2006 |
thorpej <thorpej@NetBSD.org> |
Use device_private().
|
#
39cd836e |
| 28-Mar-2006 |
thorpej <thorpej@NetBSD.org> |
Use device_unit().
|
#
95e1ffb1 |
| 11-Dec-2005 |
christos <christos@NetBSD.org> |
merge ktrace-lwp.
|
#
f31bd063 |
| 27-Feb-2005 |
perry <perry@NetBSD.org> |
nuke trailing whitespace
|
#
aad01611 |
| 07-Aug-2003 |
agc <agc@NetBSD.org> |
Move UCB-licensed code from 4-clause to 3-clause licence.
Patches provided by Joel Baker in PR 22364, verified by myself.
|
#
cb93fa6a |
| 02-Jun-2003 |
gmcgarry <gmcgarry@NetBSD.org> |
Port the hp300 ppi driver to the MI gpib framework.
|