Re: [patch 1/3] mm: protect set_page_dirty() from ongoing truncation

2017-04-10 Thread Jan Kara
On Mon 10-04-17 15:07:58, alexander.le...@verizon.com wrote: > On Mon, Apr 10, 2017 at 02:06:38PM +0200, Jan Kara wrote: > > On Mon 10-04-17 02:22:33, alexander.le...@verizon.com wrote: > > > On Fri, Dec 05, 2014 at 09:52:44AM -0500, Johannes Weiner wrote: > > > > Tejun, while reviewing the code, s

Re: [patch 1/3] mm: protect set_page_dirty() from ongoing truncation

2017-04-10 Thread alexander . levin
On Mon, Apr 10, 2017 at 02:06:38PM +0200, Jan Kara wrote: > On Mon 10-04-17 02:22:33, alexander.le...@verizon.com wrote: > > On Fri, Dec 05, 2014 at 09:52:44AM -0500, Johannes Weiner wrote: > > > Tejun, while reviewing the code, spotted the following race condition > > > between the dirtying and tr

Re: [patch 1/3] mm: protect set_page_dirty() from ongoing truncation

2017-04-10 Thread Jan Kara
On Mon 10-04-17 02:22:33, alexander.le...@verizon.com wrote: > On Fri, Dec 05, 2014 at 09:52:44AM -0500, Johannes Weiner wrote: > > Tejun, while reviewing the code, spotted the following race condition > > between the dirtying and truncation of a page: > > > > __set_page_dirty_nobuffers() __

Re: [patch 1/3] mm: protect set_page_dirty() from ongoing truncation

2017-04-09 Thread alexander . levin
On Fri, Dec 05, 2014 at 09:52:44AM -0500, Johannes Weiner wrote: > Tejun, while reviewing the code, spotted the following race condition > between the dirtying and truncation of a page: > > __set_page_dirty_nobuffers() __delete_from_page_cache() > if (TestSetPageDirty(page)) >

Re: [patch 1/3] mm: protect set_page_dirty() from ongoing truncation

2014-12-09 Thread Jan Kara
On Fri 05-12-14 09:52:44, Johannes Weiner wrote: > Tejun, while reviewing the code, spotted the following race condition > between the dirtying and truncation of a page: > > __set_page_dirty_nobuffers() __delete_from_page_cache() > if (TestSetPageDirty(page)) >

Re: [patch 1/3] mm: protect set_page_dirty() from ongoing truncation

2014-12-02 Thread Johannes Weiner
On Tue, Dec 02, 2014 at 01:56:52PM +0200, Kirill A. Shutemov wrote: > On Mon, Dec 01, 2014 at 05:58:00PM -0500, Johannes Weiner wrote: > > Tejun, while reviewing the code, spotted the following race condition > > between the dirtying and truncation of a page: > > > > __set_page_dirty_nobuffers()

Re: [patch 1/3] mm: protect set_page_dirty() from ongoing truncation

2014-12-02 Thread Johannes Weiner
On Tue, Dec 02, 2014 at 10:12:12AM +0100, Jan Kara wrote: > On Mon 01-12-14 17:58:00, Johannes Weiner wrote: > > Tejun, while reviewing the code, spotted the following race condition > > between the dirtying and truncation of a page: > > > > __set_page_dirty_nobuffers() __delete_from_page_ca

Re: [patch 1/3] mm: protect set_page_dirty() from ongoing truncation

2014-12-02 Thread Kirill A. Shutemov
On Mon, Dec 01, 2014 at 05:58:00PM -0500, Johannes Weiner wrote: > Tejun, while reviewing the code, spotted the following race condition > between the dirtying and truncation of a page: > > __set_page_dirty_nobuffers() __delete_from_page_cache() > if (TestSetPageDirty(page)) >

Re: [patch 1/3] mm: protect set_page_dirty() from ongoing truncation

2014-12-02 Thread Jan Kara
On Mon 01-12-14 17:58:00, Johannes Weiner wrote: > Tejun, while reviewing the code, spotted the following race condition > between the dirtying and truncation of a page: > > __set_page_dirty_nobuffers() __delete_from_page_cache() > if (TestSetPageDirty(page)) >