Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752895Ab2JOLhZ (ORCPT ); Mon, 15 Oct 2012 07:37:25 -0400 Received: from mga02.intel.com ([134.134.136.20]:48571 "EHLO mga02.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752664Ab2JOLhT (ORCPT ); Mon, 15 Oct 2012 07:37:19 -0400 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="4.80,587,1344236400"; d="scan'208";a="227002382" From: dragos.tatulea@intel.com To: cbouatmailru@gmail.com, linux-kernel@vger.kernel.org Cc: adrian.hunter@intel.com, octavian.purdila@intel.com Subject: [PATCH 6/8] pstore: do not run timer while pstore is not mounted Date: Mon, 15 Oct 2012 14:40:08 +0300 Message-Id: <1350301210-16953-7-git-send-email-dragos.tatulea@intel.com> X-Mailer: git-send-email 1.7.9.5 In-Reply-To: <1350301210-16953-1-git-send-email-dragos.tatulea@intel.com> References: <1350301210-16953-1-git-send-email-dragos.tatulea@intel.com> Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 3390 Lines: 119 From: Adrian Hunter A timer is used to periodically check for new dumps. The timer is not needed unless pstore is mounted, so disable it otherwise. Signed-off-by: Adrian Hunter --- fs/pstore/inode.c | 3 +++ fs/pstore/internal.h | 2 ++ fs/pstore/platform.c | 36 +++++++++++++++++++++++++++++------- 3 files changed, 34 insertions(+), 7 deletions(-) diff --git a/fs/pstore/inode.c b/fs/pstore/inode.c index 4800c09..5e20a8d 100644 --- a/fs/pstore/inode.c +++ b/fs/pstore/inode.c @@ -397,6 +397,8 @@ static int pstore_fill_super(struct super_block *sb, void *data, int silent) pstore_get_records(0); + pstore_add_timer(); + return 0; } @@ -408,6 +410,7 @@ static struct dentry *pstore_mount(struct file_system_type *fs_type, static void pstore_kill_sb(struct super_block *sb) { + pstore_del_timer(); kill_litter_super(sb); pstore_sb = NULL; } diff --git a/fs/pstore/internal.h b/fs/pstore/internal.h index 4847f58..c7b8197 100644 --- a/fs/pstore/internal.h +++ b/fs/pstore/internal.h @@ -54,4 +54,6 @@ extern int pstore_mkfile(enum pstore_type_id, char *psname, u64 id, struct timespec time, struct pstore_info *psi); extern int pstore_is_mounted(void); +extern void pstore_add_timer(void); +extern void pstore_del_timer(void); #endif diff --git a/fs/pstore/platform.c b/fs/pstore/platform.c index 63ff377..3a4c38f 100644 --- a/fs/pstore/platform.c +++ b/fs/pstore/platform.c @@ -54,6 +54,8 @@ static int pstore_new_entry; static void pstore_timefunc(unsigned long); static DEFINE_TIMER(pstore_timer, pstore_timefunc, 0, 0); +static DEFINE_SPINLOCK(pstore_timer_lock); +static int pstore_timer_on; static void pstore_dowork(struct work_struct *); static DECLARE_WORK(pstore_work, pstore_dowork); @@ -351,12 +353,6 @@ int pstore_register(struct pstore_info *psi) pstore_register_console(); pstore_register_ftrace(); - if (pstore_update_ms >= 0) { - pstore_timer.expires = jiffies + - msecs_to_jiffies(pstore_update_ms); - add_timer(&pstore_timer); - } - pstore_debugfs_init(); return 0; @@ -411,12 +407,38 @@ static void pstore_dowork(struct work_struct *work) static void pstore_timefunc(unsigned long dummy) { + unsigned long flags; + if (pstore_new_entry) { pstore_new_entry = 0; schedule_work(&pstore_work); } - mod_timer(&pstore_timer, jiffies + msecs_to_jiffies(pstore_update_ms)); + spin_lock_irqsave(&pstore_timer_lock, flags); + if (pstore_timer_on) + mod_timer(&pstore_timer, + jiffies + msecs_to_jiffies(pstore_update_ms)); + spin_unlock_irqrestore(&pstore_timer_lock, flags); +} + +void pstore_add_timer(void) +{ + pstore_timer_on = 1; + if (pstore_update_ms >= 0) { + pstore_timer.expires = jiffies + + msecs_to_jiffies(pstore_update_ms); + add_timer(&pstore_timer); + } +} + +void pstore_del_timer(void) +{ + unsigned long flags; + + spin_lock_irqsave(&pstore_timer_lock, flags); + pstore_timer_on = 0; + spin_unlock_irqrestore(&pstore_timer_lock, flags); + del_timer_sync(&pstore_timer); } /* pstore_write must only be called from PSTORE_DUMP notifier callbacks */ -- 1.7.9.5 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/