lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Date:	Thu, 08 Apr 2010 11:43:10 +0900
From:	Tejun Heo <tj@...nel.org>
To:	Sonic Zhang <sonic.adi@...il.com>
CC:	graff yang <graff.yang@...il.com>, dhowells@...hat.com,
	linux-kernel@...r.kernel.org, akpm@...ux-foundation.org,
	uclinux-dist-devel@...ckfin.uclinux.org
Subject: Re: [PATCH] mm/nommu.c:Dynamic alloc/free percpu area for nommu

Hello, Sonic.

> I tested your patch on NOMMU bf561 with SMP enabled. It is compiled
> and boots without problem. Because there are few percpu data defined
> in bf561 SMP kernel, the functions in mm/percpu-km.c may not be
> executed with a simple test. But, since these functions are simple
> malloc/free, I don't see any problem.

Great, thanks for testing.  Just in case, can you please test with the
attached module?  In test-pcpu.c, the cmds table directs the module
what to allocate and free.  { size > 0, tag } entry makes it allocate
an area with the specified size and tag and { 0, tag } entry makes it
free all areas with the matching tag.  The existing table makes pretty
large amount of allocations and might not work very well on nommu
configuration.  There are also several DEFINE_PER_CPU() instances to
test module static percpu area alloc/free.  Please insmod/rmmod in
loop and make sure it doesn't leak any memory or crashes the machine.

Thanks.

-- 
tejun

Download attachment "test-pcpu.tar.gz" of type "application/x-gzip" (1590 bytes)

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