Re: [RFC]: tfrc_tx_hist_rtt

[Arnaldo Carvalho de Melo <acme@redhat.com>]

Em Fri, Nov 30, 2007 at 12:19:36PM +0000, Gerrit Renker escreveu:
> Sorry I only got this email today and it is a busy day, too.
> 
> The changes look good and are in general a further improvement on the
> code. I like the idea of hiding the internals of the list structure in 
> the source file.
> 
> I wonder if one could go one step further and also take the timestamp
> directly when looking up the previous history sample, e.g.
> 
> 	u32 tfrc_tx_hist_rtt(struct tfrc_tx_hist_entry *head, const u64 seqno)
> 	{
> 		u32 rtt = 0;
> 		struct tfrc_tx_hist_entry *packet = tfrc_tx_hist_find_entry(head, seqno);
> 	
> 		if (packet != NULL) {
> 			rtt = ktime_us_delta(ktime_get_real(), packet->stamp);
> 			/*
> 			 * Garbage-collect older (irrelevant) entries:
> 			 */
> 			tfrc_tx_hist_purge(&packet->next);
> 		}
> 	
> 		return rtt;
> 	}
> 
> Just a suggestion.

I thought about that, but ccid3_hc_tx_packet_recv needs a timestamp some
lines below, when we reuse the ktime_get_real call.

- Arnaldo