Re: [RFC PATCH 1/4] IPVS: Prepare for transferring firewall marks (fwmark) to the backup daemon.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hello,

On Thursday 28 October 2010 14:08:47 Simon Horman wrote:
> On Tue, Oct 26, 2010 at 01:00:20PM +0200, Hans Schillstrom wrote:
> > Two structs will have fwmark added:
> >  * ip_vs_conn
> >  * ip_vs_conn_param
> >
> > ip_vs_conn_fill_param will have an extra param - fwmark
> > The effects of that is in this patch.
>
> This looks good to me, with the exception of
> the minor formatting nit below.
>
> I have fixed it as I describe below and queued it up for nf-next-2.6
> once it opens.
>
> > Signed-off-by: Hans Schillstrom <hans.schillstrom@xxxxxxxxxxxx>
>
> [ snip ]
>
> > diff --git a/net/netfilter/ipvs/ip_vs_core.c b/net/netfilter/ipvs/ip_vs_core.c
> > index b4e51e9..61abf39 100644
> > --- a/net/netfilter/ipvs/ip_vs_core.c
> > +++ b/net/netfilter/ipvs/ip_vs_core.c
>
> [ snip ]
>
> > @@ -485,8 +486,8 @@ int ip_vs_leave(struct ip_vs_service *svc, struct sk_buff *skb,
> >  		{
> >  			struct ip_vs_conn_param p;
> >  			ip_vs_conn_fill_param(svc->af, iph.protocol,
> > -					      &iph.saddr, pptr[0],
> > -					      &iph.daddr, pptr[1], &p);
> > +					      &iph.saddr, pptr[0],&iph.daddr,
> > +					      pptr[1], skb->mark, &p);
>
> Could you please put a space after the ,?
> For the record, the following is my preferred formatting.

I prefer that too ;-)

I will prepare and send a version 2 of the patch set today, with
the comments included from you and Julian.

--
Regards
Hans Schillstrom <hans.schillstrom@xxxxxxxxxxxx>
--
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


[Index of Archives]     [Linux Filesystem Devel]     [Linux NFS]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux SCSI]     [X.Org]

  Powered by Linux