Bug #10691
closed
ceph-dencoder links to libtcmalloc, and shouldn't
Added by Ken Dreyer over 9 years ago.
Updated over 8 years ago.
Description
The /usr/bin/ceph-dencoder binary links to libtcmalloc, and it has no need to link to it.
From a packaging perspective, since /usr/bin/ceph-dencoder ships in the "ceph-common" RPM, this means that ceph-common requires the gperftools-libs RPM to be installed. Ideally we'd avoid this dependency for ceph-common.
- Status changed from New to 12
Looks like the linkage is sneaking in via the autotools LIBMDS variable (which itself includes LIBPERFGLUE). We don't actually use LIBMDS very often, and most of the places also don't want tcmalloc, so we can probably just drop LIBPERFGLUE from LIBMDS and include it explicitly in the MDS build.
Thanks Greg, I think you're right. Yesterday evening when I talked with Dan about this, he was musing whether it would make sense to split LIBMDS to LIBMDS_TYPES, similar to what was done in 1c170776cb8c0266f0f54f049ed07bbdb9c9ab5e ? The commit log is a little terse there but we were thinking that commit was done in order to reduce ceph-dencoder linkage.
I'd love to have a LIBMDS_TYPES, but the reason we didn't do so already is that it's much harder than for the other systems. The MDS base types, their in-memory representations, and their code are a lot more tangled together than in the other systems. We're closer than we were back then but I wouldn't try out that avenue. :(
- Status changed from 12 to Fix Under Review
- Status changed from Fix Under Review to Won't Fix
fixed downstream. if we want to resolve this later we can use their approach (two version of libmds.la basically)
@Sage Weil, this is still an issue - could we keep this tracker open?
- Status changed from Won't Fix to Fix Under Review
- Status changed from Fix Under Review to Resolved
- Status changed from Resolved to Pending Backport
- Backport set to hammer
- Status changed from Pending Backport to Resolved
- Regression set to No
- Status changed from Resolved to Pending Backport
- Backport changed from hammer to hammer,firefly
- Status changed from Pending Backport to Resolved
Also available in: Atom
PDF