Skip to contentSkip to conversation

Use correct kzalloc

PATCH New Kyungmin Park 2 participants 2 messages Oct 6, 2008 Memory technology devices (MTD)

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 Use correct kzalloc

I'm not sure calloc returns all zero values.
Actually the calloc uses the __libc_alloc.

Instead of ambiguity, it uses the correct kzalloc.

---

include/linux/mtd/compat.h +6 -1
index 9036b74..840de4c 100644
@@ -18,7 +18,12 @@
 #define KERN_DEBUG
 
 #define kmalloc(size, flags)	malloc(size)
-#define kzalloc(size, flags)	calloc(size, 1)
+#define kzalloc(size, flags)	({					\
+	void *__ret = malloc(size);					\
+	if (__ret)							\
+		memset(__ret, 0, size);					\
+	__ret;								\
+})
 #define vmalloc(size)			malloc(size)
 #define kfree(ptr)				free(ptr)
 #define vfree(ptr)				free(ptr)
#1 lore raw

PATCH 1/1 Use correct kzalloc

Dear Kyungmin Park,

Quoted 1 line · In message <20081006022028.GA5522@july> you wrote:

I'm not sure calloc returns all zero values.

Why aren't you sure? Check the code...

Quoted 1 line

Actually the calloc uses the __libc_alloc.

Ooops? What exactly are you talking about?

It makes no sense to reinvent the wheel here.

If calloc() should have a problem (which I don't believe) it needs to be fixed.

NAK for this patch.

Best regards,

Wolfgang Denk

Signature · 5 lines
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
The human mind  ordinarily  operates  at  only  ten  percent  of  its
capacity. The rest is overhead for the operating system.