All of lore.kernel.org
 help / color / mirror / Atom feed
From: kernel test robot <lkp@intel.com>
To: kbuild-all@lists.01.org
Subject: Re: [PATCH v11 09/10] fs/ntfs3: Add NTFS3 in fs/Kconfig and fs/Makefile
Date: Tue, 03 Nov 2020 11:06:14 +0800	[thread overview]
Message-ID: <202011031109.d2r2hcCI-lkp@intel.com> (raw)
In-Reply-To: <20201030150239.3957156-10-almaz.alexandrovich@paragon-software.com>

[-- Attachment #1: Type: text/plain, Size: 19188 bytes --]

Hi Konstantin,

Thank you for the patch! Yet something to improve:

[auto build test ERROR on linus/master]
[also build test ERROR on v5.10-rc2 next-20201102]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch]

url:    https://github.com/0day-ci/linux/commits/Konstantin-Komarov/NTFS-read-write-driver-GPL-implementation-by-Paragon-Software/20201031-220904
base:   https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git 5fc6b075e165f641fbc366b58b578055762d5f8c
config: alpha-allmodconfig (attached as .config)
compiler: alpha-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
        # https://github.com/0day-ci/linux/commit/0868596d71532d31ce16ee71c5e73c154878416e
        git remote add linux-review https://github.com/0day-ci/linux
        git fetch --no-tags linux-review Konstantin-Komarov/NTFS-read-write-driver-GPL-implementation-by-Paragon-Software/20201031-220904
        git checkout 0868596d71532d31ce16ee71c5e73c154878416e
        # save the attached .config to linux build tree
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross ARCH=alpha 

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 >>):

   In file included from fs/ntfs3/frecord.c:15:
   fs/ntfs3/ntfs.h:430:1: warning: 'inline' is not at beginning of declaration [-Wold-style-declaration]
     430 | static const inline __le16 *attr_name(const struct ATTRIB *attr)
         | ^~~~~~
   fs/ntfs3/ntfs.h:547:1: warning: 'inline' is not at beginning of declaration [-Wold-style-declaration]
     547 | static const inline __le16 *le_name(const struct ATTR_LIST_ENTRY *le)
         | ^~~~~~
   fs/ntfs3/frecord.c: In function 'ni_read_frame':
>> fs/ntfs3/frecord.c:2247:17: error: implicit declaration of function 'vmap'; did you mean 'kmap'? [-Werror=implicit-function-declaration]
    2247 |  frame_ondisk = vmap(pages_disk, npages_disk, VM_MAP, PAGE_KERNEL);
         |                 ^~~~
         |                 kmap
>> fs/ntfs3/frecord.c:2247:47: error: 'VM_MAP' undeclared (first use in this function); did you mean 'VM_MTE'?
    2247 |  frame_ondisk = vmap(pages_disk, npages_disk, VM_MAP, PAGE_KERNEL);
         |                                               ^~~~~~
         |                                               VM_MTE
   fs/ntfs3/frecord.c:2247:47: note: each undeclared identifier is reported only once for each function it appears in
>> fs/ntfs3/frecord.c:2278:2: error: implicit declaration of function 'vunmap'; did you mean 'kunmap'? [-Werror=implicit-function-declaration]
    2278 |  vunmap(frame_mem);
         |  ^~~~~~
         |  kunmap
   fs/ntfs3/frecord.c: In function 'ni_write_frame':
   fs/ntfs3/frecord.c:2375:51: error: 'VM_MAP' undeclared (first use in this function); did you mean 'VM_MTE'?
    2375 |  frame_ondisk = vmap(pages_disk, pages_per_frame, VM_MAP, PAGE_KERNEL);
         |                                                   ^~~~~~
         |                                                   VM_MTE
   cc1: some warnings being treated as errors

vim +2247 fs/ntfs3/frecord.c

