From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753382AbcBJTJc (ORCPT ); Wed, 10 Feb 2016 14:09:32 -0500 Received: from mail-wm0-f67.google.com ([74.125.82.67]:32917 "EHLO mail-wm0-f67.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751949AbcBJTJT (ORCPT ); Wed, 10 Feb 2016 14:09:19 -0500 From: Christian Brauner To: tj@kernel.org, gregkh@linuxfoundation.org, linux-kernel@vger.kernel.org Cc: Christian Brauner Subject: [PATCH v2] let kernfs_name_locked & kernfs_name return size_t Date: Wed, 10 Feb 2016 20:07:06 +0100 Message-Id: <1455131226-6836-2-git-send-email-christian.brauner@mailbox.org> X-Mailer: git-send-email 2.7.1 In-Reply-To: <1455131226-6836-1-git-send-email-christian.brauner@mailbox.org> References: <1455131226-6836-1-git-send-email-christian.brauner@mailbox.org> Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org They return the length of kn->parent or kn->name and should be aligned with kernfs_path_len which also returns size_t. Signed-off-by: Christian Brauner --- Changelog: 20150210: - Change return type of kernfs_name in kernfs.h as well. - Build tested against Tejun's for-4.5 tree, 4.5-rc2 and mainline 4.5-rc3. --- fs/kernfs/dir.c | 4 ++-- include/linux/kernfs.h | 6 +++--- 2 files changed, 5 insertions(+), 5 deletions(-) diff --git a/fs/kernfs/dir.c b/fs/kernfs/dir.c index 742bf4a..a3553b8 100644 --- a/fs/kernfs/dir.c +++ b/fs/kernfs/dir.c @@ -39,7 +39,7 @@ static bool kernfs_lockdep(struct kernfs_node *kn) #endif } -static int kernfs_name_locked(struct kernfs_node *kn, char *buf, size_t buflen) +static size_t kernfs_name_locked(struct kernfs_node *kn, char *buf, size_t buflen) { return strlcpy(buf, kn->parent ? kn->name : "/", buflen); } @@ -80,7 +80,7 @@ static char * __must_check kernfs_path_locked(struct kernfs_node *kn, char *buf, * * This function can be called from any context. */ -int kernfs_name(struct kernfs_node *kn, char *buf, size_t buflen) +size_t kernfs_name(struct kernfs_node *kn, char *buf, size_t buflen) { unsigned long flags; int ret; diff --git a/include/linux/kernfs.h b/include/linux/kernfs.h index af51df3..982a198 100644 --- a/include/linux/kernfs.h +++ b/include/linux/kernfs.h @@ -265,7 +265,7 @@ static inline bool kernfs_ns_enabled(struct kernfs_node *kn) return kn->flags & KERNFS_NS; } -int kernfs_name(struct kernfs_node *kn, char *buf, size_t buflen); +size_t kernfs_name(struct kernfs_node *kn, char *buf, size_t buflen); size_t kernfs_path_len(struct kernfs_node *kn); char * __must_check kernfs_path(struct kernfs_node *kn, char *buf, size_t buflen); @@ -332,8 +332,8 @@ static inline void kernfs_enable_ns(struct kernfs_node *kn) { } static inline bool kernfs_ns_enabled(struct kernfs_node *kn) { return false; } -static inline int kernfs_name(struct kernfs_node *kn, char *buf, size_t buflen) -{ return -ENOSYS; } +static inline size_t kernfs_name(struct kernfs_node *kn, char *buf, size_t buflen) +{ return 0; } static inline size_t kernfs_path_len(struct kernfs_node *kn) { return 0; } -- 2.7.1