{"action":"created","issue":{"url":"https://api.github.com/repos/wavesoft/nanomsg-transport-ofi/issues/1","repository_url":"https://api.github.com/repos/wavesoft/nanomsg-transport-ofi","labels_url":"https://api.github.com/repos/wavesoft/nanomsg-transport-ofi/issues/1/labels{/name}","comments_url":"https://api.github.com/repos/wavesoft/nanomsg-transport-ofi/issues/1/comments","events_url":"https://api.github.com/repos/wavesoft/nanomsg-transport-ofi/issues/1/events","html_url":"https://github.com/wavesoft/nanomsg-transport-ofi/issues/1","id":150991955,"number":1,"title":"posix_memalign","user":{"login":"mdcb","id":66446,"avatar_url":"https://avatars.githubusercontent.com/u/66446?v=3","gravatar_id":"","url":"https://api.github.com/users/mdcb","html_url":"https://github.com/mdcb","followers_url":"https://api.github.com/users/mdcb/followers","following_url":"https://api.github.com/users/mdcb/following{/other_user}","gists_url":"https://api.github.com/users/mdcb/gists{/gist_id}","starred_url":"https://api.github.com/users/mdcb/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mdcb/subscriptions","organizations_url":"https://api.github.com/users/mdcb/orgs","repos_url":"https://api.github.com/users/mdcb/repos","events_url":"https://api.github.com/users/mdcb/events{/privacy}","received_events_url":"https://api.github.com/users/mdcb/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":2,"created_at":"2016-04-25T21:56:31Z","updated_at":"2016-04-25T22:09:18Z","closed_at":null,"body":"In nn_sofi_init, the argument to posix_memalign should be the address of the ingress pointer, not the pointer itself. Patch below.\r\n\r\n```\r\n--- a/src/transports/ofi/sofi.c\r\n+++ b/src/transports/ofi/sofi.c\r\n@@ -1212,7 +1212,7 @@ int nn_sofi_init ( struct nn_sofi *self, struct ofi_domain *domain, int offset,\r\n     /* Allocate ingress buffers */\r\n #if _POSIX_C_SOURCE >= 200112L\r\n     self->ingress_buffers = NULL;\r\n-    ret = posix_memalign( (void**)self->ingress_buffers, sysconf(_SC_PAGESIZE), \r\n+    ret = posix_memalign( (void**) &self->ingress_buffers, sysconf(_SC_PAGESIZE), \r\n         ibufsz * rx_queue );\r\n```"},"comment":{"url":"https://api.github.com/repos/wavesoft/nanomsg-transport-ofi/issues/comments/214545859","html_url":"https://github.com/wavesoft/nanomsg-transport-ofi/issues/1#issuecomment-214545859","issue_url":"https://api.github.com/repos/wavesoft/nanomsg-transport-ofi/issues/1","id":214545859,"user":{"login":"mdcb","id":66446,"avatar_url":"https://avatars.githubusercontent.com/u/66446?v=3","gravatar_id":"","url":"https://api.github.com/users/mdcb","html_url":"https://github.com/mdcb","followers_url":"https://api.github.com/users/mdcb/followers","following_url":"https://api.github.com/users/mdcb/following{/other_user}","gists_url":"https://api.github.com/users/mdcb/gists{/gist_id}","starred_url":"https://api.github.com/users/mdcb/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mdcb/subscriptions","organizations_url":"https://api.github.com/users/mdcb/orgs","repos_url":"https://api.github.com/users/mdcb/repos","events_url":"https://api.github.com/users/mdcb/events{/privacy}","received_events_url":"https://api.github.com/users/mdcb/received_events","type":"User","site_admin":false},"created_at":"2016-04-25T22:09:18Z","updated_at":"2016-04-25T22:09:18Z","body":"Hi @wavesoft,\r\ngdb'd through the demo and send this, just to find your commit 2h ago ... I'll retry later.\r\nI'm x86_64 / linux. Might have a few more question as I go through the new transport. Are you content to use github issue for that, any other ways would be fine."}}