|
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
|