8f609ab22280366 Konstantin Komarov 2020-10-30  2099  
8f609ab22280366 Konstantin Komarov 2020-10-30  2100  /*
8f609ab22280366 Konstantin Komarov 2020-10-30  2101   * ni_read_frame
8f609ab22280366 Konstantin Komarov 2020-10-30  2102   *
8f609ab22280366 Konstantin Komarov 2020-10-30  2103   * pages - array of locked pages
8f609ab22280366 Konstantin Komarov 2020-10-30  2104   */
8f609ab22280366 Konstantin Komarov 2020-10-30  2105  int ni_read_frame(struct ntfs_inode *ni, u64 frame_vbo, struct page **pages,
8f609ab22280366 Konstantin Komarov 2020-10-30  2106  		  u32 pages_per_frame)
8f609ab22280366 Konstantin Komarov 2020-10-30  2107  {
8f609ab22280366 Konstantin Komarov 2020-10-30  2108  	int err;
8f609ab22280366 Konstantin Komarov 2020-10-30  2109  	struct ntfs_sb_info *sbi = ni->mi.sbi;
8f609ab22280366 Konstantin Komarov 2020-10-30  2110  	char *frame_ondisk = NULL;
8f609ab22280366 Konstantin Komarov 2020-10-30  2111  	char *frame_mem = NULL;
8f609ab22280366 Konstantin Komarov 2020-10-30  2112  	struct page **pages_disk = NULL;
8f609ab22280366 Konstantin Komarov 2020-10-30  2113  	u64 valid_size = ni->i_valid;
8f609ab22280366 Konstantin Komarov 2020-10-30  2114  	u32 frame_size;
8f609ab22280366 Konstantin Komarov 2020-10-30  2115  	size_t unc_size;
8f609ab22280366 Konstantin Komarov 2020-10-30  2116  	u32 ondisk_size, i, npages_disk;
8f609ab22280366 Konstantin Komarov 2020-10-30  2117  	struct page *pg;
8f609ab22280366 Konstantin Komarov 2020-10-30  2118  	struct ATTRIB *attr;
8f609ab22280366 Konstantin Komarov 2020-10-30  2119  	CLST frame, clst_data;
8f609ab22280366 Konstantin Komarov 2020-10-30  2120  
8f609ab22280366 Konstantin Komarov 2020-10-30  2121  	attr = ni_find_attr(ni, NULL, NULL, ATTR_DATA, NULL, 0, NULL, NULL);
8f609ab22280366 Konstantin Komarov 2020-10-30  2122  	if (!attr) {
8f609ab22280366 Konstantin Komarov 2020-10-30  2123  		err = -ENOENT;
8f609ab22280366 Konstantin Komarov 2020-10-30  2124  		goto out;
8f609ab22280366 Konstantin Komarov 2020-10-30  2125  	}
8f609ab22280366 Konstantin Komarov 2020-10-30  2126  
8f609ab22280366 Konstantin Komarov 2020-10-30  2127  	if (!attr->non_res) {
8f609ab22280366 Konstantin Komarov 2020-10-30  2128  		u32 data_size = le32_to_cpu(attr->res.data_size);
8f609ab22280366 Konstantin Komarov 2020-10-30  2129  		void *kaddr = kmap_atomic(pages[0]);
8f609ab22280366 Konstantin Komarov 2020-10-30  2130  
8f609ab22280366 Konstantin Komarov 2020-10-30  2131  		memset(kaddr, 0, PAGE_SIZE);
8f609ab22280366 Konstantin Komarov 2020-10-30  2132  		if (frame_vbo < data_size) {
8f609ab22280366 Konstantin Komarov 2020-10-30  2133  			err = data_size - frame_vbo;
8f609ab22280366 Konstantin Komarov 2020-10-30  2134  			if (err > PAGE_SIZE)
8f609ab22280366 Konstantin Komarov 2020-10-30  2135  				err = PAGE_SIZE;
8f609ab22280366 Konstantin Komarov 2020-10-30  2136  			memcpy(kaddr, resident_data(attr) + frame_vbo, err);
8f609ab22280366 Konstantin Komarov 2020-10-30  2137  		}
8f609ab22280366 Konstantin Komarov 2020-10-30  2138  		flush_dcache_page(pages[0]);
8f609ab22280366 Konstantin Komarov 2020-10-30  2139  		kunmap_atomic(kaddr);
8f609ab22280366 Konstantin Komarov 2020-10-30  2140  		err = 0;
8f609ab22280366 Konstantin Komarov 2020-10-30  2141  		goto out;
8f609ab22280366 Konstantin Komarov 2020-10-30  2142  	}
8f609ab22280366 Konstantin Komarov 2020-10-30  2143  
8f609ab22280366 Konstantin Komarov 2020-10-30  2144  	if (frame_vbo >= valid_size) {
8f609ab22280366 Konstantin Komarov 2020-10-30  2145  all_zero:
8f609ab22280366 Konstantin Komarov 2020-10-30  2146  		for (i = 0; i < pages_per_frame; i++)
8f609ab22280366 Konstantin Komarov 2020-10-30  2147  			zero_user_segment(pages[i], 0, PAGE_SIZE);
8f609ab22280366 Konstantin Komarov 2020-10-30  2148  		err = 0;
8f609ab22280366 Konstantin Komarov 2020-10-30  2149  		goto out;
8f609ab22280366 Konstantin Komarov 2020-10-30  2150  	}
8f609ab22280366 Konstantin Komarov 2020-10-30  2151  
8f609ab22280366 Konstantin Komarov 2020-10-30  2152  	if (ni->ni_flags & NI_FLAG_COMPRESSED_MASK) {
8f609ab22280366 Konstantin Komarov 2020-10-30  2153  		u32 cmpr = ((ni->ni_flags & NI_FLAG_COMPRESSED_MASK) >> 8) - 1;
8f609ab22280366 Konstantin Komarov 2020-10-30  2154  
8f609ab22280366 Konstantin Komarov 2020-10-30  2155  		switch (cmpr) {
8f609ab22280366 Konstantin Komarov 2020-10-30  2156  		case WOF_COMPRESSION_XPRESS4K:
8f609ab22280366 Konstantin Komarov 2020-10-30  2157  			/* xpress 4k*/
8f609ab22280366 Konstantin Komarov 2020-10-30  2158  			frame_size = 0x1000;
8f609ab22280366 Konstantin Komarov 2020-10-30  2159  			break;
8f609ab22280366 Konstantin Komarov 2020-10-30  2160  		case WOF_COMPRESSION_LZX:
8f609ab22280366 Konstantin Komarov 2020-10-30  2161  			/* lzx 32k*/
8f609ab22280366 Konstantin Komarov 2020-10-30  2162  			frame_size = 0x8000;
8f609ab22280366 Konstantin Komarov 2020-10-30  2163  			break;
8f609ab22280366 Konstantin Komarov 2020-10-30  2164  		case WOF_COMPRESSION_XPRESS8K:
8f609ab22280366 Konstantin Komarov 2020-10-30  2165  			/* xpress 8k*/
8f609ab22280366 Konstantin Komarov 2020-10-30  2166  			frame_size = 0x2000;
8f609ab22280366 Konstantin Komarov 2020-10-30  2167  			break;
8f609ab22280366 Konstantin Komarov 2020-10-30  2168  		case WOF_COMPRESSION_XPRESS16K:
8f609ab22280366 Konstantin Komarov 2020-10-30  2169  			/* xpress 16k*/
8f609ab22280366 Konstantin Komarov 2020-10-30  2170  			frame_size = 0x4000;
8f609ab22280366 Konstantin Komarov 2020-10-30  2171  			break;
8f609ab22280366 Konstantin Komarov 2020-10-30  2172  		}
8f609ab22280366 Konstantin Komarov 2020-10-30  2173  		/* TODO: port lzx/xpress */
8f609ab22280366 Konstantin Komarov 2020-10-30  2174  		err = -EOPNOTSUPP;
8f609ab22280366 Konstantin Komarov 2020-10-30  2175  		goto out;
8f609ab22280366 Konstantin Komarov 2020-10-30  2176  	} else if (is_attr_compressed(attr)) {
8f609ab22280366 Konstantin Komarov 2020-10-30  2177  		/* lznt compression*/
8f609ab22280366 Konstantin Komarov 2020-10-30  2178  		if (sbi->cluster_size > NTFS_LZNT_MAX_CLUSTER) {
8f609ab22280366 Konstantin Komarov 2020-10-30  2179  			err = -EOPNOTSUPP;
8f609ab22280366 Konstantin Komarov 2020-10-30  2180  			goto out;
8f609ab22280366 Konstantin Komarov 2020-10-30  2181  		}
8f609ab22280366 Konstantin Komarov 2020-10-30  2182  
8f609ab22280366 Konstantin Komarov 2020-10-30  2183  		if (attr->nres.c_unit != NTFS_LZNT_CUNIT) {
8f609ab22280366 Konstantin Komarov 2020-10-30  2184  			err = -EOPNOTSUPP;
8f609ab22280366 Konstantin Komarov 2020-10-30  2185  			goto out;
8f609ab22280366 Konstantin Komarov 2020-10-30  2186  		}
8f609ab22280366 Konstantin Komarov 2020-10-30  2187  
8f609ab22280366 Konstantin Komarov 2020-10-30  2188  		down_write(&ni->file.run_lock);
8f609ab22280366 Konstantin Komarov 2020-10-30  2189  		run_truncate_around(&ni->file.run,
8f609ab22280366 Konstantin Komarov 2020-10-30  2190  				    le64_to_cpu(attr->nres.svcn));
8f609ab22280366 Konstantin Komarov 2020-10-30  2191  		frame = frame_vbo >> (sbi->cluster_bits + NTFS_LZNT_CUNIT);
8f609ab22280366 Konstantin Komarov 2020-10-30  2192  		err = attr_is_frame_compressed(ni, attr, frame, &clst_data);
8f609ab22280366 Konstantin Komarov 2020-10-30  2193  		up_write(&ni->file.run_lock);
8f609ab22280366 Konstantin Komarov 2020-10-30  2194  		if (err)
8f609ab22280366 Konstantin Komarov 2020-10-30  2195  			goto out;
8f609ab22280366 Konstantin Komarov 2020-10-30  2196  
8f609ab22280366 Konstantin Komarov 2020-10-30  2197  		if (!clst_data)
8f609ab22280366 Konstantin Komarov 2020-10-30  2198  			goto all_zero;
8f609ab22280366 Konstantin Komarov 2020-10-30  2199  
8f609ab22280366 Konstantin Komarov 2020-10-30  2200  		frame_size = sbi->cluster_size << NTFS_LZNT_CUNIT;
8f609ab22280366 Konstantin Komarov 2020-10-30  2201  		ondisk_size = clst_data << sbi->cluster_bits;
8f609ab22280366 Konstantin Komarov 2020-10-30  2202  
8f609ab22280366 Konstantin Komarov 2020-10-30  2203  		if (clst_data >= NTFS_LZNT_CLUSTERS) {
8f609ab22280366 Konstantin Komarov 2020-10-30  2204  			/* frame is not compressed */
8f609ab22280366 Konstantin Komarov 2020-10-30  2205  			down_read(&ni->file.run_lock);
8f609ab22280366 Konstantin Komarov 2020-10-30  2206  			err = ntfs_bio_pages(sbi, &ni->file.run, pages,
8f609ab22280366 Konstantin Komarov 2020-10-30  2207  					     pages_per_frame, frame_vbo,
8f609ab22280366 Konstantin Komarov 2020-10-30  2208  					     ondisk_size, REQ_OP_READ);
8f609ab22280366 Konstantin Komarov 2020-10-30  2209  			up_read(&ni->file.run_lock);
8f609ab22280366 Konstantin Komarov 2020-10-30  2210  			goto out;
8f609ab22280366 Konstantin Komarov 2020-10-30  2211  		}
8f609ab22280366 Konstantin Komarov 2020-10-30  2212  	} else {
8f609ab22280366 Konstantin Komarov 2020-10-30  2213  		__builtin_unreachable();
8f609ab22280366 Konstantin Komarov 2020-10-30  2214  		err = -EINVAL;
8f609ab22280366 Konstantin Komarov 2020-10-30  2215  		goto out;
8f609ab22280366 Konstantin Komarov 2020-10-30  2216  	}
8f609ab22280366 Konstantin Komarov 2020-10-30  2217  
8f609ab22280366 Konstantin Komarov 2020-10-30  2218  	npages_disk = (ondisk_size + PAGE_SIZE - 1) >> PAGE_SHIFT;
8f609ab22280366 Konstantin Komarov 2020-10-30  2219  	pages_disk = ntfs_alloc(npages_disk * sizeof(struct page *), 1);
8f609ab22280366 Konstantin Komarov 2020-10-30  2220  	if (!pages_disk) {
8f609ab22280366 Konstantin Komarov 2020-10-30  2221  		err = -ENOMEM;
8f609ab22280366 Konstantin Komarov 2020-10-30  2222  		goto out;
8f609ab22280366 Konstantin Komarov 2020-10-30  2223  	}
8f609ab22280366 Konstantin Komarov 2020-10-30  2224  
8f609ab22280366 Konstantin Komarov 2020-10-30  2225  	for (i = 0; i < npages_disk; i++) {
8f609ab22280366 Konstantin Komarov 2020-10-30  2226  		pg = alloc_page(GFP_KERNEL);
8f609ab22280366 Konstantin Komarov 2020-10-30  2227  		if (!pg) {
8f609ab22280366 Konstantin Komarov 2020-10-30  2228  			err = -ENOMEM;
8f609ab22280366 Konstantin Komarov 2020-10-30  2229  			goto out1;
8f609ab22280366 Konstantin Komarov 2020-10-30  2230  		}
8f609ab22280366 Konstantin Komarov 2020-10-30  2231  		pages_disk[i] = pg;
8f609ab22280366 Konstantin Komarov 2020-10-30  2232  		lock_page(pg);
8f609ab22280366 Konstantin Komarov 2020-10-30  2233  		kmap(pg);
8f609ab22280366 Konstantin Komarov 2020-10-30  2234  	}
8f609ab22280366 Konstantin Komarov 2020-10-30  2235  
8f609ab22280366 Konstantin Komarov 2020-10-30  2236  	/* read 'ondisk_size' bytes from disk */
8f609ab22280366 Konstantin Komarov 2020-10-30  2237  	down_read(&ni->file.run_lock);
8f609ab22280366 Konstantin Komarov 2020-10-30  2238  	err = ntfs_bio_pages(sbi, &ni->file.run, pages_disk, npages_disk,
8f609ab22280366 Konstantin Komarov 2020-10-30  2239  			     frame_vbo, ondisk_size, REQ_OP_READ);
8f609ab22280366 Konstantin Komarov 2020-10-30  2240  	up_read(&ni->file.run_lock);
8f609ab22280366 Konstantin Komarov 2020-10-30  2241  	if (err)
8f609ab22280366 Konstantin Komarov 2020-10-30  2242  		goto out1;
8f609ab22280366 Konstantin Komarov 2020-10-30  2243  
8f609ab22280366 Konstantin Komarov 2020-10-30  2244  	/*
8f609ab22280366 Konstantin Komarov 2020-10-30  2245  	 * To simplify decompress algorithm do vmap for source and target pages
8f609ab22280366 Konstantin Komarov 2020-10-30  2246  	 */
8f609ab22280366 Konstantin Komarov 2020-10-30 @2247  	frame_ondisk = vmap(pages_disk, npages_disk, VM_MAP, PAGE_KERNEL);
8f609ab22280366 Konstantin Komarov 2020-10-30  2248  	if (!frame_ondisk) {
8f609ab22280366 Konstantin Komarov 2020-10-30  2249  		err = -ENOMEM;
8f609ab22280366 Konstantin Komarov 2020-10-30  2250  		goto out1;
8f609ab22280366 Konstantin Komarov 2020-10-30  2251  	}
8f609ab22280366 Konstantin Komarov 2020-10-30  2252  
8f609ab22280366 Konstantin Komarov 2020-10-30  2253  	for (i = 0; i < pages_per_frame; i++)
8f609ab22280366 Konstantin Komarov 2020-10-30  2254  		kmap(pages[i]);
8f609ab22280366 Konstantin Komarov 2020-10-30  2255  
8f609ab22280366 Konstantin Komarov 2020-10-30  2256  	frame_mem = vmap(pages, pages_per_frame, VM_MAP, PAGE_KERNEL);
8f609ab22280366 Konstantin Komarov 2020-10-30  2257  	if (!frame_mem) {
8f609ab22280366 Konstantin Komarov 2020-10-30  2258  		err = -ENOMEM;
8f609ab22280366 Konstantin Komarov 2020-10-30  2259  		goto out2;
8f609ab22280366 Konstantin Komarov 2020-10-30  2260  	}
8f609ab22280366 Konstantin Komarov 2020-10-30  2261  
8f609ab22280366 Konstantin Komarov 2020-10-30  2262  	/* decompress: frame_ondisk -> frame_mem */
8f609ab22280366 Konstantin Komarov 2020-10-30  2263  	unc_size = decompress_lznt(frame_ondisk, ondisk_size, frame_mem,
8f609ab22280366 Konstantin Komarov 2020-10-30  2264  				   frame_size);
8f609ab22280366 Konstantin Komarov 2020-10-30  2265  
8f609ab22280366 Konstantin Komarov 2020-10-30  2266  	if ((ssize_t)unc_size < 0) {
8f609ab22280366 Konstantin Komarov 2020-10-30  2267  		err = unc_size;
8f609ab22280366 Konstantin Komarov 2020-10-30  2268  	} else if (!unc_size || unc_size > frame_size) {
8f609ab22280366 Konstantin Komarov 2020-10-30  2269  		err = -EINVAL;
8f609ab22280366 Konstantin Komarov 2020-10-30  2270  	} else {
8f609ab22280366 Konstantin Komarov 2020-10-30  2271  		if (valid_size < frame_vbo + frame_size) {
8f609ab22280366 Konstantin Komarov 2020-10-30  2272  			size_t ok = valid_size - frame_vbo;
8f609ab22280366 Konstantin Komarov 2020-10-30  2273  
8f609ab22280366 Konstantin Komarov 2020-10-30  2274  			memset(frame_mem + ok, 0, frame_size - ok);
8f609ab22280366 Konstantin Komarov 2020-10-30  2275  		}
8f609ab22280366 Konstantin Komarov 2020-10-30  2276  	}
8f609ab22280366 Konstantin Komarov 2020-10-30  2277  
8f609ab22280366 Konstantin Komarov 2020-10-30 @2278  	vunmap(frame_mem);
8f609ab22280366 Konstantin Komarov 2020-10-30  2279  out2:
8f609ab22280366 Konstantin Komarov 2020-10-30  2280  	for (i = 0; i < pages_per_frame; i++) {
8f609ab22280366 Konstantin Komarov 2020-10-30  2281  		pg = pages[i];
8f609ab22280366 Konstantin Komarov 2020-10-30  2282  		kunmap(pg);
8f609ab22280366 Konstantin Komarov 2020-10-30  2283  	}
8f609ab22280366 Konstantin Komarov 2020-10-30  2284  	vunmap(frame_ondisk);
8f609ab22280366 Konstantin Komarov 2020-10-30  2285  out1:
8f609ab22280366 Konstantin Komarov 2020-10-30  2286  	for (i = 0; i < npages_disk; i++) {
8f609ab22280366 Konstantin Komarov 2020-10-30  2287  		pg = pages_disk[i];
8f609ab22280366 Konstantin Komarov 2020-10-30  2288  		if (pg) {
8f609ab22280366 Konstantin Komarov 2020-10-30  2289  			kunmap(pg);
8f609ab22280366 Konstantin Komarov 2020-10-30  2290  			unlock_page(pg);
8f609ab22280366 Konstantin Komarov 2020-10-30  2291  			put_page(pg);
8f609ab22280366 Konstantin Komarov 2020-10-30  2292  		}
8f609ab22280366 Konstantin Komarov 2020-10-30  2293  	}
8f609ab22280366 Konstantin Komarov 2020-10-30  2294  	ntfs_free(pages_disk);
8f609ab22280366 Konstantin Komarov 2020-10-30  2295  out:
8f609ab22280366 Konstantin Komarov 2020-10-30  2296  	for (i = 0; i < pages_per_frame; i++) {
8f609ab22280366 Konstantin Komarov 2020-10-30  2297  		pg = pages[i];
8f609ab22280366 Konstantin Komarov 2020-10-30  2298  		ClearPageError(pg);
8f609ab22280366 Konstantin Komarov 2020-10-30  2299  		SetPageUptodate(pg);
8f609ab22280366 Konstantin Komarov 2020-10-30  2300  	}
8f609ab22280366 Konstantin Komarov 2020-10-30  2301  
8f609ab22280366 Konstantin Komarov 2020-10-30  2302  	return err;
8f609ab22280366 Konstantin Komarov 2020-10-30  2303  }
8f609ab22280366 Konstantin Komarov 2020-10-30  2304  

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org

