From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755922AbdDGKC0 (ORCPT ); Fri, 7 Apr 2017 06:02:26 -0400 Received: from mail.fireflyinternet.com ([109.228.58.192]:57319 "EHLO fireflyinternet.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1753246AbdDGKCS (ORCPT ); Fri, 7 Apr 2017 06:02:18 -0400 X-Default-Received-SPF: pass (skip=forwardok (res=PASS)) x-ip-name=78.156.65.138; Date: Fri, 7 Apr 2017 11:02:11 +0100 From: Chris Wilson To: Andrea Arcangeli Cc: Martin Kepplinger , Thorsten Leemhuis , daniel.vetter@intel.com, Dave Airlie , intel-gfx@lists.freedesktop.org, linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org Subject: Re: [PATCH 2/5] i915: flush gem obj freeing workqueues to add accuracy to the i915 shrinker Message-ID: <20170407100211.GG10496@nuc-i3427.alporthouse.com> Mail-Followup-To: Chris Wilson , Andrea Arcangeli , Martin Kepplinger , Thorsten Leemhuis , daniel.vetter@intel.com, Dave Airlie , intel-gfx@lists.freedesktop.org, linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org References: <87pogtplxr.fsf@intel.com> <20170406232347.988-1-aarcange@redhat.com> <20170406232347.988-3-aarcange@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20170406232347.988-3-aarcange@redhat.com> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, Apr 07, 2017 at 01:23:44AM +0200, Andrea Arcangeli wrote: > Waiting a RCU grace period only guarantees the work gets queued, but > until after the queued workqueue returns, there's no guarantee the > memory was actually freed. So flush the work to provide better > guarantees to the reclaim code in addition of waiting a RCU grace > period to pass. We are not allowed to call flush_work() from the shrinker, the workqueue doesn't have and can't have the right reclaim flags. -Chris -- Chris Wilson, Intel Open Source Technology Centre From mboxrd@z Thu Jan 1 00:00:00 1970 From: Chris Wilson Subject: Re: [PATCH 2/5] i915: flush gem obj freeing workqueues to add accuracy to the i915 shrinker Date: Fri, 7 Apr 2017 11:02:11 +0100 Message-ID: <20170407100211.GG10496@nuc-i3427.alporthouse.com> References: <87pogtplxr.fsf@intel.com> <20170406232347.988-1-aarcange@redhat.com> <20170406232347.988-3-aarcange@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: Content-Disposition: inline In-Reply-To: <20170406232347.988-3-aarcange@redhat.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" To: Andrea Arcangeli Cc: intel-gfx@lists.freedesktop.org, linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org, Thorsten Leemhuis , Martin Kepplinger , daniel.vetter@intel.com List-Id: dri-devel@lists.freedesktop.org T24gRnJpLCBBcHIgMDcsIDIwMTcgYXQgMDE6MjM6NDRBTSArMDIwMCwgQW5kcmVhIEFyY2FuZ2Vs aSB3cm90ZToKPiBXYWl0aW5nIGEgUkNVIGdyYWNlIHBlcmlvZCBvbmx5IGd1YXJhbnRlZXMgdGhl IHdvcmsgZ2V0cyBxdWV1ZWQsIGJ1dAo+IHVudGlsIGFmdGVyIHRoZSBxdWV1ZWQgd29ya3F1ZXVl IHJldHVybnMsIHRoZXJlJ3Mgbm8gZ3VhcmFudGVlIHRoZQo+IG1lbW9yeSB3YXMgYWN0dWFsbHkg ZnJlZWQuIFNvIGZsdXNoIHRoZSB3b3JrIHRvIHByb3ZpZGUgYmV0dGVyCj4gZ3VhcmFudGVlcyB0 byB0aGUgcmVjbGFpbSBjb2RlIGluIGFkZGl0aW9uIG9mIHdhaXRpbmcgYSBSQ1UgZ3JhY2UKPiBw ZXJpb2QgdG8gcGFzcy4KCldlIGFyZSBub3QgYWxsb3dlZCB0byBjYWxsIGZsdXNoX3dvcmsoKSBm cm9tIHRoZSBzaHJpbmtlciwgdGhlIHdvcmtxdWV1ZQpkb2Vzbid0IGhhdmUgYW5kIGNhbid0IGhh dmUgdGhlIHJpZ2h0IHJlY2xhaW0gZmxhZ3MuCi1DaHJpcwoKLS0gCkNocmlzIFdpbHNvbiwgSW50 ZWwgT3BlbiBTb3VyY2UgVGVjaG5vbG9neSBDZW50cmUKX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX18KSW50ZWwtZ2Z4IG1haWxpbmcgbGlzdApJbnRlbC1nZnhA bGlzdHMuZnJlZWRlc2t0b3Aub3JnCmh0dHBzOi8vbGlzdHMuZnJlZWRlc2t0b3Aub3JnL21haWxt YW4vbGlzdGluZm8vaW50ZWwtZ2Z4Cg==