LKML Archive on lore.kernel.org
help / color / mirror / Atom feed
* Re: Linux 2.4.23-rc5
       [not found] <VK1t.6TB.1@gated-at.bofh.it>
@ 2003-11-25 16:34 ` Ihar 'Philips' Filipau
  0 siblings, 0 replies; 9+ messages in thread
From: Ihar 'Philips' Filipau @ 2003-11-25 16:34 UTC (permalink / raw)
  To: Marcelo Tosatti; +Cc: Linux Kernel Mailing List

Hi!

   I was investigating memory allocation problems on my systems.
   And I only figured out that I'm using quite up-to-date kernel - 2.4.22.

   This is my post from another thread:

>    Vanilla 2.4.22 (this is x86) (with HZ=1024, if it does matter).
> 
>    after '# echo -1 >/proc/sys/vm/overcommit_memory'
>    1. test app with memory touch still gets killed by 
>       oom_killer. (so no malloc() == NULL)
>    2. test app w/o memory touch still can happily allocate 2.8GB 
>       of memory (x86 - looks like 3/1 memory split) and only then 
>       gets NULL pointer - oom_killer is silent.

   "overcommit_memory < 0" supposed to not allow apps to overallocate 
memory - but still it works not like it is said in 
Documentation/filesystems/proc.txt.
    So apps which try to be MM friendly can silently break due to 
very-very lazy memory allocation in kernel. Not nice when malloc() says 
"everything is fine!".

   [ Just checked: with overcommit==-1, as soon app trying to touch 
those magic 2.8GB of memory, it gets killed by oom_killer. This is 
totally wrong... ]

   Probably fix docs at least... :-(

Marcelo Tosatti wrote:
> 
> Hi, 
> 
> Yet another thing showed up (possible data corruption on x86-64), so here 
> goes -rc5.
> 
> 
> Summary of changes from v2.4.23-rc4 to v2.4.23-rc5
> ============================================
> 
> Andi Kleen:
>   o Fix 32bit truncate64 on x86-64
> 
> Marcelo Tosatti:
>   o Felix Radensky: Remove debugging printk from agpgart
>   o Changed EXTRAVERSION to -rc5
> 
> 
> 
> -
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at  http://www.tux.org/lkml/



^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: Linux 2.4.23-rc5
  2003-11-25 17:07   ` Pascal Schmidt
@ 2003-11-26  9:18     ` Ihar 'Philips' Filipau
  0 siblings, 0 replies; 9+ messages in thread
From: Ihar 'Philips' Filipau @ 2003-11-26  9:18 UTC (permalink / raw)
  To: Pascal Schmidt; +Cc: linux-kernel

Pascal Schmidt wrote:
> On Tue, 25 Nov 2003 17:50:20 +0100, you wrote in linux.kernel:
> 
>>   "overcommit_memory < 0" supposed to not allow apps to overallocate 
>>memory - but still it works not like it is said in 
>>Documentation/filesystems/proc.txt.
> 
> 
> That file mentiones setting overcommit_memory to 0 to disable overcommit.
> Have you tried that?
> 

   Yes. This is kernel default on all my systems.

-- 
Ihar 'Philips' Filipau  / with best regards from Saarbruecken.
--                                                           _ _ _
  Because the kernel depends on it existing. "init"          |_|*|_|
  literally _is_ special from a kernel standpoint,           |_|_|*|
  because its' the "reaper of zombies" (and, may I add,      |*|*|*|
  that would be a great name for a rock band).
                                 -- Linus Torvalds


^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: Linux 2.4.23-rc5
  2003-11-25 20:06   ` Marcelo Tosatti
