[ovs-dev] Support for MCAST_Snooping

[Gandharva S Murthy]

Hi,
Sorry it was mistake with the local system. Its working fine now.

Thanks,
Gandharva

On Sun, Dec 14, 2014 at 1:11 PM, Gandharva S Murthy <gandharva666 at gmail.com>
wrote:
>
> Hi,
> We tried to patch the email which you sent, RFC[1/2] went well and patch
> applied without any errors. But, RFC[2/2] patch has got some error it
> seems, it shows me failed to apply some patch on mcast-snooping.c. The log
> of patch is as shown below,
>
> patching file lib/mcast-snooping.c
> Hunk #3 FAILED at 597.
> 1 out of 4 hunks FAILED -- saving rejects to file lib/mcast-snooping.c.rej
> patching file lib/mcast-snooping.h
> patching file ofproto/ofproto-dpif-xlate.c
> patching file ofproto/ofproto-dpif.c
> Hunk #1 succeeded at 3063 (offset -37 lines).
> patching file ofproto/ofproto-provider.h
> patching file ofproto/ofproto.c
> patching file ofproto/ofproto.h
> patching file utilities/ovs-vsctl.8.in
> patching file vswitchd/bridge.c
> patching file vswitchd/vswitch.xml
>
> I have also attached the original file, file after applying patch and the
> rejected patch set file. Can you please assist us with solving the issue.
>
> Thanks,
> Gandharva S
>
>
>
> On Fri, Dec 12, 2014 at 8:47 PM, Flavio Leitner <fbl at redhat.com> wrote:
>>
>> On Fri, Dec 12, 2014 at 08:25:48PM +0530, Neethi Shashidhar wrote:
>> > Hi,
>> >
>> > we have cloned the master branch,but the patch doesn't seem to be a
>> part of
>> > it and it works same as earlier. can you please explain how do we get it
>> > using the list that you have sent.
>>
>> You have to get the patches from the e-mail and apply manually
>> on your cloned repo.  See this documentation:
>>
>> http://git-scm.com/book/en/v2/Distributed-Git-Maintaining-a-Project#Applying-Patches-from-E-mail
>>
>> Thanks,
>> fbl
>>
>> >
>> > Thanks,
>> > Neethi
>> > On 12-Dec-2014 6:27 pm, "Flavio Leitner" <fbl at redhat.com> wrote:
>> >
>> > > On Fri, Dec 12, 2014 at 02:46:50PM +0530, Neethi Shashidhar wrote:
>> > > > Hi,
>> > > >
>> > > > we are unable to find the patch against the master branch.can you
>> tell us
>> > > > how to get it?
>> > >
>> > > I sent to the list with copies to everyone in the thread.
>> > > Here is the archive url:
>> > > http://openvswitch.org/pipermail/dev/2014-December/049620.html
>> > >
>> > > fbl
>> > >
>> > >
>> > > >
>> > > > Thanks,
>> > > > Neethi
>> > > >
>> > > >
>> > > >
>> > > > On Thu, Dec 11, 2014 at 9:55 PM, Aman Kumar <amank3107 at gmail.com>
>> wrote:
>> > > > >
>> > > > > Thanks Flavio,
>> > > > >
>> > > > > Tomorrow i will test it and once it will over i will inform you.
>> > > > >
>> > > > > Thanks
>> > > > > Aman Kumar
>> > > > >
>> > > > > On Thu, Dec 11, 2014 at 9:49 PM, Aman Kumar <amank3107 at gmail.com>
>> > > wrote:
>> > > > >
>> > > > >>
>> > > > >> ---------- Forwarded message ----------
>> > > > >> From: Flavio Leitner <fbl at redhat.com>
>> > > > >> Date: Thu, Dec 11, 2014 at 5:16 PM
>> > > > >> Subject: Re: [ovs-dev] Support for MCAST_Snooping
>> > > > >> To: Aman Kumar <amank3107 at gmail.com>
>> > > > >> Cc: gangadhar.singh at hp.com, Neethi Shashidhar <
>> neethi209 at gmail.com>,
>> > > > >> Gandharva S Murthy <gandharva666 at gmail.com>, "<
>> dev at openvswitch.org>"
>> > > <
>> > > > >> dev at openvswitch.org>, Ben Pfaff <blp at nicira.com>
>> > > > >>
>> > > > >>
>> > > > >> On Wed, Dec 10, 2014 at 08:41:34AM +0530, Aman Kumar wrote:
>> > > > >> > Yes, Please do the patch against branch master and inform me,
>> i will
>> > > > >> test
>> > > > >> > it with branch master.
>> > > > >> > Currently i am using the branch master only.
>> > > > >>
>> > > > >> OK, I've done the patches and sent them as a new thread:
>> > > > >> "[RFC 0/2] Add support to control Reports forwarding"
>> > > > >>
>> > > > >> It's pretty much *compile tested only* because my lack of
>> > > > >> time for this, so please help me providing feedback.
>> > > > >>
>> > > > >> This is RFC because I am not happy with the vlan handling.
>> > > > >> I think it should be per vlan, so instead of 'true'/'false',
>> > > > >> it could be a list of vlans or 'trunk'.
>> > > > >>
>> > > > >> E.g.:
>> > > > >> for specific vlans:
>> > > > >> other_config:mcast-snooping-flood-reports="10,12,48"
>> > > > >>
>> > > > >> or for all vlans:
>> > > > >> other_config:mcast-snooping-flood-reports="trunk"
>> > > > >>
>> > > > >> to be decided yet...
>> > > > >>
>> > > > >> Thanks,
>> > > > >> fbl
>> > > > >>
>> > > > >>
>> > > > >>
>> > > > >
>> > >
>>
>