Hi, Deepa Dinamani wrote: >>> include/linux/ip.h | 2 ++ >>> include/linux/time64.h | 3 +++ >>> kernel/time/time.c | 26 ++++++++++++++++++++++++++ >>> 3 files changed, 31 insertions(+) >>> >> Since net/ipv4/* are the only users, it is enough to put >> it in under net/ipv4/. > > time.c hosts functions that are used by individual subsystems like > current_fs_time() used by filesystems > (sometimes used by other subsystems also). > > The network timestamp function is used for both source route ip option > and timestamp icmp messages. > So it makes it difficult for it to be owned by a single layer. > This is the reason it was chosen to include here. > > Another option is to include it in the lowest layer its used: > af_inet.c. Is this what you were suggesting? >
Yes, that's right. --yoshfuji > -Deepa >
