all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* bug#51308: 29.0.50; A number of tests fail with AOT
@ 2021-10-20 18:20 Lars Ingebrigtsen
  2021-10-20 18:28 ` Eli Zaretskii
                   ` (2 more replies)
  0 siblings, 3 replies; 10+ messages in thread
From: Lars Ingebrigtsen @ 2021-10-20 18:20 UTC (permalink / raw)
  To: 51308; +Cc: Andrea Corallo


With

NATIVE_FULL_AOT=1 make -j16 bootstrap

a number of tests fail:

SUMMARY OF TEST RESULTS
-----------------------
Files examined: 375
Ran 5645 tests, 5513 results as expected, 8 unexpected, 124 skipped
5 files contained unexpected results:
  lisp/net/socks-tests.log
  lisp/loadhist-tests.log
  lisp/hi-lock-tests.log
  lisp/emacs-lisp/ert-tests.log
  lisp/calendar/todo-mode-tests.log

I have not tried to debug this further, but a cursory look at the
backtraces seems to suggest the failures might have something to do with
redefining functions -- but that's just a guess.


In GNU Emacs 29.0.50 (build 39, x86_64-pc-linux-gnu, GTK+ Version 3.24.30, cairo version 1.16.0)
 of 2021-10-20 built on elva
Repository revision: 62591c164cd6d0b0555e11b160ffa81bd3bb010f
Repository branch: master
Windowing system distributor 'The X.Org Foundation', version 11.0.12011000
System Description: Debian GNU/Linux bookworm/sid


-- 
(domestic pets only, the antidote for overdose, milk.)
   bloggy blog: http://lars.ingebrigtsen.no






^ permalink raw reply	[flat|nested] 10+ messages in thread

* bug#51308: 29.0.50; A number of tests fail with AOT
  2021-10-20 18:20 bug#51308: 29.0.50; A number of tests fail with AOT Lars Ingebrigtsen
@ 2021-10-20 18:28 ` Eli Zaretskii
  2021-10-21  3:04   ` Lars Ingebrigtsen
  2021-10-21  4:20 ` Lars Ingebrigtsen
  2021-10-21  4:31 ` bug#51308: 29.0.50; A number of tests fail with AOT Lars Ingebrigtsen
  2 siblings, 1 reply; 10+ messages in thread
From: Eli Zaretskii @ 2021-10-20 18:28 UTC (permalink / raw)
  To: Lars Ingebrigtsen; +Cc: 51308, akrl

> From: Lars Ingebrigtsen <larsi@gnus.org>
> Date: Wed, 20 Oct 2021 20:20:44 +0200
> Cc: Andrea Corallo <akrl@sdf.org>
> 
> Files examined: 375
> Ran 5645 tests, 5513 results as expected, 8 unexpected, 124 skipped
> 5 files contained unexpected results:
>   lisp/net/socks-tests.log
>   lisp/loadhist-tests.log
>   lisp/hi-lock-tests.log
>   lisp/emacs-lisp/ert-tests.log
>   lisp/calendar/todo-mode-tests.log
> 
> I have not tried to debug this further, but a cursory look at the
> backtraces seems to suggest the failures might have something to do with
> redefining functions -- but that's just a guess.

Code that redefines functions which are in *.eln files should disable
native-compilation.





^ permalink raw reply	[flat|nested] 10+ messages in thread

* bug#51308: 29.0.50; A number of tests fail with AOT
  2021-10-20 18:28 ` Eli Zaretskii
@ 2021-10-21  3:04   ` Lars Ingebrigtsen
  0 siblings, 0 replies; 10+ messages in thread
From: Lars Ingebrigtsen @ 2021-10-21  3:04 UTC (permalink / raw)
  To: Eli Zaretskii; +Cc: 51308, akrl

Eli Zaretskii <eliz@gnu.org> writes:

> Code that redefines functions which are in *.eln files should disable
> native-compilation.

Isn't it possible to flet natively compiled functions?

