This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH] i386: Insert ENDBR to trampoline for -fcf-protection=branch -mibt


On Tue, Mar 27, 2018 at 10:08 AM, Uros Bizjak <ubizjak@gmail.com> wrote:
> On Mon, Mar 26, 2018 at 10:42 PM, Tsimbalist, Igor V
> <igor.v.tsimbalist@intel.com> wrote:
>>> -----Original Message-----
>>> From: H.J. Lu [mailto:hjl.tools@gmail.com]
>>> Sent: Monday, March 26, 2018 5:59 PM
>>> To: Tsimbalist, Igor V <igor.v.tsimbalist@intel.com>
>>> Cc: gcc-patches@gcc.gnu.org; Uros Bizjak <ubizjak@gmail.com>
>>> Subject: Re: [PATCH] i386: Insert ENDBR to trampoline for -fcf-
>>> protection=branch -mibt
>>>
>>> On Mon, Mar 26, 2018 at 8:23 AM, Tsimbalist, Igor V
>>> <igor.v.tsimbalist@intel.com> wrote:
>>> >> -----Original Message-----
>>> >> From: Lu, Hongjiu
>>> >> Sent: Sunday, March 25, 2018 12:50 AM
>>> >> To: gcc-patches@gcc.gnu.org; Uros Bizjak <ubizjak@gmail.com>;
>>> Tsimbalist,
>>> >> Igor V <igor.v.tsimbalist@intel.com>
>>> >> Subject: [PATCH] i386: Insert ENDBR to trampoline for -fcf-
>>> >> protection=branch -mibt
>>> >>
>>> >> When -fcf-protection=branch -mibt are used, we need to insert ENDBR
>>> >> to trampoline.  TRAMPOLINE_SIZE is creased by 4 bytes to accommodate
>>> >> 4-byte ENDBR instruction.
>>> >>
>>> >> OK for trunk?
>>> >
>>> > Regarding the test. Is it possible to check what is generated in a
>>> trampoline? In particular, that endbr is generated.
>>> >
>>>
>>> I think run-time test is sufficient.
>>
>> Ok then.
>
> Rubber-stamp OK.
>

Done.

Thanks.

-- 
H.J.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]