Re: [testsuite patch] for: [PATCH] [PR corefiles/17808] i386: Fix internal error when prstatus in core file is too big

Andreas Arnez
2015-01-09T09:47:23Z

On Thu, Jan 08 2015, Jan Kratochvil wrote:

> On Thu, 08 Jan 2015 17:16:20 +0100, Andreas Arnez wrote:
>> Note that this behavior deviates from the default policy: In general, if
>> some future kernel adds new registers to a register set, then a GDB
>> unaware of this extension would read the known subset and just ignore
>> the unknown bytes.
>
> This patch is about 'assert' vs. 'if' so I find this paragraph outside of the
> topic of this thread/regression.

Right, it seems a bit off-topic. What I really mean is that "this
behavior *still* deviates from the default policy", and I just wanted to
make sure that we all agree to that. Anyway, I will remove this
paragraph from the commit message.

Any other comments? After adjusting the commit message, is the patch
then OK to apply?