Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1672302

Re: [lkp-robot] [mm] 1be7107fbe: kernel_BUG_at_mm/mmap.c

Show all headers | View raw

On Wed, 21 Jun 2017, Oleg Nesterov wrote:
> On 06/21, Linus Torvalds wrote:
> >
> > Hugh, Michal - I also merged Helge's drop-up cleanup, is there
> > anything I've missed? I think Oleg had something, but I can't recall
> > right now, and I might just have missed it.
> 
> Well, I meant, perhaps we need a bit more changes to ensure that a new
> GROWSDOWN vma can't come without a gap below. But this is really minor,
> we can do this later even if I am right.

I'm mortified.  At last I understand you, and see that you spelt it out
perfectly clearly in your "unmapped_gap" mail earlier in another thread,
when I was in a rush to prioritize what bugs needed looking at first,
and brusquely persuaded you to say that this is only a minor defect.

Well, yes, it's not a new vulnerability and it's not a new regression,
and the users of MAP_GROWSDOWN are few and far between, and often the
assignment of holes will work out just fine; but it's an embarrassing
oversight on my part, that everything was geared to inflating the
size of the existing VM_GROWS vmas, completely forgetting to inflate
the size of the area to be added.

Without reading you thoroughly (and all the fault mine not yours),
I had thought you were referring to the way that a MAP_GROWSDOWN
area may be assigned a place in which the stack_guard_gap would
immediately prevent it from growing down afterwards (and I'm not
sure what to do about that).  But your point is that it may be
assigned a place in which there is not even a stack_guard_gap
below it.  (And so the bug that trinity found would not even
depend upon MAP_FIXED.)

I'll go back to read you again and think on the best way to
correct that, I hope it won't need lots of plumbing through
different levels.

Hugh

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

Re: [lkp-robot] [mm]  1be7107fbe: kernel_BUG_at_mm/mmap.c Hugh Dickins <hughd@google.com> - 2017-06-21 04:50 +0200
  Re: [lkp-robot] [mm] 1be7107fbe: kernel_BUG_at_mm/mmap.c Linus Torvalds <torvalds@linux-foundation.org> - 2017-06-21 20:30 +0200
    Re: [lkp-robot] [mm] 1be7107fbe: kernel_BUG_at_mm/mmap.c Linus Torvalds <torvalds@linux-foundation.org> - 2017-06-21 21:40 +0200
      Re: [lkp-robot] [mm] 1be7107fbe: kernel_BUG_at_mm/mmap.c Oleg Nesterov <oleg@redhat.com> - 2017-06-21 22:30 +0200
        Re: [lkp-robot] [mm] 1be7107fbe: kernel_BUG_at_mm/mmap.c Linus Torvalds <torvalds@linux-foundation.org> - 2017-06-21 22:40 +0200
          Re: [lkp-robot] [mm] 1be7107fbe: kernel_BUG_at_mm/mmap.c Oleg Nesterov <oleg@redhat.com> - 2017-06-21 23:00 +0200
            Re: [lkp-robot] [mm] 1be7107fbe: kernel_BUG_at_mm/mmap.c Linus Torvalds <torvalds@linux-foundation.org> - 2017-06-22 00:20 +0200
              Re: [lkp-robot] [mm] 1be7107fbe: kernel_BUG_at_mm/mmap.c Hugh Dickins <hughd@google.com> - 2017-06-22 03:10 +0200
                Re: [lkp-robot] [mm] 1be7107fbe: kernel_BUG_at_mm/mmap.c Oleg Nesterov <oleg@redhat.com> - 2017-06-22 17:20 +0200
                Re: [lkp-robot] [mm] 1be7107fbe: kernel_BUG_at_mm/mmap.c Hugh Dickins <hughd@google.com> - 2017-06-22 20:10 +0200
                Re: [lkp-robot] [mm] 1be7107fbe: kernel_BUG_at_mm/mmap.c Oleg Nesterov <oleg@redhat.com> - 2017-06-22 23:00 +0200
    Re: [lkp-robot] [mm] 1be7107fbe: kernel_BUG_at_mm/mmap.c Oleg Nesterov <oleg@redhat.com> - 2017-06-21 21:40 +0200
      Re: [lkp-robot] [mm] 1be7107fbe: kernel_BUG_at_mm/mmap.c Hugh Dickins <hughd@google.com> - 2017-06-22 06:30 +0200
    Re: [lkp-robot] [mm] 1be7107fbe: kernel_BUG_at_mm/mmap.c Hugh Dickins <hughd@google.com> - 2017-06-21 21:50 +0200

csiph-web