all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Eli Zaretskii <eliz@gnu.org>
To: Fabrice Popineau <fabrice.popineau@gmail.com>
Cc: 22526@debbugs.gnu.org, andrewjmoreton@gmail.com
Subject: bug#22526: 25.0.90; Crash starting gnus
Date: Mon, 15 Feb 2016 13:39:02 +0200	[thread overview]
Message-ID: <83io1qxkx5.fsf@gnu.org> (raw)
In-Reply-To: <CAFgFV9OFcwKCwKMkVq8a=eC78wP1dwYv55FVCzLD-iW4GYUxsA@mail.gmail.com> (message from Fabrice Popineau on Mon, 15 Feb 2016 09:09:09 +0100)

> From: Fabrice Popineau <fabrice.popineau@gmail.com>
> Date: Mon, 15 Feb 2016 09:09:09 +0100
> Cc: andrewjmoreton@gmail.com, 22526@debbugs.gnu.org
> 
>  > As vmmap shows it, the next region is unusable.
>  > The reserved region is 4k and the 60k after are lost.
> 
>  But we only asked for 800 bytes, which could have been satisfied by
>  those 4KB that were reserved there, no? So why dod the commit fail?
> 
> As I read it, the first bloc has been reserved and committed for 4k = 0x10000 starting at 0x1f0000, and then
> 800 bytes
> have been asked for at 0x1f0000+0x10000 = 0x200000, but we failedl to extend the block (mmap_realloc
> failed
> to commit further pages to accommodate the 800 bytes).

No.  We had a buffer whose size was 0xf804, in a block starting at
0x1f0000, which means we already had 0x10000 committed there.  We
needed to enlarge buffer text by another 0x1000 (4KB), which exceeded
the 64KB committed size by 807 bytes, so we asked for a commit of 807
bytes of the next region, starting at 0x200000.  That should have
given us a single 4KB page, since the state of that region was
"reserved".  Instead, it failed with ERROR_INVALID_PARAMETER.

IOW, all of the 64KB of the 0x1f0000 block were already committed, and
all we needed was a single page of the next block.





  reply	other threads:[~2016-02-15 11:39 UTC|newest]

Thread overview: 42+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-02-01 22:13 bug#22526: 25.0.90; Crash starting gnus Andy Moreton
2016-02-07  5:42 ` Lars Ingebrigtsen
2016-02-07 16:00   ` Eli Zaretskii
2016-02-07 20:58     ` Andy Moreton
2016-02-07 21:05       ` Eli Zaretskii
2016-02-11  2:06         ` Andy Moreton
2016-02-11 20:27           ` Eli Zaretskii
2016-02-11 21:20             ` Andy Moreton
2016-02-11 21:26               ` Eli Zaretskii
2016-02-12 13:34                 ` Andy Moreton
2016-02-12 16:16                   ` Eli Zaretskii
2016-02-12 22:26                     ` Andy Moreton
2016-02-13  8:28                       ` Eli Zaretskii
2016-02-13 10:44                         ` Eli Zaretskii
2016-02-13 16:08                           ` Fabrice Popineau
2016-02-13 16:42                             ` Eli Zaretskii
2016-02-13 21:35                               ` Fabrice Popineau
2016-02-13 22:11                                 ` Eli Zaretskii
2016-02-13 23:44                                   ` Fabrice Popineau
2016-02-14  5:49                                     ` Eli Zaretskii
2016-02-14  9:05                                       ` Fabrice Popineau
2016-02-14 16:57                                         ` Eli Zaretskii
2016-02-14  5:41                                   ` Eli Zaretskii
2016-02-14 14:17                                     ` Andy Moreton
2016-02-14 16:55                                       ` Eli Zaretskii
2016-02-14 17:51                                         ` Eli Zaretskii
2016-02-14 21:04                                           ` Fabrice Popineau
2016-02-14 21:29                                             ` Eli Zaretskii
2016-02-14 21:31                                               ` Fabrice Popineau
2016-02-14 21:34                                             ` Eli Zaretskii
2016-02-14 21:41                                               ` Fabrice Popineau
2016-02-15  3:32                                                 ` Eli Zaretskii
2016-02-15  8:09                                                   ` Fabrice Popineau
2016-02-15 11:39                                                     ` Eli Zaretskii [this message]
2016-02-13 15:16                         ` Andy Moreton
2016-02-13 15:52                           ` Eli Zaretskii
2016-02-13 21:26                             ` Andy Moreton
2016-02-16  1:18                               ` Andy Moreton
2016-02-16  3:46                                 ` Eli Zaretskii
2016-02-20 11:08                                   ` Eli Zaretskii
2016-02-20 16:17                                     ` Andy Moreton
2016-02-20 17:01                                       ` Eli Zaretskii

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=83io1qxkx5.fsf@gnu.org \
    --to=eliz@gnu.org \
    --cc=22526@debbugs.gnu.org \
    --cc=andrewjmoreton@gmail.com \
    --cc=fabrice.popineau@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/emacs.git
	https://git.savannah.gnu.org/cgit/emacs/org-mode.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.