[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20131130154440.GZ16735@n2100.arm.linux.org.uk>
Date: Sat, 30 Nov 2013 15:44:40 +0000
From: Russell King - ARM Linux <linux@....linux.org.uk>
To: Anurag Aggarwal <anurag19aggarwal@...il.com>
Cc: Dave Martin <Dave.Martin@....com>,
"naveen.sel@...sung.com" <naveen.sel@...sung.com>,
"narendra.m1@...sung.com" <narendra.m1@...sung.com>,
"nico@...aro.org" <nico@...aro.org>,
Anurag Aggarwal <a.anurag@...sung.com>,
Catalin Marinas <Catalin.Marinas@....com>,
Will Deacon <Will.Deacon@....com>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
"ashish.kalra@...sung.com" <ashish.kalra@...sung.com>,
"cpgs@...sung.com" <cpgs@...sung.com>,
"naveenkrishna.ch@...il.com" <naveenkrishna.ch@...il.com>,
"rajat.suri@...sung.com" <rajat.suri@...sung.com>,
"poorva.s@...sung.com" <poorva.s@...sung.com>,
"linux-arm-kernel@...ts.infradead.org"
<linux-arm-kernel@...ts.infradead.org>,
"mohammad.a2@...sung.com" <mohammad.a2@...sung.com>
Subject: Re: [PATCH V2] ARM : unwinder : Prevent data abort due to stack
overflow while executing unwind instructions Signed-off-by: Anurag
Aggarwal <a.anurag@...sung.com>
On Sat, Nov 30, 2013 at 08:39:02PM +0530, Anurag Aggarwal wrote:
> > You can calculate low, high and high - low once and store them in
> > unwind_ctrl_block. No need to recalculate them every time.
>
> I don't think it is feasible to store high and low in unwind_ctrl_block,
> we will have to recalculate them every time in this case also as the
> value of sp is which change every time and depending on the value
> of sp the value of high and low will also change.
low may change, but high definitely won't do. The SP is not allowed to
move outside the 8K page.
Also, please learn to cut text which is not relevant to your reply,
rather than leaving lots of lines of unnecessary quoted text in.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists