Re: [git pull] drm tree for 3.12-rc1

2013-09-10 Thread Sean V Kelley
On Thu, Sep 5, 2013 at 12:18 PM, Linus Torvalds
torva...@linux-foundation.org wrote:
 On Thu, Sep 5, 2013 at 3:41 AM, Dave Airlie airl...@linux.ie wrote:

 i915: Haswell PC8+ support and eLLC support, HDMI 4K support, initial 
 per-process VMA pieces,
   watermark reworks, convert to generic hdmi infoframes, encoder 
 reworking, fastboot support,

 Hmm.

 The first time I booted this, I just got a black screen on my Haswell
 desktop when X11 started up.  I could ctrl-alt-BS and ctrl-alt-del to
 reboot the machine, and neither the Xorg.0.log nor the dmesg contained
 anything interesting.

 I was about to try to bisect it, but decided to see how repeatable it
 was, and it didn't happen again. But it also hasn't ever happened
 before, so I'm a bit worried.

 This is with the DP cable, which has made my other Haswell issues go away.

 I'm also a bit bummed that hw acceleration of video doesn't seem to
 work on Haswell, meaning that full-screen is now a jerky mess. I fear
 that that is user-space libraries/X.org, but I thought I'd mention it
 in the hope of getting a oh, it's working for us, you'll get a fix
 for it soon.

Linus,

Can you give a little more detail about video not working?  Video
accel should work fine with the current versions of libva/intel-driver
available in Fedora 19 - assuming that's what you're using.  I am
running that on my MacBook Air 2013 and  HSW based Asus quad core i7
desktop.   A quick way to exercise the decoders once you install
libva/intel-driver is to use VLC enabling GPU acceleration in
preferences.

Sean


 Because my shiny new 65W haswell is really nice and does a make
 allmodconfig in half the time of my old machine, but the GPU side has
 been something of a step backwards...

Linus
 ___
 dri-devel mailing list
 dri-devel@lists.freedesktop.org
 http://lists.freedesktop.org/mailman/listinfo/dri-devel



-- 
Sean V. Kelley sean.v.kel...@intel.com
Open Source Technology Center / SSG
Intel Corp.
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel


Re: [PATCH] drm/nouveau: do not move buffers when not needed

2013-09-10 Thread Ben Skeggs
On Wed, Sep 4, 2013 at 11:25 PM, Maarten Lankhorst
maarten.lankho...@canonical.com wrote:
 Op 04-09-13 03:24, Ben Skeggs schreef:
 On Mon, Jul 15, 2013 at 6:39 PM, Maarten Lankhorst
 maarten.lankho...@canonical.com wrote:
 Op 15-07-13 08:05, Ben Skeggs schreef:
 On Fri, Jul 12, 2013 at 10:45 PM, Maarten Lankhorst
 maarten.lankho...@canonical.com wrote:
 I have no idea what this bogus restriction on placement is, but it breaks 
 decoding 1080p
 VDPAU at boot speed. With this patch applied I only need to bump the vdec 
 clock to
 get real-time 1080p decoding. It prevents a lot of VRAM - VRAM buffer 
 moves.
 It's not bogus, and is required for pre-GF8 boards with VRAM  BAR size.

 What configuration does the buffer that's getting moved here have
 exactly?  The placement restriction isn't necessary on GF8, the rest
 of the restrictions may currently be required still however.

 = vdpau on NVC0 with tiling. I upload the raw bitstream to a tiling bo. 
 This is ok because
 the vm hides all the tiling translations, and the engines will read the raw 
 bitstream correctly.
 Why would you be doing such a thing in the first place?  It seems
 pointless, and quite possibly counter-productive to use a tiled layout
 for a linear data structure...
 Initially I just allocated everything I didn't need to access directly tiled, 
 and it seems I did the same for
 the bitstream bo. I only found out later about the bug with excessive moves 
 causing a major slowdown.

 8---
 This prevents buffer moves from being done on NV50+, where remapping is not 
 needed because
 the bar has its own VM, instead of only having the first BAR1-size chunk of 
 VRAM accessible.
 nouveau_bo_tile_layout is always 0 on  NV_50.

 Signed-off-by: Maarten Lankhorst maarten.lankho...@canonical.com
 ---
 diff --git a/drivers/gpu/drm/nouveau/nouveau_bo.c 
 b/drivers/gpu/drm/nouveau/nouveau_bo.c
 index d506da5..762bfcd 100644
 --- a/drivers/gpu/drm/nouveau/nouveau_bo.c
 +++ b/drivers/gpu/drm/nouveau/nouveau_bo.c
 @@ -1346,14 +1361,13 @@ nouveau_ttm_fault_reserve_notify(struct 
 ttm_buffer_object *bo)
 struct nouveau_device *device = nv_device(drm-device);
 u32 mappable = pci_resource_len(device-pdev, 1)  PAGE_SHIFT;

 -   /* as long as the bo isn't in vram, and isn't tiled, we've got
 -* nothing to do here.
 +   /*
 +* if the bo is not in vram, or remapping can be done (nv50+)
 +* do not worry about placement, any location is valid
  */
 -   if (bo-mem.mem_type != TTM_PL_VRAM) {
 -   if (nv_device(drm-device)-card_type  NV_50 ||
 -   !nouveau_bo_tile_layout(nvbo))
 -   return 0;
 -   }
 +   if (nv_device(drm-device)-card_type = NV_50 ||
 +   bo-mem.mem_type != TTM_PL_VRAM)
 +   return 0;
 I get what you're trying to do here, and we should definitely avoid
 the mappable vram check on GF8, but I suspect this condition is too
 broad.  I'll think about it more after I finish reviewing the rest of
 the patches on the list..

 I think this relaxed check is fine. If it's !VRAM, the host can always access 
 it because it has direct access to the
 pages without needing anything from the gpu. On = NV50 the move can always 
 be skipped too because the
 memory is mapped to the vm, and always accessible.
Yeah, I think the check is sane, after thinking it through.  We shall
find out :)

But first, if you're going to not move tiled system memory to vram
first, you're going to need to deal with mapping it into bar1 so the
vm deals with some of the reordering.  See what's done in
io_mem_reserve() for TTM_PL_VRAM.

Ben.


 ~Maarten

___
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel


Re: [PATCH 1/7] drm/exynos: move hdmiphy related code to hdmiphy driver

2013-09-10 Thread Rahul Sharma
On 6 September 2013 19:21, Sean Paul seanp...@chromium.org wrote:
 On Thu, Sep 5, 2013 at 11:37 PM, Rahul Sharma r.sh.o...@gmail.com wrote:
 On 5 September 2013 19:20, Inki Dae inki@samsung.com wrote:


 -Original Message-
 From: Sean Paul [mailto:seanp...@chromium.org]
 Sent: Thursday, September 05, 2013 10:20 PM
 To: Inki Dae
 Cc: Rahul Sharma; Rahul Sharma; linux-samsung-soc; dri-devel; kgene.kim;
 sw0312.kim; Lucas Stach; Tomasz Figa; Sylwester Nawrocki; sunil joshi;
 Shirish S
 Subject: Re: [PATCH 1/7] drm/exynos: move hdmiphy related code to hdmiphy
 driver

 On Thu, Sep 5, 2013 at 2:19 AM, Inki Dae inki@samsung.com wrote:
 
 
  -Original Message-
  From: linux-samsung-soc-ow...@vger.kernel.org [mailto:linux-samsung-
 soc-
  ow...@vger.kernel.org] On Behalf Of Rahul Sharma
  Sent: Thursday, September 05, 2013 3:04 PM
  To: Inki Dae
  Cc: Sean Paul; Rahul Sharma; linux-samsung-soc; dri-devel; kgene.kim;
  sw0312.kim; Lucas Stach; Tomasz Figa; Sylwester Nawrocki; sunil joshi;
  Shirish S
  Subject: Re: [PATCH 1/7] drm/exynos: move hdmiphy related code to
 hdmiphy
  driver
 
  On 5 September 2013 10:52, Inki Dae inki@samsung.com wrote:
  +static struct hdmiphy_config hdmiphy_4210_configs[] = {
  +   {
  +   .pixel_clock = 2700,
  +   .conf = {
  +   0x01, 0x05, 0x00, 0xD8, 0x10, 0x1C,
  0x30,
0x40,
  +   0x6B, 0x10, 0x02, 0x51, 0xDF, 0xF2,
  0x54,
0x87,
  +   0x84, 0x00, 0x30, 0x38, 0x00, 0x08,
  0x10,
0xE0,
  +   0x22, 0x40, 0xE3, 0x26, 0x00, 0x00,
  0x00,
0x00,
  +   },
  +   },
  +   {
  +   .pixel_clock = 27027000,
  +   .conf = {
  +   0x01, 0x05, 0x00, 0xD4, 0x10, 0x9C,
  0x09,
0x64,
  +   0x6B, 0x10, 0x02, 0x51, 0xDF, 0xF2,
  0x54,
0x87,
  +   0x84, 0x00, 0x30, 0x38, 0x00, 0x08,
  0x10,
0xE0,
  +   0x22, 0x40, 0xE3, 0x26, 0x00, 0x00,
  0x00,
0x00,
  +   },
  +   },
  +   {
  +   .pixel_clock = 74176000,
  +   .conf = {
  +   0x01, 0x05, 0x00, 0xD8, 0x10, 0x9C,
  0xef,
0x5B,
  +   0x6D, 0x10, 0x01, 0x51, 0xef, 0xF3,
  0x54,
0xb9,
  +   0x84, 0x00, 0x30, 0x38, 0x00, 0x08,
  0x10,
0xE0,
  +   0x22, 0x40, 0xa5, 0x26, 0x01, 0x00,
  0x00,
0x00,
  +   },
  +   },
  +   {
  +   .pixel_clock = 7425,
  +   .conf = {
  +   0x01, 0x05, 0x00, 0xd8, 0x10, 0x9c,
  0xf8,
0x40,
  +   0x6a, 0x10, 0x01, 0x51, 0xff, 0xf1,
  0x54,
0xba,
  +   0x84, 0x00, 0x10, 0x38, 0x00, 0x08,
  0x10,
0xe0,
  +   0x22, 0x40, 0xa4, 0x26, 0x01, 0x00,
  0x00,
0x00,
  +   },
  +   },
  +   {
  +   .pixel_clock = 14850,
  +   .conf = {
  +   0x01, 0x05, 0x00, 0xD8, 0x10, 0x9C,
  0xf8,
0x40,
  +   0x6A, 0x18, 0x00, 0x51, 0xff, 0xF1,
  0x54,
0xba,
  +   0x84, 0x00, 0x10, 0x38, 0x00, 0x08,
  0x10,
0xE0,
  +   0x22, 0x40, 0xa4, 0x26, 0x02, 0x00,
  0x00,
0x00,
  +   },
  +   },
  +};
  +
 
  Are you aware of the effort to move these to dt? Since these
  are
  board-specific values, it seems incorrect to apply them
   universally.
  Shirish has uploaded a patch to the chromium review site to
  push
