Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 15 Jun 2018 08:54:21 +0200
From:      Emmanuel Vadot <manu@bidouilliste.com>
To:        "Jonathan T. Looney" <jtl@freebsd.org>
Cc:        cognet@ci0.org, src-committers <src-committers@freebsd.org>, svn-src-all@freebsd.org, svn-src-head@freebsd.org, owner-src-committers@freebsd.org
Subject:   Re: svn commit: r335068 - in head: share/man/man9 sys/amd64/amd64 sys/i386/i386 sys/kern sys/net sys/sys sys/vm
Message-ID:  <0c47db8b787641e6569e5b90d604005c@megadrive.org>
In-Reply-To: <CADrOrmspbNDEaWL9ZFsn-4GFQE0xP8_LK5iEezyxvqr1QYGKug@mail.gmail.com>
References:  <201806131704.w5DH4gLv048134@repo.freebsd.org> <20180615002336.fcb966efe812cd60af59f68f@bidouilliste.com> <20180614231157.GA70834@ci0.org> <CADrOrmspbNDEaWL9ZFsn-4GFQE0xP8_LK5iEezyxvqr1QYGKug@mail.gmail.com>

next in thread | previous in thread | raw e-mail | index | archive | help
On 2018-06-15 04:04, Jonathan T. Looney wrote:
> On Thu, Jun 14, 2018 at 7:12 PM Olivier Houchard <cognet@ci0.org>
> wrote:
> 
>> On Fri, Jun 15, 2018 at 12:23:36AM +0200, Emmanuel Vadot wrote:
>>> This brake module loading on armv7 and arm64
>> 
>> I think I fixed it with r335182.
>> Jonathan probably missed it because modules are differnt on amd64
>> (and
>> mips), and the code that handles those are different, and doesn't
>> use
>> malloc().
> 
> Hi Olivier,
> 
> Thanks for fixing that!

  Indeed thanks.

> FWIW, I do have plans to make some further enhancements to fine-tune
> the permissions used for kernel modules. Once I have a patch, I would
> appreciate it if a few of the people impacted by this change would be
> willing to test it.
> 
> Jonathan

  Don't hesitate to put me on review so I can test on arm/arm64.

-- 
Emmanuel Vadot <manu@bidouilliste.com> <manu@freebsd.org>



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?0c47db8b787641e6569e5b90d604005c>