@ 2003-11-26  2:01     ` William Lee Irwin III
  0 siblings, 0 replies; 9+ messages in thread
From: William Lee Irwin III @ 2003-11-26  2:01 UTC (permalink / raw)
  To: Marcelo Tosatti; +Cc: Mike Fedyk, linux-kernel

On Tue, 25 Nov 2003, Mike Fedyk wrote:
>> Will 2.4.23 have the oom killer?

On Tue, Nov 25, 2003 at 06:06:31PM -0200, Marcelo Tosatti wrote:
> No. Andrea removed the oom killer because it had problems (deadlocks, it
> can "guess" wrong in some cases).
> It seems that in most cases killing the allocator (what 2.4.23 does)  
> works fine.
> Having it configurable might be desired. 

Some of the bad behavior was due to races that disturbed the heuristics.
2.6 got those fixed and it made a large difference wrt. correcting its
bad behaviors. It's not infallible, but relatively well-behaved. (Not
that I could be arsed to debate whether that makes it worth keeping.)


-- wli

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: Linux 2.4.23-rc5
  2003-11-25 19:04 ` Mike Fedyk
@ 2003-11-25 20:06   ` Marcelo Tosatti
  2003-11-26  2:01     ` William Lee Irwin III
  0 siblings, 1 reply; 9+ messages in thread
From: Marcelo Tosatti @ 2003-11-25 20:06 UTC (permalink / raw)
  To: Mike Fedyk; +Cc: Marcelo Tosatti, linux-kernel



On Tue, 25 Nov 2003, Mike Fedyk wrote:

> On Tue, Nov 25, 2003 at 09:42:21AM -0200, Marcelo Tosatti wrote:
> > 
> > 
> > Hi, 
> > 
> > Yet another thing showed up (possible data corruption on x86-64), so here 
> > goes -rc5.
> 
> Sorry for not reading the entire 100+ message thread, but maybe someone can
> answer anyway.
> 
> Will 2.4.23 have the oom killer?

No. Andrea removed the oom killer because it had problems (deadlocks, it
can "guess" wrong in some cases).

It seems that in most cases killing the allocator (what 2.4.23 does)  
works fine.

Having it configurable might be desired. 



^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: Linux 2.4.23-rc5
  2003-11-25 11:42 Marcelo Tosatti
  2003-11-25 13:34 ` Marcos D. Marado Torres
@ 2003-11-25 19:04 ` Mike Fedyk
  2003-11-25 20:06   ` Marcelo Tosatti
  1 sibling, 1 reply; 9+ messages in thread
From: Mike Fedyk @ 2003-11-25 19:04 UTC (permalink / raw)
  To: Marcelo Tosatti; +Cc: linux-kernel

On Tue, Nov 25, 2003 at 09:42:21AM -0200, Marcelo Tosatti wrote:
> 
> 
> Hi, 
> 
> Yet another thing showed up (possible data corruption on x86-64), so here 
> goes -rc5.

Sorry for not reading the entire 100+ message thread, but maybe someone can
answer anyway.

Will 2.4.23 have the oom killer?

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: Linux 2.4.23-rc5
       [not found] ` <VOyg.w9.11@gated-at.bofh.it>
  2003-11-25 17:07   ` Pascal Schmidt
@ 2003-11-25 17:15   ` Pascal Schmidt
  1 sibling, 0 replies; 9+ messages in thread
From: Pascal Schmidt @ 2003-11-25 17:15 UTC (permalink / raw)
  To: Ihar 'Philips' Filipau; +Cc: linux-kernel

On Tue, 25 Nov 2003 17:50:20 +0100, you wrote in linux.kernel:

>    "overcommit_memory < 0" supposed to not allow apps to overallocate 
> memory - but still it works not like it is said in 
> Documentation/filesystems/proc.txt.

Actually, looking at vm_enough_memory in mm/mmap.c in 2.4, which seems to
be the only user of sysctl_overcommit_memory, it is just used directly in
an if clause:

	if (sysctl_overcommit_memory)
		return 1;
		
This means any value different from 0 will turn overcommit on, no matter
whether positive or negative value.

Documentation bug, I would say.

-- 
Ciao,
Pascal

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: Linux 2.4.23-rc5
       [not found] ` <VOyg.w9.11@gated-at.bofh.it>
@ 2003-11-25 17:07   ` Pascal Schmidt
  2003-11-26  9:18     ` Ihar 'Philips' Filipau
  2003-11-25 17:15   ` Pascal Schmidt
  1 sibling, 1 reply; 9+ messages in thread
