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

koreader crashes on Kindle DXG #1868

Closed
elektronaut0815 opened this issue Mar 5, 2016 · 26 comments
Closed

koreader crashes on Kindle DXG #1868

elektronaut0815 opened this issue Mar 5, 2016 · 26 comments

Comments

@elektronaut0815
Copy link

@elektronaut0815 elektronaut0815 commented Mar 5, 2016

  • KOReader version: v2015.11-214-g58e7103 (upload date 2016-Mar-03 02:18:36)
  • Device: Kindle DXG

Issue

KOReader won't start, it immediately crashes.

./luajit: frontend/ui/uimanager.lua:312: attempt to index local 'self' (a nil value)
stack traceback:
    frontend/ui/uimanager.lua:312: in function 'quit'
    frontend/apps/reader/readerui.lua:353: in function 'action'
    frontend/ui/uimanager.lua:374: in function '_checkTasks'
    frontend/ui/uimanager.lua:518: in function 'handleInput'
    frontend/ui/uimanager.lua:608: in function 'run'
    ./reader.lua:164: in main chunk
    [C]: at 0x0001459c

Steps to reproduce

Simply start KOReader from KUAL.

@NiLuJe
Copy link
Member

@NiLuJe NiLuJe commented Mar 5, 2016

Is that when opening a book directly or the File Manager?

Opening a CRe-handled book currently most likely crashes in fun and interesting ways in my builds, for reasons beyond my comprehension (and that I couldn't manage to debug).

@houqp
Copy link
Member

@houqp houqp commented Mar 5, 2016

Looks like it crashed in readerui, so it's trying to open a book. @elektronaut0815 , did you get the release directly from our release page on github? Our official kindle release only works on touch based kindle devices right now. For k3/dxg/k4/k2, you will need to compile your own version with kindle-legacy as the target.

If you don't know how to setup cross compile environment, you can take https://github.com/koreader/virdevenv/tree/master/vm/ubuntu a try. Feel free to ping us if you got stuck at any step ;)

@elektronaut0815
Copy link
Author

@elektronaut0815 elektronaut0815 commented Mar 5, 2016

Correct, it crashed directly after trying to open a book.
I used the first build from this post which should work with DXG according to NiLuJe.

@houqp
Copy link
Member

@houqp houqp commented Mar 5, 2016

Oh, I see. NiLuJe's build should work then. I thought you downloaded it from github instead. Does the crash happen to any book?

@elektronaut0815
Copy link
Author

@elektronaut0815 elektronaut0815 commented Mar 5, 2016

Yes, any book.

@NiLuJe
Copy link
Member

@NiLuJe NiLuJe commented Mar 6, 2016

For curiosity's sake, does it happen with books handled by another engine than CRe (pdf or djvu for instance)?

@elektronaut0815
Copy link
Author

@elektronaut0815 elektronaut0815 commented Mar 6, 2016

Those books were all Epub's, I just tried a PDF now, which is displayed without crash.

@frankyifei
Copy link
Contributor

@frankyifei frankyifei commented Mar 6, 2016

@NiLuJe , I tested your build on my kpw1, it crashes on epubs and only works for pdf.
file name is koreader-kindle-arm-kindle5-linux-gnueabi-v2015.11-214-g58e7103.zip

@NiLuJe
Copy link
Member

@NiLuJe NiLuJe commented Mar 6, 2016

Yeah, that's why I was asking, that might just be a side-effect of the CRe
crash... Guess I'll see how the next stable linaro release behaves, and/or
try a gcc 4.9 build...
On Mar 6, 2016 11:16, "Yifei(Frank) ZHU" notifications@github.com wrote:

@NiLuJe https://github.com/NiLuJe , I tested your build on my kpw1, it
crashes on epubs and only works for pdf.


Reply to this email directly or view it on GitHub
#1868 (comment).

@houqp houqp added this to the next stable release milestone Mar 8, 2016
@NiLuJe
Copy link
Member

@NiLuJe NiLuJe commented Apr 11, 2016

Okay, the just-released snapshots should now have a properly working CRe...

(Stupid ABI mismatch, I had some crap in my env...)

TL;DR: Can you reproduce with my current >= -309 snapshots?

@elektronaut0815
Copy link
Author

@elektronaut0815 elektronaut0815 commented Apr 13, 2016

Still not working with current snapshot.

@NiLuJe
Copy link
Member

@NiLuJe NiLuJe commented Apr 13, 2016

Okay, so there's a definite issue somewhere in the codebase, and not only a silly mistake on my part, thanks :).

@houqp
Copy link
Member

@houqp houqp commented Apr 14, 2016

@elektronaut0815 are you still getting the same error? Fun fact, I tried a new kindle-legacy build with the latest master, EPUB is working fine for me, but now it says no engine found for PDFs... I will probably take a closer look at weekend to see what's actually happening.

@NiLuJe
Copy link
Member

@NiLuJe NiLuJe commented Apr 14, 2016

Okay on PDF & ePub on a K4 over here.

@NiLuJe
Copy link
Member

@NiLuJe NiLuJe commented Apr 14, 2016

Okay also on a K3.

@NiLuJe
Copy link
Member

@NiLuJe NiLuJe commented Apr 14, 2016

... And okay also on a K2.

:?

@NiLuJe
Copy link
Member

@NiLuJe NiLuJe commented Apr 14, 2016

Even checked on a clean install to make sure, and, yep, works fine on my end... :?

@houqp
Copy link
Member

@houqp houqp commented Apr 18, 2016

After updated https://github.com/koreader/koxtoolchain to use @NiLuJe 's ct-ng kindle branch, everything is working again on my K3 now. @elektronaut0815 can you give @NiLuJe 's latest build a try?

PS: @NiLuJe, the kernel version you specified in the sample file does not exist in ct-ng. Did you do anything special to make ct-ng fetch your version? I ended up using 2_6_32 for kindle-legacy TC.

@NiLuJe
Copy link
Member

@NiLuJe NiLuJe commented Apr 18, 2016

That kernel version is in my kindle, branch, make sure you don't have an
older ct-ng version installed somewhere in your path ?
On Apr 18, 2016 10:43 AM, "Qingping Hou" notifications@github.com wrote:

After updated https://github.com/koreader/koxtoolchain to use @NiLuJe
https://github.com/NiLuJe 's ct-ng kindle branch, everything is working
again on my K3 now. @elektronaut0815 https://github.com/elektronaut0815
can you give @NiLuJe https://github.com/NiLuJe 's latest build a try?

PS: @NiLuJe https://github.com/NiLuJe, the kernel version you specified
in the sample file does not exist in ct-ng. Did you do anything special to
make ct-ng fetch your version? I ended up using 2_6_32 for kindle-legacy TC.


You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
#1868 (comment)

@NiLuJe
Copy link
Member

@NiLuJe NiLuJe commented Apr 18, 2016

... On the tip of the branch, that is, whereas your script is checking out
the latest tag ;).
On Apr 18, 2016 11:36 AM, "NiLuJe" ninuje@gmail.com wrote:

