Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 20 May 2020 14:39:55 -0600
From:      Scott Long <scottl@samsco.org>
To:        "Conrad E. Meyer" <cem@FreeBSD.org>
Cc:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org
Subject:   Re: svn commit: r361284 - in head/sys/amd64: include vmm
Message-ID:  <0790972B-65C1-4C88-A4F2-FEFE656F2E66@samsco.org>
In-Reply-To: <202005201727.04KHRsVL047061@repo.freebsd.org>
References:  <202005201727.04KHRsVL047061@repo.freebsd.org>

index | next in thread | previous in thread | raw e-mail

There’s a better way to do this.  People have been wanting larger names to
be merged into 12.x and 11.x, and that was going to be hard with how I
originally did this.  Your compat shim doesn’t make it any easier.  What I was
going to do was return the ioctl in HEAD to the original size, and create a new
ioctl for the larger size,  That would fix the compatibility issue (since we don’t
support a newer userland on an older kernel) and make it possible to merge
to -stable branches.  I’d like to revert this commit and go in that direction.

Scott


> On May 20, 2020, at 11:27 AM, Conrad Meyer <cem@FreeBSD.org> wrote:
> 
> Author: cem
> Date: Wed May 20 17:27:54 2020
> New Revision: 361284
> URL: https://svnweb.freebsd.org/changeset/base/361284
> 
> Log:
>  vmm(4): Add 12 user ABI compat after r349948
> 
>  Reported by:	kp
>  Reviewed by:	jhb, kp
>  Tested by:	kp
>  Differential Revision:	https://reviews.freebsd.org/D24929
> 
> Modified:
>  head/sys/amd64/include/vmm_dev.h
>  head/sys/amd64/vmm/vmm_dev.c


help

Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?0790972B-65C1-4C88-A4F2-FEFE656F2E66>