On Fri, 15 Apr 2016 23:07:47 -0700, Philip Guenther wrote: > The current code is simply wrong: sizeof(struct timeval) > > sizeof(struct bpf_timeval). I think we should make this change (ok > guenther@) and then work to update struct bpf_hdr or a replacement > thereof to not be limited to 32bit time_t values. Maybe it should be > using a timespec instead too? Have any other free OSes added timespec > (or bintime) APIs? > > (BPF has been around since 1990; the time from then to now is less > than the time from now to 2038 when 32bit time_t fails...)
I agree that the change should be committed but since the values in struct bpf_timeval are unsigned it doesn't have a year 2038 issue. - todd