[Ocfs2-devel] [PATCH] ocfs2/trivial: Add default mask log for heartbeat.c

Sunil Mushran sunil.mushran at oracle.com
Mon Dec 21 14:37:15 PST 2009


Maybe better if we changed the two mlog(0,... to mlog(ML_HEARTBEAT,...

Now I don't know why we don't have a default mask in that file but instead
have distinct mlogs for HEARTBEAT and HB_BIO. One reason could have been to
limit the logging of hb_bio. If that is correct, then adding a default mask
will defeat the purpose.

Tao Ma wrote:
> I just noticed today that we have no default mask
> prefix for fs/ocfs2/cluster/heartbeat.c, but we have
> 2 places that use "mlog(0,...)". So add the default
> prefix so that these "mlog(0, ...)" can work.
>
> Signed-off-by: Tao Ma <tao.ma at oracle.com>
> ---
>  fs/ocfs2/cluster/heartbeat.c |    1 +
>  1 files changed, 1 insertions(+), 0 deletions(-)
>
> diff --git a/fs/ocfs2/cluster/heartbeat.c b/fs/ocfs2/cluster/heartbeat.c
> index c452d11..edec9bf 100644
> --- a/fs/ocfs2/cluster/heartbeat.c
> +++ b/fs/ocfs2/cluster/heartbeat.c
> @@ -40,6 +40,7 @@
>  #include "nodemanager.h"
>  #include "quorum.h"
>  
> +#define MLOG_MASK_PREFIX ML_HEARTBEAT
>  #include "masklog.h"
>  
>  
>   




More information about the Ocfs2-devel mailing list