[-- Attachment #2: config.gz --]
[-- Type: application/gzip, Size: 66534 bytes --]

WARNING: multiple messages have this Message-ID (diff)
From: kernel test robot <lkp@intel.com>
To: Konstantin Komarov <almaz.alexandrovich@paragon-software.com>,
	linux-fsdevel@vger.kernel.org
Cc: kbuild-all@lists.01.org, viro@zeniv.linux.org.uk,
	linux-kernel@vger.kernel.org, pali@kernel.org, dsterba@suse.cz,
	aaptel@suse.com, willy@infradead.org, rdunlap@infradead.org,
	joe@perches.com, mark@harmstone.com
Subject: Re: [PATCH v11 09/10] fs/ntfs3: Add NTFS3 in fs/Kconfig and fs/Makefile
Date: Tue, 3 Nov 2020 11:06:14 +0800	[thread overview]
Message-ID: <202011031109.d2r2hcCI-lkp@intel.com> (raw)
In-Reply-To: <20201030150239.3957156-10-almaz.alexandrovich@paragon-software.com>

[-- Attachment #1: Type: text/plain, Size: 18919 bytes --]

Hi Konstantin,

Thank you for the patch! Yet something to improve:

[auto build test ERROR on linus/master]
[also build test ERROR on v5.10-rc2 next-20201102]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch]

url:    https://github.com/0day-ci/linux/commits/Konstantin-Komarov/NTFS-read-write-driver-GPL-implementation-by-Paragon-Software/20201031-220904
base:   https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git 5fc6b075e165f641fbc366b58b578055762d5f8c
config: alpha-allmodconfig (attached as .config)
compiler: alpha-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
        # https://github.com/0day-ci/linux/commit/0868596d71532d31ce16ee71c5e73c154878416e
        git remote add linux-review https://github.com/0day-ci/linux
        git fetch --no-tags linux-review Konstantin-Komarov/NTFS-read-write-driver-GPL-implementation-by-Paragon-Software/20201031-220904
        git checkout 0868596d71532d31ce16ee71c5e73c154878416e
        # save the attached .config to linux build tree
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross ARCH=alpha 

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 >>):

   In file included from fs/ntfs3/frecord.c:15:
   fs/ntfs3/ntfs.h:430:1: warning: 'inline' is not at beginning of declaration [-Wold-style-declaration]
     430 | static const inline __le16 *attr_name(const struct ATTRIB *attr)
         | ^~~~~~
   fs/ntfs3/ntfs.h:547:1: warning: 'inline' is not at beginning of declaration [-Wold-style-declaration]
     547 | static const inline __le16 *le_name(const struct ATTR_LIST_ENTRY *le)
         | ^~~~~~
   fs/ntfs3/frecord.c: In function 'ni_read_frame':