From: Pascal Schmidt @ 2003-11-25 17:07 UTC (permalink / raw)
  To: Ihar 'Philips' Filipau; +Cc: linux-kernel

On Tue, 25 Nov 2003 17:50:20 +0100, you wrote in linux.kernel:

>    "overcommit_memory < 0" supposed to not allow apps to overallocate 
> memory - but still it works not like it is said in 
> Documentation/filesystems/proc.txt.

That file mentiones setting overcommit_memory to 0 to disable overcommit.
Have you tried that?

-- 
Ciao,
Pascal

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: Linux 2.4.23-rc5
  2003-11-25 11:42 Marcelo Tosatti
@ 2003-11-25 13:34 ` Marcos D. Marado Torres
  2003-11-25 19:04 ` Mike Fedyk
  1 sibling, 0 replies; 9+ messages in thread
From: Marcos D. Marado Torres @ 2003-11-25 13:34 UTC (permalink / raw)
  To: Marcelo Tosatti; +Cc: linux-kernel

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

It's not on kernel.org yet...
Did you forgot to upload it again? O:-)

Marcos Marado

- --
==================================================
Marcos Daniel Marado Torres AKA Mind Booster Noori
/"\               http://student.dei.uc.pt/~marado
\ /                       marado@student.dei.uc.pt
 X   ASCII Ribbon Campaign
/ \  against HTML e-mail and Micro$oft attachments
==================================================

On Tue, 25 Nov 2003, Marcelo Tosatti wrote:

>
>
> Hi,
>
> Yet another thing showed up (possible data corruption on x86-64), so here
> goes -rc5.
>
>
> Summary of changes from v2.4.23-rc4 to v2.4.23-rc5
> ============================================
>
> Andi Kleen:
>   o Fix 32bit truncate64 on x86-64
>
> Marcelo Tosatti:
>   o Felix Radensky: Remove debugging printk from agpgart
>   o Changed EXTRAVERSION to -rc5
>
>
>
> -
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at  http://www.tux.org/lkml/
>
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.2.1 (GNU/Linux)
Comment: Made with pgp4pine 1.76

iD8DBQE/w1pOmNlq8m+oD34RAhE5AKCXsmUR1AM6Is76HSO6VxUma9O9TwCdFEvo
p11Y6XEbIxeVDUYTAB+xdXo=
=ggWy
-----END PGP SIGNATURE-----


^ permalink raw reply	[flat|nested] 9+ messages in thread

* Linux 2.4.23-rc5
@ 2003-11-25 11:42 Marcelo Tosatti
  2003-11-25 13:34 ` Marcos D. Marado Torres
  2003-11-25 19:04 ` Mike Fedyk
  0 siblings, 2 replies; 9+ messages in thread
From: Marcelo Tosatti @ 2003-11-25 11:42 UTC (permalink / raw)
  To: linux-kernel



Hi, 

Yet another thing showed up (possible data corruption on x86-64), so here 
goes -rc5.


Summary of changes from v2.4.23-rc4 to v2.4.23-rc5
============================================

Andi Kleen:
  o Fix 32bit truncate64 on x86-64

Marcelo Tosatti:
  o Felix Radensky: Remove debugging printk from agpgart
  o Changed EXTRAVERSION to -rc5




^ permalink raw reply	[flat|nested] 9+ messages in thread

end of thread, other threads:[~2003-11-26  9:18 UTC | newest]

Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
     [not found] <VK1t.6TB.1@gated-at.bofh.it>
2003-11-25 16:34 ` Linux 2.4.23-rc5 Ihar 'Philips' Filipau
     [not found] <VOyg.w9.13@gated-at.bofh.it>
     [not found] ` <VOyg.w9.11@gated-at.bofh.it>
2003-11-25 17:07   ` Pascal Schmidt
2003-11-26  9:18     ` Ihar 'Philips' Filipau
2003-11-25 17:15   ` Pascal Schmidt
2003-11-25 11:42 Marcelo Tosatti
2003-11-25 13:34 ` Marcos D. Marado Torres
2003-11-25 19:04 ` Mike Fedyk
2003-11-25 20:06   ` Marcelo Tosatti
2003-11-26  2:01     ` William Lee Irwin III

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).