Bug 197403

Summary:	net-im/ejabberd - add hipe USE flag
Product:	Gentoo Linux	Reporter:	Samuel Tardieu <sam>
Component:	New packages	Assignee:	Caleb Tennis (RETIRED) <caleb>
Status:	RESOLVED NEEDINFO
Severity:	enhancement	CC:	dschridde+gentoobugs, net-im, pva
Priority:	High
Version:	unspecified
Hardware:	All
OS:	Linux
URL:	https://support.process-one.net/browse/EJAB-412
Whiteboard:
Package list:		Runtime testing required:	---
Attachments:	Add hipe USE flag

Description Samuel Tardieu 2007-10-29 11:34:39 UTC

This patch adds the ability to compile ejabberd with +native (HiPE) to get better performance (native code). The relevant part has just been submitted upstream.

Reproducible: Always

Steps to Reproduce:




Submitted upstream as https://support.process-one.net/browse/EJAB-412

Comment 1 Samuel Tardieu 2007-10-29 11:35:42 UTC

Created attachment 134626 [details, diff]
Add hipe USE flag

Patch that adds hipe USE flag to ejabberd. Apply after copying ejabberd-1.1.4.ebuild into ejabberd-1.1.4-r1.ebuild.

Comment 2 Tony Vroon (RETIRED) gentoo-dev

2007-10-29 12:02:45 UTC

Will look at this. As the first question, how will it cope with dev-lang/erlang built with USE="-hipe"? (It is still hard to gracefully depend on a USE-flag being set elsewhere)

Comment 3 Samuel Tardieu 2007-10-29 12:35:51 UTC

If erlang is not compiled with "hipe" and ejabberd is, you get a warning (not an error) at compilation time and the modules are compiled in non-native mode. Not a big deal.

Comment 4 Christian Faulhammer (RETIRED) gentoo-dev

2009-06-03 06:06:55 UTC

(In reply to comment #2)
> Will look at this. As the first question, how will it cope with dev-lang/erlang
> built with USE="-hipe"? (It is still hard to gracefully depend on a USE-flag
> being set elsewhere)

 We have EAPI 2 meanwhile to handle those cases.

Comment 5 Peter Volkov (RETIRED) gentoo-dev

2010-01-26 07:50:56 UTC

According to upstream, this feature produces no measurable performance gain but it adds problems (e.g sometimes this produces segfaults (probably fixed or hipe+SMP). Although patch taken upstream they clearly state that this is not recommended:

  --enable-hipe           compile natively with HiPE, not recommended

Thus I see no reason adding this into ebuild. In case you want to experiment, just put EXTRA_ECONF="--enable-hipe" into /etc/portage/env/net-im/ejabberd and rebuild ejabberd to get it built with hipe. If you want this feature, please, provide test result of how this is useful. Currently this is NEEDINFO.