From mboxrd@z Thu Jan 1 00:00:00 1970 From: Mike Galbraith Subject: Re: kernel spew from nouveau/ swiotlb Date: Fri, 11 May 2018 09:13:50 +0200 Message-ID: <1526022830.6288.3.camel@gmx.de> References: <1525943427.8013.11.camel@gmx.de> <1525948096.29445.4.camel@gmx.de> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: In-Reply-To: <1525948096.29445.4.camel-Mmb7MZpHnFY@public.gmane.org> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: nouveau-bounces-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org Sender: "Nouveau" To: nouveau Cc: Christian Koenig , Christoph Hellwig , Roger He , LKML , Ben Skeggs List-Id: nouveau.vger.kernel.org T24gVGh1LCAyMDE4LTA1LTEwIGF0IDEyOjI4ICswMjAwLCBNaWtlIEdhbGJyYWl0aCB3cm90ZToK PiBPbiBUaHUsIDIwMTgtMDUtMTAgYXQgMTE6MTAgKzAyMDAsIE1pa2UgR2FsYnJhaXRoIHdyb3Rl Ogo+ID4gR3JlZXRpbmdzLAo+ID4gCj4gPiBXaGVuIGJveCBpcyBlYXJuaW5nIGl0cyBrZWVwLCBu b3V2ZWF1L3N3aW90bGIgZ3J1bWJsZS4uIGEgTE9ULiAgVGhlCj4gPiBiZWxvdyBpcyBmcm9tIG1h c3Rlci50b2RheS4KPiA+IAo+ID4gWzEyNTk0LjY0MDk1OV0gbm91dmVhdSAwMDAwOjAxOjAwLjA6 IHN3aW90bGIgYnVmZmVyIGlzIGZ1bGwgKHN6OiAyMDk3MTUyIGJ5dGVzKQo+ID4gWzEyNTk0LjY5 MzAwMF0gbm91dmVhdSAwMDAwOjAxOjAwLjA6IHN3aW90bGIgYnVmZmVyIGlzIGZ1bGwgKHN6OiAy MDk3MTUyIGJ5dGVzKQo+ID4gWzEyNTk0LjcxMzc4N10gbm91dmVhdSAwMDAwOjAxOjAwLjA6IHN3 aW90bGIgYnVmZmVyIGlzIGZ1bGwgKHN6OiAyMDk3MTUyIGJ5dGVzKQo+ID4gWzEyNTk0Ljc0MzQx M10gbm91dmVhdSAwMDAwOjAxOjAwLjA6IHN3aW90bGIgYnVmZmVyIGlzIGZ1bGwgKHN6OiAyMDk3 MTUyIGJ5dGVzKQo+ID4gWzEyNTk0Ljc5Njc0MF0gbm91dmVhdSAwMDAwOjAxOjAwLjA6IHN3aW90 bGIgYnVmZmVyIGlzIGZ1bGwgKHN6OiAyMDk3MTUyIGJ5dGVzKQo+ID4gWzEyNjA3LjAwMDc3NF0g c3dpb3RsYl90YmxfbWFwX3NpbmdsZTogNTQgY2FsbGJhY2tzIHN1cHByZXNzZWQKPiA+IFsxMjYw Ny4wMDA3NzZdIG5vdXZlYXUgMDAwMDowMTowMC4wOiBzd2lvdGxiIGJ1ZmZlciBpcyBmdWxsIChz ejogMjA5NzE1MiBieXRlcykKPiA+IFsxMjYwNy4zNDc5NDFdIG5vdXZlYXUgMDAwMDowMTowMC4w OiBzd2lvdGxiIGJ1ZmZlciBpcyBmdWxsIChzejogMjA5NzE1MiBieXRlcykKPiA+IFsxMjYwOC42 NzcwMzhdIG5vdXZlYXUgMDAwMDowMTowMC4wOiBzd2lvdGxiIGJ1ZmZlciBpcyBmdWxsIChzejog MjA5NzE1MiBieXRlcykKPiA+IGhvbWVyOi9ub3ZlbGwvc3NoICMgZG1lc2d8Z3JlcCAnc3dpb3Rs YiBidWZmZXIgaXMgZnVsbCd8d2MgLWwKPiA+IDIwNTIKPiA+IGhvbWVyOi9ub3ZlbGwvc3NoICMg ZG1lc2d8Z3JlcCAnY2FsbGJhY2tzIHN1cHByZXNzZWQnfHdjIC1sCj4gPiAxNzEKPiA+IAo+ID4g bGliL3N3aW90bGIuYzoKPiA+ICA1NzMgbm90X2ZvdW5kOgo+ID4gIDU3NCDCoMKgwqDCoMKgwqDC oMKgc3Bpbl91bmxvY2tfaXJxcmVzdG9yZSgmaW9fdGxiX2xvY2ssIGZsYWdzKTsKPiA+ICA1NzUg wqDCoMKgwqDCoMKgwqDCoGlmICghKGF0dHJzICYgRE1BX0FUVFJfTk9fV0FSTikgJiYgcHJpbnRr X3JhdGVsaW1pdCgpKQo+ID4gIDU3NiDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoGRl dl93YXJuKGh3ZGV2LCAic3dpb3RsYiBidWZmZXIgaXMgZnVsbCAoc3o6ICV6ZCBieXRlcylcbiIs IHNpemUpOwo+ID4gCj4gPiBEb2VzIG5vdXZlYXUgcGVyaGFwcyB3YW50IG9uZSBvZiB0aG9zZSBE TUFfQVRUUl9OT19XQVJOIHRoaW5naWVzPwo+IAo+IE9yIHNob3VsZCB0dG0gcGVyaGFwcyBhbHdh eXMgdXNlIHRoZSBvbmUgb24gaGFuZD8gIChzZWVtcyB0byB3b3JrKQoKTm8gaXQgZGlkbid0LCBJ IGp1c3QgZGlkbid0IHdhaXQgbG9uZyBlbm91Z2ggZm9yIHNwZXcgdG8gc3RhcnQuLi4KCj4gLS0t Cj4gIGRyaXZlcnMvZ3B1L2RybS90dG0vdHRtX3BhZ2VfYWxsb2NfZG1hLmMgfCAgICAyICstCj4g IDEgZmlsZSBjaGFuZ2VkLCAxIGluc2VydGlvbigrKSwgMSBkZWxldGlvbigtKQo+IAo+IC0tLSBh L2RyaXZlcnMvZ3B1L2RybS90dG0vdHRtX3BhZ2VfYWxsb2NfZG1hLmMKPiArKysgYi9kcml2ZXJz L2dwdS9kcm0vdHRtL3R0bV9wYWdlX2FsbG9jX2RtYS5jCj4gQEAgLTM0Miw3ICszNDIsNyBAQCBz dGF0aWMgc3RydWN0IGRtYV9wYWdlICpfX3R0bV9kbWFfYWxsb2NfCj4gIAlpZiAoIWRfcGFnZSkK PiAgCQlyZXR1cm4gTlVMTDsKPiAgCj4gLQlpZiAocG9vbC0+dHlwZSAmIElTX0hVR0UpCj4gKwlp ZiAoMSB8fCBwb29sLT50eXBlICYgSVNfSFVHRSkKPiAgCQlhdHRycyA9IERNQV9BVFRSX05PX1dB Uk47Cj4gIAo+ICAJdmFkZHIgPSBkbWFfYWxsb2NfYXR0cnMocG9vbC0+ZGV2LCBwb29sLT5zaXpl LCAmZF9wYWdlLT5kbWEsCgpXaGlsZSBJU19IVUdFIGlzIGluZGVlZCBmYWxzZSBvbiBteSBib3gs IGl0IGp1c3QgZG9lc24ndCBtYXR0ZXIsCmJlY2F1c2Ugd2hlbiB3ZSBnZXQgdG8gZWl0aGVyIHRo ZSBvbGQgb3IgdGhlIG5ldyBhbGxvYygpLCBpdCBjYWxscwpzd2lvdGxiX2FsbG9jX2J1ZmZlcigp LCB3aGljaCBkcm9wcyBhdHRycyBwYXNzZWQgdG8gaXQgb24gdGhlIGZsb29yLAptYWtpbmcgaXQg dW5saWtlbHkgdGhhdCBhbGxvYygpIGNhbGxlciB3aXNoZXMgYXJlIGdyYW50ZWQuCgoJLU1pa2UK X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18KTm91dmVhdSBt YWlsaW5nIGxpc3QKTm91dmVhdUBsaXN0cy5mcmVlZGVza3RvcC5vcmcKaHR0cHM6Ly9saXN0cy5m cmVlZGVza3RvcC5vcmcvbWFpbG1hbi9saXN0aW5mby9ub3V2ZWF1Cg== From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752437AbeEKHOG convert rfc822-to-8bit (ORCPT ); Fri, 11 May 2018 03:14:06 -0400 Received: from mout.gmx.net ([212.227.15.18]:41849 "EHLO mout.gmx.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750858AbeEKHOF (ORCPT ); Fri, 11 May 2018 03:14:05 -0400 Message-ID: <1526022830.6288.3.camel@gmx.de> Subject: Re: kernel spew from nouveau/ swiotlb From: Mike Galbraith To: nouveau Cc: LKML , Ben Skeggs , Christian Koenig , Roger He , Christoph Hellwig Date: Fri, 11 May 2018 09:13:50 +0200 In-Reply-To: <1525948096.29445.4.camel@gmx.de> References: <1525943427.8013.11.camel@gmx.de> <1525948096.29445.4.camel@gmx.de> Content-Type: text/plain; charset="ISO-8859-15" X-Mailer: Evolution 3.22.6 Mime-Version: 1.0 Content-Transfer-Encoding: 8BIT X-Provags-ID: V03:K1:TfLmtzf/9EXSGYv41bUv4Ek8hktFDG8uMQyCKDrNSyZ0sSpMYKH EwcB3JCUW/XDqyzsXgFxjvNF9F3GSiISX/LVBA0KjppeirzMa974PYu7LCgzcGLykL7cNXh qXm0XIquWRLhZsoc0lKB+/TOeYWpzR1xtpuHokYaa9Fj0KPIKwOMS7lA9WAXaCFGSrPaUwB Zfya1pnDSl0S+olC2DDig== X-UI-Out-Filterresults: notjunk:1;V01:K0:bcFGEiCjmqM=:TDB2jK160VoRw7rIsTLYo2 Zn5UduA7haiGji74NE7A3g5QojIQO4GbsGKOLhDew4J8gwXzJnXABM0ZI8KtmWO1eX2AGTp+n dsjcJX0f2Ev8PwKy3a3qsT5ZG8PbGnPuN4Vp76GZqSnTuv3Ig9qlKMLq2yjF5rl3daFTJqQeX HX5vtrT/R7JXW1Kovrty9KuNv9HNzVFGjk/BGNPtrI7bU3RJkttODVCZNwjHOgtKe8lLsN9vS WiWOoFxRXb3jHwc2LrAJV/XiCBTXkb9y7mlN9RUWJQMrpgKg/nwxhOB/ymzyMCkwG11z80vy/ rCxy4S9jx2li81B7QRkXB3zBYaFZtR8nG76aKIejGN2nvjNQAaWuRm7+k/Voz0sg2Xkzod0ZL JyuJejCXB7gqJI4ax0Qbcxc1EDzkomu4ihxAJ8kHp3xLqDKofKXJIkkFwUIUzGJXaKEAp8fCa QDM2npryHRi1H/ZZE5fmsqf0dvHCZG4slh+GfQMkU0jEWiMVQJ0okLKvkYp0mzAv8zzhZtJR1 xTOOBnoyNcBfcx/Z+UEwD8QbkX+QVjzQK1Y1IeQ95ihhCZ0sxKtxxTHQfu5tVoJlS8oGyZTOq aq63/f7TALioakOvOug6ug4KWQ9QcpcxZ7cTgOrypHdeIS/nHM2ljKYYbXYbDia/zD5Ws5mbQ KGt786Y9KDUccThNpyrnP5eK7LzXGynQ3jDDOTFue87tXbIysXZVsMtFpNS/OmrIHx4BAWblc oLGhcTrYENLFJDRPWt0QuAgWuLX0bEsmGstqqQzVy6fv1jWPUElrscHB9ljL43PvAHd+Ltj8n PP5Preu Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thu, 2018-05-10 at 12:28 +0200, Mike Galbraith wrote: > On Thu, 2018-05-10 at 11:10 +0200, Mike Galbraith wrote: > > Greetings, > > > > When box is earning its keep, nouveau/swiotlb grumble.. a LOT. The > > below is from master.today. > > > > [12594.640959] nouveau 0000:01:00.0: swiotlb buffer is full (sz: 2097152 bytes) > > [12594.693000] nouveau 0000:01:00.0: swiotlb buffer is full (sz: 2097152 bytes) > > [12594.713787] nouveau 0000:01:00.0: swiotlb buffer is full (sz: 2097152 bytes) > > [12594.743413] nouveau 0000:01:00.0: swiotlb buffer is full (sz: 2097152 bytes) > > [12594.796740] nouveau 0000:01:00.0: swiotlb buffer is full (sz: 2097152 bytes) > > [12607.000774] swiotlb_tbl_map_single: 54 callbacks suppressed > > [12607.000776] nouveau 0000:01:00.0: swiotlb buffer is full (sz: 2097152 bytes) > > [12607.347941] nouveau 0000:01:00.0: swiotlb buffer is full (sz: 2097152 bytes) > > [12608.677038] nouveau 0000:01:00.0: swiotlb buffer is full (sz: 2097152 bytes) > > homer:/novell/ssh # dmesg|grep 'swiotlb buffer is full'|wc -l > > 2052 > > homer:/novell/ssh # dmesg|grep 'callbacks suppressed'|wc -l > > 171 > > > > lib/swiotlb.c: > > 573 not_found: > > 574         spin_unlock_irqrestore(&io_tlb_lock, flags); > > 575         if (!(attrs & DMA_ATTR_NO_WARN) && printk_ratelimit()) > > 576                 dev_warn(hwdev, "swiotlb buffer is full (sz: %zd bytes)\n", size); > > > > Does nouveau perhaps want one of those DMA_ATTR_NO_WARN thingies? > > Or should ttm perhaps always use the one on hand? (seems to work) No it didn't, I just didn't wait long enough for spew to start... > --- > drivers/gpu/drm/ttm/ttm_page_alloc_dma.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > --- a/drivers/gpu/drm/ttm/ttm_page_alloc_dma.c > +++ b/drivers/gpu/drm/ttm/ttm_page_alloc_dma.c > @@ -342,7 +342,7 @@ static struct dma_page *__ttm_dma_alloc_ > if (!d_page) > return NULL; > > - if (pool->type & IS_HUGE) > + if (1 || pool->type & IS_HUGE) > attrs = DMA_ATTR_NO_WARN; > > vaddr = dma_alloc_attrs(pool->dev, pool->size, &d_page->dma, While IS_HUGE is indeed false on my box, it just doesn't matter, because when we get to either the old or the new alloc(), it calls swiotlb_alloc_buffer(), which drops attrs passed to it on the floor, making it unlikely that alloc() caller wishes are granted. -Mike