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