these
  into dt
 (https://chromium-review.googlesource.com/#/c/65581).
   Maybe
  you can work that into your patch set?
 

 Are these really board-specific values?
   
According to your hardware people:
   
If the signal pattern doesn't change for new board, the phy
 setting
is same as the current board. But if changed, you need to confirm
  with
measurement of signals, because it may vary slightly by
 resistance
  of
board pattern
   
   
Right. it seems that the phy configuration should be adjusted
  according
   to
PCB environment: OSC clock rate, 24MHz or 27MHz, could be decided
 by
  PCB
even though most PCBs use 27MHz.
   
That indicates to me that we might need to tweak these on a per-
  board
basis.
   
In the 5420 datasheet, there are a few register descriptions
  available
for the phy. 0x145D0004 is CLK_SEL which seems like it would be
board-specific, same with TX_* registers.
   
   
And we can select HDMI Tx PHY internal PLL input clock by setting
   CLK_SEL.
Ok, Shirish's patch set is reasonable to me. However, that patch
 

Re: drm/radeon: ring test failed on PA-RISC Linux

2013-09-10 Thread Alex Ivanov
Alex,

09.09.2013, в 21:43, Alex Deucher alexdeuc...@gmail.com написал(а):

 On Mon, Sep 9, 2013 at 12:44 PM, Alex Ivanov gnido...@p0n4ik.tk wrote:
 Folks,
 
 We (people at linux-parisc @ vger.kernel.org mail list) are trying to make
 native video options of the latest PA-RISC servers and workstations
 (these are ATIs, most of which are based on R100/R300/R420 chips) work
 correctly on this platform (big endian pa-risc).
 
 However, we hadn't much success. DRM fails every time with
 ring test failed for both AGP  PCI.
 
 Maybe you would give us some suggestions that we could check?
 
 Topic started here:
 http://www.spinics.net/lists/linux-parisc/msg04908.html
 And continued there:
 http://www.spinics.net/lists/linux-parisc/msg04995.html
 http://www.spinics.net/lists/linux-parisc/msg05006.html
 
 Problems we've already resolved without any signs of progress:
 - Checked the successful microcode load
 parisc AGP GART code writes IOMMU entries in the wrong byte order and
 doesn't add the coherency information SBA code adds
 our PCI BAR setup doesn't really work very well together with the Radeon
 DRM address setup. DRM will generate addresses, which are even outside
 of the connected LBA
 
 Things planned for a check:
 The drivers/video/aty uses
 an endian config bit DRM doesn't use, but I haven't tested whether
 this makes a difference and how it is connected to the overall picture.
 
 I don't think that will any difference.  radeon kms works fine on
 other big endian platforms such as powerpc.

Good! I'll opt it out then.

 
 
 The Rage128 product revealed a weakness in some motherboard
 chipsets in that there is no mechanism to guarantee
 that data written by the CPU to memory is actually in a readable
 state before the Graphics Controller receives an
 update to its copy of the Write Pointer. In an effort to alleviate this
 problem, weve introduced a mechanism into the
 Graphics Controller that will delay the actual write to the Write Pointer
 for some programmable amount of time, in
 order to give the chipset time to flush its internal write buffers to
 memory.
 There are two register fields that control this mechanism:
 PRE_WRITE_TIMER and PRE_WRITE_LIMIT.
 
 In the radeon DRM codebase I didn't found anyone using/setting
 those registers. Maybe PA-RISC has some problem here?...
 
 I doubt it.  If you are using AGP, I'd suggest disabling it and first
 try to get things working using the on chip gart rather than AGP.
 Load radeon with agpmode=-1.  

Already tried this without any luck. Anyway, a radeon driver fallbacks
to the PCI mode in our case, so does it really matter?

In addition, people with PCI cards experiencing the same issue...

 The on chip gart always uses cache
 snooped pci transactions and the driver assumes pci is cache coherent.
 On AGP/PCI chips, the on-chip gart mechanism stores the gart table in
 system ram.  On PCIE asics, the gart table is stored in vram.  The
 gart page table maps system pages to a contiguous aperture in the
 GPU's address space.  The ring lives in gart memory.  The GPU sees a
 contiguous buffer and the gart mechanism handles the access to the
 backing pages via the page table.  I'd suggest verifying that the
 entries written to the gart page table are valid and then the
 information written to the ring buffer is valid before updating the
 ring's wptr in radeon_ring_unlock_commit().  Changing the wptr is what
 causes the CP to start fetching data from the ring.

Thanks! I'll try. Meanwhile i've tried a switch from page_alloc() to
dma_alloc_coherent() in radeon_dummy_page_*(), which didn't help :(

--- radeon_device.c.orig2013-09-10 08:55:05.0 +
+++ radeon_device.c 2013-09-10 09:12:17.0 +
@@ -673,15 +673,13 @@ int radeon_dummy_page_init(struct radeon
 {
if (rdev-dummy_page.page)
return 0;
-   rdev-dummy_page.page = alloc_page(GFP_DMA32 | GFP_KERNEL | __GFP_ZERO);
-   if (rdev-dummy_page.page == NULL)
+   rdev-dummy_page.page = dma_alloc_coherent(rdev-pdev-dev, PAGE_SIZE,
+   rdev-dummy_page.addr, GFP_DMA32|GFP_KERNEL);
+   if (!rdev-dummy_page.page)
return -ENOMEM;
-   rdev-dummy_page.addr = pci_map_page(rdev-pdev, rdev-dummy_page.page,
-   0, PAGE_SIZE, PCI_DMA_BIDIRECTIONAL);
if (pci_dma_mapping_error(rdev-pdev, rdev-dummy_page.addr)) {
dev_err(rdev-pdev-dev, Failed to DMA MAP the dummy page\n);
-   __free_page(rdev-dummy_page.page);
-   rdev-dummy_page.page = NULL;
+   radeon_dummy_page_fini(rdev);
return -ENOMEM;
}
return 0;
@@ -698,9 +696,8 @@ void radeon_dummy_page_fini(struct radeo
 {
if (rdev-dummy_page.page == NULL)
return;
-   pci_unmap_page(rdev-pdev, rdev-dummy_page.addr,
-   PAGE_SIZE, PCI_DMA_BIDIRECTIONAL);
-   __free_page(rdev-dummy_page.page);
+   

Re: [PATCH] drm/nouveau: avoid null deref on bad arguments to nouveau_vma_getmap

2013-09-10 Thread Pasi Kärkkäinen
On Wed, Sep 04, 2013 at 08:59:13AM +0200, Maarten Lankhorst wrote:
 Op 04-09-13 05:41, Ben Skeggs schreef:
  On Thu, Aug 22, 2013 at 5:12 PM, Maarten Lankhorst
  maarten.lankho...@canonical.com wrote:
  Op 22-08-13 02:10, Ilia Mirkin schreef:
  The code expects non-VRAM mem nodes to have a pages list. If that's not
  set, it will do a null deref down the line. Warn on that condition and
  return an error.
 
  See https://bugs.freedesktop.org/show_bug.cgi?id=64774
 
  Reported-by: Pasi Kärkkäinen pa...@iki.fi
  Tested-by: Pasi Kärkkäinen pa...@iki.fi
  Signed-off-by: Ilia Mirkin imir...@alum.mit.edu
  Cc: sta...@vger.kernel.org # 3.8+
  ---
 
  I don't exactly understand what's going on, but this is just a
  straightforward way to avoid a null deref that you see happens in the
  bug. I haven't figured out the root cause of this, but it's getting
  well into the I have no idea how TTM works space. However this seems
  like a bit of defensive programming -- nouveau_vm_map_sg will pass
  node-pages as a list down, which will be dereferenced by
  nvc0_vm_map_sg. Perhaps the other arguments should make that
  dereferencing not happen, but it definitely was happening here, as you
  can see in the bug.
 
  Ben/Maarten, I'll let you judge whether this check is appropriate,
  since like I hope I was able to convey above, I'm just not really sure :)
  Not it really isn't appropriate..
 
  You'd have to call call nouveau_vm_map_sg_table instead, the only place 
  that doesn't handle that correctly
  is where it's not expected to be called.
 
  Here, have a completely untested patch to fix things...
 
  diff --git a/drivers/gpu/drm/nouveau/nouveau_display.c 
  b/drivers/gpu/drm/nouveau/nouveau_display.c
  --- a/drivers/gpu/drm/nouveau/nouveau_display.c
  +++ b/drivers/gpu/drm/nouveau/nouveau_display.c
  @@ -138,17 +143,26 @@ nouveau_user_framebuffer_create(struct drm_device 
  *dev,
   {
  struct nouveau_framebuffer *nouveau_fb;
  struct drm_gem_object *gem;
  +   struct nouveau_bo *nvbo;
  int ret = -ENOMEM;
 
  gem = drm_gem_object_lookup(dev, file_priv, mode_cmd-handles[0]);
  if (!gem)
  return ERR_PTR(-ENOENT);
 
  +   nvbo = nouveau_gem_object(gem);
  +   if (!(nvbo-valid_domains  NOUVEAU_GEM_DOMAIN_VRAM)) {
  +   nv_warn(nouveau_drm(dev), Trying to create a fb in vram 
  with
  +valid_domains=%08x\n, nvbo-valid_domains);
  +   ret = -EINVAL;
  +   goto err_unref;
  +   }
  +
  Definitely the right idea, we can't handle this case right now.
  However, we may someday want/need to be able to scan out of system
  memory, so this is the wrong place.
 
  I suspect the correct thing to do (which'll also handle the
  defensive part) is to bail in nouveau_bo_move() on attempts to move
  a DMA-BUF backed object into VRAM.
 
  Sound OK?
 
 If it has a WARN_ON or something that would be ok, I didn't find any other 
 places that attempt to move buffers to VRAM though, so it's probably harmless.
 

So hmm.. I guess another patch is needed for the original issue in this thread.
Is someone going to submit that? 


 When looking into this bug I noticed that nouveau_bo_vma_add needs to have a 
 check for nvbo-page_shift == vma-vm-vmm-spg_shift,
 and only if the check is true it should map the page in TTM_PL_TT. Patch 
 below.
 Should probably also be cc'd to stable.


Thanks! Is this patch ready to be merged?


-- Pasi
 

 ~Maarten
 
 diff --git a/drivers/gpu/drm/nouveau/nouveau_bo.c 
 b/drivers/gpu/drm/nouveau/nouveau_bo.c
 index 89b992e..355a1b7 100644
 --- a/drivers/gpu/drm/nouveau/nouveau_bo.c
 +++ b/drivers/gpu/drm/nouveau/nouveau_bo.c
 @@ -1560,7 +1560,8 @@ nouveau_bo_vma_add(struct nouveau_bo *nvbo, struct 
 nouveau_vm *vm,
  
   if (nvbo-bo.mem.mem_type == TTM_PL_VRAM)
   nouveau_vm_map(vma, nvbo-bo.mem.mm_node);
 - else if (nvbo-bo.mem.mem_type == TTM_PL_TT) {
 + else if (nvbo-bo.mem.mem_type == TTM_PL_TT 
 +  nvbo-page_shift == vma-vm-vmm-spg_shift) {
   if (node-sg)
   nouveau_vm_map_sg_table(vma, 0, size, node);
   else
 
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel


[Bug 64810] EGL/Gles/Weston give segfault on RADEONSI with egl_gallium.so

2013-09-10 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=64810

Johannes Obermayr johannesoberm...@gmx.de changed:

   What|Removed |Added

  Attachment #84756|0   |1
is obsolete||
  Attachment #84840|0   |1
is obsolete||

--- Comment #19 from Johannes Obermayr johannesoberm...@gmx.de ---
Created attachment 85550
  -- https://bugs.freedesktop.org/attachment.cgi?id=85550action=edit
link libradeon only once

Please try this patch.
libradeon will be linked in egl_gallium.so only once.

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel


Re: drm/radeon: ring test failed on PA-RISC Linux

2013-09-10 Thread Alex Deucher
On Tue, Sep 10, 2013 at 5:20 AM, Alex Ivanov gnido...@p0n4ik.tk wrote:
 Alex,

 09.09.2013, в 21:43, Alex Deucher alexdeuc...@gmail.com написал(а):

 On Mon, Sep 9, 2013 at 12:44 PM, Alex Ivanov gnido...@p0n4ik.tk wrote:
 Folks,

 We (people at linux-parisc @ vger.kernel.org mail list) are trying to make
 native video options of the latest PA-RISC servers and workstations
 (these are ATIs, most of which are based on R100/R300/R420 chips) work
 correctly on this platform (big endian pa-risc).

 However, we hadn't much success. DRM fails every time with
 ring test failed for both AGP  PCI.

 Maybe you would give us some suggestions that we could check?

 Topic started here:
 http://www.spinics.net/lists/linux-parisc/msg04908.html
 And continued there:
 http://www.spinics.net/lists/linux-parisc/msg04995.html
 http://www.spinics.net/lists/linux-parisc/msg05006.html

 Problems we've already resolved without any signs of progress:
 - Checked the successful microcode load
 parisc AGP GART code writes IOMMU entries in the wrong byte order and
 doesn't add the coherency information SBA code adds
 our PCI BAR setup doesn't really work very well together with the Radeon
 DRM address setup. DRM will generate addresses, which are even outside
 of the connected LBA

 Things planned for a check:
 The drivers/video/aty uses
 an endian config bit DRM doesn't use, but I haven't tested whether
 this makes a difference and how it is connected to the overall picture.

 I don't think that will any difference.  radeon kms works fine on
 other big endian platforms such as powerpc.

 Good! I'll opt it out then.



 The Rage128 product revealed a weakness in some motherboard
 chipsets in that there is no mechanism to guarantee
 that data written by the CPU to memory is actually in a readable
 state before the Graphics Controller receives an
 update to its copy of the Write Pointer. In an effort to alleviate this
 problem, weve introduced a mechanism into the
 Graphics Controller that will delay the actual write to the Write Pointer
 for some programmable amount of time, in
 order to give the chipset time to flush its internal write buffers to
 memory.
 There are two register fields that control this mechanism:
 PRE_WRITE_TIMER and PRE_WRITE_LIMIT.

 In the radeon DRM codebase I didn't found anyone using/setting
 those registers. Maybe PA-RISC has some problem here?...

 I doubt it.  If you are using AGP, I'd suggest disabling it and first
 try to get things working using the on chip gart rather than AGP.
 Load radeon with agpmode=-1.

 Already tried this without any luck. Anyway, a radeon driver fallbacks
 to the PCI mode in our case, so does it really matter?

 In addition, people with PCI cards experiencing the same issue...

 The on chip gart always uses cache
 snooped pci transactions and the driver assumes pci is cache coherent.
 On AGP/PCI chips, the on-chip gart mechanism stores the gart table in
 system ram.  On PCIE asics, the gart table is stored in vram.  The
 gart page table maps system pages to a contiguous aperture in the
 GPU's address space.  The ring lives in gart memory.  The GPU sees a
 contiguous buffer and the gart mechanism handles the access to the
 backing pages via the page table.  I'd suggest verifying that the
 entries written to the gart page table are valid and then the
 information written to the ring buffer is valid before updating the
 ring's wptr in radeon_ring_unlock_commit().  Changing the wptr is what
 causes the CP to start fetching data from the ring.

 Thanks! I'll try. Meanwhile i've tried a switch from page_alloc() to
 dma_alloc_coherent() in radeon_dummy_page_*(), which didn't help :(


The dummy page isn't really going to help much.  That page is just
used as a safety placeholder for gart entries that aren't mapped on
the GPU.  TTM (drivers/gpu/drm/ttm) actually does the allocation of
the backing pages for the gart.  You may want to look there.

Alex

 --- radeon_device.c.orig2013-09-10 08:55:05.0 +
 +++ radeon_device.c 2013-09-10 09:12:17.0 +
 @@ -673,15 +673,13 @@ int radeon_dummy_page_init(struct radeon
  {
 if (rdev-dummy_page.page)
 return 0;
 -   rdev-dummy_page.page = alloc_page(GFP_DMA32 | GFP_KERNEL | 
 __GFP_ZERO);
 -   if (rdev-dummy_page.page == NULL)
 +   rdev-dummy_page.page = dma_alloc_coherent(rdev-pdev-dev, 
 PAGE_SIZE,
 +   rdev-dummy_page.addr, GFP_DMA32|GFP_KERNEL);
 +   if (!rdev-dummy_page.page)
 return -ENOMEM;
 -   rdev-dummy_page.addr = pci_map_page(rdev-pdev, 
 rdev-dummy_page.page,
 -   0, PAGE_SIZE, PCI_DMA_BIDIRECTIONAL);
 if (pci_dma_mapping_error(rdev-pdev, rdev-dummy_page.addr)) {
 dev_err(rdev-pdev-dev, Failed to DMA MAP the dummy 
 page\n);
 -   __free_page(rdev-dummy_page.page);
 -   rdev-dummy_page.page = NULL;
 +   radeon_dummy_page_fini(rdev);

Re: drm/radeon: ring test failed on PA-RISC Linux

2013-09-10 Thread Hans Verkuil
On 09/10/2013 02:37 PM, Alex Deucher wrote:
 On Tue, Sep 10, 2013 at 5:20 AM, Alex Ivanov gnido...@p0n4ik.tk wrote:
 Alex,

 09.09.2013, в 21:43, Alex Deucher alexdeuc...@gmail.com написал(а):

 On Mon, Sep 9, 2013 at 12:44 PM, Alex Ivanov gnido...@p0n4ik.tk wrote:
 Folks,

 We (people at linux-parisc @ vger.kernel.org mail list) are trying to make
 native video options of the latest PA-RISC servers and workstations
 (these are ATIs, most of which are based on R100/R300/R420 chips) work
 correctly on this platform (big endian pa-risc).

 However, we hadn't much success. DRM fails every time with
 ring test failed for both AGP  PCI.

 Maybe you would give us some suggestions that we could check?

 Topic started here:
 http://www.spinics.net/lists/linux-parisc/msg04908.html
 And continued there:
 http://www.spinics.net/lists/linux-parisc/msg04995.html
 http://www.spinics.net/lists/linux-parisc/msg05006.html

 Problems we've already resolved without any signs of progress:
 - Checked the successful microcode load
 parisc AGP GART code writes IOMMU entries in the wrong byte order and
 doesn't add the coherency information SBA code adds
 our PCI BAR setup doesn't really work very well together with the Radeon
 DRM address setup. DRM will generate addresses, which are even outside
 of the connected LBA

 Things planned for a check:
 The drivers/video/aty uses
 an endian config bit DRM doesn't use, but I haven't tested whether
 this makes a difference and how it is connected to the overall picture.

 I don't think that will any difference.  radeon kms works fine on
 other big endian platforms such as powerpc.

 Good! I'll opt it out then.

Actually, I am experiencing exactly the same problem on a Sam460ex ppc
system, at least as of 3.9 (the last time I tried it).

Very rarely the ringtest would pass, but then it would fail somewhere else.
I never could figure it out since as far as I could tell all the addresses
and logic was correct. It wasn't important enough for me to work more on it,
but I'd be happy to test code. I'm travelling for the next week and a half,
so I can't do anything right now.

One bug I found when working on drm/kms support for the ppc was that in
struct ttm_bus_placement the base address type was wrong: it should be
phys_addr_t, not unsigned long. The PPC460 is in 32-bit mode but physical
addresses are 32 bits.

The patch below fixes that. I always wanted to post this fix, but never got
around to it...

Regards,

Hans

Signed-off-by: Hans Verkuil hans.verk...@cisco.com
---
 arch/powerpc/sysdev/ppc4xx_msi.c   |6 +++---
 drivers/gpu/drm/radeon/radeon_device.c |2 +-
 include/drm/ttm/ttm_bo_api.h   |2 +-
 3 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/drivers/gpu/drm/radeon/radeon_device.c 
b/drivers/gpu/drm/radeon/radeon_device.c
index 49b0659..fa33568 100644
--- a/drivers/gpu/drm/radeon/radeon_device.c
+++ b/drivers/gpu/drm/radeon/radeon_device.c
@@ -1066,7 +1066,7 @@ int radeon_device_init(struct radeon_device *rdev,
if (rdev-rmmio == NULL) {
return -ENOMEM;
}
-   DRM_INFO(register mmio base: 0x%08X\n, (uint32_t)rdev-rmmio_base);
+   DRM_INFO(register mmio base: 0x%llx\n, (uint64_t)rdev-rmmio_base);
DRM_INFO(register mmio size: %u\n, (unsigned)rdev-rmmio_size);
 
/* io port mapping */
diff --git a/include/drm/ttm/ttm_bo_api.h b/include/drm/ttm/ttm_bo_api.h
index 3cb5d84..fcdb208 100644
--- a/include/drm/ttm/ttm_bo_api.h
+++ b/include/drm/ttm/ttm_bo_api.h
@@ -81,7 +81,7 @@ struct ttm_placement {
  */
 struct ttm_bus_placement {
void*addr;
-   unsigned long   base;
+   phys_addr_t base;
unsigned long   size;
unsigned long   offset;
boolis_iomem;
-- 
1.7.10.4


___
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel


Re: drm/radeon: ring test failed on PA-RISC Linux

2013-09-10 Thread Konrad Rzeszutek Wilk
On Tue, Sep 10, 2013 at 01:20:57PM +0400, Alex Ivanov wrote:
 Alex,
 
 09.09.2013, в 21:43, Alex Deucher alexdeuc...@gmail.com написал(а):
 
  On Mon, Sep 9, 2013 at 12:44 PM, Alex Ivanov gnido...@p0n4ik.tk wrote:
  Folks,
  
  We (people at linux-parisc @ vger.kernel.org mail list) are trying to make
  native video options of the latest PA-RISC servers and workstations
  (these are ATIs, most of which are based on R100/R300/R420 chips) work
  correctly on this platform (big endian pa-risc).
  
  However, we hadn't much success. DRM fails every time with
  ring test failed for both AGP  PCI.
  
  Maybe you would give us some suggestions that we could check?
  
  Topic started here:
  http://www.spinics.net/lists/linux-parisc/msg04908.html
  And continued there:
  http://www.spinics.net/lists/linux-parisc/msg04995.html
  http://www.spinics.net/lists/linux-parisc/msg05006.html
  
  Problems we've already resolved without any signs of progress:
  - Checked the successful microcode load
  parisc AGP GART code writes IOMMU entries in the wrong byte order and
  doesn't add the coherency information SBA code adds
  our PCI BAR setup doesn't really work very well together with the Radeon
  DRM address setup. DRM will generate addresses, which are even outside
  of the connected LBA
  
  Things planned for a check:
  The drivers/video/aty uses
  an endian config bit DRM doesn't use, but I haven't tested whether
  this makes a difference and how it is connected to the overall picture.
  
  I don't think that will any difference.  radeon kms works fine on
  other big endian platforms such as powerpc.
 
 Good! I'll opt it out then.
 
  
  
  The Rage128 product revealed a weakness in some motherboard
  chipsets in that there is no mechanism to guarantee
  that data written by the CPU to memory is actually in a readable
  state before the Graphics Controller receives an
  update to its copy of the Write Pointer. In an effort to alleviate this
  problem, weve introduced a mechanism into the
  Graphics Controller that will delay the actual write to the Write Pointer
  for some programmable amount of time, in
  order to give the chipset time to flush its internal write buffers to
  memory.
  There are two register fields that control this mechanism:
  PRE_WRITE_TIMER and PRE_WRITE_LIMIT.
  
  In the radeon DRM codebase I didn't found anyone using/setting
  those registers. Maybe PA-RISC has some problem here?...
  
  I doubt it.  If you are using AGP, I'd suggest disabling it and first
  try to get things working using the on chip gart rather than AGP.
  Load radeon with agpmode=-1.  
 
 Already tried this without any luck. Anyway, a radeon driver fallbacks
 to the PCI mode in our case, so does it really matter?
 
 In addition, people with PCI cards experiencing the same issue...
 
  The on chip gart always uses cache
  snooped pci transactions and the driver assumes pci is cache coherent.
  On AGP/PCI chips, the on-chip gart mechanism stores the gart table in
  system ram.  On PCIE asics, the gart table is stored in vram.  The
  gart page table maps system pages to a contiguous aperture in the
  GPU's address space.  The ring lives in gart memory.  The GPU sees a
  contiguous buffer and the gart mechanism handles the access to the
  backing pages via the page table.  I'd suggest verifying that the
  entries written to the gart page table are valid and then the
  information written to the ring buffer is valid before updating the
  ring's wptr in radeon_ring_unlock_commit().  Changing the wptr is what
  causes the CP to start fetching data from the ring.
 
 Thanks! I'll try. Meanwhile i've tried a switch from page_alloc() to
 dma_alloc_coherent() in radeon_dummy_page_*(), which didn't help :(

Is this platform enabling the SWIOTLB layer? The reason I am asking is
b/c if you do indeed enable it you end up using the TTM DMA pool
which allocates pages using the dma_alloc_coherent - which means that
all of the pages that come out of TTM are already 'DMA' mapped.

And that means the radeon_gart_bind and all its friends 
use the DMA addresses that have been constructed by SWIOTLB IOMMU.

Perhaps the PA-RISC IOMMU creates the DMA addresses differently?

When the card gets programmed, you do end up using ttm_agp_bind right?
I am wondering if something like this:

https://lkml.org/lkml/2010/12/6/512

is needed to pass in the right DMA address?

 
 --- radeon_device.c.orig  2013-09-10 08:55:05.0 +
 +++ radeon_device.c   2013-09-10 09:12:17.0 +
 @@ -673,15 +673,13 @@ int radeon_dummy_page_init(struct radeon
  {
   if (rdev-dummy_page.page)
   return 0;
 - rdev-dummy_page.page = alloc_page(GFP_DMA32 | GFP_KERNEL | __GFP_ZERO);
 - if (rdev-dummy_page.page == NULL)
 + rdev-dummy_page.page = dma_alloc_coherent(rdev-pdev-dev, PAGE_SIZE,
 + rdev-dummy_page.addr, GFP_DMA32|GFP_KERNEL);
 + if (!rdev-dummy_page.page)
   return -ENOMEM;
 - 

[Bug 69120] With dpm=1 vdpau is not usable

2013-09-10 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=69120

--- Comment #10 from Alex Deucher ag...@yahoo.com ---
Created attachment 85563
  -- https://bugs.freedesktop.org/attachment.cgi?id=85563action=edit
possible fix

The attached patch should fix the issue.

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel


Re: [PATCH 2/2] ACPI / video / i915: Remove ACPI backlight if firmware expects Windows 8

2013-09-10 Thread Jani Nikula
On Mon, 09 Sep 2013, Rafael J. Wysocki r...@sisk.pl wrote:
 On Monday, September 09, 2013 05:21:18 PM Daniel Vetter wrote:
 On Mon, Sep 09, 2013 at 02:16:12PM +0200, Rafael J. Wysocki wrote:
  Hi,
  
  On Monday, September 09, 2013 11:32:10 AM Daniel Vetter wrote:
   Hi Aaaron,
   
   Have we grown any clue meanwhile about which Intel boxes need this and 
   for
   which we still need to keep the acpi backlight around?
  
  First of all, there is a bunch of boxes where ACPI backlight works 
  incorrectly
  because of the Win8 compatibility issue.  [In short, if we say we are 
  compatible
  with Win8, the backlight AML goes into a special code path that is broken 
  on
  those machines. Presumably Win8 uses native backlight control on them and 
  that
  AML code path is never executed there.]  The collection of machines with 
  this
  problem appears to be kind of random (various models from various 
  vendors), but
  I *think* they are machines that originally shipped with Win7 with a Win8
  upgrade option (which in practice requires the BIOS to be updated 
  anyway).
  
  Because of that, last time we tried to switch all of the systems using i915
  and telling the BIOS that they are compatible with Win8 over to native 
  backlight
  control, but that didn't work for two reasons.  The first reason is that 
  some
  user space doesn't know how to use intel_backlight and needs to be taught 
  about
  that (so some systems are just not ready for that switch).  The second and 
  more
  fundamental reason is that the native backlight control simply doesn't 
  work on
  some machines and we don't seem to have any idea why and how to debug this
  particular problem (mostly because we don't have enough information and we
  don't know what to ask for).
  
   I've grown _very_ reluctant to just adding tons of quirks to our driver 
   for
   the backlight.
  
   Almost all the quirks we have added recently (or that have been proposed
   to be added) are for the backlight. Imo that indicates we get something
   fundamentally wrong ...
  
  This thing isn't really a quirk.  It rather is an option for the users of
  the systems where ACPI backlight doesn't work to switch over to the native
  backlight control using a command line switch.  This way they can at least
  *see* if the native backlight control works for them and (hopefully) report
  problems if that's not the case.  This gives us a chance to get more
  information about what the problem is and possibly to make some progress
  without making changes for everyone, reverting those changes when they 
  don't
  work etc.
  
  An alternative for them is to pass acpi_osi=!Windows 2012 which will 
  probably
  make the ACPI backlight work for them again, but this rather is a step 
  back,
  because we can't possibly learn anything new from that.
 
 If Win8 is as broken as we are I'm ok with the module option. It just
 sounded to me like right now we don't know of a way to make all machines
 somewhat happy, combined with the other pile of random backlight issues
 the assumption that we do something (maybe something a bit racy) that
 windows doesn't do isn't too far-fetched. So I'm not wary of the machines
 where the aml is busted for acpi_os=win8, but for the others where this
 broke stuff.
 
 Or do I miss something here?

 The ACPI video driver doesn't do anything new now.  It only does things that
 did work before we started to tell BIOSes we're compatible with Win8.  The
 reason why they don't work on some machines now is not related to whether or
 not Win8 is broken, but to what is there in the ACPI tables on those machines.
 That *is* pure garbage, but Win8 users don't see that (presumably, because
 Win8 does't execute the AML in question).  We don't see that either with
 acpi_osi=!Windows 2012 (because then we don't execute that AML either).

 Whether or not Win8 is broken doesn't matter here.  What matters is that we
 have to deal with broken AML somehow.

 One way may be to tell everyone affected by this to pass
 acpi_osi=!Windows 2012 in the kernel command line or possibly create a
 blacklist of those machines in the kernel (which Felipe has been pushing for
 recently) and wash our hands clean of this, but that leaves some exceptionally
 bad taste in my mouth.

 The alternative is to try to use native backlight in i915, which we did, but
 that didn't work on some machines.  I don't think we will know why it didn't
 work there before we collect some more information and that's not possible
 without user testing.  So, the idea is to make that testing possible without
 hacking the kernel and that's why we're introducing the new command line
 switch.  And we're going to ask users to try it and report back.

The thing that slightly bugs me with the proposed patches is that
they're adding a module parameter to i915 to tell ACPI video driver
whether to quirk the backlight or not. Before you know, we *will* have
requests to add quirks to i915 to tell ACPI video 

Re: Stereo 3D v3

2013-09-10 Thread Chris Wilson
On Sun, Sep 08, 2013 at 04:03:52PM +0100, Damien Lespiau wrote:
 On Sun, Sep 08, 2013 at 03:46:43PM +0200, David Herrmann wrote:
  The series implements SET_CAP as a per _file_ capability set, not per
  master. I like it this way. Note that with SET_VERSION, we already
  have a per _master_ capability set. Compared to SET_CAP it only allows
  incremental capability changes, but that's fine I think.
  
  However, the problem with per-master capabilities (SET_VERSION) is
  that we currently have no way to control which master a
  graphics-server gets assigned to. If it's started in background, it
  will get the same master as the foreground compositor. Therefore, we
  don't want per-master client-capabilities. It's wrong and breaks
  existing setups (same as SET_VERSION, and everyone knows that). I also
  don't see a reason to bind capabilities to a master object.
  
  SET_CAP describes what the *calling client* understands and can work
  with. And this is logically bound to drm_file (as it represents a
  client). On the other hand, GET_CAP describes what the *device*
  understands and provides. This is obviously bound to a drm_device. A
  drm_master object allows to split GET_CAP capabilities and resources
  across multiple logical master objects. But these resemble a
  drm_device much more than a drm_file.
  
  So no, this capability is not dropped with a change in master. It's
  independent of the active/bound master. It just describes what a
  client sees or not sees.
 
 Right, that sums it up. Note that while I've made stereo_allowed a per
 fd thing (which is what I wanted in that case, alter the reality viewed
 by the process opening the file), SET_CAP itself it marked as master
 only. This can be changed in the future to provide per-cap access
 restrictions if needed.

This could be renamed to SET_CLIENT_CAP and also drop the master
requirement. (That some capabilities only affect master ioctls is
irrelevant I think, as the client will be master at that time.)
That would reduce the confusion between the device caps and the session
caps.
-Chris

-- 
Chris Wilson, Intel Open Source Technology Centre
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel


Re: [PATCH 2/2] ACPI / video / i915: Remove ACPI backlight if firmware expects Windows 8

2013-09-10 Thread Jani Nikula
On Tue, 10 Sep 2013, Matthew Garrett matthew.garr...@nebula.com wrote:
 On Tue, 2013-09-10 at 16:53 +0300, Jani Nikula wrote:

 I think the parameter Does the ACPI backlight interface work or not
 belongs to the ACPI video driver.

 That depends on how Windows 8 works. If Windows 8 policy is handled by
 the GPU drivers then it belongs in i915. If it's handled by the ACPI
 code then it belongs in the ACPI code.

I fail to see the logic. Windows 8 policy dictates whether we can use
the AML code or not. IMHO, ACPI code is in the best position to figure
this out and quirk as necessary. It's the part that knows about Windows
8, not i915.

 But I have no way of determining that, whereas you work for a company
 that produces a Windows 8 video driver…

Here I do see the logic. However see signature; I'm not in the best of
positions to figure out things about Windows 8 video drivers. ;) But
I'll see what I can do.

BR,
Jani.


-- 
Jani Nikula, Intel Open Source Technology Center
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel


[Bug 60791] Display corruption with Radeon driver during boot and on desktop

2013-09-10 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=60791

Alex Deucher alexdeuc...@gmail.com changed:

   What|Removed |Added

 Attachment #107911|0   |1
is obsolete||

--- Comment #29 from Alex Deucher alexdeuc...@gmail.com ---
Created attachment 108011
  -- https://bugzilla.kernel.org/attachment.cgi?id=108011action=edit
possible fix

Sorry, I had the wrong chip check in the last version.  This patch should be
correct.

-- 
You are receiving this mail because:
You are watching the assignee of the bug.
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel


[Bug 69120] With dpm=1 vdpau is not usable

2013-09-10 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=69120

--- Comment #11 from John john.etted...@gmail.com ---
Hi Alex,

Thanks for the quick turn around.

Looking at the patch it seems to be for RV770 and RV780 will it still work for
our older cards?

I am not home right now so I cannot test it to get the answer.

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel


[Bug 69120] With dpm=1 vdpau is not usable

2013-09-10 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=69120

--- Comment #12 from Alex Deucher ag...@yahoo.com ---
(In reply to comment #11)
 Hi Alex,
 
 Thanks for the quick turn around.
 
 Looking at the patch it seems to be for RV770 and RV780 will it still work
 for our older cards?

Yes.  It covers the entire 7xx family of cards.

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel


[Bug 68235] Display freezes after login with kernel 3.11.0-rc5 on Cayman with dpm=1

2013-09-10 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=68235

--- Comment #14 from Alex Deucher ag...@yahoo.com ---
Created attachment 85578
  -- https://bugs.freedesktop.org/attachment.cgi?id=85578action=edit
disable various dpm features

I would suggest disabling various dpm features and see if you can narrow down
which, if any, help.  This patch disables just about everything.

ni_dpm_force_performance_level(rdev, RADEON_DPM_FORCED_LEVEL_AUTO) is what
actually sets the dynamic performance switching into motion.  Prior to that the
hw is locked into the low performance level.  I sounds like there is some bad
parameter that is causing a lock up when the smc enables state switching.

Separate from the patch can you also try changing the
ni_dpm_force_performance_level(rdev, RADEON_DPM_FORCED_LEVEL_AUTO) call in
ni_dpm_set_power_state() to low (RADEON_DPM_FORCED_LEVEL_LOW) or high
(RADEON_DPM_FORCED_LEVEL_HIGH) rather than auto?  See if you still get a lock
up.

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel


[Bug 68235] Display freezes after login with kernel 3.11.0-rc5 on Cayman with dpm=1

2013-09-10 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=68235

--- Comment #15 from Alex Deucher ag...@yahoo.com ---
Another thing worth checking, what is the value of module_index passed to
radeon_atom_init_mc_reg_table() in ni_initialize_mc_reg_table() in ni_dpm.c on
your system?

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel


[Bug 64201] OpenCL usage result segmentation fault on r600g with HD6850.

2013-09-10 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=64201

Erdem U. Altınyurt spamjunkea...@gmail.com changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution|--- |FIXED

--- Comment #47 from Erdem U. Altınyurt spamjunkea...@gmail.com ---
Hi Aaron,
Well, without the patch, I can verify that bfgminer started to work on my
HD6850, using mesa/llvm/libclc trunk.

Thanks.

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel


[Bug 64225] bfgminer --scyte generates Segmentation Fault on Northern Island

2013-09-10 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=64225

--- Comment #5 from Erdem U. Altınyurt spamjunkea...@gmail.com ---
Since bfgminer --benchmark --scrypt  command produces the error message of:
Cannot use benchmark mode with scrypt

I cannot report if scrypt is start to work or not.

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel


[Bug 69196] New: gpu lockup and full crash when starting some games in wine

2013-09-10 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=69196

  Priority: medium
Bug ID: 69196
  Assignee: dri-devel@lists.freedesktop.org
   Summary: gpu lockup and full crash when starting some games in
wine
  Severity: normal
Classification: Unclassified
OS: Linux (All)
  Reporter: maxmust...@gmail.com
  Hardware: x86-64 (AMD64)
Status: NEW
   Version: git
 Component: Drivers/DRI/R600
   Product: Mesa

Created attachment 85587
  -- https://bugs.freedesktop.org/attachment.cgi?id=85587action=edit
kernel log from the crash

First of I searched the bugs and found some similar looking bugs but none which
stroke me as the same bug.
lspci tells me my graphics card is 
01:00.0 VGA compatible controller: Advanced Micro Devices, Inc. [AMD/ATI] Cape
Verde XT [Radeon HD 7770 GHz Edition]
I always use everything mesa and kernel vanilla from git.
My kernel has the version 3.11.0-1-08716-g26b0332-dirty.The mesa checkout is 4
days old.
glxinfo tells me Mesa 9.3.0-devel (git-0f6fce1) (if this is relevant)
I tested it with hyperz enabled and disabled both time it failed the same way.
First the display turns dark (no signal) and then after being responsive for a
few seconds the pc crashes completly.
If I kill the process in time it doesn't die completly on me.
I don't know if this is of any relevance but the firmware for the card was
missing and I grabbed the file from here:
http://people.freedesktop.org/~agd5f/radeon_ucode/VERDE_smc.bin
I'll refrain from parsin anything specific from the kernel log since I don't
know which lines are relevant

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel


Re: [git pull] drm tree for 3.12-rc1

2013-09-10 Thread Linus Torvalds
[ Dave - your linux.ie email generates bounces for me, trying redhat instead ]

On Mon, Sep 9, 2013 at 11:25 PM, Sean V Kelley sean.v.kel...@intel.com wrote:

 I'm also a bit bummed that hw acceleration of video doesn't seem to
 work on Haswell, meaning that full-screen is now a jerky mess. I fear
 that that is user-space libraries/X.org, but I thought I'd mention it
 in the hope of getting a oh, it's working for us, you'll get a fix
 for it soon.

 Can you give a little more detail about video not working?  Video
 accel should work fine with the current versions of libva/intel-driver
 available in Fedora 19 - assuming that's what you're using.

It is indeed F19.

Easy test: go to youtube, and watch things that are in 1080p HD. They
play fine in a window (using about 70% CPU), but full-screened to
2560x1440 they play at about one or two frames per second.

Non-HD content seems to be fine even full-screen. Either just because
it's so much easier to do, or because some level of scaling is
hw-accelerated.

It may well be that I'm using chrome (and chrome seems to tend to use
its own library versions), and firefox indeed seems to be a bit
better. But by a bit better I mean closer to full frame rate in
full-screen, but lots of tearing - and it was stil using 70% CPU when
displaying in a window. So I think firefox is also still doing
everything in software but may be better about using threads for it.

My previous i5-670 which was inferior in almost every other way didn't
have these problems.. It had the same 2560x1440 display.

  Linus
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel


Re: [PATCH 2/2] ACPI / video / i915: Remove ACPI backlight if firmware expects Windows 8

2013-09-10 Thread Rafael J. Wysocki
On Tuesday, September 10, 2013 04:53:40 PM Jani Nikula wrote:
 On Mon, 09 Sep 2013, Rafael J. Wysocki r...@sisk.pl wrote:
  On Monday, September 09, 2013 05:21:18 PM Daniel Vetter wrote:
  On Mon, Sep 09, 2013 at 02:16:12PM +0200, Rafael J. Wysocki wrote:
   Hi,
   
   On Monday, September 09, 2013 11:32:10 AM Daniel Vetter wrote:
Hi Aaaron,

Have we grown any clue meanwhile about which Intel boxes need this and 
for
which we still need to keep the acpi backlight around?
   
   First of all, there is a bunch of boxes where ACPI backlight works 
   incorrectly
   because of the Win8 compatibility issue.  [In short, if we say we are 
   compatible
   with Win8, the backlight AML goes into a special code path that is 
   broken on
   those machines. Presumably Win8 uses native backlight control on them 
   and that
   AML code path is never executed there.]  The collection of machines with 
   this
   problem appears to be kind of random (various models from various 
   vendors), but
   I *think* they are machines that originally shipped with Win7 with a Win8
   upgrade option (which in practice requires the BIOS to be updated 
   anyway).
   
   Because of that, last time we tried to switch all of the systems using 
   i915
   and telling the BIOS that they are compatible with Win8 over to native 
   backlight
   control, but that didn't work for two reasons.  The first reason is that 
   some
   user space doesn't know how to use intel_backlight and needs to be 
   taught about
   that (so some systems are just not ready for that switch).  The second 
   and more
   fundamental reason is that the native backlight control simply doesn't 
   work on
   some machines and we don't seem to have any idea why and how to debug 
   this
   particular problem (mostly because we don't have enough information and 
   we
   don't know what to ask for).
   
I've grown _very_ reluctant to just adding tons of quirks to our 
driver for
the backlight.
   
Almost all the quirks we have added recently (or that have been 
proposed
to be added) are for the backlight. Imo that indicates we get something
fundamentally wrong ...
   
   This thing isn't really a quirk.  It rather is an option for the users of
   the systems where ACPI backlight doesn't work to switch over to the 
   native
   backlight control using a command line switch.  This way they can at 
   least
   *see* if the native backlight control works for them and (hopefully) 
   report
   problems if that's not the case.  This gives us a chance to get more
   information about what the problem is and possibly to make some progress
   without making changes for everyone, reverting those changes when they 
   don't
   work etc.
   
   An alternative for them is to pass acpi_osi=!Windows 2012 which will 
   probably
   make the ACPI backlight work for them again, but this rather is a step 
   back,
   because we can't possibly learn anything new from that.
  
  If Win8 is as broken as we are I'm ok with the module option. It just
  sounded to me like right now we don't know of a way to make all machines
  somewhat happy, combined with the other pile of random backlight issues
  the assumption that we do something (maybe something a bit racy) that
  windows doesn't do isn't too far-fetched. So I'm not wary of the machines
  where the aml is busted for acpi_os=win8, but for the others where this
  broke stuff.
  
  Or do I miss something here?
 
  The ACPI video driver doesn't do anything new now.  It only does things that
  did work before we started to tell BIOSes we're compatible with Win8.  The
  reason why they don't work on some machines now is not related to whether or
  not Win8 is broken, but to what is there in the ACPI tables on those 
  machines.
  That *is* pure garbage, but Win8 users don't see that (presumably, because
  Win8 does't execute the AML in question).  We don't see that either with
  acpi_osi=!Windows 2012 (because then we don't execute that AML either).
 
  Whether or not Win8 is broken doesn't matter here.  What matters is that we
  have to deal with broken AML somehow.
 
  One way may be to tell everyone affected by this to pass
  acpi_osi=!Windows 2012 in the kernel command line or possibly create a
  blacklist of those machines in the kernel (which Felipe has been pushing for
  recently) and wash our hands clean of this, but that leaves some 
  exceptionally
  bad taste in my mouth.
 
  The alternative is to try to use native backlight in i915, which we did, but
  that didn't work on some machines.  I don't think we will know why it didn't
  work there before we collect some more information and that's not possible
  without user testing.  So, the idea is to make that testing possible without
  hacking the kernel and that's why we're introducing the new command line
  switch.  And we're going to ask users to try it and report back.
 
 The thing that slightly bugs me with the proposed patches 

[Bug 68235] Display freezes after login with kernel 3.11.0-rc5 on Cayman with dpm=1

2013-09-10 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=68235

--- Comment #18 from Alex Deucher ag...@yahoo.com ---
(In reply to comment #17)
 (In reply to comment #15)
  Another thing worth checking, what is the value of module_index passed to
  radeon_atom_init_mc_reg_table() in ni_initialize_mc_reg_table() in ni_dpm.c
  on your system?
 
 How can I get it? Should I print it in dmesg?

yes, that would be great.

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel


Re: [git pull] drm tree for 3.12-rc1

2013-09-10 Thread Mike Lothian
Hi

The regular flash plugin 10.2 doesn't have vaapi (Intel) support

The Chrome flash plugin (10.7?) doesn't have any vaapi or vdpau support

You best option is to install the vdpau to vaapi wrapper and disable the
Chrome flash plugin https://github.com/i-rinat/libvdpau-va-gl

This will revert you back to the older 10.2 flash version which should work

I think I also had to override a few settings in the flash dot file to get
it to skip GPU validation along with following the instruction for the
above project

Its quite convoluted but least it makes YouTube work

Cheers

Mike
[ Dave - your linux.ie email generates bounces for me, trying redhat
instead ]

On Mon, Sep 9, 2013 at 11:25 PM, Sean V Kelley sean.v.kel...@intel.com
wrote:

 I'm also a bit bummed that hw acceleration of video doesn't seem to
 work on Haswell, meaning that full-screen is now a jerky mess. I fear
 that that is user-space libraries/X.org, but I thought I'd mention it
 in the hope of getting a oh, it's working for us, you'll get a fix
 for it soon.

 Can you give a little more detail about video not working?  Video
 accel should work fine with the current versions of libva/intel-driver
 available in Fedora 19 - assuming that's what you're using.

It is indeed F19.

Easy test: go to youtube, and watch things that are in 1080p HD. They
play fine in a window (using about 70% CPU), but full-screened to
2560x1440 they play at about one or two frames per second.

Non-HD content seems to be fine even full-screen. Either just because
it's so much easier to do, or because some level of scaling is
hw-accelerated.

It may well be that I'm using chrome (and chrome seems to tend to use
its own library versions), and firefox indeed seems to be a bit
better. But by a bit better I mean closer to full frame rate in
full-screen, but lots of tearing - and it was stil using 70% CPU when
displaying in a window. So I think firefox is also still doing
everything in software but may be better about using threads for it.

My previous i5-670 which was inferior in almost every other way didn't
have these problems.. It had the same 2560x1440 display.

  Linus
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel


[Bug 60182] X.Org Server terminate when I close video player

2013-09-10 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=60182

--- Comment #22 from Alex Deucher ag...@yahoo.com ---
Can you try a vanilla xf86-video-ati rather than the ubuntu version?  This
looks like it may be specific to their version.

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel


Re: [git pull] drm tree for 3.12-rc1

2013-09-10 Thread Sean V Kelley
On Tue, Sep 10, 2013 at 08:49:58AM -0700, Linus Torvalds writes :
 [ Dave - your linux.ie email generates bounces for me, trying redhat instead ]
 
 On Mon, Sep 9, 2013 at 11:25 PM, Sean V Kelley sean.v.kel...@intel.com 
 wrote:
 
  I'm also a bit bummed that hw acceleration of video doesn't seem to
  work on Haswell, meaning that full-screen is now a jerky mess. I fear
  that that is user-space libraries/X.org, but I thought I'd mention it
  in the hope of getting a oh, it's working for us, you'll get a fix
  for it soon.
 
  Can you give a little more detail about video not working?  Video
  accel should work fine with the current versions of libva/intel-driver
  available in Fedora 19 - assuming that's what you're using.
 
 It is indeed F19.
 
 Easy test: go to youtube, and watch things that are in 1080p HD. They
 play fine in a window (using about 70% CPU), but full-screened to
 2560x1440 they play at about one or two frames per second.

Yep.  I see the same.  Chrome is brain dead and appears to simply
rely on Flash + FFmpeg software codecs.  At least with Firefox you
know it will suck and not be hw accelerated. Unless that has changed...

 
 Non-HD content seems to be fine even full-screen. Either just because
 it's so much easier to do, or because some level of scaling is
 hw-accelerated.

SD content should be relatively fine with software codecs. Ideally,
full-screen would disable composition and use hw-overlay in addition
to an actual hw accel codec, h.264 or vp8 for html5 video.  But that
is not the case.  That being said HSW does not support vp8 hw decode.

 
 It may well be that I'm using chrome (and chrome seems to tend to use
 its own library versions), and firefox indeed seems to be a bit
 better. But by a bit better I mean closer to full frame rate in
 full-screen, but lots of tearing - and it was stil using 70% CPU when
 displaying in a window. So I think firefox is also still doing
 everything in software but may be better about using threads for it.

Sadly none of those would be hw accel without some specific flavor
of flash plugin that favors hw accel over FFmpeg software accel (i.e.,
has older flash + wrapper for vdpau or vaapi).  As someone else in this
thread mentioned, it is quite a pain in the arse.

Ironically, it is relatively trivial for me to do a chromium browser
build form source myself and enable libva/intel-driver HW accel, h.264.  And say
screw you to flash et al, and leave that out.  And that will
work fine for html5 video on youtube with fallback to SW accel for VP8
based content.  Why it is a chore for Google is not clear to me...

Sean

 
 My previous i5-670 which was inferior in almost every other way didn't
 have these problems.. It had the same 2560x1440 display.
 
   Linus


pgpPtT56X6tii.pgp
Description: PGP signature
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel


[Bug 68235] Display freezes after login with kernel 3.11.0-rc5 on Cayman with dpm=1

2013-09-10 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=68235

--- Comment #16 from Alexandre Demers alexandre.f.dem...@gmail.com ---
(In reply to comment #14)
 Created attachment 85578 [details] [review]
 disable various dpm features
 
 I would suggest disabling various dpm features and see if you can narrow
 down which, if any, help.  This patch disables just about everything.
 
 ni_dpm_force_performance_level(rdev, RADEON_DPM_FORCED_LEVEL_AUTO) is what
 actually sets the dynamic performance switching into motion.  Prior to that
 the hw is locked into the low performance level.  I sounds like there is
 some bad parameter that is causing a lock up when the smc enables state
 switching.
 
 Separate from the patch can you also try changing the
 ni_dpm_force_performance_level(rdev, RADEON_DPM_FORCED_LEVEL_AUTO) call in
 ni_dpm_set_power_state() to low (RADEON_DPM_FORCED_LEVEL_LOW) or high
 (RADEON_DPM_FORCED_LEVEL_HIGH) rather than auto?  See if you still get a
 lock up.

I'll try it later today.

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel


[Bug 60791] Display corruption with Radeon driver during boot and on desktop

2013-09-10 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=60791

--- Comment #30 from Quallenauge hams...@freenet.de ---
The last patch works for me :)
Thanks!

-- 
You are receiving this mail because:
You are watching the assignee of the bug.
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel


[Bug 60182] X.Org Server terminate when I close video player

2013-09-10 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=60182

--- Comment #21 from Weber K. weber...@yahoo.com.br ---
Hello!
I think I found a workaround!

At this file:
https://github.com/RAOF/xf86-video-ati/blob/master/src/radeon_dri2.c

At line 608:
Changed from:
xorg_list_for_each_entry(ref, pClientEventsPriv-reference_list,
link) {
ref-valid = FALSE;
radeon_dri2_unref_buffer(ref-front);
radeon_dri2_unref_buffer(ref-back);
}

Replaced by:
xorg_list_for_each_entry(ref, pClientEventsPriv-reference_list,
link) {
if (ref-valid) {
ref-valid = FALSE;
radeon_dri2_unref_buffer(ref-front);
radeon_dri2_unref_buffer(ref-back);
}
}

I just checked valid before the calls, but I still cannot find what is making
double calls...

Maybe its because I have two boards? Onboard HD4250 e PCIE HD6850?

Thanks!
Weber Kai

bash-4.2# lspci
01:05.0 VGA compatible controller: Advanced Micro Devices [AMD] nee ATI RS880
[Radeon HD 4250]
01:05.1 Audio device: Advanced Micro Devices [AMD] nee ATI RS880 HDMI Audio
[Radeon HD 4200 Series]
02:00.0 VGA compatible controller: Advanced Micro Devices [AMD] nee ATI Barts
PRO [ATI Radeon HD 6800 Series]
02:00.1 Audio device: Advanced Micro Devices [AMD] nee ATI Barts HDMI Audio
[Radeon HD 6800 Series]

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel


Re: [PATCH 0/3] drm/radeon kexec fixes

2013-09-10 Thread Alex Deucher
On Tue, Sep 10, 2013 at 2:27 PM, Eric W. Biederman
ebied...@xmission.com wrote:
 Alex Deucher alexdeuc...@gmail.com writes:

 On Mon, Sep 9, 2013 at 5:21 AM, Markus Trippelsdorf
 mar...@trippelsdorf.de wrote:

 IIRC Alex said the sanity checks are expensive and boot-time could be
 improved by dropping them. Maybe he can chime in?

 They shouldn't be necessary with a proper shutdown, but in this
 particular case, they are not very expensive.  What is expensive is
 having a separate sanity check functions for all the various hw blocks
 to teardown everything on startup prior to starting it up in case
 kexec, etc. left the system in a bad state.  It ends up amounting to a
 full tear down sequence followed by a full start up sequence every
 time you load the driver.

 I can't really comment on the first patch, but the rest seem fine.

 Let me reask the question just a little bit.

 Is it the sanity checks that are expensive?  Or is it the
 reinitialization that is triggered by the sanity checks that is
 expensive?

 From what Christian said in the other reply it sounds like this is a
 game we will never completely win, but it would be nice to have half a
 chance in the kexec on panic case to have video.  So I am curious to
 know if the checks are expensive when we are coming at hardware in a
 clean state.

The particular sanity checks from this patch set are not expensive,
but we had previously discussed more extensive sanity checks for other
aspects of the chips in prior conversations.  Prior to this patch set,
the hw is not torn down properly (might have been in the middle of DMA
for example) when kexec happens.  That's why the sanity checks were
added in the first place.  With this patch set, the sanity checks
shouldn't be necessary.

Alex
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel


[Bug 68235] Display freezes after login with kernel 3.11.0-rc5 on Cayman with dpm=1

2013-09-10 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=68235

--- Comment #17 from Alexandre Demers alexandre.f.dem...@gmail.com ---
(In reply to comment #15)
 Another thing worth checking, what is the value of module_index passed to
 radeon_atom_init_mc_reg_table() in ni_initialize_mc_reg_table() in ni_dpm.c
 on your system?

How can I get it? Should I print it in dmesg?

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel


[Bug 60182] X.Org Server terminate when I close video player

2013-09-10 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=60182

--- Comment #24 from Weber K. weber...@yahoo.com.br ---
Looking at neuromancer Xorg log, I found out:
[ 7.278] (--) RADEON(0): Chipset: SUMO (ChipID = 0x9641)
[ 7.548] (--) RADEON(G0): Chipset: TURKS (ChipID = 0x6741)

I think its a problem only with two devices with radeon driver...
After closing client, the destroy buffer is called twice, and causes the
error...

I can workaround, but I cant fix it the right way... I have no experience with
Xorg or radeon driver... I dont know either how to edit the subject of the
bug...

Please... Help us...

Thanks in advance!
Weber Kai

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel


[Bug 60182] X.Org Server terminate when I close video player

2013-09-10 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=60182

--- Comment #23 from Weber K. weber...@yahoo.com.br ---
Hi!

Ive found that (for me) this layout is causing the error:
Section ServerLayout
  Identifier My X Server
  Screen  0  LeftScreen 0 0
  Screen  1  RightScreen RightOf LeftScreen
InputDevice Mouse1 CorePointer
InputDevice Keyboard1 CoreKeyboard
EndSection

If I remove Screen 1 then the error dont occur... Both devices are using
radeon driver...

I cannot test if the error occur only when both devices are radeon, or if
having two Screens of any type...

I think the destroy buffer is called twice because of two screens...

Anyway, the workaround of checking valid value fixes this issue for me...

Sorry, I cant test vanilla, I am using Slackware64... Ive downloaded the source
DVD and changed x11.SlackBuild... Only changed CFLAGS to -O0 -g3 and removed
strip symbols...

Thanks...
Weber Kai

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel


Re: [git pull] drm tree for 3.12-rc1

2013-09-10 Thread Sean V Kelley
On Tue, Sep 10, 2013 at 1:50 PM, Sean V Kelley sean.v.kel...@intel.com wrote:
 On Tue, Sep 10, 2013 at 08:49:58AM -0700, Linus Torvalds writes :
 [ Dave - your linux.ie email generates bounces for me, trying redhat instead 
 ]

 On Mon, Sep 9, 2013 at 11:25 PM, Sean V Kelley sean.v.kel...@intel.com 
 wrote:
 
  I'm also a bit bummed that hw acceleration of video doesn't seem to
  work on Haswell, meaning that full-screen is now a jerky mess. I fear
  that that is user-space libraries/X.org, but I thought I'd mention it
  in the hope of getting a oh, it's working for us, you'll get a fix
  for it soon.
 
  Can you give a little more detail about video not working?  Video
  accel should work fine with the current versions of libva/intel-driver
  available in Fedora 19 - assuming that's what you're using.

 It is indeed F19.

 Easy test: go to youtube, and watch things that are in 1080p HD. They
 play fine in a window (using about 70% CPU), but full-screened to
 2560x1440 they play at about one or two frames per second.

 Yep.  I see the same.  Chrome is brain dead and appears to simply
 rely on Flash + FFmpeg software codecs.  At least with Firefox you
 know it will suck and not be hw accelerated. Unless that has changed...



 Ironically, it is relatively trivial for me to do a chromium browser
 build form source myself and enable libva/intel-driver HW accel, h.264.  And 
 say
 screw you to flash et al, and leave that out.  And that will
 work fine for html5 video on youtube with fallback to SW accel for VP8
 based content.  Why it is a chore for Google is not clear to me...


Google actually blacklists hardware video acceleration for Linux x86
in Chromium.  My
builds are based on ChromeOS' browser.  That's how I get around it.

For more details...
https://codereview.chromium.org/16430003/

Sean



 Sean


 My previous i5-670 which was inferior in almost every other way didn't
 have these problems.. It had the same 2560x1440 display.

   Linus



-- 
Sean V. Kelley sean.v.kel...@intel.com
Open Source Technology Center / SSG
Intel Corp.
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel


Re: [git pull] drm tree for 3.12-rc1

2013-09-10 Thread Dave Airlie
On Wed, Sep 11, 2013 at 1:49 AM, Linus Torvalds
torva...@linux-foundation.org wrote:
 [ Dave - your linux.ie email generates bounces for me, trying redhat instead ]

Uggh no worries, I read my list mail from gmail mostly anyways, so if
its on a list I'll probably see it.

 Can you give a little more detail about video not working?  Video
 accel should work fine with the current versions of libva/intel-driver
 available in Fedora 19 - assuming that's what you're using.

 It is indeed F19.

 Easy test: go to youtube, and watch things that are in 1080p HD. They
 play fine in a window (using about 70% CPU), but full-screened to
 2560x1440 they play at about one or two frames per second.

It doesn't sound like you are using hw accelerated video at all and its just
sw renderered flash,

In which case it could be a CPU issue, is the cpu freq scaling working
correctly?

Dave.
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel


[Bug 60182] X.Org Server terminate when I close video player

2013-09-10 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=60182

--- Comment #25 from lbdkm...@sharklasers.com ---
Yes, this, aparently, only happens in dual AMD graphics laptops (with llano)...
and I found a workaround:
you have to disable the second (discrete) GPU:

echo OFF  /sys/kernel/debug/vgaswitcheroo/switch

and it will not crash anymore... Sadly, vgaswitcheroo throws a big warning in
dmesg: http://pastebin.com/rXj8rrZa (I'm using kernel 3.11.0-6 from ubuntu's
proposed repos) , after which /sys/kernel/debug/vgaswitcheroo/ disappears
completely and I'm not able to use the dGPU until the next boot. I will have to
report this.. dunno where though. (and also, I always get a black screen after
resume from suspend to RAM).

as I said before, the patch helped, i.e., made it harder to crash, but apps
(glxgears, vlc, mplayer + gl) will crash nonetheless.

I hope this helps. And I want to thank the devs for their great job with the
radeon FOSS driver!

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel


Re: [PATCH] drm/msm: fix return value check in ERR_PTR()

2013-09-10 Thread Rob Clark
On Tue, Sep 10, 2013 at 6:56 PM, Wei Yongjun weiyj...@gmail.com wrote:
 From: Wei Yongjun yongjun_...@trendmicro.com.cn

 In case of error, the function drm_prime_pages_to_sg() returns ERR_PTR()
 and never returns NULL. The NULL test in the return value check should
 be replaced with IS_ERR().

 Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn

good catch, thanks

Signed-off-by: Rob Clark robdcl...@gmail.com

 ---
  drivers/gpu/drm/msm/msm_gem.c | 4 ++--
  1 file changed, 2 insertions(+), 2 deletions(-)

 diff --git a/drivers/gpu/drm/msm/msm_gem.c b/drivers/gpu/drm/msm/msm_gem.c
 index 6b5a6c8..3462def 100644
 --- a/drivers/gpu/drm/msm/msm_gem.c
 +++ b/drivers/gpu/drm/msm/msm_gem.c
 @@ -40,9 +40,9 @@ static struct page **get_pages(struct drm_gem_object *obj)
 }

 msm_obj-sgt = drm_prime_pages_to_sg(p, npages);
 -   if (!msm_obj-sgt) {
 +   if (IS_ERR(msm_obj-sgt)) {
 dev_err(dev-dev, failed to allocate sgt\n);
 -   return ERR_PTR(-ENOMEM);
 +   return ERR_CAST(msm_obj-sgt);
 }

 msm_obj-pages = p;

___
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel


[Bug 69120] With dpm=1 vdpau is not usable

2013-09-10 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=69120

--- Comment #13 from Dieter Nützel die...@nuetzel-hh.de ---
Hurray Alex,

your possible fix from Comment 10 solve it halfway or maybe  fully even on my
slow Duron 1800, RV730 AGP (4650), if we separate the mosaic into a new bug
entry.

The SPEED is there, now.
I can play 2 H.264 VDPAU accelerated videos (854x480 and 1280x720)
simultaneously with ~35% idle rest on this little system.
Have a look at 2-times-mplayer-konsole.png attachment.

I've tested it with AGP and PCIE mode, both with same results.

The mosaic problem IS mouse cursor (hardware cursor?) related!
If the (big/fullscreen) video window coming up 'under' (over?) the mouse cursor
position I get immediately mosaic. When a smaller video window starts from
mouse cursor postion away UVD is excellent. Mosaic starts then not until mouse
cursor move over the video window. With mplayer this could be faster reached if
one seek into the video window.
See 2-times-mplayer-mosaic.png attachment.
Tested with AGP and PCIE mode.

Very nice!

-Dieter

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel


[Bug 69120] With dpm=1 vdpau is not usable

2013-09-10 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=69120

--- Comment #14 from Dieter Nützel die...@nuetzel-hh.de ---
Created attachment 85591
  -- https://bugs.freedesktop.org/attachment.cgi?id=85591action=edit
2-times-mplayer-konsole.png

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel


[Bug 69120] With dpm=1 vdpau is not usable

2013-09-10 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=69120

--- Comment #15 from Dieter Nützel die...@nuetzel-hh.de ---
Created attachment 85592
  -- https://bugs.freedesktop.org/attachment.cgi?id=85592action=edit
2-times-mplayer-mosaic.png

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel


[Bug 69120] With dpm=1 vdpau is not usable

2013-09-10 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=69120

--- Comment #16 from Dieter Nützel die...@nuetzel-hh.de ---
Created attachment 85593
  -- https://bugs.freedesktop.org/attachment.cgi?id=85593action=edit
dmesg-drm-next-3.12-AGP-dpm-1-fix-fallback-for-empty-UVD-clocks.log

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel


[Bug 69120] With dpm=1 vdpau is not usable

2013-09-10 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=69120

--- Comment #17 from Dieter Nützel die...@nuetzel-hh.de ---
Created attachment 85594
  -- https://bugs.freedesktop.org/attachment.cgi?id=85594action=edit
dmesg-drm-next-3.12-PCIE-dpm-1-fix-fallback-for-empty-UVD-clocks.log

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel


[Bug 68235] Display freezes after login with kernel 3.11.0-rc5 on Cayman with dpm=1

2013-09-10 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=68235

--- Comment #19 from Alexandre Demers alexandre.f.dem...@gmail.com ---
(In reply to comment #16)
 (In reply to comment #14)
  Created attachment 85578 [details] [review] [review]
  disable various dpm features
  
  I would suggest disabling various dpm features and see if you can narrow
  down which, if any, help.  This patch disables just about everything.
  
  ni_dpm_force_performance_level(rdev, RADEON_DPM_FORCED_LEVEL_AUTO) is what
  actually sets the dynamic performance switching into motion.  Prior to that
  the hw is locked into the low performance level.  I sounds like there is
  some bad parameter that is causing a lock up when the smc enables state
  switching.
  
  Separate from the patch can you also try changing the
  ni_dpm_force_performance_level(rdev, RADEON_DPM_FORCED_LEVEL_AUTO) call in
  ni_dpm_set_power_state() to low (RADEON_DPM_FORCED_LEVEL_LOW) or high
  (RADEON_DPM_FORCED_LEVEL_HIGH) rather than auto?  See if you still get a
  lock up.
 
 I'll try it later today.

I had time for now to play with forcing RADEON_DPM_FORCED_LEVEL_LOW and
RADEON_DPM_FORCED_LEVEL_HIGH. The first one works fine, the second triggers the
problem.

I'm about to play with the suggested patch.

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel


[Bug 69120] With dpm=1 vdpau is not usable

2013-09-10 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=69120

--- Comment #18 from Dieter Nützel die...@nuetzel-hh.de ---
(In reply to comment #10)
 Created attachment 85563 [details] [review]
 possible fix
 
 The attached patch should fix the issue.

Candidate for stable?

-Dieter

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel


[Bug 60182] X.Org Server terminate when I close video player

2013-09-10 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=60182

--- Comment #26 from Weber K. weber...@yahoo.com.br ---
During a test, I needed to issue 2 cont commands at gdb to advance just one
frame at one breakpoint I trapped. And when closing the player, I issued twice
cont at gdb, and only after the segfault occured.

I suspect the function radeon_dri2_screen_init at radeon_dri2.c is called
twice, one for each Screen, and then this routine is scheduling the
radeon_dri2_client_state_changed routine twice with AddCallBack.

This way, when the client state is changed to gone, the destroy buffers
routine is called twice either, and the error occur.

I also want to thank the devs for the radeon driver!
Thank you!

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel


[Bug 60182] X.Org Server terminate when I close video player

2013-09-10 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=60182

--- Comment #27 from Weber K. weber...@yahoo.com.br ---
I think Ive got it! :)

Changed this file:
https://github.com/RAOF/xf86-video-ati/blob/master/src/radeon_dri2.c

Line 1404 from:
AddCallback(ClientStateCallback, radeon_dri2_client_state_changed, 0);

Replaced by:
if (pScrn-scrnIndex == 0) {
AddCallback(ClientStateCallback, radeon_dri2_client_state_changed, 0);
}

The callback is called only once, even if multiple screens! And the error is
gone!
Please, can someone check if it need to be repeated in other parts as well?
If correct, can someone commit it?

Thank you very much! :)
Weber Kai

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel


[Bug 68775] RV635 (r600) 600_DEBUG=sb causes GPU reset playing Second Life

2013-09-10 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=68775

Shawn Starr shawn.st...@rogers.com changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution|--- |INVALID

--- Comment #5 from Shawn Starr shawn.st...@rogers.com ---
Multiple issues... GPU reset issue was fixed, this is GLSL error, reopening
with new bug, not SB related

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel