diff mbox series

Fortran: invoke.texi - link to OpenCoarrays.org + mention libcaf_single

Message ID 18e91ab2-7ddb-47d8-9383-d4c7689eecd9@baylibre.com
State New
Headers show
Series Fortran: invoke.texi - link to OpenCoarrays.org + mention libcaf_single | expand

Commit Message

Tobias Burnus May 19, 2024, 8:01 a.m. UTC 
I noticed that gfortran's coarray support did not link to the 
http://www.opencoarrays.org/

As that library is needed to support parallelization, it makes sense to 
have the link.

Motivated by someone claiming at ISC-HPC that GCC only supports a single 
image.

And also motivated by Damian's presentation, which showed that 
gfortran's coarrays could successfully run the ICAR atmospheric model 
with 25,600 processes (OpenCoarrays with OpenSHMEM backend), which 
definitely is more than one image :-)

I think mentioning the existing libcaf_single is still useful, even 
though it is only of limited use (except that it does ship with GCC and 
permits to do some testings. Especially, it is used by GCC's testsuite).

OK for mainline?

Tobias

Comments

FX Coudert May 19, 2024, 10:24 a.m. UTC | #1 
Hi Tobias,

> OK for mainline?

Seems reasonable, OK to push in 48 hours unless someone has suggestions related to wording.

FX
Sandra Loosemore May 20, 2024, 12:31 a.m. UTC | #2 
On 5/19/24 02:01, Tobias Burnus wrote:
> I noticed that gfortran's coarray support did not link to the 
> http://www.opencoarrays.org/ >
> [snip]
> 
> diff --git a/gcc/fortran/invoke.texi b/gcc/fortran/invoke.texi
> index 40e8e4a7cdd..78a2910b8d8 100644
> --- a/gcc/fortran/invoke.texi
> +++ b/gcc/fortran/invoke.texi
> @@ -1753,7 +1753,10 @@ Single-image mode, i.e. @code{num_images()} is always one.
>  
>  @item @samp{lib}
>  Library-based coarray parallelization; a suitable GNU Fortran coarray
> -library needs to be linked.
> +library needs to be linked such as @url{http://opencoarrays.org}.

This would read better as

library such as @url{http://opencoarrays.org} needs to be linked.

> +Alternatively, GCC's @code{libcaf_single} library can be linked,
> +albeit it only supports a single image.
> +
>  @end table

OK with that tweak.

-Sandra
Bernhard Reutner-Fischer May 21, 2024, 7:28 a.m. UTC | #3 
On 20 May 2024 02:31:27 CEST, Sandra Loosemore <sloosemore@baylibre.com> wrote:
>On 5/19/24 02:01, Tobias Burnus wrote:
>> I noticed that gfortran's coarray support did not link to the http://www.opencoarrays.org/ >
>> [snip]
>> 
>> diff --git a/gcc/fortran/invoke.texi b/gcc/fortran/invoke.texi
>> index 40e8e4a7cdd..78a2910b8d8 100644
>> --- a/gcc/fortran/invoke.texi
>> +++ b/gcc/fortran/invoke.texi
>> @@ -1753,7 +1753,10 @@ Single-image mode, i.e. @code{num_images()} is always one.
>>   @item @samp{lib}
>>  Library-based coarray parallelization; a suitable GNU Fortran coarray
>> -library needs to be linked.
>> +library needs to be linked such as @url{http://opencoarrays.org}.
>
>This would read better as
>
>library such as @url{http://opencoarrays.org} needs to be linked.

Maybe use https?

thanks

>
>> +Alternatively, GCC's @code{libcaf_single} library can be linked,
>> +albeit it only supports a single image.
>> +
>>  @end table
>
>OK with that tweak.
>
>-Sandra
>
>
>
Tobias Burnus May 21, 2024, 8:20 a.m. UTC | #4 
Hi Bernhard,

rep.dot.nop@gmail.com wrote:
>> library such as @url{http://opencoarrays.org} needs to be linked.
> Maybe use https?

Works, but as the certificate is not valid, it requires to ignore the 
errors in a browser, which is a worse user experience.

The error is, e.g.,

"curl: (60) SSL certificate problem: self-signed certificate"

Or at 
https://www.ssllabs.com/ssltest/analyze.html?d=www.opencoarrays.org&ignoreMismatch=on&latest

"Common names: invalid-sni.invalid / Issuer: invalid-sni.invalid  
(Self-signed)"

@Damian: Can you fix the server to actually have a valid certificate?

Tobias
diff mbox series

Patch

Fortran: invoke.texi - link to OpenCoarrays.org + mention libcaf_single

gcc/fortran/ChangeLog:

	* invoke.texi (fcoarray): Link to OpenCoarrays.org;
	mention libcaf_single.

 gcc/fortran/invoke.texi | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/gcc/fortran/invoke.texi b/gcc/fortran/invoke.texi
index 40e8e4a7cdd..78a2910b8d8 100644
--- a/gcc/fortran/invoke.texi
+++ b/gcc/fortran/invoke.texi
@@ -1753,7 +1753,10 @@  Single-image mode, i.e. @code{num_images()} is always one.
 
 @item @samp{lib}
 Library-based coarray parallelization; a suitable GNU Fortran coarray
-library needs to be linked.
+library needs to be linked such as @url{http://opencoarrays.org}.
+Alternatively, GCC's @code{libcaf_single} library can be linked,
+albeit it only supports a single image.
+
 @end table