-- 
(domestic pets only, the antidote for overdose, milk.)
   bloggy blog: http://lars.ingebrigtsen.no





^ permalink raw reply	[flat|nested] 10+ messages in thread

* bug#51308: 29.0.50; A number of tests fail with AOT
  2021-10-20 18:20 bug#51308: 29.0.50; A number of tests fail with AOT Lars Ingebrigtsen
  2021-10-20 18:28 ` Eli Zaretskii
@ 2021-10-21  4:20 ` Lars Ingebrigtsen
  2022-03-12 21:32   ` bug#51308: bug#52138: 29.0.50; Some tests fail on emba when run with native compilation Lars Ingebrigtsen
  2021-10-21  4:31 ` bug#51308: 29.0.50; A number of tests fail with AOT Lars Ingebrigtsen
  2 siblings, 1 reply; 10+ messages in thread
From: Lars Ingebrigtsen @ 2021-10-21  4:20 UTC (permalink / raw)
  To: 51308; +Cc: Andrea Corallo

Lars Ingebrigtsen <larsi@gnus.org> writes:

> 5 files contained unexpected results:
>   lisp/net/socks-tests.log

This is now fixed.

>   lisp/loadhist-tests.log

This is a bug-ish.  If dired isn't natively-compiled:

(locate-file "dired" load-path (get-load-suffixes))
=> "/home/larsi/src/emacs/trunk/lisp/dired.elc"

But if it is, then we return the .el file instead:

(locate-file "dired" load-path (get-load-suffixes))
=> "/home/larsi/src/emacs/trunk/lisp/dired.el"

