linux-f2fs-devel.lists.sourceforge.net archive mirror
 help / color / mirror / Atom feed
From: Aravind Ramesh <Aravind.Ramesh@wdc.com>
To: Jaegeuk Kim <jaegeuk@kernel.org>
Cc: Niklas Cassel <Niklas.Cassel@wdc.com>,
	Damien Le Moal <Damien.LeMoal@wdc.com>,
	"kbuild-all@lists.01.org" <kbuild-all@lists.01.org>,
	kernel test robot <lkp@intel.com>,
	"linux-f2fs-devel@lists.sourceforge.net"
	<linux-f2fs-devel@lists.sourceforge.net>
Subject: Re: [f2fs-dev] [f2fs:dev-test 37/38] fs/f2fs/super.c:3058:48: error: 'struct blk_zone' has no member named 'capacity'
Date: Tue, 21 Jul 2020 13:42:28 +0000	[thread overview]
Message-ID: <BY5PR04MB699535E4F62BEC16568A3E058C780@BY5PR04MB6995.namprd04.prod.outlook.com> (raw)
In-Reply-To: <202007211524.zrSvup2X%lkp@intel.com>

Hello Jaegeuk,
The patch that adds the capacity struct member has been merged to linux-block tree's for-next branch.
So either:
1. f2fs-dev branch has to carry the same patch, from here:
https://git.kernel.org/pub/scm/linux/kernel/git/axboe/linux-block.git/log/include/uapi/linux/blkzoned.h?h=for-next
I had mentioned this is the cover letter of this patch.
2. linux-block pull request has to be merged before f2fs pull request to Linus
3. Or this f2fs patch has to wait until next merge window (the one after kernel 5.9 merge window)

Thanks
Aravind

> -----Original Message-----
> From: kernel test robot <lkp@intel.com>
> Sent: Tuesday, July 21, 2020 12:43 PM
> To: Aravind Ramesh <Aravind.Ramesh@wdc.com>
> Cc: kbuild-all@lists.01.org; linux-f2fs-devel@lists.sourceforge.net; Jaegeuk Kim
> <jaegeuk@kernel.org>; Damien Le Moal <Damien.LeMoal@wdc.com>; Niklas Cassel
> <Niklas.Cassel@wdc.com>; Chao Yu <yuchao0@huawei.com>; Chao Yu
> <chao@kernel.org>
> Subject: [f2fs:dev-test 37/38] fs/f2fs/super.c:3058:48: error: 'struct blk_zone' has
> no member named 'capacity'
> 
> tree:   https://git.kernel.org/pub/scm/linux/kernel/git/jaegeuk/f2fs.git dev-test
> head:   439633ba3673a5d8983529426df144c64a23d181
> commit: d787c5eec8b58bb47d4bf750758e0e87d667bf56 [37/38] f2fs: support
> zone capacity less than zone size
> config: nds32-allyesconfig (attached as .config)
> compiler: nds32le-linux-gcc (GCC) 9.3.0
> reproduce (this is a W=1 build):
>         wget https://raw.githubusercontent.com/intel/lkp-
> tests/master/sbin/make.cross -O ~/bin/make.cross
>         chmod +x ~/bin/make.cross
>         git checkout d787c5eec8b58bb47d4bf750758e0e87d667bf56
>         # save the attached .config to linux build tree
>         COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross
> ARCH=nds32
> 
> If you fix the issue, kindly add following tag as appropriate
> Reported-by: kernel test robot <lkp@intel.com>
> 
> All errors (new ones prefixed by >>):
> 
>    fs/f2fs/super.c: In function 'f2fs_report_zone_cb':
> >> fs/f2fs/super.c:3058:48: error: 'struct blk_zone' has no member named 'capacity'
>     3058 |  rz_args->dev->zone_capacity_blocks[idx] = zone->capacity >>
>          |                                                ^~
>    fs/f2fs/super.c:3060:23: error: 'struct blk_zone' has no member named 'capacity'
>     3060 |  if (zone->len != zone->capacity && !rz_args->zone_cap_mismatch)
>          |                       ^~
> 
> vim +3058 fs/f2fs/super.c
> 
>   3048
>   3049	static int f2fs_report_zone_cb(struct blk_zone *zone, unsigned int idx,
>   3050				      void *data)
>   3051	{
>   3052		struct f2fs_report_zones_args *rz_args = data;
>   3053
>   3054		if (zone->type == BLK_ZONE_TYPE_CONVENTIONAL)
>   3055			return 0;
>   3056
>   3057		set_bit(idx, rz_args->dev->blkz_seq);
> > 3058		rz_args->dev->zone_capacity_blocks[idx] = zone->capacity >>
>   3059
> 	F2FS_LOG_SECTORS_PER_BLOCK;
>   3060		if (zone->len != zone->capacity && !rz_args->zone_cap_mismatch)
>   3061			rz_args->zone_cap_mismatch = true;
>   3062
>   3063		return 0;
>   3064	}
>   3065
> 
> ---
> 0-DAY CI Kernel Test Service, Intel Corporation
> https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org


_______________________________________________
Linux-f2fs-devel mailing list
Linux-f2fs-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linux-f2fs-devel

       reply	other threads:[~2020-07-21 13:42 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <202007211524.zrSvup2X%lkp@intel.com>
2020-07-21 13:42 ` Aravind Ramesh [this message]
2020-07-21 19:57   ` [f2fs-dev] [f2fs:dev-test 37/38] fs/f2fs/super.c:3058:48: error: 'struct blk_zone' has no member named 'capacity' Jaegeuk Kim
2020-08-10 14:18     ` Niklas Cassel
2020-08-10 19:49       ` Jaegeuk Kim

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=BY5PR04MB699535E4F62BEC16568A3E058C780@BY5PR04MB6995.namprd04.prod.outlook.com \
    --to=aravind.ramesh@wdc.com \
    --cc=Damien.LeMoal@wdc.com \
    --cc=Niklas.Cassel@wdc.com \
    --cc=jaegeuk@kernel.org \
    --cc=kbuild-all@lists.01.org \
    --cc=linux-f2fs-devel@lists.sourceforge.net \
    --cc=lkp@intel.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).