CentOS virt - Mar 2019 - CentOS virt-sig Linux kernel 4.9.155 build

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

Karl
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
<http://lists.centos.org/pipermail/centos-virt/attachments/20190323/ac075f18/attachment-0002.html>

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

[root at node-tmp1 ~]# cat /proc/version
Linux version 4.9.165-35.el6.x86_64 (mockbuild at build.aerisnetwork.net) (gcc
version 4.4.7 20120313 (Red Hat 4.4.7-23) (GCC) ) #1 SMP Sat Mar 23
14:44:16 EDT 2019

Test rpms: https://repo.aerisnetwork.com/archive/

Anthony will be able to build it on the CBS as devtoolset has been removed
from the spec.

Karl
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
<http://lists.centos.org/pipermail/centos-virt/attachments/20190323/b7fb038d/attachment-0002.html>

On Sat, Mar 23, 2019 at 05:00:00PM -0400, Karl Johnson
wrote:
>    On Sat, Mar 23, 2019 at 2:07 PM Karl Johnson <[1]karljohnson.it at
gmail.com>
>    wrote:
> 
> 
>      Sure, I will update the PR in the next few days.
>      Karl
> 
>    I've updated my PR and tested a build, it boots well:
>    [root at node-tmp1 ~]# cat /proc/version
>    Linux version 4.9.165-35.el6.x86_64 ([4]mockbuild at
build.aerisnetwork.net)
>    (gcc version 4.4.7 20120313 (Red Hat 4.4.7-23) (GCC) ) #1 SMP Sat Mar 23
>    14:44:16 EDT 2019
>    Test rpms: [5]https://repo.aerisnetwork.com/archive/
>    Anthony will be able to build it on the CBS as devtoolset has been
removed
>    from the spec.
>
With quick testing the 4.9.165-35.el6 kernel seems to work OK for me!

Anthony: Can you please review the updated PR on github, merge it and kick of
the builds for el6 and el7 ?


>    Karl
> 
Thanks,

-- Pasi