When I've been reworking vznetstat mem management I've added destroying of VE's statistics upon containers stop. This is incorrect, we need to keep it while module is alive.
Still one can destroy statistics with VZCTL_TC_DESTROY_STAT or VZCTL_TC_DESTROY_ALL_STAT (of course it clears it for non running VEs only). https://jira.sw.ru/browse/PSBM-35178 v2 (by vdavydov@): - clear ve::stat in net_exit_acct instead of bunch cleanup on module exit Signed-off-by: Cyrill Gorcunov <gorcu...@virtuozzo.com> CC: Andrey Vagin <ava...@virtuozzo.com> CC: Vladimir Davydov <vdavy...@virtuozzo.com> CC: Konstantin Khorenko <khore...@virtuozzo.com> CC: Pavel Emelyanov <xe...@virtuozzo.com> CC: Igor Sukhih <i...@parallels.com> --- kernel/ve/vznetstat/vznetstat.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) Index: linux-pcs7.git/kernel/ve/vznetstat/vznetstat.c =================================================================== --- linux-pcs7.git.orig/kernel/ve/vznetstat/vznetstat.c +++ linux-pcs7.git/kernel/ve/vznetstat/vznetstat.c @@ -1098,10 +1098,8 @@ static void __net_exit net_exit_acct(str if (ve->stat) { venet_acct_put_stat(ve->stat); - if (atomic_read(&ve->stat->users) == 0) { - venet_acct_destroy_stat(ve->veid); + if (atomic_read(&ve->stat->users) == 0) ve->stat = NULL; - } } } _______________________________________________ Devel mailing list Devel@openvz.org https://lists.openvz.org/mailman/listinfo/devel
