-
Notifications
You must be signed in to change notification settings - Fork 104
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
Windows 10 error #311
Comments
Which CCL version are you using? |
Seems like a duplicate of #201 |
Yes, it seems a duplicate of #201 |
So the issue happens only after you recompile the kernel? Huh, interesting! Which toolchain did you use to do so? |
No, it happens whether I recompile the kernel or I use the one I can download. I am not able to launch any of them, and neither 1.11.5 nor 1.11. |
More on this: it is perfectly possible that I allocate the needed TLS slots, only they are not contiguous. In my case, allocation start @ 14, but 16 is unavailable and so is 64. So I don't see an easy solution but rewrite the handling of these indexes with a level of indirection. |
It only happens on Windows 10 version 2004, version 1909 dosen't have this problem. |
Le jeu. 14 mai 2020 à 18:15, Alkitor <[email protected]> a écrit :
It only happens on Windows 10 version 2004, version 1909 dosen't have this
problem.
I suspected that but it is nice to have a confirmation.
Regards,
… |
The commit message for a079670 explains the strategy of using the slots in TEB block. But, on 64-bit Windows, we dedicate a GPR as the TCR pointer, so I don't see why it would be necessary to reserve any TEB slots there. A simple test would be to comment out the call to |
Le jeu. 14 mai 2020 à 21:28, R. Matthew Emerson <[email protected]>
a écrit :
The commit message for a079670
<a079670>
explains the strategy of using the slots in TEB block.
But, on 64-bit Windows, we dedicate a GPR as the TCR pointer, so I don't
see why is would be necessary to reserve any TEB slots there.
I was wondering about that, but not being familiar with the code, I was not
sure.
A simple test would be to comment out the call to reserve_tls_slots() in
lisp-kernel/pmcl-kernel.c and rebuild the lisp kernel and see if it
works. (See
https://github.com/Clozure/ccl/blob/master/lisp-kernel/pmcl-kernel.c#L1905
)
$ ./wx86cl64.exe
Clozure Common Lisp Version 1.12 (v1.12-4-g579ed109) WindowsX8664
For more information about CCL, please see http://ccl.clozure.com.
CCL is free software. It is distributed under the terms of the Apache
Licence, Version 2.0.
?
It works at least to the point it was able to rebuild the image.
Regards,
Fabrice
|
It seems to be an overly strict check fort continuous TLS indices. CCL seems to require 30-63 to be continuous, but the reserve_tls_slots() also checks if those below are reservable in one continuous block. Totally unnecessary since they get freed afterwards. |
I have this issue. Any help? |
You have to build from scratch for now unfortunately... @xrme Is it possible to release an official 1.12-r1 build for windows, with just this issue fixed? |
Thank you
…On Thu, 29 Oct 2020 12:34:46 -0700, astrangeguy ***@***.***> wrote:
You have to build from scratch for now unfortunately...
@xrme Is it possible to release an official 1.12-r1 build for windows, with just this issue fixed?
--
You are receiving this because you commented.
Reply to this email directly or view it on GitHub:
#311 (comment)
|
Expecting the official release. ccl can't work on the windows 10 20H2. Unfortunately i use ccl on some projects. |
Is there an update on 1.12.1 or how to get the fix deployed? I installed it from scoop on windows and it just dies. this is almost a year old and if rebuilding from source works it would be nice if the project did it once for everyone |
Not a clozure dev, but this is the build I use at work. |
I don't want to do a full rebuild, I want a working build so the scoop ccl maintainer has a reason to update the package to something that works. I do not want to manage one-offs on my laptop when a customer is not paying me to do so. I would like the maintainers to maintain the project. This is a hard fail bug, ccl does not start. |
+1 for this issue. It's been going on nearly year and I've stopped testing CCL builds because of it. Ironic because for many years CCL was the 'go-to' implementation on Windows. |
I know this issue has lingered for far too long, but I'll find time to make a release (or at the very least provide updated Windows binaries) this weekend. |
Thank you |
Scoop installed it and (+ 1 1 ) gave me 2 Thanks for fixing this |
Thanks for fixing this. |
make reserve_tls_slots() more lenient (fixes #311)
Thanks a lot for fixing this. |
As of fa2bbe8, we don't bother to reserve (uselessly) TEB slots on 64-bit Windows. |
Hi,
Trying CCL and I get this weird error:
I cannot run the binary.
This is with Microsoft Windows [version 10.0.19041.208]
Any idea what is going on here?
Thanks,
The text was updated successfully, but these errors were encountered: