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
