add digital TV framework which implements a subset of Linux DVB APIs

drop advertising clause

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!)

branches: 1.8.8; 1.8.10;
remove checks for failure after memory allocation calls that cannot fail:

kmem_alloc() with KM_SLEEP
kmem_zalloc() with KM_SLEEP
percpu_alloc()
pserialize_create()
psref_class_create()

all of these paths include an assertion that the allocation has not failed,
so callers should not assert that again.

branches: 1.7.12; 1.7.30;
modify the dtv device api so hardware drivers can be loaded independently
of the dtv module

Fix a locking problem with the demux, and while here do a bit of
housekeeping and documentation.

add section filter support

- use separate locks for ingress and egress queues
- increase ts block and buffer sizes

use %zu to print size_t

don't kmem_alloc with IPL_VM mutex locked, spotted by rmind

add digital TV framework which implements a subset of Linux DVB APIs

drop advertising clause

allocate memory for dtv_ts_section using kmem_alloc() in dtv_demux_read(),
instead of on-stack

XXX compile-tested only

Pass 0xff instead of true to memset when setting "all PIDs" mode in the PID filter

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!)

branches: 1.8.2; 1.8.4;
add fo_name so we can identify the fileops in a simple way.

remove checks for failure after memory allocation calls that cannot fail:

kmem_alloc() with KM_SLEEP
kmem_zalloc() with KM_SLEEP
percpu_alloc()
pserialize_create()
psref_class_create()

all of these paths include an assertion that the allocation has not failed,
so callers should not assert that again.

branches: 1.6.4;
IPL_VM -> IPL_SCHED to avoid taking kernel lock

remove unused variable

branches: 1.4.2; 1.4.12; 1.4.16;
Fix a locking problem with the demux, and while here do a bit of
housekeeping and documentation.

sections can't be > 4KB

dtv_demux_write: make sure to unlock muted in all error paths

add section filter support

drop advertising clause

For device modules that provide both auto-config and /dev/xxx
interfaces, make sure that initialization and destruction
follow the proper sequence. This is triggered by the recent
changes to the devsw stuff; per riastradh@ the required call
sequence is:

devsw_attach()
config_init_component() or config_cf*_attach()
...
config_fini_component() or config_cf*_detach()
devsw_detach()

While here, add a few missing calls to some of the detach
routines.

Testing of these changes has been limited to:
1. compile without build break
2. no related test failures from atf
3. modload/modunload work as well as
before.

No functional device testing done, since I don't have any
of these devices. Let me know of any damage I might cause
here!

XXX Some of the modules affected by this commit are already
XXX broken; see kern/56772. This commit does not break
any additional modules (as far as I know).

#include <sys/param.h> for COHERENCY_UNIT and some KNF while I'm here.

branches: 1.12.22;
Kill some more extern struct cfdriver declarations.

Down with externs in .c!

IPL_VM -> IPL_SCHED to avoid taking kernel lock

Add d_discard to all struct cdevsw instances I could find.

All have been set to "nodiscard"; some should get a real implementation.

branches: 1.9.2;
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.

branches: 1.8.2; 1.8.12; 1.8.16;
modify the dtv device api so hardware drivers can be loaded independently
of the dtv module

Fix a locking problem with the demux, and while here do a bit of
housekeeping and documentation.

add missing mutex_destroy on detach

add section filter support

- use separate locks for ingress and egress queues
- increase ts block and buffer sizes

allow this to be compiled into the kernel

don't kmem_alloc with IPL_VM mutex locked, spotted by rmind

add digital TV framework which implements a subset of Linux DVB APIs

drop advertising clause

add section filter support

don't kmem_alloc with IPL_VM mutex locked, spotted by rmind

add digital TV framework which implements a subset of Linux DVB APIs

modify the dtv device api so hardware drivers can be loaded independently
of the dtv module

Reinstate #include <dev/dtv/dtvif.h> which was incorrectly
removed in the previous commit.

Instead of returning an integer between 0 and 31 (scaled by 1<<24), use
(0.5 + x/2 - 1/x) as an approximation to log2(x) for x from 1 to 2, and
scale the input to fit this range. Now the error is always less than
0.2%.

Also add an test program, hidden behind #ifdef TEST_DTV_MATH, to print
a table of expected and actual results, and the errors.

add sys/types.h

add dtv_intlog10 implementation, from apb@

drop advertising clause

add dtv_math.h

s/transfered/transferred/

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!)

branches: 1.4.2; 1.4.4;
Guess pool_cache_get(pc, 0) means PR_WAITOK here.

Earlier on in the same context we use kmem_alloc(sz, KM_SLEEP).

branches: 1.3.2;
remove checks for failure after memory allocation calls that cannot fail:

kmem_alloc() with KM_SLEEP
kmem_zalloc() with KM_SLEEP
percpu_alloc()
pserialize_create()
psref_class_create()

all of these paths include an assertion that the allocation has not failed,
so callers should not assert that again.

branches: 1.2.4;
IPL_VM -> IPL_SCHED to avoid taking kernel lock

branches: 1.1.12;
add digital TV framework which implements a subset of Linux DVB APIs

drop advertising clause

add digital TV framework which implements a subset of Linux DVB APIs

drop advertising clause

s/static inline/static __inline/g for consistency.

branches: 1.3.52;
modify the dtv device api so hardware drivers can be loaded independently
of the dtv module

add dtv_intlog10 implementation, from apb@

add digital TV framework which implements a subset of Linux DVB APIs

drop advertising clause

add digital TV framework which implements a subset of Linux DVB APIs

drop advertising clause

Add some #includes to make a few more header files compile.

branches: 1.2.12;
add DMX_PES_foo -> DMX_PES_foo0 aliases for dmx_pes_type_t types

add digital TV framework which implements a subset of Linux DVB APIs

drop advertising clause

Add some #includes to make a few more header files compile.

branches: 1.2.12;
fe_sec_voltage_t: add SEC_VOLTAGE_OFF

add digital TV framework which implements a subset of Linux DVB APIs

allow this to be compiled into the kernel

add digital TV framework which implements a subset of Linux DVB APIs

drop advertising clause

modify the dtv device api so hardware drivers can be loaded independently
of the dtv module

Fix a locking problem with the demux, and while here do a bit of
housekeeping and documentation.

add section filter support

- use separate locks for ingress and egress queues
- increase ts block and buffer sizes

don't kmem_alloc with IPL_VM mutex locked, spotted by rmind

add digital TV framework which implements a subset of Linux DVB APIs

modify the dtv device api so hardware drivers can be loaded independently
of the dtv module

add dtv_intlog10 implementation, from apb@

add section filter support

add digital TV framework which implements a subset of Linux DVB APIs