Bug 164243

Comment 1 Edwin Groothuis 2012-01-17 15:10:27 UTC

Responsible Changed
From-To: freebsd-ports-bugs->miwi

miwi@ wants his PRs (via the GNATS Auto Assign Tool)

Comment 2 Edwin Groothuis 2012-01-17 15:10:29 UTC

Maintainer of emulators/dosbox,

Please note that PR ports/164243 has just been submitted.

If it contains a patch for an upgrade, an enhancement or a bug fix
you agree on, reply to this email stating that you approve the patch
and a committer will take care of it.

The full text of the PR can be found at:
    http://www.freebsd.org/cgi/query-pr.cgi?pr=ports/164243

-- 
Edwin Groothuis via the GNATS Auto Assign Tool
edwin@FreeBSD.org

Comment 3 Edwin Groothuis 2012-01-17 15:10:31 UTC

State Changed
From-To: open->feedback

Awaiting maintainers feedback (via the GNATS Auto Assign Tool)

Hi,

Thanks for this.

I've had a word with one of the devs. From the info you've given he
isn't sure if it's a problem with DOSBox or with clang.

Can you reply with the patch you made (even if it breaks things) and
the relevant compiler output?
I'm concerned that disabling optimisation (even if only for clang) is
not the right way to fix this for such a resource-hungry program.

Thanks,
Tom

Sure.

These are errors from svn trunk as they are more informative (exactly
the same with ports' version):
cpu/libcpu.a(core_dynrec.o): In function `CreateCacheBlock(CodePageHandlerDynRec*, unsigned int, unsigned long)':
/home/yuri/dosbox/src/cpu/./core_dynrec/risc_x64.h:88: relocation truncated to fit: R_X86_64_32S against symbol `Segs' defined in .bss section in cpu/libcpu.a(cpu.o)
/home/yuri/dosbox/src/cpu/./core_dynrec/risc_x64.h:88: relocation truncated to fit: R_X86_64_32S against symbol `cpu_regs' defined in .bss section in cpu/libcpu.a(cpu.o)
/home/yuri/dosbox/src/cpu/./core_dynrec/risc_x64.h:88: relocation truncated to fit: R_X86_64_32S against symbol `cpu_regs' defined in .bss section in cpu/libcpu.a(cpu.o)

Patch attached (I'm pretty sure I've broken the logic there but that's
the best I could come with).


Yuri

Thanks.

The devs are looking into it.

I'll get back to you if/when I hear back from them.

Otherwise I'll chase up in a week or two and depending on how things
go I'll try to decide if your original Makefile patch is a good
interim fix.

Tom

Comment 7 Martin Wilke 2012-03-04 06:32:40 UTC

State Changed
From-To: feedback->open

Maintainer has approved.

Comment 8 Martin Wilke 2012-03-04 06:32:56 UTC

State Changed
From-To: open->feedback

tom any news here?

Yes, this doesn't seem to be a very high priority for the devs, so I
think the patch is okay for now, but would be nice to include a
warning that the performance may be affected.

Comment 10 Martin Wilke 2012-05-25 16:50:45 UTC

State Changed
From-To: feedback->closed

Committed. Thanks!

Comment 11 dfilter service 2012-05-25 16:50:58 UTC

miwi        2012-05-25 15:50:37 UTC

  FreeBSD ports repository

  Added files:
    emulators/dosbox/files patch-src-cpu_core_dynrec_risc_x64.h 
  Log:
  - Fix build with clang
  
  PR:             164243
  Submitted by:   Yuri Pankov <yuri.pankov@gmail.com>
  Approved by:    maintainer
  
  Revision  Changes    Path
  1.1       +14 -0     ports/emulators/dosbox/files/patch-src-cpu_core_dynrec_risc_x64.h (new)
_______________________________________________
cvs-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/cvs-all
To unsubscribe, send any mail to "cvs-all-unsubscribe@freebsd.org"