{"action":"created","issue":{"url":"https://api.github.com/repos/testppl/charmtest/issues/298","repository_url":"https://api.github.com/repos/testppl/charmtest","labels_url":"https://api.github.com/repos/testppl/charmtest/issues/298/labels{/name}","comments_url":"https://api.github.com/repos/testppl/charmtest/issues/298/comments","events_url":"https://api.github.com/repos/testppl/charmtest/issues/298/events","html_url":"https://github.com/testppl/charmtest/issues/298","id":334906280,"node_id":"MDU6SXNzdWUzMzQ5MDYyODA=","number":298,"title":"Avoid warning in sockRoutines","user":{"login":"pplimport","id":40410281,"node_id":"MDQ6VXNlcjQwNDEwMjgx","avatar_url":"https://avatars2.githubusercontent.com/u/40410281?v=4","gravatar_id":"","url":"https://api.github.com/users/pplimport","html_url":"https://github.com/pplimport","followers_url":"https://api.github.com/users/pplimport/followers","following_url":"https://api.github.com/users/pplimport/following{/other_user}","gists_url":"https://api.github.com/users/pplimport/gists{/gist_id}","starred_url":"https://api.github.com/users/pplimport/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/pplimport/subscriptions","organizations_url":"https://api.github.com/users/pplimport/orgs","repos_url":"https://api.github.com/users/pplimport/repos","events_url":"https://api.github.com/users/pplimport/events{/privacy}","received_events_url":"https://api.github.com/users/pplimport/received_events","type":"User","site_admin":false},"labels":[{"id":972436640,"node_id":"MDU6TGFiZWw5NzI0MzY2NDA=","url":"https://api.github.com/repos/testppl/charmtest/labels/bug","name":"bug","color":"d73a4a","default":true}],"state":"open","locked":false,"assignee":{"login":"pplimport","id":40410281,"node_id":"MDQ6VXNlcjQwNDEwMjgx","avatar_url":"https://avatars2.githubusercontent.com/u/40410281?v=4","gravatar_id":"","url":"https://api.github.com/users/pplimport","html_url":"https://github.com/pplimport","followers_url":"https://api.github.com/users/pplimport/followers","following_url":"https://api.github.com/users/pplimport/following{/other_user}","gists_url":"https://api.github.com/users/pplimport/gists{/gist_id}","starred_url":"https://api.github.com/users/pplimport/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/pplimport/subscriptions","organizations_url":"https://api.github.com/users/pplimport/orgs","repos_url":"https://api.github.com/users/pplimport/repos","events_url":"https://api.github.com/users/pplimport/events{/privacy}","received_events_url":"https://api.github.com/users/pplimport/received_events","type":"User","site_admin":false},"assignees":[{"login":"pplimport","id":40410281,"node_id":"MDQ6VXNlcjQwNDEwMjgx","avatar_url":"https://avatars2.githubusercontent.com/u/40410281?v=4","gravatar_id":"","url":"https://api.github.com/users/pplimport","html_url":"https://github.com/pplimport","followers_url":"https://api.github.com/users/pplimport/followers","following_url":"https://api.github.com/users/pplimport/following{/other_user}","gists_url":"https://api.github.com/users/pplimport/gists{/gist_id}","starred_url":"https://api.github.com/users/pplimport/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/pplimport/subscriptions","organizations_url":"https://api.github.com/users/pplimport/orgs","repos_url":"https://api.github.com/users/pplimport/repos","events_url":"https://api.github.com/users/pplimport/events{/privacy}","received_events_url":"https://api.github.com/users/pplimport/received_events","type":"User","site_admin":false}],"milestone":null,"comments":2,"created_at":"2013-10-04T14:55:33Z","updated_at":"2018-06-27T14:33:33Z","closed_at":null,"author_association":"NONE","body":"*Original author: Nikhil Jain*\n\n---\nOn most machine, I see the following warning (that we should get rid of):\r\n\r\n/usr/lib/../lib64/libpthread.a(sem_open.o): In function `sem_open':\r\n/usr/src/packages/BUILD/glibc-2.11.3/nptl/sem_open.c:333: warning: the\r\nuse of `mktemp' is dangerous, better use `mkstemp'\r\n/users/fminiati/Charm++/gni-crayxc/bin/../lib/libconv-util.a(sockRoutines.o):\r\nIn function `skt_lookup_ip':\r\nsockRoutines.c:(.text+0x344): warning: Using 'gethostbyname' in\r\nstatically linked applications requires at runtime the shared libraries\r\nfrom the glibc version used for linking"},"comment":{"url":"https://api.github.com/repos/testppl/charmtest/issues/comments/400694672","html_url":"https://github.com/testppl/charmtest/issues/298#issuecomment-400694672","issue_url":"https://api.github.com/repos/testppl/charmtest/issues/298","id":400694672,"node_id":"MDEyOklzc3VlQ29tbWVudDQwMDY5NDY3Mg==","user":{"login":"pplimport","id":40410281,"node_id":"MDQ6VXNlcjQwNDEwMjgx","avatar_url":"https://avatars2.githubusercontent.com/u/40410281?v=4","gravatar_id":"","url":"https://api.github.com/users/pplimport","html_url":"https://github.com/pplimport","followers_url":"https://api.github.com/users/pplimport/followers","following_url":"https://api.github.com/users/pplimport/following{/other_user}","gists_url":"https://api.github.com/users/pplimport/gists{/gist_id}","starred_url":"https://api.github.com/users/pplimport/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/pplimport/subscriptions","organizations_url":"https://api.github.com/users/pplimport/orgs","repos_url":"https://api.github.com/users/pplimport/repos","events_url":"https://api.github.com/users/pplimport/events{/privacy}","received_events_url":"https://api.github.com/users/pplimport/received_events","type":"User","site_admin":false},"created_at":"2018-06-27T14:33:33Z","updated_at":"2018-06-27T14:33:33Z","author_association":"NONE","body":"*Original author: Sam White*\n*Original date: 2016-12-17 18:42:19*\n\n---\nThese warnings still appear on Cray builds and possibly elsewhere.\r\n\r\nThe warning about mktemp comes from our use of pthread's sem_open(). We currently only call that routine from pxshm/xpmem machine layers:\r\n```\r\nsrc/arch/net/machine-pxshm.c:    (*bufs)[i].mutex = sem_open(bufNames[i], O_CREAT, S_IRUSR | S_IWUSR, 1);\r\nsrc/arch/util/machine-pxshm.c:   (*bufs)[i].mutex = sem_open(bufNames[i], O_CREAT, S_IRUSR | S_IWUSR, 1);\r\nsrc/arch/util/machine-xpmem.c\t(*bufs)[i].mutex = sem_open(bufNames[i],O_CREAT, S_IRUSR | S_IWUSR,1);\r\nsrc/arch/util/machine-xpmem.c:   (*bufs)[i].mutex = sem_open(bufNames[i],O_CREAT, S_IRUSR | S_IWUSR,1);\r\n\n```\r\nThe following commit turns off that support by default for Cray MPI SMP builds, but it appears to be the default for GNI builds: https://charm.cs.illinois.edu/gerrit/#/c/2054/\r\nWe would like for the pxshm queues to use atomics/fences (PXSHM_FENCE) rather than locks (PXSHM_LOCK), but the current fence implementation appears to be buggy: https://charm.cs.illinois.edu/redmine/issues/571\r\n\r\nThe latter issue, about gethostbyname, is not seen on any BGQ builds because we have set CMK_NO_SOCKETS on {pami,pamilrts,mpi}-bluegeneq.\r\nInvestigate if we can define that flag without losing CCS support on crayx{c,e} builds."}}
{"actor":{"display_login":"pplimport"}}