Revert of Revert of Disable yama on 3.2 kernels with 32bit userland and 64bit kernel.

Revert of Revert of Disable yama on 3.2 kernels with 32bit userland and 64bit kernel. (https://codereview.chromium.org/374933002/)

Reason for revert:
[Sheriff] This was fixed already in 281762.

Original issue's description:
> Revert of Disable yama on 3.2 kernels with 32bit userland and 64bit kernel. (https://codereview.chromium.org/371113003/)
> 
> Reason for revert:
> Broke clang builders:
> https://build.chromium.org/p/chromium.linux/builders/Linux%20Clang%20(dbg)/builds/61819/steps/compile/logs/stdio
> 
> Original issue's description:
> > Disable two yama tests on 3.2 kernels with 32bit userland and 64bit kernel.
> > 
> > Disabling yama restrictions is broken there.
> > 
> > BUG=391916
> > R=jorgelo@chromium.org
> > 
> > Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=281666
> > 
> > Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=281757
> 
> TBR=jorgelo@chromium.org,keescook@chromium.org,thakis@chromium.org
> NOTREECHECKS=true
> NOTRY=true
> BUG=391916
> 
> Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=281763

TBR=jorgelo@chromium.org,keescook@chromium.org,thakis@chromium.org,eseidel@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=391916
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=281764

[Michael Achenbach, 2014-07-08 16:20:44]

Created Revert of Revert of Disable yama on 3.2 kernels with 32bit userland and
64bit kernel.

[commit-bot: I haz the power, 2014-07-08 16:22:26]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/machenbach@chromium.org/374943002/1

[commit-bot: I haz the power, 2014-07-08 16:23:58]

Change committed as 281764

[eseidel, 2014-07-08 16:29:03]

Strange that this would make it in at all..

[Michael Achenbach, 2014-07-08 16:30:33]

On 2014/07/08 16:29:03, eseidel wrote:
> Strange that this would make it in at all..

The fix probably touched different code and the patch applied. With NOTRY there
is no testing at all...

[Nico, 2014-07-08 16:33:18]

Oh, it's passive-aggressive tuesday! It's a 32bit change, and the CQ
doesn't test that. So the first time it landed it had a green try set. I
sadly screwed things up when relanding with the 32bit fix, but I was around
and fixed the issue within seconds. Now, what's _your_ excuse?
On Jul 8, 2014 9:29 AM, <eseidel@chromium.org> wrote:

> Strange that this would make it in at all..
>
> https://codereview.chromium.org/374943002/
>

To unsubscribe from this group and stop receiving emails from it, send an email
to chromium-reviews+unsubscribe@chromium.org.

[eseidel, 2014-07-08 16:37:39]

Woh woh woh.  I'm not trying to be aggressive.  Passive or otherwise.

I'm sorry to have caused you a headache.  I was attempting to be helpful.

Please remember our many years of working together. :)

[eseidel, 2014-07-08 16:38:28]

I see now.  My "strange that this would make it in at all" was about *this*
change, which I had assumed would conflict with your fix and thus not land.

[Nico, 2014-07-08 16:59:00]

Oh sorry, this was missing a ":-P" or similar; it was meant in a jokular
way. Reading it again, that wasn't clear at all, sorry.

I'm always glad when folks help with keeping the tree green (which you
often do!) and generally very supportive of reverting early.


On Tue, Jul 8, 2014 at 9:37 AM, <eseidel@chromium.org> wrote:

> Woh woh woh.  I'm not trying to be aggressive.  Passive or otherwise.
>
> I'm sorry to have caused you a headache.  I was attempting to be helpful.
>
> Please remember our many years of working together. :)
>
> https://codereview.chromium.org/374943002/
>

To unsubscribe from this group and stop receiving emails from it, send an email
to chromium-reviews+unsubscribe@chromium.org.

[eseidel, 2014-07-08 17:01:12]

Ah text based communication. :)

Thanks for fixing. :)