That kernel version is in my kindle, branch, make sure you don't have an
older ct-ng version installed somewhere in your path ?
On Apr 18, 2016 10:43 AM, "Qingping Hou" notifications@github.com wrote:

After updated https://github.com/koreader/koxtoolchain to use @NiLuJe
https://github.com/NiLuJe 's ct-ng kindle branch, everything is working
again on my K3 now. @elektronaut0815 https://github.com/elektronaut0815
can you give @NiLuJe https://github.com/NiLuJe 's latest build a try?

PS: @NiLuJe https://github.com/NiLuJe, the kernel version you specified
in the sample file does not exist in ct-ng. Did you do anything special to
make ct-ng fetch your version? I ended up using 2_6_32 for kindle-legacy TC.


You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
#1868 (comment)

@houqp
Copy link
Member

@houqp houqp commented Apr 18, 2016

haha, I am an idiot. Thanks for catching that, I forgot to change the commit hash ;)

@houqp
Copy link
Member

@houqp houqp commented Apr 19, 2016

@NiLuJe btw, which version of binutils should we use for kindle-legacy? linaro-2.24.0-2014.11-2?

@NiLuJe
Copy link
Member

@NiLuJe NiLuJe commented Apr 19, 2016

The latest when using gold, but with bfd we don't really care. That said,
it doesn't hurt to go with something up to date, so 2.26 (mainline) right
now ;).
On Apr 19, 2016 9:19 AM, "Qingping Hou" notifications@github.com wrote:

@NiLuJe https://github.com/NiLuJe btw, which version of binutils should
we use? linaro-2.24.0-2014.11-2?


You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
#1868 (comment)

@NiLuJe
Copy link
Member

@NiLuJe NiLuJe commented Apr 19, 2016

@houqp: Any particular reason for sticking w/ GCC 4.8? (If the idea is to avoid GCC 5, which is not such a crazy idea, GCC 4.9 is also a solid choice ;)).

Same question for glibc 2.11 in the Kobo TC?

@houqp
Copy link
Member

@houqp houqp commented Apr 20, 2016

I haven't finished up the kindle tc clean up yet. GCC 5 for some reason does not work for me. I need to dig further into it. The rest should be in sync with your sample setting.

As for Kobo TC, I haven't started rebuilding it based on your sample yet ;)

@NiLuJe
Copy link
Member

@NiLuJe NiLuJe commented Jul 8, 2018

This should definitely be fixed by now (updated TCs, proper targets, and the only GCC >4 issue we actually had fixed) :).

@NiLuJe NiLuJe closed this Jul 8, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
4 participants