Fix SLURM bug in rank integer sign expansion
See: https://bugs.schedmd.com/show_bug.cgi?id=19324 Reviewed-by: Aleix Roca Nonell <aleix.rocanonell@bsc.es>
This commit is contained in:
		
							parent
							
								
									8652b7403c
								
							
						
					
					
						commit
						067919c135
					
				| @ -30,5 +30,12 @@ final: prev: | ||||
|     ]; | ||||
|   }); | ||||
| 
 | ||||
|   slurm = prev.slurm.overrideAttrs (old: { | ||||
|     patches = (old.patches or []) ++ [ | ||||
|       # See https://bugs.schedmd.com/show_bug.cgi?id=19324 | ||||
|       ./slurm-rank-expansion.patch | ||||
|     ]; | ||||
|   }); | ||||
| 
 | ||||
|   prometheus-slurm-exporter = prev.callPackage ./slurm-exporter.nix { }; | ||||
| } | ||||
|  | ||||
							
								
								
									
										11
									
								
								pkgs/slurm-rank-expansion.patch
									
									
									
									
									
										Normal file
									
								
							
							
						
						
									
										11
									
								
								pkgs/slurm-rank-expansion.patch
									
									
									
									
									
										Normal file
									
								
							| @ -0,0 +1,11 @@ | ||||
| --- a/src/plugins/mpi/pmix/pmixp_dmdx.c	2024-03-15 13:05:24.815313882 +0100
 | ||||
| +++ b/src/plugins/mpi/pmix/pmixp_dmdx.c	2024-03-15 13:09:53.936900823 +0100
 | ||||
| @@ -314,7 +314,7 @@ static void _dmdx_req(buf_t *buf, int no
 | ||||
|  	} | ||||
|   | ||||
|  	nsptr = pmixp_nspaces_local(); | ||||
| -	if (nsptr->ntasks <= rank) {
 | ||||
| +	if ((long) nsptr->ntasks <= (long) rank) {
 | ||||
|  		char *nodename = pmixp_info_job_host(nodeid); | ||||
|  		PMIXP_ERROR("Bad request from %s: nspace \"%s\" has only %d ranks, asked for %d", | ||||
|  			    nodename, ns, nsptr->ntasks, rank); | ||||
		Loading…
	
	
			
			x
			
			
		
	
		Reference in New Issue
	
	Block a user