From aaf649ab32bfe9a6e215c2534d0a94834c9d445e Mon Sep 17 00:00:00 2001 From: Pierre Veyre Date: Tue, 1 Apr 2014 08:25:59 +0200 Subject: [PATCH] Add bug-17132 tesh suite --- teshsuite/bug-17132/CMakeLists.txt | 37 +++++++++++++++++++++++ teshsuite/bug-17132/README | 8 +++++ teshsuite/bug-17132/bug-17132 | Bin 0 -> 18131 bytes teshsuite/bug-17132/bug-17132.c | 40 +++++++++++++++++++++++++ teshsuite/bug-17132/bug-17132.tesh | 8 +++++ teshsuite/bug-17132/hostfile.txt | 2 ++ teshsuite/bug-17132/small_platform.xml | 15 ++++++++++ 7 files changed, 110 insertions(+) create mode 100644 teshsuite/bug-17132/CMakeLists.txt create mode 100644 teshsuite/bug-17132/README create mode 100755 teshsuite/bug-17132/bug-17132 create mode 100644 teshsuite/bug-17132/bug-17132.c create mode 100644 teshsuite/bug-17132/bug-17132.tesh create mode 100644 teshsuite/bug-17132/hostfile.txt create mode 100644 teshsuite/bug-17132/small_platform.xml diff --git a/teshsuite/bug-17132/CMakeLists.txt b/teshsuite/bug-17132/CMakeLists.txt new file mode 100644 index 0000000000..d2d8163401 --- /dev/null +++ b/teshsuite/bug-17132/CMakeLists.txt @@ -0,0 +1,37 @@ +cmake_minimum_required(VERSION 2.6) + +if(enable_smpi) + if(WIN32) + set(CMAKE_C_FLAGS "-include ${CMAKE_HOME_DIRECTORY}/include/smpi/smpi_main.h") + else() + set(CMAKE_C_COMPILER "${CMAKE_BINARY_DIR}/smpi_script/bin/smpicc") + endif() + + + set(EXECUTABLE_OUTPUT_PATH "${CMAKE_CURRENT_BINARY_DIR}") + + add_executable(bug-17132 ${CMAKE_HOME_DIRECTORY}/teshsuite/bug-17132/bug-17132.c) + target_link_libraries(bug-17132 simgrid) + + set(tesh_files + ${tesh_files} + ${CMAKE_CURRENT_SOURCE_DIR}/bug-17132.tesh + PARENT_SCOPE) + set(xml_files + ${xml_files} + ${CMAKE_CURRENT_SOURCE_DIR}/small_platform.xml + PARENT_SCOPE) + set(teshsuite_src + ${teshsuite_src} + ${CMAKE_CURRENT_SOURCE_DIR}/bug-17132.c + PARENT_SCOPE) + set(bin_files + ${bin_files} + PARENT_SCOPE) + set(txt_files + ${txt_files} + ${CMAKE_CURRENT_SOURCE_DIR}/hostfile.txt + ${CMAKE_CURRENT_SOURCE_DIR}/README + PARENT_SCOPE) + +endif(enable_smpi) diff --git a/teshsuite/bug-17132/README b/teshsuite/bug-17132/README new file mode 100644 index 0000000000..22e35b9bb3 --- /dev/null +++ b/teshsuite/bug-17132/README @@ -0,0 +1,8 @@ +This is the bug #17132 described on gforge [1]. This small SMPI code +triggers an issue in SURF, which is still to be debugged. + +The problem seems to be related to the order of events, as changing it +(with another platform or another message size or a MPI_barrier in +between) fixes the problem. + +[1] https://gforge.inria.fr/tracker/index.php?func=detail&aid=17132&group_id=12&atid=165 \ No newline at end of file diff --git a/teshsuite/bug-17132/bug-17132 b/teshsuite/bug-17132/bug-17132 new file mode 100755 index 0000000000000000000000000000000000000000..66d42543ec087c206c974be266d6c0f76d3cf107 GIT binary patch literal 18131 zcmeHOeRN#Kb)UB%(yl&MD}Nzl%U&B1M2c6Eag6w~SC(wAK(-ZI0#k6FR=X={gS0F5 z!`Rd*3Ic-SI58=6H z?!JAyBFP{9qvvSsojdn8_ujcPZ{EDoeB8XbXN$uy1P6!MAc$+O4oOJGLL6Ir|bBL-G(=*yMQ;(K&g1Yc)cPOCU5t0d}%M~n`vWG;eyg~@+q@pkT z6w6eIfCNa7e0{W0vY-Pp!L*qiLI`QfdgLDIO>4brt;f`_{b$Pcqq@M!C(ECC*M(OEV$%2Mtg6TR13#P31e&~^(e`;Vqck1%yYL~+EQI`2Rw2USO z*RN|Cjjb6?By;=L>}y}YX8pQgCKX&O>+Od)$!^`=FBQ2R8^Dpzi}0hSru^#fKjWNS z|I|+=W}VBQtu`Kiu<=)K)FY2Cv1*aGYP0*R_qjtMKr(*;=tBIK;77KX z;&(QF)Nk}76DyInhJa>X`su5_>uqPeVfXTXx$^mA4?h3N z1J`>yfBwY4mv>&59k{e}^Uq$cUG&@+)($@N$}>NH{tLC2e5U$^mhTT+?OM|}gIR3_ zboyJMR5yF*0uZo+;HjO;;hkt)s=Ga0GzWg}9QZ3RP|nY0-~qdF@Ehi^vuh6dgTTw# z9|K;_|DVkvzhaJZKO%9TxJ$n$P?upX%j0F5Y?i+D#VbapF+G^)QZ*`g+HNl zpOSWJgA-;GIXVQqUet*r+8=&DuhsINRmJjuEAa*~VYBm3(oTao zqVZl`z*j))E#>li9M*R1{(K1d0@UlU_Q#>^+^caO=QS)&Xk2LdDV^~J-~qg0c0Zrf z{+!W%vOfajLnbjcoKD1OP`EJI)+Wp=_L@86!--5bp6(otWHRxLFuVJWS@G3VN$;M;mSR|1&Wp?L|ZQIPNckJxx5|mF~v%UCmD!tz{)2WoQx+Rf} zj3#c1OTDfg{hN9=%Vakdi%Jhiqh^Mx+(z7Zb|f8-$R=ndK1w<3#dtiOPNh*nf^-GR z8!;9c9Zf~WP&ytL@qMsp#$%Ce1d@Z9j1mNtjER6(2b$CxlBBUY_iGL%GVd|fyT00lI=Q2>nLf{jHZ&AY&sXsrqXy|TYI`U zb((8~YlH9O?gi?Q{|Ml|NGWtL{EGis5t64s5{n>A-v$b~vlCv@cWGm1Rt`vk$-<_PC?%d4&0v!4=@PYp!+#}lgJi_fCabCPns>b_50jv!O0efb_#2my-EG13R7v6k7QD`4XT*YAzgywSmp#)9(E{Pg?|F_F z6lULd|Kfm{J@@N~>sE%4B7TGfW=_o_ock5xsp)1;O8#Er>58o$q$65o(oU@`gCt!_laXPqJ~dAOao>3#0i1# z!}LZ7zA`B>3bu00af8ckp*dJa)<8*lPj4wR(!i|nS#~(oX zQuvKM4^h_ask)UDggrnU%RhX^3Hk8UBjJZ%+88!YhNpfIe)63h^uJBA;oE+aW&P*0 z{>0@=Q6G`3?}z5}U&FyYKX6w7GG0GU&nG;2`P20f$S%vfP~ZR1Da>*qKYMb|%wIue zW-p>4>3{z&)UILbKvPZl5Jgk#n!-~rPWQhvJNpp55z~L6QZDEppI!l`=b-x|)Dj05 z-vQG0*uT^J6rTFe@YMIGHz5DEW4S9K6IC)XEi*wfuTD2AfsXLO%Qqos`m1lx&Q3Q2 zgn{YPU`S}1=;Zer+Kyej=lDZ^j|Vcn7$VaG(c}}e({~{Mp|27A4yn%|BKQW-!{MpX zrl)(RVoj&Qht{0%2Sm@IbB_`MKHW2wZ93h9%AN^NUDt%C_;P#C`7d-#z1=zdFdMw|RS z2V+eQJxB$54rZI0dk!9GYK8CZco?X%5U(HC{YSr621cvR^sxwRqEA?$qpFDAR3-FC zy)B-}M26#;zz}4!BZ*947e0c4Y#iS`1Y?==(siKp;*7v$nol&N0qz40qn-AFHsC2A z1Eu~x4N75=BEq<7r!e;Ujb$~Jp2K*1n}E|C+YejENXF;i;%m5~uKGsLgxGlYrSHA4 zX*IzY;D>H69z|JC5YXZC-|pz}HB36Ye65a8SNR$`ef|!gr=yy3WqIWAKG@~C7U5Lh z$MEZguX0ZE{*SqDuP}X~$;wYSKJGl^a_sieOHTfN9lr-4XC3DO&4>9Gm~Vmk7MO2= z`4*UOf%z7gZ-Mz1m~Vmk7MO2=`4;$p-vYcpA@u&mDXd3X11-2wuv3=?7^k&WtK6H4 z#g2vw13I7A8hIau*BE&pW~r9v{ncff=XG0NYkgYpr}928{S8myx9`lRh~Di85y|iQ z?ygWgucd~Zisv;^-iKAI32IM}^cPcP2~JL9d5r?E6?k<0c-@27X3vsRLbQ5AGR62w zkK)4 ze!jR2KB#Wxk>1$17WtLsE@F525Z3CnzOe!cV^6-*+C?F_8#Q#i(T*IaX9Hr_CrQPz z8L{K@J0#ps@V$r(2W`DNe>Q~J?WxQ_wZi3Eb3fS115gl^TM-$RtI1~N#}GR!PXcjO z(rfLmj6gWe%?0EF5b|=n~r7br_Iwz4IuNPS4{c^A9l7>v9<%2KC;C z$mKS^OOodZ@2^R?LbD$4k14lOvwrUY<$9!|RLkUqFCvV( z$Dlzq^$iNcPt5aR{8#GYKJUV+KMJZh`W<&(S8wz=ZbWXs&OL$LLn_x9x~xg~2Q3+2 z=8O%#B78{>q z8?=X@Z2Zn*Vk71&Xi$oiK4#{PDY-LHqy)>WPt9c0Xkpr;sJ1M%hVMc$OQNQbG4?BlP4ONvM&u8@pggrN}GT*#bL`Z)joa z8I3YyrST220$-#n0b>`m-P3T5I)F4N0>(9vskqW<81$4HX%GUqnIr?yq-K(3iB(1f zpzAtqt&xVPMj9F6d%bb4Z(+65*Iz+>){i-eV~KG3E~&n>x*La(oOPmFCR_`FUWO@> zy9vk!XwU#_R9BOv0m;f{2rRF~sJPH@`Xufq+*_;6xw{v(V@6Z;9zk}iwH16uYJ$eb z8dc(XHQfvAjRi)i;xn6SNV(RX_r(F%eCzN4lxxJ+71-sdtbghj1NCWJNSgH@y#@PB zRd-@HNi0AHI!PPgg)Dnx7Sf3F%&ycCVpP zBfG1wfsv)k_@_42P*G=VeNYX#n8}(&!R&LL{Y<=!p*gJS`pCv1`YzU%dMAYdtq? zyk@88!5!bZX7X0&*kPyFGvqjn#w)Lb2dgE9qX9@F(}!~}nUU_~NIZ=uiir%4%$Qlh z2X)5Ni4-#Sn>R+%$wYEk#LbedGSb;}B#LuE<&Uh$BRVBkq9`U2jf{#&dU$W)qz(?# z&|x6@(P5bE{_%LJbBQD!pUIZbN&Sf@hsvLgDe*HF$CDGWcm@uSXA`NUI{yA{&t^TYQv+Eslt^c?og)cUXpRo4cjlub5ghd@-)GBqjv$^~HrT3~LLXv_ z#A4{jd-2MMmh9MgOEkK5_x_9fqQOzT6Uj_#FqMB0`Sw61(R&l}xLS;lmho%dre`n? zaifRRYYUj4)9VXZ1E+sfz*;%|L;-8y^vMF|=k${W%)@D(K*5(*`p{Yf`@2!=8KR$) z1+_b`v#L;<)wbl-`Z#G(>$R)#WcFUI<`v_@6KL`T-t$GC!`@$-#r61QX%^2fUn|YB zm%T@eL^L%-FDJP{4Y7;M;#vT*8~Tto@Zo|XPV!SQ#8st+uCg2AM&z=1l;_!{U0wt; z`^U18LC|bk(GgBzM2e@!MKGDl#)BQ3y4PeQ!XV5qo5Q8Whu%cj%SJ>jVv_e2IXK5q{07eU>P)MYJ&-Z4DF7Prjq3ETPRM+a+>Hl z_|e)I>-RZA;6(nOLqKwc2y=|0&<4(`pLK*}KK&(1p-GW^Y}fBX&YM^(XivK>E674!awoFQ-`fA6CKvm*+aG0S#heO{!YcZ5W^d|r>-O$rFS z{;0Up_M<<#Xzh>ndA(1bP(!+koGi$s4$AMA*)6JDXhfja*Q1B^|kL}NY3OQE& z-l~w4?bZ6nBv*+1I>}V7r2ymdc}Vk9r$D%s8%DADbH{pVzzZ!XI-KbTcRh zvEBb8z(}8q;d=r{wEkw~S%Y2wK4e<;`Cbs;7g}dkM%u1V%XL=$fG;F90=nUUB)LMw z&ySH~NRX(0O)U4Qlmxf4;tJ27d9q)JAM5k_|I>|1!d{-8e;Q)sKkL&;RSKv6MEz$W zOV6J5dHtL3ck%nn{e#Qrdc6o;Yx#WM{LD&ZAh5jR3OwV#L55Xd1O}97K#_4p@-gc% zeFHgG{Ryo<5l||2eGRKj#Bbq8wz&RuyqLo2fF$w`l}f3Y>&+Og98vkL3~0*wG(K4a z$x>iS;{&Hvj*72crD(_sB5jvFYYzR-Y5hBS;L>*N`sd7{f5CZ5;g>cA8!Ou8(62aO z>G#?cY^=Ck>)YG!s`n~=T5Gfh)@K^l`c@fAzb2Q$aNd>&7A?SfEKh4=c8u~GwL^Bl z?6UQcwYexwZ_|pjcxDYYR+epZQHr0wL}`U|M{fe#jvx1X{{GVVTCV)oPNn}PTc*8y zB~?Cl!n<}ys3g;g(YmiJ?!*duIeAxp-_mN#iCIa=Sx&w}@I0bez7jVV%E@~KfA@>! zy@J2z#duYI-?A7ly$476PJFrfyIL$?xCd8^m)^fqcIxu?;ELt_`FpL!c)j57Q8B(C z|D7qu7YhD<6yuBVJs7Y;GVR1JFpu}vOeb!S@;F|MW7jt@V1;DbS>_%bWjHPO@~leE zqMJp99H*e!cxiB!%)*t7Gk>GCSjQ<=6a$)bmfU=0JApamSC!!O{$k(e=_aVXzWX%J=QP;PL20M-eYp$xY8JB4BftY?-V53tEq7n_ z+rZ1+OZ}n7?Mn<#N;{SUb@CP9fl^rfVGf+;cMHnMuaie?<3_UhVVmfzxxLYi$&!WQ@O= z@m(dH|6bznG3~pSy9eC`e6`lKlI_xt6;bJdIpoK*efz!X^c;3>m-40W#UbF-5BXjv z&nxa`JGOD-^Aa!fef;B_H^E?w`d_lO6HR9mSg6Hr#Hi(JvnaEc3&~72H#CGz9Mim_ zbEnzUy{ivbY6~}|&Fq*NrMuD@!FQr95U%*9%;C}0U}V&cVVyf;MsoWE)~Cit80cNpny0c8HdAcxpBV3qW z*KF_D*4d7Y}e^6jU#tMQ1zm#@b9`wfkz;3t5XBY wmDP8Zw%-M}USOw+sx2PtK9Mp?dHGEkdDC5O`PeF{ZpD+ZzJf1r&npxE3l-|71^@s6 literal 0 HcmV?d00001 diff --git a/teshsuite/bug-17132/bug-17132.c b/teshsuite/bug-17132/bug-17132.c new file mode 100644 index 0000000000..cecb3e4607 --- /dev/null +++ b/teshsuite/bug-17132/bug-17132.c @@ -0,0 +1,40 @@ +#include "xbt/log.h" +#include +#include + +XBT_LOG_NEW_DEFAULT_CATEGORY(MM_mpi, "Messages for this SMPI test"); + +int main(int argc, char ** argv) +{ + size_t err; + size_t M = 8*1024; + size_t N = 32*1024; + + MPI_Init(&argc, &argv); + + double *a = malloc(sizeof(double) * M); + double *b = malloc(sizeof(double) * N); + + // A broadcast + err = MPI_Bcast(a, M, MPI_DOUBLE, 0, MPI_COMM_WORLD); + if (err != MPI_SUCCESS) { + perror("Error Bcast A\n"); MPI_Finalize(); exit(-1); + } + +// Uncommenting this barrier fixes it! +// MPI_Barrier(MPI_COMM_WORLD); + + // Another broadcast + err = MPI_Bcast(b, N, MPI_DOUBLE, 0, MPI_COMM_WORLD ); + if (err != MPI_SUCCESS) { + perror("Error Bcast B\n"); MPI_Finalize(); exit(-1); + } + + // Commenting out this barrier fixes it!! + MPI_Barrier(MPI_COMM_WORLD); + + MPI_Finalize(); + free(a); + free(b); + return 0; +} diff --git a/teshsuite/bug-17132/bug-17132.tesh b/teshsuite/bug-17132/bug-17132.tesh new file mode 100644 index 0000000000..d167f08b25 --- /dev/null +++ b/teshsuite/bug-17132/bug-17132.tesh @@ -0,0 +1,8 @@ +$ ../../smpi_script/bin/smpirun -np 16 -platform ${srcdir:=.}/small_platform.xml -hostfile ${srcdir:=.}/hostfile.txt ./bug-17132 --cfg=smpi/cpu_threshold:-1 --log=smpi_kernel.thres:warning +> You requested to use 16 processes, but there is only 2 processes in your hostfile... +> [0.000000] [xbt_cfg/INFO] Configuration change: Set 'maxmin/precision' to '1e-9' +> [0.000000] [xbt_cfg/INFO] Configuration change: Set 'network/model' to 'SMPI' +> [0.000000] [xbt_cfg/INFO] Configuration change: Set 'network/TCP_gamma' to '4194304' +> [0.000000] [xbt_cfg/INFO] Configuration change: Set 'smpi/cpu_threshold' to '-1' +> [0.000000] [surf_config/INFO] Switching workstation model to compound since you changed the network and/or cpu model(s) +> (some sensible output) diff --git a/teshsuite/bug-17132/hostfile.txt b/teshsuite/bug-17132/hostfile.txt new file mode 100644 index 0000000000..994b3e2cfc --- /dev/null +++ b/teshsuite/bug-17132/hostfile.txt @@ -0,0 +1,2 @@ +host1 +host2 diff --git a/teshsuite/bug-17132/small_platform.xml b/teshsuite/bug-17132/small_platform.xml new file mode 100644 index 0000000000..a2b420d810 --- /dev/null +++ b/teshsuite/bug-17132/small_platform.xml @@ -0,0 +1,15 @@ + + + + + + + + + + + + + + + -- 2.20.1