All of lore.kernel.org
 help / color / mirror / Atom feed
From: Vladimir 'phcoder' Serbinenko <phcoder@gmail.com>
To: The development of GNU GRUB <grub-devel@gnu.org>
Subject: Re: [PATCH] Re: Bug in read_handler_list (autoloading) ???
Date: Fri, 13 Nov 2009 23:43:52 +0100	[thread overview]
Message-ID: <4AFDE128.6050408@gmail.com> (raw)
In-Reply-To: <1258058194.4779.6.camel@fz.local>

[-- Attachment #1: Type: text/plain, Size: 906 bytes --]

Felix Zielcke wrote:
> Am Donnerstag, den 12.11.2009, 21:46 +0530 schrieb BVK:
>   
>> Valgrind log is attached. It also reported invalid free for the same place.
>>
>> BTW, valgrind is run as
>>
>>     sudo valgrind -v --log-file=/tmp/valgrind.log ./grub-emu
>>     
>
> Here's a grub-emu tested patch to fix this
>
>
>   
Why are all handlers are removed on normal.mod unload? If user e.g.
relies on special terminal he may lose terminal completely. I would say
it's better not to touch already loaded modules. I temporarily unmerged
it from experimental. We can remerge it once issues are fixed
>   
> ------------------------------------------------------------------------
>
> _______________________________________________
> Grub-devel mailing list
> Grub-devel@gnu.org
> http://lists.gnu.org/mailman/listinfo/grub-devel


-- 
Regards
Vladimir 'phcoder' Serbinenko



[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 293 bytes --]

  parent reply	other threads:[~2009-11-13 22:44 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-11-11  4:00 Bug in read_handler_list (autoloading) ??? BVK
2009-11-12 16:16 ` BVK
2009-11-12 20:36   ` [PATCH] " Felix Zielcke
2009-11-12 20:51     ` Robert Millan
2009-11-13 22:43     ` Vladimir 'phcoder' Serbinenko [this message]
2009-11-14 15:35       ` Robert Millan
2009-11-14 21:11         ` Felix Zielcke
2009-11-15 11:10           ` Robert Millan
2010-01-07 18:48             ` Robert Millan

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=4AFDE128.6050408@gmail.com \
    --to=phcoder@gmail.com \
    --cc=grub-devel@gnu.org \
    /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.
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.