From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Date: Mon, 27 Jan 2014 05:19:50 -0800 From: Greg KH To: Weijie Yang Cc: hughd@google.com, 'Andrew Morton' , 'Minchan Kim' , shli@kernel.org, 'Bob Liu' , weijie.yang.kh@gmail.com, 'Seth Jennings' , 'Linux-MM' , 'linux-kernel' , stable@vger.kernel.org, 'Heesub Shin' , mguzik@redhat.com Subject: Re: [PATCH 0/8] mm/swap: fix some rare issues in swap subsystem Message-ID: <20140127131950.GD16027@kroah.com> References: <000501cf1b46$b899edb0$29cdc910$%yang@samsung.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <000501cf1b46$b899edb0$29cdc910$%yang@samsung.com> Sender: linux-kernel-owner@vger.kernel.org List-ID: On Mon, Jan 27, 2014 at 06:00:03PM +0800, Weijie Yang wrote: > This patch series focus on some tiny and rare issues in swap subsystem. > These issues happen rarely, so it is just for the correctness of the code. > > It firstly add some comments to try to make swap flag/lock usage in > swapfile.c more clear and readable, > and fix some rare issues in swap subsystem that cause race condition among > swapon, swapoff and frontswap_register_ops. > and fix some not race issues. > > Please see individual patch for details, any complaint and suggestion > are welcome. > > Regards > > patch 1/8: add some comments for swap flag/lock usage > > patch 2/8: fix race on swap_info reuse between swapoff and swapon > This patch has been in akpm -mm tree, however I improve it according > to Heesub Shin and Mateusz Guzik's suggestion. So, that old patch need > to be dropped. > > patch 3/8: prevent concurrent swapon on the same S_ISBLK blockdev > > patch 4/8: fix race among frontswap_register_ops, swapoff and swapon > > patch 5/8: drop useless and bug frontswap_shrink codes > > patch 6/8: remove swap_lock to simplify si_swapinfo() > > patch 7/8: check swapfile blocksize greater than PAGE_SIZE > > patch 8/8: add missing handle on a dup-store failure > > include/linux/blkdev.h | 4 +++- > include/linux/frontswap.h | 2 -- > include/linux/swapfile.h | 4 +--- > mm/frontswap.c | 127 +++++++------------------------------------------------------------------------------------------------------------------------ > mm/page_io.c | 2 ++ > mm/rmap.c | 2 +- > mm/swapfile.c | 138 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++---------------------------------------- > 7 files changed, 112 insertions(+), 167 deletions(-) This is not the correct way to submit patches for inclusion in the stable kernel tree. Please read Documentation/stable_kernel_rules.txt for how to do this properly.