>> fs/ntfs3/frecord.c:2247:17: error: implicit declaration of function 'vmap'; did you mean 'kmap'? [-Werror=implicit-function-declaration]
    2247 |  frame_ondisk = vmap(pages_disk, npages_disk, VM_MAP, PAGE_KERNEL);
         |                 ^~~~
         |                 kmap
>> fs/ntfs3/frecord.c:2247:47: error: 'VM_MAP' undeclared (first use in this function); did you mean 'VM_MTE'?
    2247 |  frame_ondisk = vmap(pages_disk, npages_disk, VM_MAP, PAGE_KERNEL);
         |                                               ^~~~~~
         |                                               VM_MTE
   fs/ntfs3/frecord.c:2247:47: note: each undeclared identifier is reported only once for each function it appears in
>> fs/ntfs3/frecord.c:2278:2: error: implicit declaration of function 'vunmap'; did you mean 'kunmap'? [-Werror=implicit-function-declaration]
    2278 |  vunmap(frame_mem);
         |  ^~~~~~
         |  kunmap
   fs/ntfs3/frecord.c: In function 'ni_write_frame':
   fs/ntfs3/frecord.c:2375:51: error: 'VM_MAP' undeclared (first use in this function); did you mean 'VM_MTE'?
    2375 |  frame_ondisk = vmap(pages_disk, pages_per_frame, VM_MAP, PAGE_KERNEL);
         |                                                   ^~~~~~
         |                                                   VM_MTE
   cc1: some warnings being treated as errors

