On Fri, Oct 29, 2021 at 8:42 AM Daniel P. Berrangé <berra...@redhat.com>
wrote:

> On Fri, Oct 29, 2021 at 09:32:16PM +0900, Mamoru TASAKA wrote:
> > Daniel P. Berrangé wrote on 2021/10/29 21:15:
> > > On Fri, Oct 29, 2021 at 07:55:08AM -0400, Kaleb Keithley wrote:
> > > > On Fri, Oct 29, 2021 at 6:52 AM Richard W.M. Jones <
> rjo...@redhat.com>
> > > > wrote:
> > > >
> > > > > On Thu, Oct 28, 2021 at 11:16:42AM -0400, Kaleb Keithley wrote:
> > > > > >
> > > > > > Perfect. Thank you Mamoru-san.
> > > > > >
> > > > > > armv7hl is now back in.
> > > > >
> > > > > glusterfs library is totally broken on armv7, aarch64, s390x at
> the moment:
> > > > >
> > > >
> > > > Are you saying it wasn't broken on aarch64 and s390x before adding
> Mamoru's
> > > > armv7hl fix?  I.e. specifically 10.0-0.1rc0 ?
> > >
> > > I thought it was triggered by the introduction of libtcmalloc, but now
> > > I look back at the previuous libvirt build
> > >
> > > https://koji.fedoraproject.org/koji/taskinfo?taskID=77969720
> > >
> > >    glusterfs-10.0-0.1rc0.fc36
> > >
> > > was ok on aarch64 and s390x.
> > >
> > > so in fact I think it may well be that change of Mamoru that
> > > has somehow triggered increased static TLS usage and thus
> > > broken ability to dlopen libgluestfs.so
> > >
> >
> > But my suggestion is only applied on armv7hl, not on aarch64 or s390x:
> >
> https://src.fedoraproject.org/rpms/glusterfs/c/8e29421e267d21194f9d7300c12b47f451d9e81a?branch=rawhide
> >
> > So I don't think this is relevant. My guess is this is related to
> tcmalloc side.
>
> The gperftools NEVR is the same in both builds, so I wonder what else
> has changed that might cause it. I see changes in gcc, glibc, and
> libatomic between the two glusterfs builds but not sure if any of
> those are culprits
>

I'm having a really tough time understanding how adding a -D... to CFLAGS
on armv7hl _only_ could possibly have changed the aarch64 and s390x builds.
Did gcc, glibc, and libatomic really all change from one day to the next in
rawhide? I guess I'll go have a look. ;-)

Regardless, there are new -0.3rc0 builds now with tcmalloc disabled on all
arches except x86_64.

-- 

Kaleb
_______________________________________________
devel mailing list -- devel@lists.fedoraproject.org
To unsubscribe send an email to devel-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/devel@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure

Reply via email to