Use QSTR() and drop strlen() in efivarfs_alloc_dentry().

Signed-off-by: Thorsten Blum <thorsten.blum@linux.dev>
---
 fs/efivarfs/super.c | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)

diff --git a/fs/efivarfs/super.c b/fs/efivarfs/super.c
index 1c5224cf183e..95a685f022e6 100644
--- a/fs/efivarfs/super.c
+++ b/fs/efivarfs/super.c
@@ -192,12 +192,9 @@ static const struct dentry_operations efivarfs_d_ops = {
 static struct dentry *efivarfs_alloc_dentry(struct dentry *parent, char *name)
 {
 	struct dentry *d;
-	struct qstr q;
+	struct qstr q = QSTR(name);
 	int err;
 
-	q.name = name;
-	q.len = strlen(name);
-
 	err = efivarfs_d_hash(parent, &q);
 	if (err)
 		return ERR_PTR(err);

On Wed, 2026-04-22 at 14:25 +0200, Thorsten Blum wrote:
> Use QSTR() and drop strlen() in efivarfs_alloc_dentry().

QSTR contains strlen(), so it's not really accurate to say it's been
dropped; this is merely changing to use a #define that does exactly the
same as the original code.

Regards,

James

Hi Thorsten,

On Wed, 22 Apr 2026, at 14:25, Thorsten Blum wrote:
> Use QSTR() and drop strlen() in efivarfs_alloc_dentry().
>
> Signed-off-by: Thorsten Blum <thorsten.blum@linux.dev>
> ---
>  fs/efivarfs/super.c | 5 +----
>  1 file changed, 1 insertion(+), 4 deletions(-)
>

I'll queue this up once -rc1 comes around. Thanks.

> diff --git a/fs/efivarfs/super.c b/fs/efivarfs/super.c
> index 1c5224cf183e..95a685f022e6 100644
> --- a/fs/efivarfs/super.c
> +++ b/fs/efivarfs/super.c
> @@ -192,12 +192,9 @@ static const struct dentry_operations efivarfs_d_ops = {
>  static struct dentry *efivarfs_alloc_dentry(struct dentry *parent, char *name)
>  {
>  	struct dentry *d;
> -	struct qstr q;
> +	struct qstr q = QSTR(name);
>  	int err;
> 
> -	q.name = name;
> -	q.len = strlen(name);
> -
>  	err = efivarfs_d_hash(parent, &q);
>  	if (err)
>  		return ERR_PTR(err);