All of lore.kernel.org
 help / color / mirror / Atom feed
* IPVS: sync_options needed (removal of some code...)?
@ 2010-10-28  9:18 Hans Schillstrom
  2010-10-28 20:29 ` Julian Anastasov
  0 siblings, 1 reply; 3+ messages in thread
From: Hans Schillstrom @ 2010-10-28  9:18 UTC (permalink / raw)
  To: Simon Horman, Julian Anastasov, wensong, lvs-devel

Hello
do we really need sync_options any more ?
(tcp seq re-calc)

I might be blid, but I can't see anyone use it.
ip_vs_ftp_in() and ip_vs_ftp_out() do set diff=0 and never change them.

Less code makes Linus happy :-)
--
Regards
Hans Schillstrom <hans.schillstrom@ericsson.com>

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: IPVS: sync_options needed (removal of some code...)?
  2010-10-28  9:18 IPVS: sync_options needed (removal of some code...)? Hans Schillstrom
@ 2010-10-28 20:29 ` Julian Anastasov
  2010-10-28 21:45   ` Hans Schillstrom
  0 siblings, 1 reply; 3+ messages in thread
From: Julian Anastasov @ 2010-10-28 20:29 UTC (permalink / raw)
  To: Hans Schillstrom; +Cc: Simon Horman, wensong, lvs-devel


 	Hello,

On Thu, 28 Oct 2010, Hans Schillstrom wrote:

> Hello
> do we really need sync_options any more ?
> (tcp seq re-calc)
>
> I might be blid, but I can't see anyone use it.
> ip_vs_ftp_in() and ip_vs_ftp_out() do set diff=0 and never change them.

 	Yes, sync for FTP is not supported anymore
starting from 2.6.36 :( But I'm not sure if any third party
IPVS modules exists, it is possible other apps to use
sync and sync_options.

Regards

--
Julian Anastasov <ja@ssi.bg>

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: IPVS: sync_options needed (removal of some code...)?
  2010-10-28 20:29 ` Julian Anastasov
@ 2010-10-28 21:45   ` Hans Schillstrom
  0 siblings, 0 replies; 3+ messages in thread
From: Hans Schillstrom @ 2010-10-28 21:45 UTC (permalink / raw)
  To: Julian Anastasov; +Cc: Hans Schillstrom, Simon Horman, wensong, lvs-devel


On Thursday, October 28, 2010 22:29:14 Julian Anastasov wrote:
> 
>  	Hello,
> 
> On Thu, 28 Oct 2010, Hans Schillstrom wrote:
> 
> > Hello
> > do we really need sync_options any more ?
> > (tcp seq re-calc)
> >
> > I might be blid, but I can't see anyone use it.
> > ip_vs_ftp_in() and ip_vs_ftp_out() do set diff=0 and never change them.
> 
>  	Yes, sync for FTP is not supported anymore
> starting from 2.6.36 :( But I'm not sure if any third party
> IPVS modules exists, it is possible other apps to use
> sync and sync_options.

OK, we leave it for now

> 
> Regards
> 
> --
> Julian Anastasov <ja@ssi.bg>

Regards
Hans Schillstrom <hans@schillstrom.com>

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2010-10-28 21:45 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2010-10-28  9:18 IPVS: sync_options needed (removal of some code...)? Hans Schillstrom
2010-10-28 20:29 ` Julian Anastasov
2010-10-28 21:45   ` Hans Schillstrom

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.