vim +2247 fs/ntfs3/frecord.c

8f609ab22280366 Konstantin Komarov 2020-10-30  2099  
8f609ab22280366 Konstantin Komarov 2020-10-30  2100  /*
8f609ab22280366 Konstantin Komarov 2020-10-30  2101   * ni_read_frame
8f609ab22280366 Konstantin Komarov 2020-10-30  2102   *
8f609ab22280366 Konstantin Komarov 2020-10-30  2103   * pages - array of locked pages
8f609ab22280366 Konstantin Komarov 2020-10-30  2104   */
8f609ab22280366 Konstantin Komarov 2020-10-30  2105  int ni_read_frame(struct ntfs_inode *ni, u64 frame_vbo, struct page **pages,
8f609ab22280366 Konstantin Komarov 2020-10-30  2106  		  u32 pages_per_frame)
8f609ab22280366 Konstantin Komarov 2020-10-30  2107  {
8f609ab22280366 Konstantin Komarov 2020-10-30  2108  	int err;
8f609ab22280366 Konstantin Komarov 2020-10-30  2109  	struct ntfs_sb_info *sbi = ni->mi.sbi;
8f609ab22280366 Konstantin Komarov 2020-10-30  2110  	char *frame_ondisk = NULL;
8f609ab22280366 Konstantin Komarov 2020-10-30  2111  	char *frame_mem = NULL;
8f609ab22280366 Konstantin Komarov 2020-10-30  2112  	struct page **pages_disk = NULL;
8f609ab22280366 Konstantin Komarov 2020-10-30  2113  	u64 valid_size = ni->i_valid;
8f609ab22280366 Konstantin Komarov 2020-10-30  2114  	u32 frame_size;
8f609ab22280366 Konstantin Komarov 2020-10-30  2115  	size_t unc_size;
8f609ab22280366 Konstantin Komarov 2020-10-30  2116  	u32 ondisk_size, i, npages_disk;
8f609ab22280366 Konstantin Komarov 2020-10-30  2117  	struct page *pg;
8f609ab22280366 Konstantin Komarov 2020-10-30  2118  	struct ATTRIB *attr;
8f609ab22280366 Konstantin Komarov 2020-10-30  2119  	CLST frame, clst_data;
8f609ab22280366 Konstantin Komarov 2020-10-30  2120  
8f609ab22280366 Konstantin Komarov 2020-10-30  2121  	attr = ni_find_attr(ni, NULL, NULL, ATTR_DATA, NULL, 0, NULL, NULL);
8f609ab22280366 Konstantin Komarov 2020-10-30  2122  	if (!attr) {
8f609ab22280366 Konstantin Komarov 2020-10-30  2123  		err = -ENOENT;
8f609ab22280366 Konstantin Komarov 2020-10-30  2124  		goto out;
8f609ab22280366 Konstantin Komarov 2020-10-30  2125  	}
8f609ab22280366 Konstantin Komarov 2020-10-30  2126  
8f609ab22280366 Konstantin Komarov 2020-10-30  2127  	if (!attr->non_res) {
8f609ab22280366 Konstantin Komarov 2020-10-30  2128  		u32 data_size = le32_to_cpu(attr->res.data_size);
8f609ab22280366 Konstantin Komarov 2020-10-30  2129  		void *kaddr = kmap_atomic(pages[0]);
8f609ab22280366 Konstantin Komarov 2020-10-30  2130  
8f609ab22280366 Konstantin Komarov 2020-10-30  2131  		memset(kaddr, 0, PAGE_SIZE);
8f609ab22280366 Konstantin Komarov 2020-10-30  2132  		if (frame_vbo < data_size) {
8f609ab22280366 Konstantin Komarov 2020-10-30  2133  			err = data_size - frame_vbo;
8f609ab22280366 Konstantin Komarov 2020-10-30  2134  			if (err > PAGE_SIZE)
8f609ab22280366 Konstantin Komarov 2020-10-30  2135  				err = PAGE_SIZE;
8f609ab22280366 Konstantin Komarov 2020-10-30  2136  			memcpy(kaddr, resident_data(attr) + frame_vbo, err);
8f609ab22280366 Konstantin Komarov 2020-10-30  2137  		}
8f609ab22280366 Konstantin Komarov 2020-10-30  2138  		flush_dcache_page(pages[0]);
8f609ab22280366 Konstantin Komarov 2020-10-30  2139  		kunmap_atomic(kaddr);
8f609ab22280366 Konstantin Komarov 2020-10-30  2140  		err = 0;
8f609ab22280366 Konstantin Komarov 2020-10-30  2141  		goto out;
8f609ab22280366 Konstantin Komarov 2020-10-30  2142  	}
8f609ab22280366 Konstantin Komarov 2020-10-30  2143  
8f609ab22280366 Konstantin Komarov 2020-10-30  2144  	if (frame_vbo >= valid_size) {
8f609ab22280366 Konstantin Komarov 2020-10-30  2145  all_zero:
8f609ab22280366 Konstantin Komarov 2020-10-30  2146  		for (i = 0; i < pages_per_frame; i++)
8f609ab22280366 Konstantin Komarov 2020-10-30  2147  			zero_user_segment(pages[i], 0, PAGE_SIZE);
8f609ab22280366 Konstantin Komarov 2020-10-30  2148  		err = 0;
8f609ab22280366 Konstantin Komarov 2020-10-30  2149  		goto out;
8f609ab22280366 Konstantin Komarov 2020-10-30  2150  	}
8f609ab22280366 Konstantin Komarov 2020-10-30  2151  
8f609ab22280366 Konstantin Komarov 2020-10-30  2152  	if (ni->ni_flags & NI_FLAG_COMPRESSED_MASK) {
8f609ab22280366 Konstantin Komarov 2020-10-30  2153  		u32 cmpr = ((ni->ni_flags & NI_FLAG_COMPRESSED_MASK) >> 8) - 1;
8f609ab22280366 Konstantin Komarov 2020-10-30  2154  
8f609ab22280366 Konstantin Komarov 2020-10-30  2155  		switch (cmpr) {
8f609ab22280366 Konstantin Komarov 2020-10-30  2156  		case WOF_COMPRESSION_XPRESS4K:
8f609ab22280366 Konstantin Komarov 2020-10-30  2157  			/* xpress 4k*/
8f609ab22280366 Konstantin Komarov 2020-10-30  2158  			frame_size = 0x1000;
8f609ab22280366 Konstantin Komarov 2020-10-30  2159  			break;
8f609ab22280366 Konstantin Komarov 2020-10-30  2160  		case WOF_COMPRESSION_LZX:
8f609ab22280366 Konstantin Komarov 2020-10-30  2161  			/* lzx 32k*/
8f609ab22280366 Konstantin Komarov 2020-10-30  2162  			frame_size = 0x8000;
8f609ab22280366 Konstantin Komarov 2020-10-30  2163  			break;
8f609ab22280366 Konstantin Komarov 2020-10-30  2164  		case WOF_COMPRESSION_XPRESS8K:
8f609ab22280366 Konstantin Komarov 2020-10-30  2165  			/* xpress 8k*/
8f609ab22280366 Konstantin Komarov 2020-10-30  2166  			frame_size = 0x2000;
8f609ab22280366 Konstantin Komarov 2020-10-30  2167  			break;
8f609ab22280366 Konstantin Komarov 2020-10-30  2168  		case WOF_COMPRESSION_XPRESS16K:
8f609ab22280366 Konstantin Komarov 2020-10-30  2169  			/* xpress 16k*/
8f609ab22280366 Konstantin Komarov 2020-10-30  2170  			frame_size = 0x4000;
8f609ab22280366 Konstantin Komarov 2020-10-30  2171  			break;
8f609ab22280366 Konstantin Komarov 2020-10-30  2172  		}
8f609ab22280366 Konstantin Komarov 2020-10-30  2173  		/* TODO: port lzx/xpress */
8f609ab22280366 Konstantin Komarov 2020-10-30  2174  		err = -EOPNOTSUPP;
8f609ab22280366 Konstantin Komarov 2020-10-30  2175  		goto out;
8f609ab22280366 Konstantin Komarov 2020-10-30  2176  	} else if (is_attr_compressed(attr)) {
8f609ab22280366 Konstantin Komarov 2020-10-30  2177  		/* lznt compression*/
8f609ab22280366 Konstantin Komarov 2020-10-30  2178  		if (sbi->cluster_size > NTFS_LZNT_MAX_CLUSTER) {
8f609ab22280366 Konstantin Komarov 2020-10-30  2179  			err = -EOPNOTSUPP;
8f609ab22280366 Konstantin Komarov 2020-10-30  2180  			goto out;
8f609ab22280366 Konstantin Komarov 2020-10-30  2181  		}
8f609ab22280366 Konstantin Komarov 2020-10-30  2182  
8f609ab22280366 Konstantin Komarov 2020-10-30  2183  		if (attr->nres.c_unit != NTFS_LZNT_CUNIT) {
8f609ab22280366 Konstantin Komarov 2020-10-30  2184  			err = -EOPNOTSUPP;
8f609ab22280366 Konstantin Komarov 2020-10-30  2185  			goto out;
8f609ab22280366 Konstantin Komarov 2020-10-30  2186  		}
8f609ab22280366 Konstantin Komarov 2020-10-30  2187  
8f609ab22280366 Konstantin Komarov 2020-10-30  2188  		down_write(&ni->file.run_lock);
8f609ab22280366 Konstantin Komarov 2020-10-30  2189  		run_truncate_around(&ni->file.run,
8f609ab22280366 Konstantin Komarov 2020-10-30  2190  				    le64_to_cpu(attr->nres.svcn));
8f609ab22280366 Konstantin Komarov 2020-10-30  2191  		frame = frame_vbo >> (sbi->cluster_bits + NTFS_LZNT_CUNIT);
8f609ab22280366 Konstantin Komarov 2020-10-30  2192  		err = attr_is_frame_compressed(ni, attr, frame, &clst_data);
8f609ab22280366 Konstantin Komarov 2020-10-30  2193  		up_write(&ni->file.run_lock);
8f609ab22280366 Konstantin Komarov 2020-10-30  2194  		if (err)
8f609ab22280366 Konstantin Komarov 2020-10-30  2195  			goto out;
8f609ab22280366 Konstantin Komarov 2020-10-30  2196  
8f609ab22280366 Konstantin Komarov 2020-10-30  2197  		if (!clst_data)
8f609ab22280366 Konstantin Komarov 2020-10-30  2198  			goto all_zero;
8f609ab22280366 Konstantin Komarov 2020-10-30  2199  
8f609ab22280366 Konstantin Komarov 2020-10-30  2200  		frame_size = sbi->cluster_size << NTFS_LZNT_CUNIT;
8f609ab22280366 Konstantin Komarov 2020-10-30  2201  		ondisk_size = clst_data << sbi->cluster_bits;
8f609ab22280366 Konstantin Komarov 2020-10-30  2202  
8f609ab22280366 Konstantin Komarov 2020-10-30  2203  		if (clst_data >= NTFS_LZNT_CLUSTERS) {
8f609ab22280366 Konstantin Komarov 2020-10-30  2204  			/* frame is not compressed */
8f609ab22280366 Konstantin Komarov 2020-10-30  2205  			down_read(&ni->file.run_lock);
8f609ab22280366 Konstantin Komarov 2020-10-30  2206  			err = ntfs_bio_pages(sbi, &ni->file.run, pages,
8f609ab22280366 Konstantin Komarov 2020-10-30  2207  					     pages_per_frame, frame_vbo,
8f609ab22280366 Konstantin Komarov 2020-10-30  2208  					     ondisk_size, REQ_OP_READ);
8f609ab22280366 Konstantin Komarov 2020-10-30  2209  			up_read(&ni->file.run_lock);
8f609ab22280366 Konstantin Komarov 2020-10-30  2210  			goto out;
8f609ab22280366 Konstantin Komarov 2020-10-30  2211  		}
8f609ab22280366 Konstantin Komarov 2020-10-30  2212  	} else {
8f609ab22280366 Konstantin Komarov 2020-10-30  2213  		__builtin_unreachable();
8f609ab22280366 Konstantin Komarov 2020-10-30  2214  		err = -EINVAL;
8f609ab22280366 Konstantin Komarov 2020-10-30  2215  		goto out;
8f609ab22280366 Konstantin Komarov 2020-10-30  2216  	}
8f609ab22280366 Konstantin Komarov 2020-10-30  2217  
8f609ab22280366 Konstantin Komarov 2020-10-30  2218  	npages_disk = (ondisk_size + PAGE_SIZE - 1) >> PAGE_SHIFT;
8f609ab22280366 Konstantin Komarov 2020-10-30  2219  	pages_disk = ntfs_alloc(npages_disk * sizeof(struct page *), 1);
8f609ab22280366 Konstantin Komarov 2020-10-30  2220  	if (!pages_disk) {
8f609ab22280366 Konstantin Komarov 2020-10-30  2221  		err = -ENOMEM;
8f609ab22280366 Konstantin Komarov 2020-10-30  2222  		goto out;
8f609ab22280366 Konstantin Komarov 2020-10-30  2223  	}
8f609ab22280366 Konstantin Komarov 2020-10-30  2224  
8f609ab22280366 Konstantin Komarov 2020-10-30  2225  	for (i = 0; i < npages_disk; i++) {
8f609ab22280366 Konstantin Komarov 2020-10-30  2226  		pg = alloc_page(GFP_KERNEL);
8f609ab22280366 Konstantin Komarov 2020-10-30  2227  		if (!pg) {
8f609ab22280366 Konstantin Komarov 2020-10-30  2228  			err = -ENOMEM;
8f609ab22280366 Konstantin Komarov 2020-10-30  2229  			goto out1;
8f609ab22280366 Konstantin Komarov 2020-10-30  2230  		}
8f609ab22280366 Konstantin Komarov 2020-10-30  2231  		pages_disk[i] = pg;
8f609ab22280366 Konstantin Komarov 2020-10-30  2232  		lock_page(pg);
8f609ab22280366 Konstantin Komarov 2020-10-30  2233  		kmap(pg);
8f609ab22280366 Konstantin Komarov 2020-10-30  2234  	}
8f609ab22280366 Konstantin Komarov 2020-10-30  2235  
8f609ab22280366 Konstantin Komarov 2020-10-30  2236  	/* read 'ondisk_size' bytes from disk */
8f609ab22280366 Konstantin Komarov 2020-10-30  2237  	down_read(&ni->file.run_lock);
8f609ab22280366 Konstantin Komarov 2020-10-30  2238  	err = ntfs_bio_pages(sbi, &ni->file.run, pages_disk, npages_disk,
8f609ab22280366 Konstantin Komarov 2020-10-30  2239  			     frame_vbo, ondisk_size, REQ_OP_READ);
8f609ab22280366 Konstantin Komarov 2020-10-30  2240  	up_read(&ni->file.run_lock);
8f609ab22280366 Konstantin Komarov 2020-10-30  2241  	if (err)
8f609ab22280366 Konstantin Komarov 2020-10-30  2242  		goto out1;
8f609ab22280366 Konstantin Komarov 2020-10-30  2243  
8f609ab22280366 Konstantin Komarov 2020-10-30  2244  	/*
8f609ab22280366 Konstantin Komarov 2020-10-30  2245  	 * To simplify decompress algorithm do vmap for source and target pages
8f609ab22280366 Konstantin Komarov 2020-10-30  2246  	 */
8f609ab22280366 Konstantin Komarov 2020-10-30 @2247  	frame_ondisk = vmap(pages_disk, npages_disk, VM_MAP, PAGE_KERNEL);
8f609ab22280366 Konstantin Komarov 2020-10-30  2248  	if (!frame_ondisk) {
8f609ab22280366 Konstantin Komarov 2020-10-30  2249  		err = -ENOMEM;
8f609ab22280366 Konstantin Komarov 2020-10-30  2250  		goto out1;
8f609ab22280366 Konstantin Komarov 2020-10-30  2251  	}
8f609ab22280366 Konstantin Komarov 2020-10-30  2252  
8f609ab22280366 Konstantin Komarov 2020-10-30  2253  	for (i = 0; i < pages_per_frame; i++)
8f609ab22280366 Konstantin Komarov 2020-10-30  2254  		kmap(pages[i]);
8f609ab22280366 Konstantin Komarov 2020-10-30  2255  
8f609ab22280366 Konstantin Komarov 2020-10-30  2256  	frame_mem = vmap(pages, pages_per_frame, VM_MAP, PAGE_KERNEL);
8f609ab22280366 Konstantin Komarov 2020-10-30  2257  	if (!frame_mem) {
8f609ab22280366 Konstantin Komarov 2020-10-30  2258  		err = -ENOMEM;
8f609ab22280366 Konstantin Komarov 2020-10-30  2259  		goto out2;
8f609ab22280366 Konstantin Komarov 2020-10-30  2260  	}
8f609ab22280366 Konstantin Komarov 2020-10-30  2261  
8f609ab22280366 Konstantin Komarov 2020-10-30  2262  	/* decompress: frame_ondisk -> frame_mem */
8f609ab22280366 Konstantin Komarov 2020-10-30  2263  	unc_size = decompress_lznt(frame_ondisk, ondisk_size, frame_mem,
8f609ab22280366 Konstantin Komarov 2020-10-30  2264  				   frame_size);
8f609ab22280366 Konstantin Komarov 2020-10-30  2265  
8f609ab22280366 Konstantin Komarov 2020-10-30  2266  	if ((ssize_t)unc_size < 0) {
8f609ab22280366 Konstantin Komarov 2020-10-30  2267  		err = unc_size;
8f609ab22280366 Konstantin Komarov 2020-10-30  2268  	} else if (!unc_size || unc_size > frame_size) {
8f609ab22280366 Konstantin Komarov 2020-10-30  2269  		err = -EINVAL;
8f609ab22280366 Konstantin Komarov 2020-10-30  2270  	} else {
8f609ab22280366 Konstantin Komarov 2020-10-30  2271  		if (valid_size < frame_vbo + frame_size) {
8f609ab22280366 Konstantin Komarov 2020-10-30  2272  			size_t ok = valid_size - frame_vbo;
8f609ab22280366 Konstantin Komarov 2020-10-30  2273  
8f609ab22280366 Konstantin Komarov 2020-10-30  2274  			memset(frame_mem + ok, 0, frame_size - ok);
8f609ab22280366 Konstantin Komarov 2020-10-30  2275  		}
8f609ab22280366 Konstantin Komarov 2020-10-30  2276  	}
8f609ab22280366 Konstantin Komarov 2020-10-30  2277  
8f609ab22280366 Konstantin Komarov 2020-10-30 @2278  	vunmap(frame_mem);
8f609ab22280366 Konstantin Komarov 2020-10-30  2279  out2:
8f609ab22280366 Konstantin Komarov 2020-10-30  2280  	for (i = 0; i < pages_per_frame; i++) {
8f609ab22280366 Konstantin Komarov 2020-10-30  2281  		pg = pages[i];
8f609ab22280366 Konstantin Komarov 2020-10-30  2282  		kunmap(pg);
8f609ab22280366 Konstantin Komarov 2020-10-30  2283  	}
8f609ab22280366 Konstantin Komarov 2020-10-30  2284  	vunmap(frame_ondisk);
8f609ab22280366 Konstantin Komarov 2020-10-30  2285  out1:
8f609ab22280366 Konstantin Komarov 2020-10-30  2286  	for (i = 0; i < npages_disk; i++) {
8f609ab22280366 Konstantin Komarov 2020-10-30  2287  		pg = pages_disk[i];
8f609ab22280366 Konstantin Komarov 2020-10-30  2288  		if (pg) {
8f609ab22280366 Konstantin Komarov 2020-10-30  2289  			kunmap(pg);
8f609ab22280366 Konstantin Komarov 2020-10-30  2290  			unlock_page(pg);
8f609ab22280366 Konstantin Komarov 2020-10-30  2291  			put_page(pg);
8f609ab22280366 Konstantin Komarov 2020-10-30  2292  		}
8f609ab22280366 Konstantin Komarov 2020-10-30  2293  	}
8f609ab22280366 Konstantin Komarov 2020-10-30  2294  	ntfs_free(pages_disk);
8f609ab22280366 Konstantin Komarov 2020-10-30  2295  out:
8f609ab22280366 Konstantin Komarov 2020-10-30  2296  	for (i = 0; i < pages_per_frame; i++) {
8f609ab22280366 Konstantin Komarov 2020-10-30  2297  		pg = pages[i];
8f609ab22280366 Konstantin Komarov 2020-10-30  2298  		ClearPageError(pg);
8f609ab22280366 Konstantin Komarov 2020-10-30  2299  		SetPageUptodate(pg);
8f609ab22280366 Konstantin Komarov 2020-10-30  2300  	}
8f609ab22280366 Konstantin Komarov 2020-10-30  2301  
8f609ab22280366 Konstantin Komarov 2020-10-30  2302  	return err;
8f609ab22280366 Konstantin Komarov 2020-10-30  2303  }
8f609ab22280366 Konstantin Komarov 2020-10-30  2304  

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org

