From: Tina Ruchandani <ruchandani.t...@gmail.com> Date: Thu, 29 Oct 2015 22:01:07 -0700
> @@ -14,9 +15,9 @@ void atm_mpoa_init_cache(struct mpoa_client *mpc); > typedef struct in_cache_entry { > struct in_cache_entry *next; > struct in_cache_entry *prev; > - struct timeval tv; > - struct timeval reply_wait; > - struct timeval hold_down; > + time64_t tv; > + time64_t reply_wait; > + time64_t hold_down; > uint32_t packets_fwded; > uint16_t entry_state; > uint32_t retry_time; > @@ -51,7 +52,7 @@ struct in_cache_ops{ > typedef struct eg_cache_entry{ > struct eg_cache_entry *next; > struct eg_cache_entry *prev; > - struct timeval tv; > + time64_t tv; Again, these structure member names like "tv" only make sense if they store a "timeval". Please rename these structure members appropriately, thanks. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/