Hi,
I tried compiling mini-os with optimization flag switched on and got some
warnings leading to errors because of the -Werror flag.
The stuff in netfront.c seems to be forgotten to remove during the update of
the orginal patch from Jacob. Maybe it's better to remove it.
Thanks.
Dietmar.
Signed-off-by: Dietmar Hahn <dietmar.hahn@xxxxxxxxxxxxxxxxxxx>
# HG changeset patch
# User dietmar.hahn@xxxxxxxxxxxxxxxxxxx
# Date 1169202107 -3600
# Node ID e9c7d2ed782fca937aa15131d42e69cbca32db02
# Parent e2539ab3580a28a6bebfe7ea4756fb8c6f90a722
Fixed warnings when compiling with -O.
diff -r e2539ab3580a -r e9c7d2ed782f extras/mini-os/arch/x86/mm.c
--- a/extras/mini-os/arch/x86/mm.c Thu Jan 18 18:54:28 2007 +0000
+++ b/extras/mini-os/arch/x86/mm.c Fri Jan 19 11:21:47 2007 +0100
@@ -61,6 +61,7 @@ void new_pt_frame(unsigned long *pt_pfn,
mmu_update_t mmu_updates[1];
struct mmuext_op pin_request;
+ prot_e = prot_t = pincmd = 0;
DEBUG("Allocating new L%d pt frame for pt_pfn=%lx, "
"prev_l_mfn=%lx, offset=%lx",
level, *pt_pfn, prev_l_mfn, offset);
diff -r e2539ab3580a -r e9c7d2ed782f extras/mini-os/netfront.c
--- a/extras/mini-os/netfront.c Thu Jan 18 18:54:28 2007 +0000
+++ b/extras/mini-os/netfront.c Fri Jan 19 11:21:47 2007 +0100
@@ -327,7 +327,7 @@ done:
printk("backend at %s\n",backend);
printk("mac is %s\n",mac);
- char *res;
+ // char *res;
char path[256];
sprintf(path,"%s/state",backend);
@@ -336,7 +336,7 @@ done:
xenbus_wait_for_value(path,"4");
//free(backend);
- free(res);
+ // free(res);
printk("**************************\n");
diff -r e2539ab3580a -r e9c7d2ed782f extras/mini-os/xenbus/xenbus.c
--- a/extras/mini-os/xenbus/xenbus.c Thu Jan 18 18:54:28 2007 +0000
+++ b/extras/mini-os/xenbus/xenbus.c Fri Jan 19 11:21:47 2007 +0100
@@ -103,7 +103,7 @@ static void xenbus_thread_func(void *ign
static void xenbus_thread_func(void *ign)
{
struct xsd_sockmsg msg;
- unsigned prod;
+ unsigned prod=0;
for (;;)
{
_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel
|
Home