From: Igor Munkin <imun@tarantool.org> To: Sergey Kaplun <skaplun@tarantool.org> Cc: tarantool-patches@dev.tarantool.org Subject: Re: [Tarantool-patches] [PATCH luajit] x64: Fix __call metamethod return dispatch. Date: Fri, 4 Dec 2020 17:35:04 +0300 [thread overview] Message-ID: <20201204143504.GA5396@tarantool.org> (raw) In-Reply-To: <20201204141408.GA20578@root> Sergey, Thanks for your review! On 04.12.20, Sergey Kaplun wrote: > Hi, Igor! > > Thanks for the patch! > LGTM except the single nit. Added your tag: | Reviewed-by: Sergey Kaplun <skaplun@tarantool.org> > > On 04.12.20, Igor Munkin wrote: > > From: Mike Pall <mike> > > > > After linking new cframe to the chain KBASEa still stores the address of > > the previous one. If the execution proceeds to <lj_vmeta_call> KBASE > > value (i.e. low 32 bits of the stored address) might be equal to the > > current BASE address value so the execution takes the invalid path. Such > > address clashing occurs only on x86_64 platform with disabled LJ_GC64, > > Typo: s/address/address parts/ (considering your message in PR). Thanks, fixed. > > > so 64-bit registers have to be compared in x64 build. > > > > NB: Though there is only 32-bit load to restore BASE value prior to the > > comparison, the high 32 bits of RDX are reset to zeros, according to x86 > > long mode semantics. > > > > Igor Munkin: > > * backported the original patch to tarantool/luajit repo > > * extended the original commit message with the rationale > > > > For more info and explanation see LuaJIT/LuaJIT#636. > > > > Relates to tarantool/tarantool#4518 > > Relates to tarantool/tarantool#4649 > > > > Signed-off-by: Igor Munkin <imun@tarantool.org> > > --- > > <snipped> > > > > -- > Best regards, > Sergey Kaplun -- Best regards, IM
next prev parent reply other threads:[~2020-12-04 14:35 UTC|newest] Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top 2020-12-04 13:42 Igor Munkin 2020-12-04 14:14 ` Sergey Kaplun 2020-12-04 14:35 ` Igor Munkin [this message] 2020-12-04 15:40 ` Sergey Ostanevich 2020-12-04 16:22 ` Igor Munkin 2020-12-04 16:24 ` Igor Munkin 2020-12-04 16:46 ` Alexander V. Tikhonov 2020-12-04 17:03 ` Igor Munkin 2020-12-07 9:37 ` Kirill Yukhin
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=20201204143504.GA5396@tarantool.org \ --to=imun@tarantool.org \ --cc=skaplun@tarantool.org \ --cc=tarantool-patches@dev.tarantool.org \ --subject='Re: [Tarantool-patches] [PATCH luajit] x64: Fix __call metamethod return dispatch.' \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: link
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox