Re: svn commit: r315701 - in head/sys: amd64/cloudabi32 amd64/cloudabi64 arm/cloudabi32 arm64/cloudabi64 i386/cloudabi32

2017-03-23 Thread Konstantin Belousov
On Thu, Mar 23, 2017 at 08:43:20AM +0100, Ed Schouten wrote: > Thanks! I just gave the patch a try, but the comparison added to > imgact_elf.c now causes the brandinfo to be skipped entirely. Attached > is a patch that does work for me. > > Index: sys/kern/imgact_elf.c >

Re: svn commit: r315701 - in head/sys: amd64/cloudabi32 amd64/cloudabi64 arm/cloudabi32 arm64/cloudabi64 i386/cloudabi32

2017-03-23 Thread Konstantin Belousov
On Thu, Mar 23, 2017 at 08:43:20AM +0100, Ed Schouten wrote: > Hi Kostik, > > 2017-03-23 0:00 GMT+01:00 Konstantin Belousov : > > On Wed, Mar 22, 2017 at 03:16:24PM +0100, Ed Schouten wrote: > >> Similarly, I seem to remember CloudABI's brandinfos set compat_3_brand > >> for

Re: svn commit: r315701 - in head/sys: amd64/cloudabi32 amd64/cloudabi64 arm/cloudabi32 arm64/cloudabi64 i386/cloudabi32

2017-03-23 Thread Ed Schouten
Hi Kostik, 2017-03-23 0:00 GMT+01:00 Konstantin Belousov : > On Wed, Mar 22, 2017 at 03:16:24PM +0100, Ed Schouten wrote: >> Similarly, I seem to remember CloudABI's brandinfos set compat_3_brand >> for a similar reason: it seems to be required by imgact_elf.c. Would >> we

Re: svn commit: r315701 - in head/sys: amd64/cloudabi32 amd64/cloudabi64 arm/cloudabi32 arm64/cloudabi64 i386/cloudabi32

2017-03-22 Thread Konstantin Belousov
On Wed, Mar 22, 2017 at 03:16:24PM +0100, Ed Schouten wrote: > Similarly, I seem to remember CloudABI's brandinfos set compat_3_brand > for a similar reason: it seems to be required by imgact_elf.c. Would > we also want to change that? Could you please try this ? diff --git

Re: svn commit: r315701 - in head/sys: amd64/cloudabi32 amd64/cloudabi64 arm/cloudabi32 arm64/cloudabi64 i386/cloudabi32

2017-03-22 Thread Ed Schouten
Hi Kostik, 2017-03-22 10:02 GMT+01:00 Konstantin Belousov : > On Wed, Mar 22, 2017 at 07:05:27AM +, Ed Schouten wrote: >> Author: ed >> Date: Wed Mar 22 07:05:27 2017 >> New Revision: 315701 >> URL: https://svnweb.freebsd.org/changeset/base/315701 >> >> Log: >> Set the

Re: svn commit: r315701 - in head/sys: amd64/cloudabi32 amd64/cloudabi64 arm/cloudabi32 arm64/cloudabi64 i386/cloudabi32

2017-03-22 Thread Konstantin Belousov
On Wed, Mar 22, 2017 at 11:02:58AM +0200, Konstantin Belousov wrote: > On Wed, Mar 22, 2017 at 07:05:27AM +, Ed Schouten wrote: > > Author: ed > > Date: Wed Mar 22 07:05:27 2017 > > New Revision: 315701 > > URL: https://svnweb.freebsd.org/changeset/base/315701 > > > > Log: > > Set the

Re: svn commit: r315701 - in head/sys: amd64/cloudabi32 amd64/cloudabi64 arm/cloudabi32 arm64/cloudabi64 i386/cloudabi32

2017-03-22 Thread Konstantin Belousov
On Wed, Mar 22, 2017 at 07:05:27AM +, Ed Schouten wrote: > Author: ed > Date: Wed Mar 22 07:05:27 2017 > New Revision: 315701 > URL: https://svnweb.freebsd.org/changeset/base/315701 > > Log: > Set the interpreter path to /nonexistent. > > CloudABI executables are statically linked and

svn commit: r315701 - in head/sys: amd64/cloudabi32 amd64/cloudabi64 arm/cloudabi32 arm64/cloudabi64 i386/cloudabi32

2017-03-22 Thread Ed Schouten
Author: ed Date: Wed Mar 22 07:05:27 2017 New Revision: 315701 URL: https://svnweb.freebsd.org/changeset/base/315701 Log: Set the interpreter path to /nonexistent. CloudABI executables are statically linked and don't have an interpreter. Setting the interpreter path to NULL used to work