Skip to contentSkip to conversation

Remove redundant #ifdef _BSD

PATCH In review Andre Przywara 2 participants 2 messages Dec 10, 2008 – Dec 11, 2008

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 Remove redundant #ifdef _BSD

Hi,

since _BSD if already handled in osdep.c:qemu_memalign(), we don't need to check it in the calling function again. getpagesize() is available in BSD.

Regards,
Andre.

Signature · 9 lines
Andre Przywara
AMD-Operating System Research Center (OSRC), Dresden, Germany
Tel: +49 351 277-84917
----to satisfy European Law for business letters:
AMD Saxony Limited Liability Company & Co. KG,
Wilschdorfer Landstr. 101, 01109 Dresden, Germany
Register Court Dresden: HRA 4896, General Partner authorized
to represent: AMD Saxony LLC (Wilmington, Delaware, US)
General Manager of AMD Saxony LLC: Dr. Hans-R. Deppe, Thomas McCoy
osdep.c +0 -4
diff -r fb954e9c4a86 osdep.c
@@ -200,11 +200,7 @@ void *qemu_vmalloc(size_t size)
     if (kqemu_allowed)
         return kqemu_vmalloc(size);
 #endif
-#ifdef _BSD
-    return valloc(size);
-#else
     return qemu_memalign(getpagesize(), size);
-#endif
 }
 
 void qemu_vfree(void *ptr)
#1 lore raw
Quoted 9 lines · Andre Przywara

Hi,

since _BSD if already handled in osdep.c:qemu_memalign(), we don't need to
check it in the calling function again. getpagesize() is available in BSD.

Regards,
Andre.

Thanks, committed in r5983.