# HG changeset patch # User Florian Pose # Date 1332150189 -3600 # Node ID d895cd1db2bf391023b9e9a57542fdf32600d065 # Parent a5052376202b8c38ea60061a2aaf5a23aa47fda3 Improved/fixed frame statistics (low-pass filters). diff -r a5052376202b -r d895cd1db2bf master/device.c --- a/master/device.c Mon Mar 19 09:56:36 2012 +0100 +++ b/master/device.c Mon Mar 19 10:43:09 2012 +0100 @@ -465,25 +465,25 @@ { unsigned int i; - u32 tx_frame_rate = - (u32) (device->tx_count - device->last_tx_count) * 1000; - u32 rx_frame_rate = - (u32) (device->rx_count - device->last_rx_count) * 1000; - u32 tx_byte_rate = - (device->tx_bytes - device->last_tx_bytes); - u32 rx_byte_rate = - (device->rx_bytes - device->last_rx_bytes); - + s32 tx_frame_rate = (device->tx_count - device->last_tx_count) * 1000; + s32 rx_frame_rate = (device->rx_count - device->last_rx_count) * 1000; + s32 tx_byte_rate = (device->tx_bytes - device->last_tx_bytes); + s32 rx_byte_rate = (device->rx_bytes - device->last_rx_bytes); + + /* Low-pass filter: + * Y_n = y_(n - 1) + T / tau * (x - y_(n - 1)) | T = 1 + * -> Y_n += (x - y_(n - 1)) / tau + */ for (i = 0; i < EC_RATE_COUNT; i++) { - unsigned int n = rate_intervals[i]; - device->tx_frame_rates[i] = - (device->tx_frame_rates[i] * (n - 1) + tx_frame_rate) / n; - device->rx_frame_rates[i] = - (device->rx_frame_rates[i] * (n - 1) + rx_frame_rate) / n; - device->tx_byte_rates[i] = - (device->tx_byte_rates[i] * (n - 1) + tx_byte_rate) / n; - device->rx_byte_rates[i] = - (device->rx_byte_rates[i] * (n - 1) + rx_byte_rate) / n; + s32 n = rate_intervals[i]; + device->tx_frame_rates[i] += + (tx_frame_rate - device->tx_frame_rates[i]) / n; + device->rx_frame_rates[i] += + (rx_frame_rate - device->rx_frame_rates[i]) / n; + device->tx_byte_rates[i] += + (tx_byte_rate - device->tx_byte_rates[i]) / n; + device->rx_byte_rates[i] += + (rx_byte_rate - device->rx_byte_rates[i]) / n; } device->last_tx_count = device->tx_count; diff -r a5052376202b -r d895cd1db2bf master/device.h --- a/master/device.h Mon Mar 19 09:56:36 2012 +0100 +++ b/master/device.h Mon Mar 19 10:43:09 2012 +0100 @@ -109,18 +109,16 @@ u64 last_rx_bytes; /**< Number of bytes received of last statistics cycle. */ u64 tx_errors; /**< Number of transmit errors. */ - unsigned int tx_frame_rates[EC_RATE_COUNT]; /**< Transmit rates in - frames/s for different - statistics cycle periods. */ - unsigned int rx_frame_rates[EC_RATE_COUNT]; /**< Receive rates in - frames/s for different - statistics cycle periods. */ - unsigned int tx_byte_rates[EC_RATE_COUNT]; /**< Transmit rates in byte/s - for different statistics - cycle periods. */ - unsigned int rx_byte_rates[EC_RATE_COUNT]; /**< Receive rates in byte/s - for different statistics - cycle periods. */ + s32 tx_frame_rates[EC_RATE_COUNT]; /**< Transmit rates in frames/s for + different statistics cycle periods. + */ + s32 rx_frame_rates[EC_RATE_COUNT]; /**< Receive rates in frames/s for + different statistics cycle periods. + */ + s32 tx_byte_rates[EC_RATE_COUNT]; /**< Transmit rates in byte/s for + different statistics cycle periods. */ + s32 rx_byte_rates[EC_RATE_COUNT]; /**< Receive rates in byte/s for + different statistics cycle periods. */ #ifdef EC_DEBUG_IF ec_debug_t dbg; /**< debug device */ diff -r a5052376202b -r d895cd1db2bf master/ioctl.h --- a/master/ioctl.h Mon Mar 19 09:56:36 2012 +0100 +++ b/master/ioctl.h Mon Mar 19 10:43:09 2012 +0100 @@ -56,7 +56,7 @@ * * Increment this when changing the ioctl interface! */ -#define EC_IOCTL_VERSION_MAGIC 15 +#define EC_IOCTL_VERSION_MAGIC 16 // Command-line tool #define EC_IOCTL_MODULE EC_IOR(0x00, ec_ioctl_module_t) @@ -170,19 +170,19 @@ uint64_t tx_bytes; uint64_t rx_bytes; uint64_t tx_errors; - uint32_t tx_frame_rates[EC_RATE_COUNT]; - uint32_t rx_frame_rates[EC_RATE_COUNT]; - uint32_t tx_byte_rates[EC_RATE_COUNT]; - uint32_t rx_byte_rates[EC_RATE_COUNT]; + int32_t tx_frame_rates[EC_RATE_COUNT]; + int32_t rx_frame_rates[EC_RATE_COUNT]; + int32_t tx_byte_rates[EC_RATE_COUNT]; + int32_t rx_byte_rates[EC_RATE_COUNT]; } devices[EC_NUM_DEVICES]; uint64_t tx_count; uint64_t rx_count; uint64_t tx_bytes; uint64_t rx_bytes; - uint32_t tx_frame_rates[EC_RATE_COUNT]; - uint32_t rx_frame_rates[EC_RATE_COUNT]; - uint32_t tx_byte_rates[EC_RATE_COUNT]; - uint32_t rx_byte_rates[EC_RATE_COUNT]; + int32_t tx_frame_rates[EC_RATE_COUNT]; + int32_t rx_frame_rates[EC_RATE_COUNT]; + int32_t tx_byte_rates[EC_RATE_COUNT]; + int32_t rx_byte_rates[EC_RATE_COUNT]; int32_t loss_rates[EC_RATE_COUNT]; uint64_t app_time; uint16_t ref_clock; diff -r a5052376202b -r d895cd1db2bf master/master.c --- a/master/master.c Mon Mar 19 09:56:36 2012 +0100 +++ b/master/master.c Mon Mar 19 10:43:09 2012 +0100 @@ -1229,9 +1229,8 @@ ) { ec_device_stats_t *s = &master->device_stats; - u32 tx_frame_rate, rx_frame_rate, tx_byte_rate, rx_byte_rate; + s32 tx_frame_rate, rx_frame_rate, tx_byte_rate, rx_byte_rate, loss_rate; u64 loss; - s32 loss_rate; unsigned int i; // frame statistics @@ -1239,27 +1238,26 @@ return; } - tx_frame_rate = (u32) (s->tx_count - s->last_tx_count) * 1000; - rx_frame_rate = (u32) (s->rx_count - s->last_rx_count) * 1000; - tx_byte_rate = (s->tx_bytes - s->last_tx_bytes); - rx_byte_rate = (s->rx_bytes - s->last_rx_bytes); + tx_frame_rate = (s->tx_count - s->last_tx_count) * 1000; + rx_frame_rate = (s->rx_count - s->last_rx_count) * 1000; + tx_byte_rate = s->tx_bytes - s->last_tx_bytes; + rx_byte_rate = s->rx_bytes - s->last_rx_bytes; loss = s->tx_count - s->rx_count; - loss_rate = (s32) (loss - s->last_loss) * 1000; - + loss_rate = (loss - s->last_loss) * 1000; + + /* Low-pass filter: + * Y_n = y_(n - 1) + T / tau * (x - y_(n - 1)) | T = 1 + * -> Y_n += (x - y_(n - 1)) / tau + */ for (i = 0; i < EC_RATE_COUNT; i++) { - unsigned int n = rate_intervals[i]; - s->tx_frame_rates[i] = - (s->tx_frame_rates[i] * (n - 1) + tx_frame_rate) / n; - s->rx_frame_rates[i] = - (s->rx_frame_rates[i] * (n - 1) + rx_frame_rate) / n; - s->tx_byte_rates[i] = - (s->tx_byte_rates[i] * (n - 1) + tx_byte_rate) / n; - s->rx_byte_rates[i] = - (s->rx_byte_rates[i] * (n - 1) + rx_byte_rate) / n; - s->loss_rates[i] = - (s->loss_rates[i] * (n - 1) + loss_rate) / n; - - } + s32 n = rate_intervals[i]; + s->tx_frame_rates[i] += (tx_frame_rate - s->tx_frame_rates[i]) / n; + s->rx_frame_rates[i] += (rx_frame_rate - s->rx_frame_rates[i]) / n; + s->tx_byte_rates[i] += (tx_byte_rate - s->tx_byte_rates[i]) / n; + s->rx_byte_rates[i] += (rx_byte_rate - s->rx_byte_rates[i]) / n; + s->loss_rates[i] += (loss_rate - s->loss_rates[i]) / n; + } + s->last_tx_count = s->tx_count; s->last_rx_count = s->rx_count; s->last_tx_bytes = s->tx_bytes; diff -r a5052376202b -r d895cd1db2bf master/master.h --- a/master/master.h Mon Mar 19 09:56:36 2012 +0100 +++ b/master/master.h Mon Mar 19 10:43:09 2012 +0100 @@ -151,19 +151,17 @@ u64 last_rx_bytes; /**< Number of bytes received of last statistics cycle. */ u64 last_loss; /**< Tx/Rx difference of last statistics cycle. */ - unsigned int tx_frame_rates[EC_RATE_COUNT]; /**< Transmit rates in - frames/s for different - statistics cycle periods. */ - unsigned int rx_frame_rates[EC_RATE_COUNT]; /**< Receive rates in - frames/s for different - statistics cycle periods. */ - unsigned int tx_byte_rates[EC_RATE_COUNT]; /**< Transmit rates in byte/s - for different statistics - cycle periods. */ - unsigned int rx_byte_rates[EC_RATE_COUNT]; /**< Receive rates in byte/s - for different statistics - cycle periods. */ - int loss_rates[EC_RATE_COUNT]; /**< Frame loss rates for different + s32 tx_frame_rates[EC_RATE_COUNT]; /**< Transmit rates in frames/s for + different statistics cycle periods. + */ + s32 rx_frame_rates[EC_RATE_COUNT]; /**< Receive rates in frames/s for + different statistics cycle periods. + */ + s32 tx_byte_rates[EC_RATE_COUNT]; /**< Transmit rates in byte/s for + different statistics cycle periods. */ + s32 rx_byte_rates[EC_RATE_COUNT]; /**< Receive rates in byte/s for + different statistics cycle periods. */ + s32 loss_rates[EC_RATE_COUNT]; /**< Frame loss rates for different statistics cycle periods. */ unsigned long jiffies; /**< Jiffies of last statistic cycle. */ } ec_device_stats_t;