2018-08-21

devnexen: Avoid directly adding to string literal

https://github.com/php/php-src/commit/0f68ff5c92924deb9401ce66458e67f250ccde74
メモリアロケータの修正
メモリアロケータが OS から追加の領域を取ってくる際（realloc 的な）、Windows 以外では mmap() を使っていた
なるべく地続きのメモリ領域を確保するために MAP_FIXED が使いたかったが、MAP_FIXED はすでに別途で使われている領域を指定されると、元からいる奴をぶっ飛ばすという恐ろしいブツだったため、使えなかった
最近めに MAP_FIXED_NOREPLACE という、先客がいたらぶっ飛ばさないで諦める奴が追加されたので、Linux ではそちらを使い、FreeBSD では同様の挙動の MAP_FIXED|MAP_EXCL を使うようになった
~~これ元実装では先客がいたら別の箇所を確保して返すだけだったけど、修正後だと単に諦めてしまうの、別によいのだろうか、という気持ち~~
- 聞いてみたところ、「元の奴ずれたら munmap() してたので意図通りやでー、でもこの際 munmap() に失敗しない限りエラーメッセージは出してなかったので、エラー時の処理はびみょそう」という話があった
- そんでスッと PR がきた https://github.com/php/php-src/pull/3456 ので親指だけ立てといた

https://github.com/php/php-src/commit/6d88e1ccd67d0b3e57679b3c6caa330de22765d8
こちらも BIND_STATIC で第 2 オペランドがいらなくなったのをうけ、オペコード/オペコードハンドラ側で第 2 オペランドを UNUSED に修正
なお、全然別の修正で array_key_exists() に専用のオペコード割り当てて最適化する、という PR が存在し、「あっちは 7.3 にはもう Feature Freeze だからダメって聞いたけど、こっちの BIND_STATIC の修正は何で入れるの！何で入れていいの！」的な会話が 20 日の元修正コミットへのコメントであり、dmitry は「自己完結してる修正は beta でも入れて良いはず、API/ABI の互換性は崩さない方いいけど、あと array_key_exists() についてはちゃんとレビューあればそっちも入れていいんじゃね？」などと言ってたりする

https://github.com/php/php-src/commit/168376e98e533f9f99159d9f5516e03ab823a302
ext/opcache で関数テーブル/クラスエントリの破棄処理をリファクタリング
元コードでは関数テーブルとクラスエントリのデストラクタをオーバーライドする所があった
関数テーブルのデストラクタは、対象がユーザ定義関数で、静的変数があって immutable でなくて、参照カウントを 1 つ減らして 0 になるようなら、静的変数のハッシュテーブルを破棄、というもの
クラスのデストラクタは、そのクラスの関数テーブルを同様の処理でオーバーライドするもの
これらのオーバーライドされたデストラクタは、free_persistent_script() の第 2 引数が 1 のケースで呼ばれる
free_persistent_script() の第 2 引数へ 1 を渡して呼ぶ箇所は同時に zend_accel_free_user_functions() を呼ぶ箇所でもあり、なのでそっちにハッシュテーブルの破棄処理を移動した形のもよう
元コードは黙って efree() な感じに見えるので、破棄処理の内容も変わってそう？ここの具体的な差異までは追ってない