* [PATCH 6/6] fdisk: add debug support
@ 2012-05-20 16:11 Davidlohr Bueso
2012-05-21 20:32 ` Davidlohr Bueso
0 siblings, 1 reply; 3+ messages in thread
From: Davidlohr Bueso @ 2012-05-20 16:11 UTC (permalink / raw)
To: Karel Zak, Petr Uzel; +Cc: util-linux
From: Davidlohr Bueso <dave@gnu.org>
Based on libmnt, this patch adds basic debugging support for fdisk. Currently only CONTEXT is debugged, yet keeps exact functionality
as libmnt/libblkid.
Signed-off-by: Davidlohr Bueso <dave@gnu.org>
---
fdisk/fdisk.c | 2 ++
fdisk/fdisk.h | 27 +++++++++++++++++++++++++++
fdisk/utils.c | 33 ++++++++++++++++++++++++++++++++-
3 files changed, 61 insertions(+), 1 deletions(-)
diff --git a/fdisk/fdisk.c b/fdisk/fdisk.c
index 400e682..9b91fa0 100644
--- a/fdisk/fdisk.c
+++ b/fdisk/fdisk.c
@@ -2168,6 +2168,8 @@ int main(int argc, char **argv)
}
}
+ fdisk_init_debug(0);
+
if (user_set_sector_size && argc-optind != 1)
printf(_("Warning: the -b (set sector size) option should"
" be used with one specified device\n"));
diff --git a/fdisk/fdisk.h b/fdisk/fdisk.h
index c43517c..2471541 100644
--- a/fdisk/fdisk.h
+++ b/fdisk/fdisk.h
@@ -32,6 +32,33 @@
#define cround(n) (display_in_cyl_units ? ((n)/units_per_sector)+1 : (n))
#define scround(x) (((x)+units_per_sector-1)/units_per_sector)
+/* fdisk debugging flags/options */
+#define FDISK_DEBUG_INIT (1 << 1)
+#define FDISK_DEBUG_CONTEXT (1 << 2)
+#define FDISK_DEBUG_ALL 0xFFFF
+
+# define ON_DBG(m, x) do { \
+ if ((FDISK_DEBUG_ ## m) & fdisk_debug_mask) { \
+ x; \
+ } \
+ } while (0)
+
+# define DBG(m, x) do { \
+ if ((FDISK_DEBUG_ ## m) & fdisk_debug_mask) { \
+ fprintf(stderr, "%d: fdisk: %8s: ", getpid(), # m); \
+ x; \
+ } \
+ } while (0)
+
+# define DBG_FLUSH do { \
+ if (fdisk_debug_mask && \
+ fdisk_debug_mask != FDISK_DEBUG_INIT) \
+ fflush(stderr); \
+ } while(0)
+
+extern int fdisk_debug_mask;
+extern void fdisk_init_debug(int mask);
+
struct partition {
unsigned char boot_ind; /* 0x80 - active */
unsigned char head; /* starting head */
diff --git a/fdisk/utils.c b/fdisk/utils.c
index ad83d7f..9f071a6 100644
--- a/fdisk/utils.c
+++ b/fdisk/utils.c
@@ -23,6 +23,35 @@
#include "common.h"
#include "fdisk.h"
+int fdisk_debug_mask;
+
+/**
+ * fdisk_init_debug:
+ * @mask: debug mask (0xffff to enable full debuging)
+ *
+ * If the @mask is not specified then this function reads
+ * FDISK_DEBUG environment variable to get the mask.
+ *
+ * Already initialized debugging stuff cannot be changed. It does not
+ * have effect to call this function twice.
+ */
+void fdisk_init_debug(int mask)
+{
+ if (fdisk_debug_mask & FDISK_DEBUG_INIT)
+ return;
+ if (!mask) {
+ char *str = getenv("FDISK_DEBUG");
+ if (str)
+ fdisk_debug_mask = strtoul(str, 0, 0);
+ } else
+ fdisk_debug_mask = mask;
+
+ if (fdisk_debug_mask)
+ printf("fdisk: debug mask set to 0x%04x.\n",
+ fdisk_debug_mask);
+ fdisk_debug_mask |= FDISK_DEBUG_INIT;
+}
+
/**
* fdisk_new_context:
*
@@ -37,9 +66,11 @@ struct fdisk_context *fdisk_new_context_from_filename(const char *fname)
* Attempt to open the device with r-w permissions
* by default, otherwise try read-only.
*/
- if ((fd = open(fname, O_RDWR)) < 0)
+ if ((fd = open(fname, O_RDWR)) < 0) {
if ((fd = open(fname, O_RDONLY)) < 0)
goto ret;
+ DBG(CONTEXT, printf("opened %s as read-only\n", fname));
+ }
cxt = calloc(1, sizeof(*cxt));
if (!cxt)
--
1.7.4.1
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH 6/6] fdisk: add debug support
2012-05-20 16:11 [PATCH 6/6] fdisk: add debug support Davidlohr Bueso
@ 2012-05-21 20:32 ` Davidlohr Bueso
2012-05-23 9:46 ` Karel Zak
0 siblings, 1 reply; 3+ messages in thread
From: Davidlohr Bueso @ 2012-05-21 20:32 UTC (permalink / raw)
To: Karel Zak; +Cc: Petr Uzel, util-linux
On Sun, 2012-05-20 at 18:11 +0200, Davidlohr Bueso wrote:
> From: Davidlohr Bueso <dave@gnu.org>
Rebase as this patch no longer applies with the changes from 5/6.
Thanks.
From: Davidlohr Bueso <dave@gnu.org>
Date: Mon, 21 May 2012 22:30:14 +0200
Subject: [PATCH] fdisk: add debug support
Based on libmnt, this patch adds basic debugging support for fdisk. Currently only CONTEXT is debugged, yet keeps exact functionality
as libmnt/libblkid.
Signed-off-by: Davidlohr Bueso <dave@gnu.org>
---
fdisk/fdisk.c | 2 ++
fdisk/fdisk.h | 27 +++++++++++++++++++++++++++
fdisk/utils.c | 33 ++++++++++++++++++++++++++++++++-
3 files changed, 61 insertions(+), 1 deletions(-)
diff --git a/fdisk/fdisk.c b/fdisk/fdisk.c
index 71c009c..1da4ffe 100644
--- a/fdisk/fdisk.c
+++ b/fdisk/fdisk.c
@@ -2168,6 +2168,8 @@ int main(int argc, char **argv)
}
}
+ fdisk_init_debug(0);
+
if (user_set_sector_size && argc-optind != 1)
printf(_("Warning: the -b (set sector size) option should"
" be used with one specified device\n"));
diff --git a/fdisk/fdisk.h b/fdisk/fdisk.h
index c43517c..2471541 100644
--- a/fdisk/fdisk.h
+++ b/fdisk/fdisk.h
@@ -32,6 +32,33 @@
#define cround(n) (display_in_cyl_units ? ((n)/units_per_sector)+1 : (n))
#define scround(x) (((x)+units_per_sector-1)/units_per_sector)
+/* fdisk debugging flags/options */
+#define FDISK_DEBUG_INIT (1 << 1)
+#define FDISK_DEBUG_CONTEXT (1 << 2)
+#define FDISK_DEBUG_ALL 0xFFFF
+
+# define ON_DBG(m, x) do { \
+ if ((FDISK_DEBUG_ ## m) & fdisk_debug_mask) { \
+ x; \
+ } \
+ } while (0)
+
+# define DBG(m, x) do { \
+ if ((FDISK_DEBUG_ ## m) & fdisk_debug_mask) { \
+ fprintf(stderr, "%d: fdisk: %8s: ", getpid(), # m); \
+ x; \
+ } \
+ } while (0)
+
+# define DBG_FLUSH do { \
+ if (fdisk_debug_mask && \
+ fdisk_debug_mask != FDISK_DEBUG_INIT) \
+ fflush(stderr); \
+ } while(0)
+
+extern int fdisk_debug_mask;
+extern void fdisk_init_debug(int mask);
+
struct partition {
unsigned char boot_ind; /* 0x80 - active */
unsigned char head; /* starting head */
diff --git a/fdisk/utils.c b/fdisk/utils.c
index 5138c04..f97aa4f 100644
--- a/fdisk/utils.c
+++ b/fdisk/utils.c
@@ -24,6 +24,35 @@
#include "common.h"
#include "fdisk.h"
+int fdisk_debug_mask;
+
+/**
+ * fdisk_init_debug:
+ * @mask: debug mask (0xffff to enable full debuging)
+ *
+ * If the @mask is not specified then this function reads
+ * FDISK_DEBUG environment variable to get the mask.
+ *
+ * Already initialized debugging stuff cannot be changed. It does not
+ * have effect to call this function twice.
+ */
+void fdisk_init_debug(int mask)
+{
+ if (fdisk_debug_mask & FDISK_DEBUG_INIT)
+ return;
+ if (!mask) {
+ char *str = getenv("FDISK_DEBUG");
+ if (str)
+ fdisk_debug_mask = strtoul(str, 0, 0);
+ } else
+ fdisk_debug_mask = mask;
+
+ if (fdisk_debug_mask)
+ printf("fdisk: debug mask set to 0x%04x.\n",
+ fdisk_debug_mask);
+ fdisk_debug_mask |= FDISK_DEBUG_INIT;
+}
+
/**
* fdisk_new_context:
*
@@ -38,9 +67,11 @@ struct fdisk_context *fdisk_new_context_from_filename(const char *fname)
* Attempt to open the device with r-w permissions
* by default, otherwise try read-only.
*/
- if ((fd = open(fname, O_RDWR)) < 0)
+ if ((fd = open(fname, O_RDWR)) < 0) {
if ((fd = open(fname, O_RDONLY)) < 0)
return NULL;
+ DBG(CONTEXT, printf("opened %s as read-only\n", fname));
+ }
cxt = calloc(1, sizeof(*cxt));
if (!cxt)
--
1.7.4.1
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH 6/6] fdisk: add debug support
2012-05-21 20:32 ` Davidlohr Bueso
@ 2012-05-23 9:46 ` Karel Zak
0 siblings, 0 replies; 3+ messages in thread
From: Karel Zak @ 2012-05-23 9:46 UTC (permalink / raw)
To: Davidlohr Bueso; +Cc: Petr Uzel, util-linux
On Mon, May 21, 2012 at 10:32:25PM +0200, Davidlohr Bueso wrote:
> fdisk/fdisk.c | 2 ++
> fdisk/fdisk.h | 27 +++++++++++++++++++++++++++
> fdisk/utils.c | 33 ++++++++++++++++++++++++++++++++-
> 3 files changed, 61 insertions(+), 1 deletions(-)
Applied, thanks.
> +void fdisk_init_debug(int mask)
> +{
> + if (fdisk_debug_mask & FDISK_DEBUG_INIT)
> + return;
> + if (!mask) {
> + char *str = getenv("FDISK_DEBUG");
> + if (str)
> + fdisk_debug_mask = strtoul(str, 0, 0);
> + } else
> + fdisk_debug_mask = mask;
> +
> + if (fdisk_debug_mask)
> + printf("fdisk: debug mask set to 0x%04x.\n",
> + fdisk_debug_mask);
this is bug (in libmount too), the message should be printed to
stderr as all others debug messages.
I have added dbgprint() inline function to standardize the output, so
> + DBG(CONTEXT, printf("opened %s as read-only\n", fname));
should be
DBG(CONTEXT, dbgprint("opened %s as read-only", fname));
All the changes are already pushed to the repository.
Karel
--
Karel Zak <kzak@redhat.com>
http://karelzak.blogspot.com
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2012-05-23 9:46 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-05-20 16:11 [PATCH 6/6] fdisk: add debug support Davidlohr Bueso
2012-05-21 20:32 ` Davidlohr Bueso
2012-05-23 9:46 ` Karel Zak
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).