Re: [Qemu-devel] New Year's starting over ... bsd-user

[Sean Bruno]

On 01/03/17 23:53, Thomas Huth wrote:
> On 03.01.2017 18:11, Sean Bruno wrote:
>>
>>
>> On 01/03/17 09:18, Sean Bruno wrote:
>>>
>>> I'm pondering where to start with getting FreeBSD's bsd-user code into
>>> shape so it could actually be reviewed and accepted now that its sort of
>>> working again (signal handling fixed finally).
>>>
>>> I almost feel like the existing code should be purged, except that it
>>> gives a good history (and this seems lazy to me).
>>>
>>> As a first pass, I guess, I'd like to at least make i386 user run on
>>> x86_64.  What would you folks like to see in a first pass?
>>>
>>> sean
>>>
>>> ref: https://github.com/seanbruno/qemu-bsd-user/tree/bsd-user
>>>
>>
>> Primitive example of what I think I should base my patchset on.  Its
>> invasive and large.
>>
>> https://github.com/seanbruno/qemu-bsd-user/tree/merge1
>>
>> That branch, is all the bsd-user changes that are pending in one large
>> "splat".  It excludes the new architectures (arm, aarch64, mips, mips64)
>> that we are actively using.  i386-bsd-user when compiled statically on
>> x86_64 will run a static (rescue) sh ... so, I think that's good.
>> x86_64 running on x86_64 just blows up.
>>
>> As for sparc/sparc64 ... I'm tempted to delete them as nobody in freebsd
>> is actively maintaining them nor do we have any expectation that they
>> will work someday.
> 
> It's broken ... nobody maintains it ... and we've got too many
> unmaintained bit-rotten files in the QEMU tree, so IMHO just go ahead
> and send a patch to remove the bsud-user sparc support. If anybody ever
> needs it again, they can revert the commit or simply submit a patch with
> the fixed code.
> 
>  Thomas
> 
> 
> 

I agree with the exception that its way easier to nuke the sparc/64 code
after I update bsd-user.  Right now, its a tangle of #ifdef in a couple
of files that I'd like to un-tangle first.

Regardless, I can maintain the sparc/64 bsd-user target outside of qemu
until such time as someone wants to make it work or the heat death of
the universe occurs.

sean

signature.asc
Description: OpenPGP digital signature