<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Mar 22, 2019 at 4:43 AM Pasi Kärkkäinen <<a href="mailto:pasik@iki.fi">pasik@iki.fi</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On Tue, Mar 12, 2019 at 01:21:01PM +0200, Pasi Kärkkäinen wrote:<br>
> > <br>
> > On Mon, Mar 04, 2019 at 04:22:19PM -0500, Karl Johnson wrote:<br>
> > >      ><br>
> > >      > I updated my PR: [3]<a href="https://github.com/CentOS-virt7/xen-kernel/pull/18" rel="noreferrer" target="_blank">https://github.com/CentOS-virt7/xen-kernel/pull/18</a><br>
> > > <br>
> > >      Hi Karl, I've build and tested your PR, but without the GCC7 patch, and<br>
> > >      when I've tested it on el6, none of the guests had network access.<br>
> > >      I had to revert the bridge changes to have them working<br>
> > >      (CONFIG_BRIDGE_NETFILTER=m and CONFIG_BRIDGE=m).<br>
> > > <br>
> > >      Why did you want the BRIDGE built-in instead of a module?<br>
> > > <br>
> > >      Maybe you have something to setup the bridge (probably filter) properly,<br>
> > >      or maybe something is different in the xen package between el6 and el7.<br>
> > >      Any idea?<br>
> > > <br>
> > >      I still have to investigate the Xen pkg, and have a closer look at those<br>
> > >      tests I'm running.<br>
> > > <br>
> > >    Hum strange. I switched from module to built-in because I always had a<br>
> > >    deprecated warning about bridge when booting CentOS 6 (and I guess<br>
> > >    everyone needs bridge?). I don't remember what was the exact warning<br>
> > >    because it's been a long time since I switched it. I don't mind to put it<br>
> > >    back as a module, both works for me but the warning is gone when built-in.<br>
> > <br>
> > Yeah better to keep things as modules, like they have been.<br>
> > <br>
> > btw upstream Linux 4.9 LTS kernel is now at 4.9.162 :)<br>
> > <br>
> <br>
> I suggest we:<br>
> <br>
> 1) Rebase to linux 4.9.162.<br>
> 2) Drop the "set bridge to built-in instead of module" changes. Let's keep those settings like they were earlier.<br>
> 3) Drop the use of gcc7 for now.<br>
> <br>
> How does that sound? <br>
> <br>
<br>
Karl: Would you like to update the github PR ? Or should someone else pick up the kernel update work?<br>
<br>
<br>
Thanks,<br>
<br>
-- Pasi<br>
<br><br></blockquote><div><br></div><div>Sure, I will update the PR in the next few days.</div><div><br></div><div>Karl<br></div></div></div>