Bug 1190870 - ceph is not built with yasm
Summary: ceph is not built with yasm
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Red Hat Ceph Storage
Classification: Red Hat Storage
Component: Build
Version: 1.2.2
Hardware: Unspecified
OS: Unspecified
unspecified
medium
Target Milestone: pre-dev-freeze
: 1.2.3
Assignee: Ken Dreyer (Red Hat)
QA Contact: ceph-qa-bugs
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2015-02-09 20:03 UTC by Ken Dreyer (Red Hat)
Modified: 2022-08-11 07:55 UTC (History)
2 users (show)

Fixed In Version: ceph-0.80.8-4.el7cp
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2015-02-10 01:31:51 UTC
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Red Hat Bugzilla 1190882 0 unspecified CLOSED ceph is not built with yasm 2021-02-22 00:41:40 UTC
Red Hat Issue Tracker RHCEPH-3396 0 None None None 2022-02-21 18:23:13 UTC

Internal Links: 1190882

Description Ken Dreyer (Red Hat) 2015-02-09 20:03:24 UTC
Description of problem:
ceph-0.80.8-3 is not built with yasm

Version-Release number of selected component (if applicable):
ceph-0.80.8-3.el6cp
ceph-0.80.8-3.el7cp

Additional info:
yasm was initially disabled in the downstream packaging in order to solve an SELinux issue (bug 1118504). Ceph v0.80.8 contains an upstream fix that should make a yasm build operate properly with SELinux.

Comment 1 Ken Dreyer (Red Hat) 2015-02-09 20:04:12 UTC
Dan mentioned to me a while back that disabling yasm impedes Ceph's performance, so it would be ideal if we re-enabled yasm. It's not clear to me how bad the performance impact would be if we continued to disable yasm - Dan knows more about this than me.

Since ICE 1.2.2 shipped with yasm enabled in the build, and product management has clarified that SELinux support is not a blocker for Red Hat Ceph 1.2.3, I think we should re-enable yasm. Boris, do you agree ?

Comment 2 Dan Mick 2015-02-09 20:39:16 UTC
Argh, yes, we definitely want it on.  CRC and erasure coding benefit greatly from assembly.

Comment 3 Boris Ranto 2015-02-09 20:45:29 UTC
Hmm, I thought this already was enabled in RHC builds. I will have to enable it in fedora as well once I will have some time to rebase the ceph packages there.

AFAIK, it is not that important for client stuff and that is why I just disabled it in base RHEL.

Comment 4 Ken Dreyer (Red Hat) 2015-02-09 21:09:17 UTC
Thanks Boris and Dan for helping me understand this issue.

I'll re-enable the BuildRequires: yasm in the 0.80.8 packages and get a new RHC build attached to the Errata today.

So that this doesn't get lost for Fedora, I've filed https://bugzilla.redhat.com/show_bug.cgi?id=1190882 in order to track that separately.


Note You need to log in before you can comment on or make changes to this bug.