Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 27 Mar 2015 01:48:05 +0000 (UTC)
From:      Rui Paulo <rpaulo@FreeBSD.org>
To:        src-committers@freebsd.org, svn-src-projects@freebsd.org
Subject:   svn commit: r280723 - projects/lua-bootloader/sys/boot/common
Message-ID:  <201503270148.t2R1m5Dg024209@svn.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: rpaulo
Date: Fri Mar 27 01:48:05 2015
New Revision: 280723
URL: https://svnweb.freebsd.org/changeset/base/280723

Log:
  Improve the debugging printfs.

Modified:
  projects/lua-bootloader/sys/boot/common/interp_lua.c

Modified: projects/lua-bootloader/sys/boot/common/interp_lua.c
==============================================================================
--- projects/lua-bootloader/sys/boot/common/interp_lua.c	Fri Mar 27 00:42:51 2015	(r280722)
+++ projects/lua-bootloader/sys/boot/common/interp_lua.c	Fri Mar 27 01:48:05 2015	(r280723)
@@ -43,13 +43,18 @@ __FBSDID("$FreeBSD$");
 struct interp_lua_softc {
 	lua_State	*luap;
 };
-static struct interp_lua_softc lua_softc = { 0 };
 
+static struct interp_lua_softc lua_softc;
+
+#ifdef LUA_DEBUG
 #define	LDBG(...)	do {			\
 	printf("%s(%d): ", __func__, __LINE__);	\
 	printf(__VA_ARGS__);			\
 	printf("\n");				\
 } while (0)
+#else
+#define	LDBG(...)
+#endif
 
 
 void
@@ -63,9 +68,11 @@ interp_lua_init(void *ctx)
 	char buf[16];
 
 	softc = ctx;
+	LDBG("creating context");
 	luap = lua_create();
 	if (luap == NULL) {
-		LDBG("problem initializing the Lua interpreter");
+		printf("problem initializing the Lua interpreter\n");
+		abort();
 	}
 	softc->luap = luap;
 	register_utils(luap);
@@ -84,9 +91,9 @@ interp_lua_run(void *data, const char *l
 
 	softc = data;
 	luap = softc->luap;
-
+	LDBG("running line...");
 	if (ldo_string(luap, line, strlen(line)) != 0)
-		LDBG("failed to execute \'%s\'", line);
+		printf("failed to parse \'%s\'\n", line);
 
 	return (0);
 }
@@ -97,6 +104,7 @@ interp_lua_incl(void *ctx, const char *f
 	struct interp_lua_softc *softc;
 
 	softc = ctx;
+	LDBG("loading file %s", filename);
 
 	return (ldo_file(softc->luap, filename));
 }
@@ -108,10 +116,11 @@ interp_lua_incl(void *ctx, const char *f
 int
 interp_lua_load_config(void *ctx)
 {
+	LDBG("loading config");
+
 	return (interp_lua_incl(ctx, "/boot/loader.lua"));
 }
 
-
 struct interp boot_interp_lua = {
 	.init = interp_lua_init,
 	.run = interp_lua_run,



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201503270148.t2R1m5Dg024209>