[Click] patchless click patches

[Roman Chertov]

You are correct, I missed Joonwoo's 4/12 patch that addresses the
NETREG_REGISTERED issue.

I am not sure on what is the exact issue with FromHost element when using
patchless click.  Hopefully, somebody more knowledgeable will chime in. 

Roman

On Tue, 09 Nov 2010 20:42:12 +0100 Sascha Alexander Jopen
<jopen at informatik.uni-bonn.de> wrote

> Hey,
> 
> i compiled for a recent kernel from ubuntu, namely 2.6.35-22-generic. I
> didn't apply your NETREG_REGISTERED patch, but it seems, everything
> compiled fine.
> I think this patch part is included in Joonwoos "[PATCH 04/12]
> Configury: prepend net_device namespace for NETREG_REGISTERED"
> As the enum in question is part of the net_device struct, everything
> should work as expected. Applying your patch leads to compiler errors on
> my system.
> 
> So basically i am right, that FromHost is completely missing, currently?
> It would be nice, if someone could point me to the right direction. What
> is the problematic part in implementing this element?
> 
> Regards,
> Sascha
> 
> Am 09.11.2010 19:35, schrieb Roman Chertov:
> > Sascha,
> >
> > What kernel version did you use?  It appears that for 2.6.35 and above the
> > NETREG_REGISTERED fix is required as well.  If you start on updating
> > FromHost, I
> > will be glad to give you a hand.
> >
> > Roman
> >
> >
> > On Fri, 05 Nov 2010 20:02:38 +0100 Sascha Alexander Jopen
> > <jopen at informatik.uni-bonn.de> wrote
> >
> >> Hello,
> >>
> >> i tried click patchless today. I used those patches from Joonwoo, which
> >> didn't make it into the repository till now. Especially the patches to
> >> fixincludes and the configure scripts were necessary.
> >>
> >> It seems the FromHost element is not included during build. I know this
> >> was the last element which had to be done, but is it really still not
> >> finished? What are the missing parts for this element to work?
> >>
> >> We are using click userlevel and kernel level isn't really necessary but
> >> still nice to have. So maybe i could use some spare time to finish the
> >> FromHost element, if there isn't that much missing.
> >>
> >> Regards,
> >> Sascha Jopen
> >>
> >> On 11/03/10 17:42, Roman Chertov wrote:
> >>> This will also require Joonwoo's PATCH1/12.  It removes #include
> >>> <linux/autoconf.h>  from configure.in
> >>>
> >>> On Tue, 02 Nov 2010 18:44:32 -0700 "Roman Chertov"<rchertov at cs.ucsb.edu>
> >>> wrote
> >>>
> >>>> Hello,
> >>>>
> >>>> This works on Fedora 13 running 2.6.35.8 SMP kernel.  The second patch
> >>>> is
> >>>> based
> >>>> on patches 2/12 and 3/12 that Joonwoo released.  I had to add the
> >>>> NETREG_REGISTERED patch due to scoping issues when an enum declared in
> >>>> net_device is used.  Otherwise, the code is same as in github master
> >>>> branch.
> >>>> I
> >>>> am not sure if there is a more elegant way to solve that issue.
> >>>>
> >>>> Roman
> >>>
> >>>
> >>> _______________________________________________
> >>> click mailing list
> >>> click at amsterdam.lcs.mit.edu
> >>> https://amsterdam.lcs.mit.edu/mailman/listinfo/click
> >>
> >> _______________________________________________
> >> click mailing list
> >> click at amsterdam.lcs.mit.edu
> >> https://amsterdam.lcs.mit.edu/mailman/listinfo/click
> >
> >
> 
> _______________________________________________
> click mailing list
> click at amsterdam.lcs.mit.edu
> https://amsterdam.lcs.mit.edu/mailman/listinfo/click