Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[WIP] Fix arm64 #228

Closed
wants to merge 8 commits into from
Closed

[WIP] Fix arm64 #228

wants to merge 8 commits into from

Conversation

k-okada
Copy link
Member

@k-okada k-okada commented Jan 5, 2017

This X program also fails...

ubuntu@tegra-ubuntu:~/prog/jskeus/eus/lisp/xwindow$ cat hoge.c
#include <stdio.h>
#include <X11/Xlib.h>

int main () {
  Display * display;
  XFontStruct * font;
  display = XOpenDisplay (NULL);
  printf("display = %d, root = %d\n", display, DefaultRootWindow(display));
  font = XLoadQueryFont (display, "*-fixed-medium-r-normal-*-14-*");
  printf("font addr %d %x %p\n", font, font, font);
  printf("font id addr %d %x %p\n", &(font->fid),&(font->fid),&(font->fid));
  printf("fid %d %x\n", font->fid, font->fid);
  XGCValues gcval;
  printf("gcfont %x\n", GCFont);
  GC gcid = XCreateGC(display, DefaultRootWindow(display), 0, &gcval);
  printf("font = %x (init)\n", gcval.font);    
  printf("font = %x\n", font);
  gcval.font = font;
  XChangeGC(display, gcid, 0x4000, &gcval);
  XGetGCValues(display, gcid, 0x4000, &gcval);
  printf("font = %x (get)\n", gcval.font);  
  //XSetFont(display, gcid, font);
  char bitmap[32];
  XCreatePixmapFromBitmapData(display, DefaultRootWindow(display),
			      bitmap, 8, 4, 0, 0, 24);
  printf("done\n");
}

@k-okada
Copy link
Member Author

k-okada commented Jan 5, 2017

@YoheiKakiuchi これでsegfaultせずに(objects)できるようになった.あとは、call_foreign を直せばちゃんと表示されるかな?
eusgl までコンパイル出来ない問題は、
074350b
がクリティカルでdefcstructが動いていなかったんだけど、これは #229 でチェックされているでしょうか.

@YoheiKakiuchi
Copy link
Member

#229 をポインタ/浮動小数点数/整数のサイズチェックを入れました。

k-okada#1 にアセンブリを含めた修正コードをPRしました。

手元のjetsonでirtviewerにモデルが表示できるところまで確認できました。

@inabajsk
Copy link
Contributor

inabajsk commented Jan 6, 2017 via email

@k-okada
Copy link
Member Author

k-okada commented Jan 8, 2017

merged in #230

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants