LVS
lvs-devel
Google
 
Web LinuxVirtualServer.org

Re: [PATCH RFC net-next] ipvs: use 64-bit rates in stats

To: Julian Anastasov <ja@xxxxxx>
Subject: Re: [PATCH RFC net-next] ipvs: use 64-bit rates in stats
Cc: lvs-devel@xxxxxxxxxxxxxxx, Chris Caputo <ccaputo@xxxxxxx>
From: Simon Horman <horms@xxxxxxxxxxxx>
Date: Tue, 3 Feb 2015 11:12:00 +0900
On Fri, Jan 30, 2015 at 10:33:26AM +0200, Julian Anastasov wrote:
> 
>       Hello,
> 
> On Fri, 30 Jan 2015, Simon Horman wrote:
> 
> > On Tue, Jan 27, 2015 at 10:46:55AM +0200, Julian Anastasov wrote:
> > > IPVS stats are limited to 2^(32-10) conns/s and packets/s,
> > > 2^(32-5) bytes/s. It is time to use 64 bits:
> > > 
> > > * Change all conn/packet kernel counters to 64-bit and update
> > > them in u64_stats_update_{begin,end} section
> > > 
> > > * In kernel use struct ip_vs_kstats instead of the user-space
> > > struct ip_vs_stats_user and use new func ip_vs_export_stats_user
> > > to export it to sockopt users to preserve compatibility with
> > > 32-bit values
> > > 
> > > * Rename cpu counters "ustats" to "cnt"
> > > 
> > > * To netlink users provide additionally 64-bit stats:
> > > IPVS_SVC_ATTR_STATS64 and IPVS_DEST_ATTR_STATS64. Old stats
> > > remain for old binaries.
> > > 
> > > * We can use ip_vs_copy_stats in ip_vs_stats_percpu_show
> > > 
> > > Thanks to Chris Caputo for providing initial patch for ip_vs_est.c
> > 
> > This seems reasonable to me, let me know if you would like me to queue it 
> > up.
> > 
> > The ipvsadm patch also seems fine to me.
> 
>       I didn't accumulated other changes to these patches,
> so they can be applied as is. If needed, I can submit the
> ipvsadm patch again because I didn't sent a copy to Jesper.
> In fact, we do not need to hurry with these patches, no problem
> if we apply them after the next merge window. Lets give
> some time, in case someone has idea for new attribute or
> other changes for stats.

I'm comfortable with waiting. Please repost when you're ready.
--
To unsubscribe from this list: send the line "unsubscribe lvs-devel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

<Prev in Thread] Current Thread [Next in Thread>
  • Re: [PATCH RFC net-next] ipvs: use 64-bit rates in stats, Simon Horman <=