[Openmp-dev] [EXTERNAL] Behavior of OMP_PROC_BIND=true

Wed Jul 8 11:36:38 PDT 2020

From the spec, for omp_get_proc_bind: "The effect of this routine is to return the value of the first element of the bind-var ICV of the current task."

OMP_PROC_BIND value is used to determine how to set bind-var ICV.  At some point, "true" has to be interpreted into an actual strategy, and that is what is stored in the ICV.

-----Original Message-----
From: Openmp-dev <openmp-dev-bounces at lists.llvm.org> On Behalf Of Lewis, Cannada via Openmp-dev
Sent: Wednesday, July 8, 2020 11:24 AM
To: Churbanov, Andrey <Andrey.Churbanov at intel.com>
Cc: openmp-dev at lists.llvm.org
Subject: Re: [Openmp-dev] [EXTERNAL] Behavior of OMP_PROC_BIND=true

Andrey, 

How do you interpret the following? 

"The behavior of the program is implementation defined if the value in the OMP_PROC_BIND environment variable is not true, false, or a comma separated list of master, close, or spread. The behavior is also implementation defined if an initial thread cannot be bound to the first place in the OpenMP place list.” 

My issue is that I have a fake omp library that needs to return the same value as a real omp library from functions like omp_get_proc_bind, omp_get_max_threads, and omp_get_num_places if it is invoked with the same environmental variables.  When implementing this fake omp, I interpreted the above quote to mean that if the OMP_PROC_BIND variable is set, then omp_get_proc_bind should return the value that corresponds to what the user requested. 

My question boils down to: When OMP_PROC_BIND is used is the return value of omp_get_proc_bind allowed to be implementation defined or not?

-Drew 

> On Jul 8, 2020, at 1:22 AM, Churbanov, Andrey <Andrey.Churbanov at intel.com> wrote:
> 
> The specification requires omp_get_proc_bind() to return the thread affinity policy to be used for parallel regions.
> Though TRUE is among allowed return values, it is actually not a policy but rather enabling of some policy the runtime applies to threads binding.
> So returning SPREAD looks more reasonable to me in this case.  If we would return TRUE users will never know what policy the runtime is actually using. 
> 
> Returning TRUE means for me:
> - implementation wants to apply some thread affinity policy, other 
> than standard MASTER, SPREAD or CLOSE;
> - implementation does not want to bother with inventing a name for the 
> policy;
> - and/or implementation does not want to show any details of the policy to users.
> 
> I don't think that hiding details from user is a good behavior.
> Though it may be possible that gcc has documentation with detailed description of the runtime behavior for OMP_PROC_BIND=true, I haven't checked.
> Once we return SPREAD - the standard policy, we don't need to add any extra documentation.
> 
> -- Andrey
> 
> -----Original Message-----
> From: Openmp-dev <openmp-dev-bounces at lists.llvm.org> On Behalf Of 
> Doerfert, Johannes via Openmp-dev
> Sent: Wednesday, July 8, 2020 2:09 AM
> To: Lewis, Cannada <canlewi at sandia.gov>
> Cc: openmp-dev at lists.llvm.org
> Subject: Re: [Openmp-dev] [EXTERNAL] Behavior of OMP_PROC_BIND=true
> 
> On 7/7/20 5:54 PM, Lewis, Cannada wrote:
>> I found where my confusion was coming from, but I would like clarification that omp_get_proc_bind is doing the right thing:
>> 
>> At line 5473 in kmp_settings.cpp
>> 
>>     } else if (__kmp_nested_proc_bind.bind_types[0] == proc_bind_true) {
>>       // OMP_PROC_BIND=true maps to OMP_PROC_BIND=spread.
>>       __kmp_nested_proc_bind.bind_types[0] = proc_bind_spread;
>>     }
>> 
>> Even if libomp uses spread internally for the case of OMP_PROC_BIND=true shouldn’t omp_get_proc_bind still report that the value true was provided instead of the value for spread?
> 
> I can see that we might want to do that but is it really more helpful?
> 
> 
> ~ Johannes
> 
> 
>> 
>> -Drew
>> 
>> On Jul 6, 2020, at 2:13 PM, Lewis, Cannada via Openmp-dev <openmp-dev at lists.llvm.org<mailto:openmp-dev at lists.llvm.org>> wrote:
>> 
>> I am writing to figure out what is the expected result from get_omp_proc_bind when the environmental variable OMP_PROC_BIND is true.  Based on https://www.openmp.org/spec-html/5.0/openmpse52.html I was expecting the result to be 1, which is what I get when using g++ version 9.2, but for clang++ I get 4, which is the same as spread.  Any help would be appreciated since I am trying to match up results from an app compiled with clang and a performance model that currently returns 1 for get_omp_proc_bind when OMP_PROC_BIND=true.
>> 
>> -Drew
>> _______________________________________________
>> Openmp-dev mailing list
>> Openmp-dev at lists.llvm.org<mailto:Openmp-dev at lists.llvm.org>
>> https://lists.llvm.org/cgi-bin/mailman/listinfo/openmp-dev
>> 
> 
> _______________________________________________
> Openmp-dev mailing list
> Openmp-dev at lists.llvm.org
> https://lists.llvm.org/cgi-bin/mailman/listinfo/openmp-dev
> 
> --------------------------------------------------------------------
> Joint Stock Company Intel A/O
> Registered legal address: Krylatsky Hills Business Park,
> 17 Krylatskaya Str., Bldg 4, Moscow 121614, Russian Federation
> 
> This e-mail and any attachments may contain confidential material for 
> the sole use of the intended recipient(s). Any review or distribution 
> by others is strictly prohibited. If you are not the intended 
> recipient, please contact the sender and delete all copies.

_______________________________________________
Openmp-dev mailing list
Openmp-dev at lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/openmp-dev