Age | Commit message (Collapse) | Author | |
---|---|---|---|
2016-01-19 | erts: Ignore unexpected messages to erts_code_purger | Sverker Eriksson | |
2016-01-13 | erts: Optimize erlang:check_process_code | Sverker Eriksson | |
by ignoring literals. erts_internal:check_process_code will be called again anyway (with option {copy_literals, true}) before the module is actually purged. No need to check literals twice. | |||
2016-01-13 | erts: Refactor check_process_code/3 | Sverker Eriksson | |
Move impl from erlang to erts_internal. Cut and paste. | |||
2016-01-13 | erts: Make copy_literals more fail safe | Sverker Eriksson | |
* Same process must do enable-disable. * System process will force it and never get 'aborted' | |||
2016-01-13 | erts: Move copy_literals/2 from erlang to erts_internal | Sverker Eriksson | |
as it's not a public interface. | |||
2016-01-13 | erts: Make erlang:purge_module/1 safe | Sverker Eriksson | |
Problem: erlang:purge_module/1 is not safe in the sense that very bad things may happen if the code to be purged is still referred to by live processes. Introduce erts_internal:purge_module which is the same as the old erlang:purge_module BIF (except it returns false if no such old module). Implement erlang:purge_module in Erlang and let it invoke erts_code_purger for safe purging where all clogging processes first are killed. | |||
2016-01-13 | erts: Refactor code:purge/1 and code:soft_purge/1 | Sverker Eriksson | |
by moving code from code_server to erts_code_purger. This is more or less a copy-paste from code_server.erl to erts_code_purger.erl. All the inner mechanics of code:purge/1 and code:soft_purge/1 are unchanged. | |||
2016-01-13 | erts: Introduce erts_code_purger | Sverker Eriksson | |
as a system process with preloaded code. | |||
2016-01-11 | Merge branch 'maint' | Henrik Nord | |
Conflicts: OTP_VERSION | |||
2016-01-11 | Merge branch 'maint-18' into maint | Henrik Nord | |
2016-01-11 | Merge branch 'maint' | Björn Gustavsson | |
* maint: Eliminate crash in v3_codegen | |||
2016-01-11 | Eliminate crash in v3_codegen | Björn Gustavsson | |
The following code would crash v3_codegen: order(From) -> catch if From#{[] => sufficient} -> saint end. Before explaining the crash, first some background on the stack frame and the Y registers. Certain instructions, most notably the 'call' instructions, clobber all X registers. Before any such instruction, all X registers that have values that will be used after the call must be saved to Y registers (i.e. to the stack frame). adjust_stack/4 will be called when X registers must be saved. There is also another situation when X registers must be saved, namely within a 'catch' if we are about to execute any instruction that may cause an exception. Examples of such instructions are some guard BIFs (such as length/1) and construction of binaries or maps. Within a 'catch', X registers must be be saved because if an exception is thrown and catched all X registers will be destroyed. The same adjust_stack/4 function will be called for those instructions, but only if they occur within a 'catch'. There is actually one more complication. If there is code in a guard within a catch, the X registers should not be saved, because the code in a guard never clobbers any X registers that were alive before the guard code was entered. v3_codegen is written with the implicit assumption that code in guards never cause anything to be saved to Y registers. The code for building maps and binaries would incorrectly save X registers within a guard inside a 'catch'. For construction of binaries, that would mean that a useless but harmelss 'move' instruction was generated. But for construction of maps, the saving of the Y register would not be harmless. There would be a crash when attempting to merge #sr{} records. #sr{} records keeps track of the contents of X and Y registers. When two separate code paths are joined (e.g. at the end of 'case' statement), the register descriptors must be reconciled. Basically, the register descriptors for both paths must be identical. The #sr{} record for one path must not claim that {y,0} contains a certain value, while another path claims that {y,0} is dead. Thus, the crash occurs in sr_merge/2 when failing to reconcile the Y registers. To fix this bug this bug we will introduce a new function called maybe_adjust_stack/5. It will save X registers on the stack only if the code is inside a catch but not inside a guard. We will change all existing code to use this new function when appropriate. Reported-by: Thomas Arts | |||
2016-01-08 | Updated OTP versionOTP-18.2.2 | Erlang/OTP | |
2016-01-08 | Update release notes | Erlang/OTP | |
2016-01-08 | Merge branch 'hans/ssh/keyboard_interactive0/OTP-13255' into maint-18 | Erlang/OTP | |
* hans/ssh/keyboard_interactive0/OTP-13255: ssh: update vsn.mk to 4.2.1 ssh: handle secondary ssh_msg_userauth_info_request message ssh: testcase for abnormal keyboard-interactive authentication | |||
2016-01-08 | ssh: update vsn.mk to 4.2.1 | Hans Nilsson | |
2016-01-08 | ssh: handle secondary ssh_msg_userauth_info_request message | Hans Nilsson | |
2016-01-08 | ssh: testcase for abnormal keyboard-interactive authentication | Hans Nilsson | |
2016-01-08 | Merge branch 'maint' | Björn Gustavsson | |
* maint: beam_bool: Fix unsafe optimization | |||
2016-01-07 | Merge branch 'sverk/proc-dict-opt' | Sverker Eriksson | |
OTP-13167 * sverk/proc-dict-opt: erts: Add new test case pdict_SUITE:mixed erts: Add 'fill_heap' to erts_debug:state_internal_state erts: Rename proc dict size to arraySize erts: Refactor proc dict with 'usedSlots' erts: Add sizeMask for faster proc dict indexing erts: Remove ProcDict.used erts: Add proc dict macros ERTS_PD_START/SIZE erts: Optimize away function "array_put" in proc dict erts: Optimize hashing in process dictionary | |||
2016-01-07 | Merge branch 'sverk/hipe-mmap-config' | Sverker Eriksson | |
* sverk/hipe-mmap-config: erts: Add config test for MAP_NORESERVE for hipe on 64-bit | |||
2016-01-07 | erts: Add config test for MAP_NORESERVE for hipe on 64-bit | Sverker Eriksson | |
to get a nicer error on FreeBSD and others that does not support MAP_NORESERVE for mmap. Q: How to support this? A: Implement the "literal tag" in hipe or another way to reserve virtual address space in erl_mmap.c | |||
2016-01-07 | Merge branch 'sverk/float_SUITE-arith' | Sverker Eriksson | |
* sverk/float_SUITE-arith: erts: Add new test cases to float_SUITE | |||
2016-01-07 | Merge branch 'sverk/check_process_code-fix' | Sverker Eriksson | |
* sverk/check_process_code-fix: erts: Fix bug in check_process_code for literals | |||
2016-01-07 | Merge branch 'bjorn/compiler/beam_bool/OTP-13208' into maint | Björn Gustavsson | |
* bjorn/compiler/beam_bool/OTP-13208: beam_bool: Fix unsafe optimization | |||
2016-01-07 | beam_bool: Fix unsafe optimization | Björn Gustavsson | |
beam_bool would make the following code unsafe (which would be reported by beam_validator): scotland(Echo) -> found(case Echo of Echo when true; Echo, Echo, Echo -> Echo; echo -> [] end, Echo = placed). found(_, _) -> million. Basically, beam_bool would see that the 'case' would always return the value of Echo. Thus: scotland(Echo) -> found(Echo, Echo = placed). The only problem is that beam_bool would also remove a 'move' instruction that would save Echo to the stack. Here is the assembly code for part of the function: {allocate_zero,1,1}. {move,{x,0},{y,0}}. %% Save Echo on stack. {bif,'=:=',{f,7},[{x,0},{atom,true}],{x,1}}. {bif,'=:=',{f,7},[{x,0},{atom,true}],{x,2}}. {bif,'=:=',{f,7},[{x,0},{atom,true}],{x,3}}. {bif,'and',{f,7},[{x,2},{x,3}],{x,2}}. {bif,'and',{f,7},[{x,1},{x,2}],{x,1}}. {jump,{f,8}}. {label,7}. {move,{atom,false},{x,1}}. {label,8}. {bif,'or',{f,6},[{atom,true},{x,1}],{x,1}}. {test,is_eq_exact,{f,6},[{x,1},{atom,true}]}. %% Jump never taken. {jump,{f,5}}. {label,6}. {test,is_eq_exact,{f,9},[{x,0},{atom,echo}]}. {move,nil,{x,0}}. {jump,{f,5}}. {label,9}. {test_heap,3,0}. {put_tuple,2,{x,0}}. {put,{atom,case_clause}}. {put,{y,0}}. {line,[{location,"t.erl",5}]}. {call_ext,1,{extfunc,erlang,error,1}}. {jump,{f,5}}. {label,5}. {test,is_eq_exact,{f,12},[{atom,placed},{y,0}]}. beam_bool would see that the is_eq_exact test at label 8 would always succeed. It could therefore remove most of the code before the jump to label 5. Unfortunately it also removed the essential move of Echo to the stack: {allocate_zero,1,1}. %% Instruction incorrectly removed: {move,{x,0},{y,0}}. {jump,{f,5}}. {label,5}. {test,is_eq_exact,{f,12},[{atom,placed},{y,0}]}. The root cause of the problem is that the 'move' instruction is included in the block of 'bif' instructions before label 8. Normally the 'move' instruction would not have been discarded, but because the left operand to the 'or' BIF is 'true', the entire block with 'bif' instructions are dropped. As far as I can see, there is no gain by including 'move' instructions in the first place. There is no way that better code will be produced. In fact, the entire optimization can be given up if 'move' instructions are found in the block. Thus we can fix this bug by never including any 'move' instructions in the block of 'bif' instructions. We can also remove all the code that deals with 'move' instructions within blocks. Reported-by: Thomas Arts | |||
2015-12-30 | Merge branch 'maint' | Rickard Green | |
* maint: Light weight statistics of run queue lengths Conflicts: erts/preloaded/ebin/erlang.beam | |||
2015-12-30 | Merge branch 'rickard/rq-len/OTP-13201' into maint | Rickard Green | |
* rickard/rq-len/OTP-13201: Light weight statistics of run queue lengths | |||
2015-12-30 | Light weight statistics of run queue lengths | Rickard Green | |
- statistics(total_run_queue_lengths) - statistics(run_queue_lengths) - statistics(total_active_tasks) - statistics(active_tasks) Conflicts: erts/emulator/beam/erl_process.c | |||
2015-12-30 | Merge branch 'maint' | Rickard Green | |
* maint: Use monotonic time for call_time trace | |||
2015-12-30 | Merge branch 'rickard/trace_call_time/OTP-13216' into maint | Rickard Green | |
* rickard/trace_call_time/OTP-13216: Use monotonic time for call_time trace | |||
2015-12-30 | Merge branch 'rickard/ohmq-fixup/OTP-13047' | Rickard Green | |
* rickard/ohmq-fixup/OTP-13047: Fix asynchronous BIF timer cancellation message reply | |||
2015-12-29 | Fix asynchronous BIF timer cancellation message reply | Rickard Green | |
2015-12-28 | Merge branch 'maint' | Zandra | |
2015-12-28 | Merge branch 'zandra/update-java-versions-doc' into maint | Zandra | |
* zandra/update-java-versions-doc: update java version in documentation OTp-13221 | |||
2015-12-28 | Merge branch 'maint' | Zandra | |
2015-12-28 | Merge branch 'binarin/maint' into maint | Zandra | |
* binarin/maint: Don't wait for twice the delay_write timeout OTP-13220 | |||
2015-12-28 | Merge branch 'maint' | Zandra | |
2015-12-28 | Merge branch 'legoscia/tls_dist_error_reporting' into maint | Zandra | |
* legoscia/tls_dist_error_reporting: Report bad options for outgoing TLS distribution Save error reasons for TLS distribution connections Report bad options for TLS distribution connections OTP-13219 | |||
2015-12-28 | Merge branch 'maint' | Zandra | |
2015-12-28 | Merge branch 'josevalim/jv-map-fun-eval-maint' into maint | Zandra | |
* josevalim/jv-map-fun-eval-maint: Use full list of bindings when matching on map keys OTP-13218 | |||
2015-12-28 | Merge branch 'maint' | Zandra | |
2015-12-28 | Merge branch 'lucafavatella/dialyzer-fun-call' into maint | Zandra | |
* lucafavatella/dialyzer-fun-call: Teach Dialyzer call to funs `M:F/A` (literal M, F, A) OTP-13217 | |||
2015-12-23 | Use monotonic time for call_time trace | Rickard Green | |
2015-12-22 | Merge branch 'maint' | Anders Svensson | |
2015-12-22 | Merge branch 'maint-17' into maint | Anders Svensson | |
2015-12-22 | Merge branch 'maint' | Hans Nilsson | |
* maint: ssh: clean test specs | |||
2015-12-22 | Merge branch 'hans/ssh/ssh_benchmark_SUITE' into maint | Hans Nilsson | |
* hans/ssh/ssh_benchmark_SUITE: ssh: clean test specs | |||
2015-12-22 | ssh: clean test specs | Hans Nilsson | |
2015-12-22 | Merge branch 'maint' | Hans Nilsson | |
* maint: ssh: add econnaborted to disconnect msgs in test suite ssh: fix the check that open-ssh supports certain pubkeys in a test suite |