There is no bug here, but some code starting on line 461 seems to be
incorrectly indented.
julia
---------- Forwarded message ----------
Date: Sat, 29 Jun 2019 19:51:04 +0800
From: kbuild test robot <[email protected]>
To: [email protected]
Cc: Julia Lawall <[email protected]>
Subject: fs/ceph/export.c:459:3-12: code aligned with following code on line 461
CC: [email protected]
CC: [email protected]
TO: "Yan, Zheng" <[email protected]>
CC: Ilya Dryomov <[email protected]>
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head: 249155c20f9b0754bc1b932a33344cfb4e0c2101
commit: 570df4e9c23f861aa3f8f2954468c534a033bf1a ceph: snapshot nfs re-export
date: 8 weeks ago
:::::: branch date: 5 days ago
:::::: commit date: 8 weeks ago
If you fix the issue, kindly add following tag
Reported-by: kbuild test robot <[email protected]>
Reported-by: Julia Lawall <[email protected]>
>> fs/ceph/export.c:459:3-12: code aligned with following code on line 461
# https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=570df4e9c23f861aa3f8f2954468c534a033bf1a
git remote add linus https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
git remote update linus
git checkout 570df4e9c23f861aa3f8f2954468c534a033bf1a
vim +459 fs/ceph/export.c
a8e63b7d Sage Weil 2009-10-06 401
570df4e9 Yan, Zheng 2017-11-15 402 static int __get_snap_name(struct dentry *parent, char *name,
570df4e9 Yan, Zheng 2017-11-15 403 struct dentry *child)
570df4e9 Yan, Zheng 2017-11-15 404 {
570df4e9 Yan, Zheng 2017-11-15 405 struct inode *inode = d_inode(child);
570df4e9 Yan, Zheng 2017-11-15 406 struct inode *dir = d_inode(parent);
570df4e9 Yan, Zheng 2017-11-15 407 struct ceph_fs_client *fsc = ceph_inode_to_client(inode);
570df4e9 Yan, Zheng 2017-11-15 408 struct ceph_mds_request *req = NULL;
570df4e9 Yan, Zheng 2017-11-15 409 char *last_name = NULL;
570df4e9 Yan, Zheng 2017-11-15 410 unsigned next_offset = 2;
570df4e9 Yan, Zheng 2017-11-15 411 int err = -EINVAL;
570df4e9 Yan, Zheng 2017-11-15 412
570df4e9 Yan, Zheng 2017-11-15 413 if (ceph_ino(inode) != ceph_ino(dir))
570df4e9 Yan, Zheng 2017-11-15 414 goto out;
570df4e9 Yan, Zheng 2017-11-15 415 if (ceph_snap(inode) == CEPH_SNAPDIR) {
570df4e9 Yan, Zheng 2017-11-15 416 if (ceph_snap(dir) == CEPH_NOSNAP) {
570df4e9 Yan, Zheng 2017-11-15 417 strcpy(name, fsc->mount_options->snapdir_name);
570df4e9 Yan, Zheng 2017-11-15 418 err = 0;
570df4e9 Yan, Zheng 2017-11-15 419 }
570df4e9 Yan, Zheng 2017-11-15 420 goto out;
570df4e9 Yan, Zheng 2017-11-15 421 }
570df4e9 Yan, Zheng 2017-11-15 422 if (ceph_snap(dir) != CEPH_SNAPDIR)
570df4e9 Yan, Zheng 2017-11-15 423 goto out;
570df4e9 Yan, Zheng 2017-11-15 424
570df4e9 Yan, Zheng 2017-11-15 425 while (1) {
570df4e9 Yan, Zheng 2017-11-15 426 struct ceph_mds_reply_info_parsed *rinfo;
570df4e9 Yan, Zheng 2017-11-15 427 struct ceph_mds_reply_dir_entry *rde;
570df4e9 Yan, Zheng 2017-11-15 428 int i;
570df4e9 Yan, Zheng 2017-11-15 429
570df4e9 Yan, Zheng 2017-11-15 430 req = ceph_mdsc_create_request(fsc->mdsc, CEPH_MDS_OP_LSSNAP,
570df4e9 Yan, Zheng 2017-11-15 431 USE_AUTH_MDS);
570df4e9 Yan, Zheng 2017-11-15 432 if (IS_ERR(req)) {
570df4e9 Yan, Zheng 2017-11-15 433 err = PTR_ERR(req);
570df4e9 Yan, Zheng 2017-11-15 434 req = NULL;
570df4e9 Yan, Zheng 2017-11-15 435 goto out;
570df4e9 Yan, Zheng 2017-11-15 436 }
570df4e9 Yan, Zheng 2017-11-15 437 err = ceph_alloc_readdir_reply_buffer(req, inode);
570df4e9 Yan, Zheng 2017-11-15 438 if (err)
570df4e9 Yan, Zheng 2017-11-15 439 goto out;
570df4e9 Yan, Zheng 2017-11-15 440
570df4e9 Yan, Zheng 2017-11-15 441 req->r_direct_mode = USE_AUTH_MDS;
570df4e9 Yan, Zheng 2017-11-15 442 req->r_readdir_offset = next_offset;
570df4e9 Yan, Zheng 2017-11-15 443 req->r_args.readdir.flags =
570df4e9 Yan, Zheng 2017-11-15 444 cpu_to_le16(CEPH_READDIR_REPLY_BITFLAGS);
570df4e9 Yan, Zheng 2017-11-15 445 if (last_name) {
570df4e9 Yan, Zheng 2017-11-15 446 req->r_path2 = last_name;
570df4e9 Yan, Zheng 2017-11-15 447 last_name = NULL;
570df4e9 Yan, Zheng 2017-11-15 448 }
570df4e9 Yan, Zheng 2017-11-15 449
570df4e9 Yan, Zheng 2017-11-15 450 req->r_inode = dir;
570df4e9 Yan, Zheng 2017-11-15 451 ihold(dir);
570df4e9 Yan, Zheng 2017-11-15 452 req->r_dentry = dget(parent);
570df4e9 Yan, Zheng 2017-11-15 453
570df4e9 Yan, Zheng 2017-11-15 454 inode_lock(dir);
570df4e9 Yan, Zheng 2017-11-15 455 err = ceph_mdsc_do_request(fsc->mdsc, NULL, req);
570df4e9 Yan, Zheng 2017-11-15 456 inode_unlock(dir);
570df4e9 Yan, Zheng 2017-11-15 457
570df4e9 Yan, Zheng 2017-11-15 458 if (err < 0)
570df4e9 Yan, Zheng 2017-11-15 @459 goto out;
570df4e9 Yan, Zheng 2017-11-15 460
570df4e9 Yan, Zheng 2017-11-15 @461 rinfo = &req->r_reply_info;
570df4e9 Yan, Zheng 2017-11-15 462 for (i = 0; i < rinfo->dir_nr; i++) {
570df4e9 Yan, Zheng 2017-11-15 463 rde = rinfo->dir_entries + i;
570df4e9 Yan, Zheng 2017-11-15 464 BUG_ON(!rde->inode.in);
570df4e9 Yan, Zheng 2017-11-15 465 if (ceph_snap(inode) ==
570df4e9 Yan, Zheng 2017-11-15 466 le64_to_cpu(rde->inode.in->snapid)) {
570df4e9 Yan, Zheng 2017-11-15 467 memcpy(name, rde->name, rde->name_len);
570df4e9 Yan, Zheng 2017-11-15 468 name[rde->name_len] = '\0';
570df4e9 Yan, Zheng 2017-11-15 469 err = 0;
570df4e9 Yan, Zheng 2017-11-15 470 goto out;
570df4e9 Yan, Zheng 2017-11-15 471 }
570df4e9 Yan, Zheng 2017-11-15 472 }
570df4e9 Yan, Zheng 2017-11-15 473
570df4e9 Yan, Zheng 2017-11-15 474 if (rinfo->dir_end)
570df4e9 Yan, Zheng 2017-11-15 475 break;
570df4e9 Yan, Zheng 2017-11-15 476
570df4e9 Yan, Zheng 2017-11-15 477 BUG_ON(rinfo->dir_nr <= 0);
570df4e9 Yan, Zheng 2017-11-15 478 rde = rinfo->dir_entries + (rinfo->dir_nr - 1);
570df4e9 Yan, Zheng 2017-11-15 479 next_offset += rinfo->dir_nr;
570df4e9 Yan, Zheng 2017-11-15 480 last_name = kstrndup(rde->name, rde->name_len, GFP_KERNEL);
570df4e9 Yan, Zheng 2017-11-15 481 if (!last_name) {
570df4e9 Yan, Zheng 2017-11-15 482 err = -ENOMEM;
570df4e9 Yan, Zheng 2017-11-15 483 goto out;
570df4e9 Yan, Zheng 2017-11-15 484 }
570df4e9 Yan, Zheng 2017-11-15 485
570df4e9 Yan, Zheng 2017-11-15 486 ceph_mdsc_put_request(req);
570df4e9 Yan, Zheng 2017-11-15 487 req = NULL;
570df4e9 Yan, Zheng 2017-11-15 488 }
570df4e9 Yan, Zheng 2017-11-15 489 err = -ENOENT;
570df4e9 Yan, Zheng 2017-11-15 490 out:
570df4e9 Yan, Zheng 2017-11-15 491 if (req)
570df4e9 Yan, Zheng 2017-11-15 492 ceph_mdsc_put_request(req);
570df4e9 Yan, Zheng 2017-11-15 493 kfree(last_name);
570df4e9 Yan, Zheng 2017-11-15 494 dout("get_snap_name %p ino %llx.%llx err=%d\n",
570df4e9 Yan, Zheng 2017-11-15 495 child, ceph_vinop(inode), err);
570df4e9 Yan, Zheng 2017-11-15 496 return err;
570df4e9 Yan, Zheng 2017-11-15 497 }
570df4e9 Yan, Zheng 2017-11-15 498
---
0-DAY kernel test infrastructure Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all Intel Corporation
On Sat, Jun 29, 2019 at 4:09 PM Julia Lawall <[email protected]> wrote:
>
> There is no bug here, but some code starting on line 461 seems to be
> incorrectly indented.
>
> julia
>
> ---------- Forwarded message ----------
> Date: Sat, 29 Jun 2019 19:51:04 +0800
> From: kbuild test robot <[email protected]>
> To: [email protected]
> Cc: Julia Lawall <[email protected]>
> Subject: fs/ceph/export.c:459:3-12: code aligned with following code on line 461
>
> CC: [email protected]
> CC: [email protected]
> TO: "Yan, Zheng" <[email protected]>
> CC: Ilya Dryomov <[email protected]>
>
> tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
> head: 249155c20f9b0754bc1b932a33344cfb4e0c2101
> commit: 570df4e9c23f861aa3f8f2954468c534a033bf1a ceph: snapshot nfs re-export
> date: 8 weeks ago
> :::::: branch date: 5 days ago
> :::::: commit date: 8 weeks ago
>
> If you fix the issue, kindly add following tag
> Reported-by: kbuild test robot <[email protected]>
> Reported-by: Julia Lawall <[email protected]>
>
> >> fs/ceph/export.c:459:3-12: code aligned with following code on line 461
>
> # https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=570df4e9c23f861aa3f8f2954468c534a033bf1a
> git remote add linus https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
> git remote update linus
> git checkout 570df4e9c23f861aa3f8f2954468c534a033bf1a
> vim +459 fs/ceph/export.c
>
> a8e63b7d Sage Weil 2009-10-06 401
> 570df4e9 Yan, Zheng 2017-11-15 402 static int __get_snap_name(struct dentry *parent, char *name,
> 570df4e9 Yan, Zheng 2017-11-15 403 struct dentry *child)
> 570df4e9 Yan, Zheng 2017-11-15 404 {
> 570df4e9 Yan, Zheng 2017-11-15 405 struct inode *inode = d_inode(child);
> 570df4e9 Yan, Zheng 2017-11-15 406 struct inode *dir = d_inode(parent);
> 570df4e9 Yan, Zheng 2017-11-15 407 struct ceph_fs_client *fsc = ceph_inode_to_client(inode);
> 570df4e9 Yan, Zheng 2017-11-15 408 struct ceph_mds_request *req = NULL;
> 570df4e9 Yan, Zheng 2017-11-15 409 char *last_name = NULL;
> 570df4e9 Yan, Zheng 2017-11-15 410 unsigned next_offset = 2;
> 570df4e9 Yan, Zheng 2017-11-15 411 int err = -EINVAL;
> 570df4e9 Yan, Zheng 2017-11-15 412
> 570df4e9 Yan, Zheng 2017-11-15 413 if (ceph_ino(inode) != ceph_ino(dir))
> 570df4e9 Yan, Zheng 2017-11-15 414 goto out;
> 570df4e9 Yan, Zheng 2017-11-15 415 if (ceph_snap(inode) == CEPH_SNAPDIR) {
> 570df4e9 Yan, Zheng 2017-11-15 416 if (ceph_snap(dir) == CEPH_NOSNAP) {
> 570df4e9 Yan, Zheng 2017-11-15 417 strcpy(name, fsc->mount_options->snapdir_name);
> 570df4e9 Yan, Zheng 2017-11-15 418 err = 0;
> 570df4e9 Yan, Zheng 2017-11-15 419 }
> 570df4e9 Yan, Zheng 2017-11-15 420 goto out;
> 570df4e9 Yan, Zheng 2017-11-15 421 }
> 570df4e9 Yan, Zheng 2017-11-15 422 if (ceph_snap(dir) != CEPH_SNAPDIR)
> 570df4e9 Yan, Zheng 2017-11-15 423 goto out;
> 570df4e9 Yan, Zheng 2017-11-15 424
> 570df4e9 Yan, Zheng 2017-11-15 425 while (1) {
> 570df4e9 Yan, Zheng 2017-11-15 426 struct ceph_mds_reply_info_parsed *rinfo;
> 570df4e9 Yan, Zheng 2017-11-15 427 struct ceph_mds_reply_dir_entry *rde;
> 570df4e9 Yan, Zheng 2017-11-15 428 int i;
> 570df4e9 Yan, Zheng 2017-11-15 429
> 570df4e9 Yan, Zheng 2017-11-15 430 req = ceph_mdsc_create_request(fsc->mdsc, CEPH_MDS_OP_LSSNAP,
> 570df4e9 Yan, Zheng 2017-11-15 431 USE_AUTH_MDS);
> 570df4e9 Yan, Zheng 2017-11-15 432 if (IS_ERR(req)) {
> 570df4e9 Yan, Zheng 2017-11-15 433 err = PTR_ERR(req);
> 570df4e9 Yan, Zheng 2017-11-15 434 req = NULL;
> 570df4e9 Yan, Zheng 2017-11-15 435 goto out;
> 570df4e9 Yan, Zheng 2017-11-15 436 }
> 570df4e9 Yan, Zheng 2017-11-15 437 err = ceph_alloc_readdir_reply_buffer(req, inode);
> 570df4e9 Yan, Zheng 2017-11-15 438 if (err)
> 570df4e9 Yan, Zheng 2017-11-15 439 goto out;
> 570df4e9 Yan, Zheng 2017-11-15 440
> 570df4e9 Yan, Zheng 2017-11-15 441 req->r_direct_mode = USE_AUTH_MDS;
> 570df4e9 Yan, Zheng 2017-11-15 442 req->r_readdir_offset = next_offset;
> 570df4e9 Yan, Zheng 2017-11-15 443 req->r_args.readdir.flags =
> 570df4e9 Yan, Zheng 2017-11-15 444 cpu_to_le16(CEPH_READDIR_REPLY_BITFLAGS);
> 570df4e9 Yan, Zheng 2017-11-15 445 if (last_name) {
> 570df4e9 Yan, Zheng 2017-11-15 446 req->r_path2 = last_name;
> 570df4e9 Yan, Zheng 2017-11-15 447 last_name = NULL;
> 570df4e9 Yan, Zheng 2017-11-15 448 }
> 570df4e9 Yan, Zheng 2017-11-15 449
> 570df4e9 Yan, Zheng 2017-11-15 450 req->r_inode = dir;
> 570df4e9 Yan, Zheng 2017-11-15 451 ihold(dir);
> 570df4e9 Yan, Zheng 2017-11-15 452 req->r_dentry = dget(parent);
> 570df4e9 Yan, Zheng 2017-11-15 453
> 570df4e9 Yan, Zheng 2017-11-15 454 inode_lock(dir);
> 570df4e9 Yan, Zheng 2017-11-15 455 err = ceph_mdsc_do_request(fsc->mdsc, NULL, req);
> 570df4e9 Yan, Zheng 2017-11-15 456 inode_unlock(dir);
> 570df4e9 Yan, Zheng 2017-11-15 457
> 570df4e9 Yan, Zheng 2017-11-15 458 if (err < 0)
> 570df4e9 Yan, Zheng 2017-11-15 @459 goto out;
> 570df4e9 Yan, Zheng 2017-11-15 460
> 570df4e9 Yan, Zheng 2017-11-15 @461 rinfo = &req->r_reply_info;
> 570df4e9 Yan, Zheng 2017-11-15 462 for (i = 0; i < rinfo->dir_nr; i++) {
> 570df4e9 Yan, Zheng 2017-11-15 463 rde = rinfo->dir_entries + i;
> 570df4e9 Yan, Zheng 2017-11-15 464 BUG_ON(!rde->inode.in);
> 570df4e9 Yan, Zheng 2017-11-15 465 if (ceph_snap(inode) ==
> 570df4e9 Yan, Zheng 2017-11-15 466 le64_to_cpu(rde->inode.in->snapid)) {
> 570df4e9 Yan, Zheng 2017-11-15 467 memcpy(name, rde->name, rde->name_len);
> 570df4e9 Yan, Zheng 2017-11-15 468 name[rde->name_len] = '\0';
> 570df4e9 Yan, Zheng 2017-11-15 469 err = 0;
> 570df4e9 Yan, Zheng 2017-11-15 470 goto out;
> 570df4e9 Yan, Zheng 2017-11-15 471 }
> 570df4e9 Yan, Zheng 2017-11-15 472 }
> 570df4e9 Yan, Zheng 2017-11-15 473
> 570df4e9 Yan, Zheng 2017-11-15 474 if (rinfo->dir_end)
> 570df4e9 Yan, Zheng 2017-11-15 475 break;
> 570df4e9 Yan, Zheng 2017-11-15 476
> 570df4e9 Yan, Zheng 2017-11-15 477 BUG_ON(rinfo->dir_nr <= 0);
> 570df4e9 Yan, Zheng 2017-11-15 478 rde = rinfo->dir_entries + (rinfo->dir_nr - 1);
> 570df4e9 Yan, Zheng 2017-11-15 479 next_offset += rinfo->dir_nr;
> 570df4e9 Yan, Zheng 2017-11-15 480 last_name = kstrndup(rde->name, rde->name_len, GFP_KERNEL);
> 570df4e9 Yan, Zheng 2017-11-15 481 if (!last_name) {
> 570df4e9 Yan, Zheng 2017-11-15 482 err = -ENOMEM;
> 570df4e9 Yan, Zheng 2017-11-15 483 goto out;
> 570df4e9 Yan, Zheng 2017-11-15 484 }
> 570df4e9 Yan, Zheng 2017-11-15 485
> 570df4e9 Yan, Zheng 2017-11-15 486 ceph_mdsc_put_request(req);
> 570df4e9 Yan, Zheng 2017-11-15 487 req = NULL;
> 570df4e9 Yan, Zheng 2017-11-15 488 }
> 570df4e9 Yan, Zheng 2017-11-15 489 err = -ENOENT;
> 570df4e9 Yan, Zheng 2017-11-15 490 out:
> 570df4e9 Yan, Zheng 2017-11-15 491 if (req)
> 570df4e9 Yan, Zheng 2017-11-15 492 ceph_mdsc_put_request(req);
> 570df4e9 Yan, Zheng 2017-11-15 493 kfree(last_name);
> 570df4e9 Yan, Zheng 2017-11-15 494 dout("get_snap_name %p ino %llx.%llx err=%d\n",
> 570df4e9 Yan, Zheng 2017-11-15 495 child, ceph_vinop(inode), err);
> 570df4e9 Yan, Zheng 2017-11-15 496 return err;
> 570df4e9 Yan, Zheng 2017-11-15 497 }
> 570df4e9 Yan, Zheng 2017-11-15 498
Fixed.
Thanks,
Ilya