Ander Juaristi <a...@juaristi.eus> wrote:
> This patch introduces meta matches in the kernel for time (a UNIX timestamp),
> day (a day of week, represented as an integer between 0-6), and
> hour (an hour in the current day, or: number of seconds since midnight).
> 
> All values are taken as unsigned 64-bit integers.
> 
> The 'time' keyword is internally converted to nanoseconds by nft in
> userspace, and hence the timestamp is taken in nanoseconds as well.

I think this is conceptually fine, thanks Ander.

Can you run this throuch scripts/checkpatch.pl and fix up the style
nits?

> +     case NFT_META_TIME_HOUR:
> +             len = sizeof(u64);

As in my other comment, I think this can be u32.

Reply via email to