This patch has been rolled into the source trunk in Subversion and will be available in the future Maui patches. The current snapshot, http://www.clusterresources.com/downloads/maui/temp/maui-3.2.6p20-snap.1182974819.tar.gz, also has this patch included.
Your name has been added to the CHANGELOG. Thanks for your contribution! Regards, -- Joshua Butikofer Cluster Resources, Inc. [EMAIL PROTECTED] Voice: (801) 717-3707 Fax: (801) 717-3738 -------------------------- Jeffrey B. Reed wrote: > The enclosed patch addresses two separate issues: > > 1. Generic Consumable Floating Resources do not work properly when > a job also requests the `mem' resource. > > 2. Typos found in the initialization of the RQ data structure in the > MPBSJobAdjustResources subroutine. > > Generic Consumable Floating Resources Issue Description > ------------------------------------------------------- > > We configured maui to have a Generic Consumable Floating Resource > > NODECFG[GLOBAL] GRES=hsimplus:1 > > We then attempted to request this resource, see following example, the > job was > promptly submitted and the hsimplus resource was allocated. > > % qsub -l software=hsimplus test_job > > However, when we also requested a particular memory size, see following > example, > the job would never get submitted due to lack of memory resource of the > GLOBAL node. > > % qsub -l software=hsimplus,mem=2G test_job > > > ------------------------------------------------------------------------ > > --- /export/home/tomr/maui-3.2.6p19.orig/src/moab/MPBSI.c 2007-03-28 > 15:05:39.000000000 -0500 > +++ /export/home/tomr/src/rpm/SOURCES/maui-3.2.6p19/src/moab/MPBSI.c > 2007-06-26 11:35:14.000000000 -0500 > @@ -4249,7 +4249,7 @@ > > /* adjust 'per task limits */ > > - if (MaxJobMem > 0) > + if ((MaxJobMem > 0) && (RQ->DRes.Procs)) > { > /* set job wide dedicated resources */ > > @@ -4261,7 +4261,7 @@ > RQ->URes.Mem /= RQ->TaskCount; > } > > - if (MaxJobSwap > 0) > + if ((MaxJobSwap > 0) && (RQ->DRes.Procs)) > { > /* set both dedicated resources AND node requirements */ > > @@ -6034,10 +6034,11 @@ > > if (TA != NULL) > { > - if (TA->JobMemLimit > 0) > + if ((TA->JobMemLimit > 0) && (RQ->DRes.Procs)) { > RQ->DRes.Mem = MAX(RQ->DRes.Mem,TA->JobMemLimit / RQ->TaskCount); > + } > > - if (TA->JobSwapLimit > 0) > + if ((TA->JobSwapLimit > 0) && (RQ->DRes.Procs)) > RQ->DRes.Swap = MAX(RQ->DRes.Swap,TA->JobSwapLimit / RQ->TaskCount); > } /* END if (TA != NULL) */ > > @@ -6054,9 +6055,9 @@ > > RQ->RequiredMemory = J->Req[0]->RequiredMemory; > RQ->MemCmp = J->Req[0]->MemCmp; > - RQ->RequiredSwap = J->Req[0]->RequiredMemory; > + RQ->RequiredSwap = J->Req[0]->RequiredSwap; > RQ->SwapCmp = J->Req[0]->SwapCmp; > - RQ->RequiredDisk = J->Req[0]->RequiredMemory; > + RQ->RequiredDisk = J->Req[0]->RequiredDisk; > RQ->DiskCmp = J->Req[0]->DiskCmp; > } > > > > ------------------------------------------------------------------------ > > _______________________________________________ > mauiusers mailing list > mauiusers@supercluster.org > http://www.supercluster.org/mailman/listinfo/mauiusers _______________________________________________ mauiusers mailing list mauiusers@supercluster.org http://www.supercluster.org/mailman/listinfo/mauiusers