On 06/18/2015 10:33 PM, Malahal Naineni wrote:
> Soumya Koduri [skod...@redhat.com] wrote:
>>
>> I thought pthread_exit() always returns a pointer which gets assigned to
>> retval of pthread_join(). I assume this is the flow -->
>
> pthread_exit() does take a "void *", since it is "void *", it is
Soumya Koduri [skod...@redhat.com] wrote:
>
> I thought pthread_exit() always returns a pointer which gets assigned to
> retval of pthread_join(). I assume this is the flow -->
pthread_exit() does take a "void *", since it is "void *", it is up to
you whether to really pass some pointer or some
On 06/18/2015 10:35 AM, Malahal Naineni wrote:
> Soumya Koduri [skod...@redhat.com] wrote:
>> Right. But even on success, "retval" need not be set until the
>> thread exits with a value other than NULL.
>>
>> From the manpage,
>
>> If retval is not NULL, then pthread_join() copies th
Soumya Koduri [skod...@redhat.com] wrote:
> Right. But even on success, "retval" need not be set until the
> thread exits with a value other than NULL.
>
> From the manpage,
> >>>
>If retval is not NULL, then pthread_join() copies the exit
> status of the target thread (i.e., the value tha
On 06/18/2015 12:45 AM, Malahal Naineni wrote:
> I didn't see GLUSTERFSAL_UP_Thread() returning anything other than
> NULL/0. I think passing NULL to pthread_join is even cleaner!
>
Agree that current GLUSTERFSAL_UP_THREAD returns only NULL. But
considering the fact that we shall add more upcal
I didn't see GLUSTERFSAL_UP_Thread() returning anything other than
NULL/0. I think passing NULL to pthread_join is even cleaner!
Regards, Malahal.
Malahal Naineni [mala...@us.ibm.com] wrote:
> Soumya Koduri [skod...@redhat.com] wrote:
> > Hi Kaleb/Malahal,
> >
> > Request you to merge below FSAL
Soumya Koduri [skod...@redhat.com] wrote:
> Hi Kaleb/Malahal,
>
> Request you to merge below FSAL_GLUSTER patches into V2.2-stable branch -
>
> 366f71c - FSAL_GLUSTER: Fixed an issue with dereferencing a NULL ponter
I just looked at the patch from your other mail. I have few questions on
this pa