[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 66534 bytes --]

  parent reply	other threads:[~2020-11-03  3:06 UTC|newest]

Thread overview: 26+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-10-30 15:02 [PATCH v11 00/10] NTFS read-write driver GPL implementation by Paragon Software Konstantin Komarov
2020-10-30 15:02 ` [PATCH v11 01/10] fs/ntfs3: Add headers and misc files Konstantin Komarov
2020-10-30 15:02 ` [PATCH v11 02/10] fs/ntfs3: Add initialization of super block Konstantin Komarov
2020-10-30 15:02 ` [PATCH v11 03/10] fs/ntfs3: Add bitmap Konstantin Komarov
2020-10-30 15:02 ` [PATCH v11 04/10] fs/ntfs3: Add file operations and implementation Konstantin Komarov
2020-10-30 15:02 ` [PATCH v11 05/10] fs/ntfs3: Add attrib operations Konstantin Komarov
2020-10-30 15:02 ` [PATCH v11 06/10] fs/ntfs3: Add compression Konstantin Komarov
2020-10-30 15:02 ` [PATCH v11 07/10] fs/ntfs3: Add NTFS journal Konstantin Komarov
2020-10-30 15:02 ` [PATCH v11 08/10] fs/ntfs3: Add Kconfig, Makefile and doc Konstantin Komarov
2020-10-30 15:02 ` [PATCH v11 09/10] fs/ntfs3: Add NTFS3 in fs/Kconfig and fs/Makefile Konstantin Komarov
2020-10-31  1:23   ` kernel test robot
2020-10-31  1:23     ` kernel test robot
2020-11-02  8:36   ` Dan Carpenter
2020-11-02  8:36     ` [kbuild] " Dan Carpenter
2020-11-02  8:36     ` Dan Carpenter
2020-11-03  3:06   ` kernel test robot [this message]
2020-11-03  3:06     ` kernel test robot
2020-10-30 15:02 ` [PATCH v11 10/10] fs/ntfs3: Add MAINTAINERS Konstantin Komarov
2020-10-30 15:24 ` [PATCH v11 00/10] NTFS read-write driver GPL implementation by Paragon Software Pali Rohár
2020-10-30 15:51   ` Konstantin Komarov
2020-10-30 16:24     ` Pali Rohár
2020-10-30 16:41 ` Pali Rohár
2020-10-31  8:51   ` Christoph Hellwig
2020-10-31  2:42 ` Eric Biggers
2020-10-31  2:48   ` Eric Biggers
  -- strict thread matches above, loose matches on Subject: below --
2020-11-02  8:00 [PATCH v11 09/10] fs/ntfs3: Add NTFS3 in fs/Kconfig and fs/Makefile kernel test robot

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=202011031109.d2r2hcCI-lkp@intel.com \
    --to=lkp@intel.com \
    --cc=kbuild-all@lists.01.org \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.