Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 309755 - sys-cluster/maui patch to correct an issue with maui not passing correct resource string to torque with recent glibc versions
Summary: sys-cluster/maui patch to correct an issue with maui not passing correct reso...
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: AMD64 Linux
: High normal (vote)
Assignee: Gentoo Cluster Team
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-03-16 13:28 UTC by Matthias Schoepfer
Modified: 2010-09-10 19:01 UTC (History)
0 users

See Also:
Package list:
Runtime testing required: ---


Attachments
adds patch that fixes wrong resource string (maui-3.2.6_p21.ebuild,1.07 KB, text/plain)
2010-03-16 13:41 UTC, Matthias Schoepfer
Details
Patch to fix the wrong sprintf into a correct snprintf which will generate the resource string (3.2.6_p21-fix-pbs_nodefile.patch,554 bytes, patch)
2010-03-16 13:42 UTC, Matthias Schoepfer
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Matthias Schoepfer 2010-03-16 13:28:42 UTC
When I request

qsub -l nodes=5:ppn=8 script.sh

maui will reserve 5 nodes, but my program is eventually only executed on
one node, and also the PBS_NODEFILE is wrong (as it only mentions one
node). 

Alexis Cousein found the problem in MPBSI.c, where the resource string is generated in an incorrect way (obviously only manifesting with a recent glibc).

I would like to submit the patch to the maui ebuild


Reproducible: Always

Steps to Reproduce:
1. emerge maui
2. qsub -l nodes=10:ppn=2 <whatever-openmpi-thing>


Actual Results:  
job will only run on 1 node.

Expected Results:  
jpb spreads according to maui's resource allocation
Comment 1 Matthias Schoepfer 2010-03-16 13:41:38 UTC
Created attachment 223879 [details]
adds patch that fixes wrong resource string
Comment 2 Matthias Schoepfer 2010-03-16 13:42:28 UTC
Created attachment 223881 [details, diff]
Patch to fix the wrong sprintf into a correct snprintf which will generate the resource string
Comment 3 Justin Bronder (RETIRED) gentoo-dev 2010-03-23 22:20:59 UTC
Has this been submitted/discussed upstream?
Comment 4 Matthias Schoepfer 2010-03-24 07:18:25 UTC
(In reply to comment #3)
> Has this been submitted/discussed upstream?
> 

It has been discussed on the maui-users mailinglist. I don't see any other possibilities right now to contact the developers. 

I cannot tell if and when the patch is merged upstream. As far as I understand, the guys at supercluster.org have a focus on moab, not maui :-(
Comment 5 Justin Bronder (RETIRED) gentoo-dev 2010-03-24 21:56:15 UTC
Found the thread.

http://www.supercluster.org/pipermail/mauiusers/2010-March/004174.html
Comment 6 Justin Bronder (RETIRED) gentoo-dev 2010-03-24 22:30:36 UTC
+*maui-3.2.6_p21-r1 (24 Mar 2010)
+
+  24 Mar 2010; Justin Bronder <jsbronder@gentoo.org>
+  +maui-3.2.6_p21-r1.ebuild, +files/maui-3.2.6_p21-pbs-nodefile.patch:
+  Fix incorrect PBS_NODEFILE with newer glibc fix. #309755
+

Thanks for reporting.