Re: [Gluster-infra] [Gluster-devel] Gluster and GCC 5.1

2015-07-06 Thread Kaleb S. KEITHLEY
-BEGIN PGP SIGNED MESSAGE- Hash: SHA256 On 07/05/2015 06:54 PM, Michael Scherer wrote: Le lundi 29 juin 2015 à 18:48 -0400, Kaleb Keithley a écrit : From: Niels de Vos nde...@redhat.com On Mon, Jun 29, 2015 at 02:45:50PM -0400, Kaleb S. KEITHLEY wrote: On 06/29/2015 02:40 PM,

Re: [Gluster-infra] [Gluster-devel] Gluster and GCC 5.1

2015-07-05 Thread Michael Scherer
Le lundi 29 juin 2015 à 18:48 -0400, Kaleb Keithley a écrit : From: Niels de Vos nde...@redhat.com On Mon, Jun 29, 2015 at 02:45:50PM -0400, Kaleb S. KEITHLEY wrote: On 06/29/2015 02:40 PM, Anoop C S wrote: Niels and I had a brief chat a while back about adding a Fedora

Re: [Gluster-infra] [Gluster-devel] Gluster and GCC 5.1

2015-07-03 Thread Kaleb Keithley
- Original Message - From: Joseph Fernandes josfe...@redhat.com Agree with Jeff But now the question is should we have this patch go through or leave it to float in time and space :) http://review.gluster.org/#/c/11214/ The patch just makes the existing inline calls static or

Re: [Gluster-infra] [Gluster-devel] Gluster and GCC 5.1

2015-07-02 Thread Jeff Darcy
Or perhaps we could just get everyone to stop using 'inline' I agree that it would be a good thing to reduce/modify our use of 'inline' significantly. Any advantage gained from avoiding normal function-all entry/exit has to be weighed against cache pollution from having the same code repeated

Re: [Gluster-infra] [Gluster-devel] Gluster and GCC 5.1

2015-07-02 Thread Joseph Fernandes
Agree with Jeff But now the question is should we have this patch go through or leave it to float in time and space :) http://review.gluster.org/#/c/11214/ The patch just makes the existing inline calls static or extern appropriately without causing any harm to the existing code but removes the

Re: [Gluster-infra] [Gluster-devel] Gluster and GCC 5.1

2015-06-30 Thread Joseph Fernandes
-devel] Gluster and GCC 5.1 From: Niels de Vos nde...@redhat.com On Mon, Jun 29, 2015 at 02:45:50PM -0400, Kaleb S. KEITHLEY wrote: On 06/29/2015 02:40 PM, Anoop C S wrote: Niels and I had a brief chat a while back about adding a Fedora 22 VM to Jenkins to catch these. I don't

Re: [Gluster-infra] [Gluster-devel] Gluster and GCC 5.1

2015-06-29 Thread Kaleb S. KEITHLEY
On 06/29/2015 02:45 PM, Kaleb S. KEITHLEY wrote: E.g. by building on Fedora Rawhide last year, reporting the bugs, and getting them fixed before Rawhide turned into Fedora 22. Immediately after the Fedora 22 release, we had a similar issue reported from a user with gcc v5 and was fixed

Re: [Gluster-infra] [Gluster-devel] Gluster and GCC 5.1

2015-06-29 Thread Kaleb Keithley
From: Niels de Vos nde...@redhat.com On Mon, Jun 29, 2015 at 02:45:50PM -0400, Kaleb S. KEITHLEY wrote: On 06/29/2015 02:40 PM, Anoop C S wrote: Niels and I had a brief chat a while back about adding a Fedora 22 VM to Jenkins to catch these. I don't think anything ever happened