But load-history has:

 ("/home/larsi/src/emacs/emacs-28/lisp/dired.elc"

Which means that we can't find the .elc file by looking into
load-history based on locate-file.

Returning the .elc file would fix this, but I don't know what other
repercussions there might be.

-- 
(domestic pets only, the antidote for overdose, milk.)
   bloggy blog: http://lars.ingebrigtsen.no





^ permalink raw reply	[flat|nested] 10+ messages in thread

* bug#51308: 29.0.50; A number of tests fail with AOT
  2021-10-20 18:20 bug#51308: 29.0.50; A number of tests fail with AOT Lars Ingebrigtsen
  2021-10-20 18:28 ` Eli Zaretskii
  2021-10-21  4:20 ` Lars Ingebrigtsen
@ 2021-10-21  4:31 ` Lars Ingebrigtsen
  2022-03-12 21:42   ` bug#51308: bug#52138: 29.0.50; Some tests fail on emba when run with native compilation Lars Ingebrigtsen
  2 siblings, 1 reply; 10+ messages in thread
From: Lars Ingebrigtsen @ 2021-10-21  4:31 UTC (permalink / raw)
  To: 51308; +Cc: Andrea Corallo

Lars Ingebrigtsen <larsi@gnus.org> writes:

> Files examined: 375
> Ran 5645 tests, 5513 results as expected, 8 unexpected, 124 skipped
> 5 files contained unexpected results:
>   lisp/net/socks-tests.log
>   lisp/loadhist-tests.log
>   lisp/hi-lock-tests.log
>   lisp/emacs-lisp/ert-tests.log
>   lisp/calendar/todo-mode-tests.log

I've now fixed

>   lisp/hi-lock-tests.log
>   lisp/calendar/todo-mode-tests.log

which were about wrong number of parameters (which native-comp is more
strict about) when redefining functions.

>   lisp/emacs-lisp/ert-tests.log

is about

(backtrace-frame-fun (car (ert-test-failed-backtrace result)))
=> ert-fail

(car (ert-test-failed-backtrace result))
=> #s(backtrace-frame t ert-fail ("foo") nil ((debugger-may-continue . t) (inhibit-redisplay) (inhibit-debugger . t) (inhibit-changing-match-data)) nil nil)

while it's `signal' when ert isn't natively compiled.

-- 
(domestic pets only, the antidote for overdose, milk.)
   bloggy blog: http://lars.ingebrigtsen.no





^ permalink raw reply	[flat|nested] 10+ messages in thread

* bug#51308: bug#52138: 29.0.50; Some tests fail on emba when run with native compilation
  2021-10-21  4:20 ` Lars Ingebrigtsen
@ 2022-03-12 21:32   ` Lars Ingebrigtsen
  2022-03-13  5:45     ` Eli Zaretskii
  0 siblings, 1 reply; 10+ messages in thread
From: Lars Ingebrigtsen @ 2022-03-12 21:32 UTC (permalink / raw)
  To: 51308; +Cc: 52138, Andrea Corallo

Lars Ingebrigtsen <larsi@gnus.org> writes:

> This is a bug-ish.  If dired isn't natively-compiled:
>
> (locate-file "dired" load-path (get-load-suffixes))
> => "/home/larsi/src/emacs/trunk/lisp/dired.elc"
>
> But if it is, then we return the .el file instead:
>
> (locate-file "dired" load-path (get-load-suffixes))
> => "/home/larsi/src/emacs/trunk/lisp/dired.el"
>
> But load-history has:
>
>  ("/home/larsi/src/emacs/emacs-28/lisp/dired.elc"
>
> Which means that we can't find the .elc file by looking into
> load-history based on locate-file.
>
> Returning the .elc file would fix this, but I don't know what other
> repercussions there might be.

I've now done this change (which fixes this test failure), and I don't
see any other regressions, so I've pushed it to Emacs 29.

-- 
(domestic pets only, the antidote for overdose, milk.)
   bloggy blog: http://lars.ingebrigtsen.no





^ permalink raw reply	[flat|nested] 10+ messages in thread

* bug#51308: bug#52138: 29.0.50; Some tests fail on emba when run with native compilation
  2021-10-21  4:31 ` bug#51308: 29.0.50; A number of tests fail with AOT Lars Ingebrigtsen
@ 2022-03-12 21:42   ` Lars Ingebrigtsen
  0 siblings, 0 replies; 10+ messages in thread
From: Lars Ingebrigtsen @ 2022-03-12 21:42 UTC (permalink / raw)
  To: 51308; +Cc: 52138, Andrea Corallo

Lars Ingebrigtsen <larsi@gnus.org> writes:

> while it's `signal' when ert isn't natively compiled.

I've now fixed this in the test, but it's not clear that that's the
correct thing to do here.  But at least this makes EMBA have one less
failure.

-- 
(domestic pets only, the antidote for overdose, milk.)
   bloggy blog: http://lars.ingebrigtsen.no





^ permalink raw reply	[flat|nested] 10+ messages in thread

* bug#51308: bug#52138: 29.0.50; Some tests fail on emba when run with native compilation
  2022-03-12 21:32   ` bug#51308: bug#52138: 29.0.50; Some tests fail on emba when run with native compilation Lars Ingebrigtsen
@ 2022-03-13  5:45     ` Eli Zaretskii
  2022-03-13 14:14       ` Lars Ingebrigtsen
  0 siblings, 1 reply; 10+ messages in thread
From: Eli Zaretskii @ 2022-03-13  5:45 UTC (permalink / raw)
  To: Lars Ingebrigtsen; +Cc: 52138, 51308, akrl

> From: Lars Ingebrigtsen <larsi@gnus.org>
> Date: Sat, 12 Mar 2022 22:32:57 +0100
> Cc: 52138@debbugs.gnu.org, Andrea Corallo <akrl@sdf.org>
> 
> > Returning the .elc file would fix this, but I don't know what other
> > repercussions there might be.
> 
> I've now done this change (which fixes this test failure), and I don't
> see any other regressions, so I've pushed it to Emacs 29.

Did you check that loading a .elc file via load-library still triggers
async native-compilation when there's no .eln file or it is not
up-to-date with the .el file?  And what about loading a .el or .elc
file with an explicit extension -- that should NOT trigger
native-compilation even if there's no up-to-date .eln file.





^ permalink raw reply	[flat|nested] 10+ messages in thread

* bug#51308: bug#52138: 29.0.50; Some tests fail on emba when run with native compilation
  2022-03-13  5:45     ` Eli Zaretskii
@ 2022-03-13 14:14       ` Lars Ingebrigtsen
  2022-03-13 16:40         ` bug#52138: " Eli Zaretskii
  0 siblings, 1 reply; 10+ messages in thread
From: Lars Ingebrigtsen @ 2022-03-13 14:14 UTC (permalink / raw)
  To: Eli Zaretskii; +Cc: 52138, 51308, akrl

Eli Zaretskii <eliz@gnu.org> writes:

> Did you check that loading a .elc file via load-library still triggers
> async native-compilation when there's no .eln file or it is not
> up-to-date with the .el file?

Yes, I didn't see any noticeable difference before or after the patch in
that regard.

> And what about loading a .el or .elc
> file with an explicit extension -- that should NOT trigger
> native-compilation even if there's no up-to-date .eln file.

Didn't try that.  Let's see...  if I do

(load "~/src/emacs/trunk/lisp/battery.el")

or

(load "~/src/emacs/trunk/lisp/battery.elc")

then no .eln compilation is started.

-- 
(domestic pets only, the antidote for overdose, milk.)
   bloggy blog: http://lars.ingebrigtsen.no





^ permalink raw reply	[flat|nested] 10+ messages in thread

* bug#52138: bug#51308: bug#52138: 29.0.50; Some tests fail on emba when run with native compilation
  2022-03-13 14:14       ` Lars Ingebrigtsen
@ 2022-03-13 16:40         ` Eli Zaretskii
  0 siblings, 0 replies; 10+ messages in thread
From: Eli Zaretskii @ 2022-03-13 16:40 UTC (permalink / raw)
  To: Lars Ingebrigtsen; +Cc: 52138, 51308, akrl

> From: Lars Ingebrigtsen <larsi@gnus.org>
> Cc: 51308@debbugs.gnu.org,  52138@debbugs.gnu.org,  akrl@sdf.org
> Date: Sun, 13 Mar 2022 15:14:56 +0100
> 
> Eli Zaretskii <eliz@gnu.org> writes:
> 
> > And what about loading a .el or .elc
> > file with an explicit extension -- that should NOT trigger
> > native-compilation even if there's no up-to-date .eln file.
> 
> Didn't try that.  Let's see...  if I do
> 
> (load "~/src/emacs/trunk/lisp/battery.el")
> 
> or
> 
> (load "~/src/emacs/trunk/lisp/battery.elc")
> 
> then no .eln compilation is started.

Great, that's the correct behavior.  Thanks.





^ permalink raw reply	[flat|nested] 10+ messages in thread

end of thread, other threads:[~2022-03-13 16:40 UTC | newest]

Thread overview: 10+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2021-10-20 18:20 bug#51308: 29.0.50; A number of tests fail with AOT Lars Ingebrigtsen
2021-10-20 18:28 ` Eli Zaretskii
2021-10-21  3:04   ` Lars Ingebrigtsen
2021-10-21  4:20 ` Lars Ingebrigtsen
2022-03-12 21:32   ` bug#51308: bug#52138: 29.0.50; Some tests fail on emba when run with native compilation Lars Ingebrigtsen
2022-03-13  5:45     ` Eli Zaretskii
2022-03-13 14:14       ` Lars Ingebrigtsen
2022-03-13 16:40         ` bug#52138: " Eli Zaretskii
2021-10-21  4:31 ` bug#51308: 29.0.50; A number of tests fail with AOT Lars Ingebrigtsen
2022-03-12 21:42   ` bug#51308: bug#52138: 29.0.50; Some tests fail on emba when run with native compilation Lars Ingebrigtsen

Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/emacs.git
	https://git.savannah.gnu.org/cgit/emacs/org-mode.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.