linux-kernel - RE: [f2fs-dev] [PATCH 0/8] some cleanup of inline flag checking

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-id: <00fe01d17b57$30b6e9c0$9224bd40$@samsung.com>
Date:	Fri, 11 Mar 2016 13:29:57 +0800
From:	Chao Yu <chao2.yu@...sung.com>
To:	'Shawn Lin' <shawn.lin@...k-chips.com>,
	'Jaegeuk Kim' <jaegeuk@...nel.org>
Cc:	linux-kernel@...r.kernel.org,
	linux-f2fs-devel@...ts.sourceforge.net
Subject: RE: [f2fs-dev] [PATCH 0/8] some cleanup of inline flag checking

Hi Shawn,

> -----Original Message-----
> From: Shawn Lin [mailto:shawn.lin@...k-chips.com]
> Sent: Friday, March 11, 2016 11:28 AM
> To: Jaegeuk Kim
> Cc: Shawn Lin; linux-kernel@...r.kernel.org; linux-f2fs-devel@...ts.sourceforge.net
> Subject: [f2fs-dev] [PATCH 0/8] some cleanup of inline flag checking
> 
> 
> This patchset is going to remove some redunant checking
> of inline data flag and also going to avoid some unnecessary
> cpu waste when doing inline stuff.

When we are accessing inline inode, inline inode conversion can happen
concurrently, we should check inline flag again under inode page's lock
to avoid accessing the wrong inline data which may have been converted.

Thanks,

> 
> Note:
> Sorry for sending previous four patches in separate, let
> drop them and make them in this thread for better review.
> 
> 
> 
> Shawn Lin (8):
>   f2fs: check inline flag ahead for f2fs_write_inline_data
>   f2fs: remove checing inline data flag for f2fs_write_data_page
>   f2fs: check inline flag ahead for f2fs_read_inline_data
>   f2fs: remove redundant checking of inline data flag
>   f2fs: f2fs: check inline flag ahead for f2fs_inline_data_fiemap
>   f2fs: remove checing inline data flag for f2fs_fiemap
>   f2fs: remove unnecessary inline checking for f2fs_convert_inline_inode
>   f2fs: check inline flag ahead for get_dnode_of_data
> 
>  fs/f2fs/data.c   | 17 +++++++----------
>  fs/f2fs/inline.c | 27 ++++++++++-----------------
>  fs/f2fs/node.c   | 12 +++++-------
>  3 files changed, 22 insertions(+), 34 deletions(-)
> 
> --
> 2.3.7
> 
> 
> 
> ------------------------------------------------------------------------------
> Transform Data into Opportunity.
> Accelerate data analysis in your applications with
> Intel Data Analytics Acceleration Library.
> Click to learn more.
> http://pubads.g.doubleclick.net/gampad/clk?id=278785111&iu=/4140
> _______________________________________________
> Linux-f2fs-devel mailing list
> Linux-f2fs-devel@...ts.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/linux-f2fs-devel