From mboxrd@z Thu Jan 1 00:00:00 1970 From: Srivatsa Vaddagiri Subject: [PATCH 1/3] debugfs: Add support to print u32 array Date: Wed, 19 Jan 2011 22:37:11 +0530 Message-ID: <20110119170710.GA726__13225.6942023107$1295457947$gmane$org@linux.vnet.ibm.com> References: <20110119164432.GA30669@linux.vnet.ibm.com> Reply-To: vatsa@linux.vnet.ibm.com Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Content-Disposition: inline In-Reply-To: <20110119164432.GA30669@linux.vnet.ibm.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: virtualization-bounces@lists.linux-foundation.org Errors-To: virtualization-bounces@lists.linux-foundation.org To: Jeremy Fitzhardinge Cc: Xen-devel , Mathieu Desnoyers , Nick Piggin , kvm@vger.kernel.org, Peter Zijlstra , Linux Kernel Mailing List , Jan Beulich , Eric Dumazet , Jeremy Fitzhardinge , suzuki@in.ibm.com, Avi Kivity , "H. Peter Anvin" , =?iso-8859-1?Q?Am=E9rico?= Wang , Linux Virtualization List-Id: virtualization@lists.linuxfoundation.org Add debugfs support to print u32-arrays. Most of this comes from Xen-hypervisor sources, which has been refactored to make the code common for other users as well. Signed-off-by: Srivatsa Vaddagiri Signed-off-by: Suzuki Poulose --- arch/x86/xen/debugfs.c | 104 -------------------------------------------- arch/x86/xen/debugfs.h | 4 - arch/x86/xen/mmu.c | 2 arch/x86/xen/multicalls.c | 6 +- arch/x86/xen/spinlock.c | 2 fs/debugfs/file.c | 108 ++++++++++++++++++++++++++++++++++++++++++++++ include/linux/debugfs.h | 11 ++++ 7 files changed, 124 insertions(+), 113 deletions(-) Index: linux-2.6.37/arch/x86/xen/debugfs.c =================================================================== --- linux-2.6.37.orig/arch/x86/xen/debugfs.c +++ linux-2.6.37/arch/x86/xen/debugfs.c @@ -19,107 +19,3 @@ struct dentry * __init xen_init_debugfs( return d_xen_debug; } -struct array_data -{ - void *array; - unsigned elements; -}; - -static int u32_array_open(struct inode *inode, struct file *file) -{ - file->private_data = NULL; - return nonseekable_open(inode, file); -} - -static size_t format_array(char *buf, size_t bufsize, const char *fmt, - u32 *array, unsigned array_size) -{ - size_t ret = 0; - unsigned i; - - for(i = 0; i < array_size; i++) { - size_t len; - - len = snprintf(buf, bufsize, fmt, array[i]); - len++; /* ' ' or '\n' */ - ret += len; - - if (buf) { - buf += len; - bufsize -= len; - buf[-1] = (i == array_size-1) ? '\n' : ' '; - } - } - - ret++; /* \0 */ - if (buf) - *buf = '\0'; - - return ret; -} - -static char *format_array_alloc(const char *fmt, u32 *array, unsigned array_size) -{ - size_t len = format_array(NULL, 0, fmt, array, array_size); - char *ret; - - ret = kmalloc(len, GFP_KERNEL); - if (ret == NULL) - return NULL; - - format_array(ret, len, fmt, array, array_size); - return ret; -} - -static ssize_t u32_array_read(struct file *file, char __user *buf, size_t len, - loff_t *ppos) -{ - struct inode *inode = file->f_path.dentry->d_inode; - struct array_data *data = inode->i_private; - size_t size; - - if (*ppos == 0) { - if (file->private_data) { - kfree(file->private_data); - file->private_data = NULL; - } - - file->private_data = format_array_alloc("%u", data->array, data->elements); - } - - size = 0; - if (file->private_data) - size = strlen(file->private_data); - - return simple_read_from_buffer(buf, len, ppos, file->private_data, size); -} - -static int xen_array_release(struct inode *inode, struct file *file) -{ - kfree(file->private_data); - - return 0; -} - -static const struct file_operations u32_array_fops = { - .owner = THIS_MODULE, - .open = u32_array_open, - .release= xen_array_release, - .read = u32_array_read, - .llseek = no_llseek, -}; - -struct dentry *xen_debugfs_create_u32_array(const char *name, mode_t mode, - struct dentry *parent, - u32 *array, unsigned elements) -{ - struct array_data *data = kmalloc(sizeof(*data), GFP_KERNEL); - - if (data == NULL) - return NULL; - - data->array = array; - data->elements = elements; - - return debugfs_create_file(name, mode, parent, data, &u32_array_fops); -} Index: linux-2.6.37/arch/x86/xen/debugfs.h =================================================================== --- linux-2.6.37.orig/arch/x86/xen/debugfs.h +++ linux-2.6.37/arch/x86/xen/debugfs.h @@ -3,8 +3,4 @@ struct dentry * __init xen_init_debugfs(void); -struct dentry *xen_debugfs_create_u32_array(const char *name, mode_t mode, - struct dentry *parent, - u32 *array, unsigned elements); - #endif /* _XEN_DEBUGFS_H */ Index: linux-2.6.37/arch/x86/xen/mmu.c =================================================================== --- linux-2.6.37.orig/arch/x86/xen/mmu.c +++ linux-2.6.37/arch/x86/xen/mmu.c @@ -2757,7 +2757,7 @@ static int __init xen_mmu_debugfs(void) debugfs_create_u32("mmu_update", 0444, d_mmu_debug, &mmu_stats.mmu_update); debugfs_create_u32("mmu_update_extended", 0444, d_mmu_debug, &mmu_stats.mmu_update_extended); - xen_debugfs_create_u32_array("mmu_update_histo", 0444, d_mmu_debug, + debugfs_create_u32_array("mmu_update_histo", 0444, d_mmu_debug, mmu_stats.mmu_update_histo, 20); debugfs_create_u32("set_pte_at", 0444, d_mmu_debug, &mmu_stats.set_pte_at); Index: linux-2.6.37/arch/x86/xen/multicalls.c =================================================================== --- linux-2.6.37.orig/arch/x86/xen/multicalls.c +++ linux-2.6.37/arch/x86/xen/multicalls.c @@ -269,11 +269,11 @@ static int __init xen_mc_debugfs(void) debugfs_create_u32("hypercalls", 0444, d_mc_debug, &mc_stats.hypercalls); debugfs_create_u32("arg_total", 0444, d_mc_debug, &mc_stats.arg_total); - xen_debugfs_create_u32_array("batch_histo", 0444, d_mc_debug, + debugfs_create_u32_array("batch_histo", 0444, d_mc_debug, mc_stats.histo, MC_BATCH); - xen_debugfs_create_u32_array("hypercall_histo", 0444, d_mc_debug, + debugfs_create_u32_array("hypercall_histo", 0444, d_mc_debug, mc_stats.histo_hypercalls, NHYPERCALLS); - xen_debugfs_create_u32_array("flush_reasons", 0444, d_mc_debug, + debugfs_create_u32_array("flush_reasons", 0444, d_mc_debug, mc_stats.flush, FL_N_REASONS); return 0; Index: linux-2.6.37/arch/x86/xen/spinlock.c =================================================================== --- linux-2.6.37.orig/arch/x86/xen/spinlock.c +++ linux-2.6.37/arch/x86/xen/spinlock.c @@ -228,7 +228,7 @@ static int __init xen_spinlock_debugfs(v debugfs_create_u64("time_blocked", 0444, d_spin_debug, &spinlock_stats.time_blocked); - xen_debugfs_create_u32_array("histo_blocked", 0444, d_spin_debug, + debugfs_create_u32_array("histo_blocked", 0444, d_spin_debug, spinlock_stats.histo_spin_blocked, HISTO_BUCKETS + 1); return 0; Index: linux-2.6.37/fs/debugfs/file.c =================================================================== --- linux-2.6.37.orig/fs/debugfs/file.c +++ linux-2.6.37/fs/debugfs/file.c @@ -18,6 +18,7 @@ #include #include #include +#include static ssize_t default_read_file(struct file *file, char __user *buf, size_t count, loff_t *ppos) @@ -534,3 +535,110 @@ struct dentry *debugfs_create_blob(const return debugfs_create_file(name, mode, parent, blob, &fops_blob); } EXPORT_SYMBOL_GPL(debugfs_create_blob); + +struct array_data { + void *array; + unsigned elements; +}; + +static int u32_array_open(struct inode *inode, struct file *file) +{ + file->private_data = NULL; + return nonseekable_open(inode, file); +} + +static size_t format_array(char *buf, size_t bufsize, const char *fmt, + u32 *array, unsigned array_size) +{ + size_t ret = 0; + unsigned i; + + for (i = 0; i < array_size; i++) { + size_t len; + + len = snprintf(buf, bufsize, fmt, array[i]); + len++; /* ' ' or '\n' */ + ret += len; + + if (buf) { + buf += len; + bufsize -= len; + buf[-1] = (i == array_size-1) ? '\n' : ' '; + } + } + + ret++; /* \0 */ + if (buf) + *buf = '\0'; + + return ret; +} + +static char * +format_array_alloc(const char *fmt, u32 *array, unsigned array_size) +{ + size_t len = format_array(NULL, 0, fmt, array, array_size); + char *ret; + + ret = kmalloc(len, GFP_KERNEL); + if (ret == NULL) + return NULL; + + format_array(ret, len, fmt, array, array_size); + return ret; +} + +static ssize_t u32_array_read(struct file *file, char __user *buf, size_t len, + loff_t *ppos) +{ + struct inode *inode = file->f_path.dentry->d_inode; + struct array_data *data = inode->i_private; + size_t size; + + if (*ppos == 0) { + if (file->private_data) { + kfree(file->private_data); + file->private_data = NULL; + } + + file->private_data = + format_array_alloc("%u", data->array, data->elements); + } + + size = 0; + if (file->private_data) + size = strlen(file->private_data); + + return simple_read_from_buffer(buf, len, ppos, + file->private_data, size); +} + +static int u32_array_release(struct inode *inode, struct file *file) +{ + kfree(file->private_data); + + return 0; +} + +static const struct file_operations u32_array_fops = { + .owner = THIS_MODULE, + .open = u32_array_open, + .release = u32_array_release, + .read = u32_array_read, + .llseek = no_llseek, +}; + +struct dentry *debugfs_create_u32_array(const char *name, mode_t mode, + struct dentry *parent, + u32 *array, unsigned elements) +{ + struct array_data *data = kmalloc(sizeof(*data), GFP_KERNEL); + + if (data == NULL) + return NULL; + + data->array = array; + data->elements = elements; + + return debugfs_create_file(name, mode, parent, data, &u32_array_fops); +} Index: linux-2.6.37/include/linux/debugfs.h =================================================================== --- linux-2.6.37.orig/include/linux/debugfs.h +++ linux-2.6.37/include/linux/debugfs.h @@ -74,6 +74,10 @@ struct dentry *debugfs_create_blob(const struct dentry *parent, struct debugfs_blob_wrapper *blob); +struct dentry *debugfs_create_u32_array(const char *name, mode_t mode, + struct dentry *parent, + u32 *array, unsigned elements); + bool debugfs_initialized(void); #else @@ -187,6 +191,13 @@ static inline struct dentry *debugfs_cre { return ERR_PTR(-ENODEV); } + +struct dentry *debugfs_create_u32_array(const char *name, mode_t mode, + struct dentry *parent, + u32 *array, unsigned elements) +{ + return ERR_PTR(-ENODEV); +} static inline bool debugfs_initialized(void) {