Re: [PATCH] Re: DRM oops with unichrome drivers

2004-10-09 Thread Dave Airlie


 Is this a left over from gamma maybe?

correct,

I've killed them in CVS, and I'll push a patch to Linus tree also..

Dave.


  leading to the null dereference.
 
I assume these lines need to be wrapped in an
  if(drm_core_check_feature(dev, DRIVER_HAVE_DMA)) { } or something, but
  I'll leave that to someone who actually understands this code.
 
  Robert
 
 
 




-- 
David Airlie, Software Engineer
http://www.skynet.ie/~airlied / airlied at skynet.ie
pam_smb / Linux DECstation / Linux VAX / ILUG person



---
This SF.net email is sponsored by: IT Product Guide on ITManagersJournal
Use IT products in your business? Tell us what you think of them. Give us
Your Opinions, Get Free ThinkGeek Gift Certificates! Click to find out more
http://productguide.itmanagersjournal.com/guidepromo.tmpl
--
___
Dri-devel mailing list
[EMAIL PROTECTED]
https://lists.sourceforge.net/lists/listinfo/dri-devel


Re: [PATCH] Re: DRM oops with unichrome drivers

2004-10-08 Thread Jon Smirl
On Sat, 09 Oct 2004 00:58:00 +0100, Robert Clark [EMAIL PROTECTED] wrote:
 On Wed, 2004-10-06 at 15:19, Robert Clark wrote:
 
  Oct  5 22:02:21 miller kernel: Oops: 0002 [#1]
 
   The attached (trivial) patch fixes this oops for me but I think this
 is only half the problem.
 
   What seems to have been happening is that, without the DRIVER_HAVE_DMA
 flag, dev-dma was not being initialised. This caused a problem with the
 following code in DRM(irq_install) from drm_irq.h:
 
 dev-dma-next_buffer = NULL;
 dev-dma-next_queue = NULL;
 dev-dma-this_buffer = NULL;

Dave, these variables don't seem to be in use any more. I commented
out their definitions and everything still compiles.

drmP.h
/** \name DMA support */
/[EMAIL PROTECTED] */
//  drm_buf_t *this_buffer; /** Buffer being sent */
//  drm_buf_t *next_buffer; /** Selected buffer to send */
//  drm_queue_t *next_queue;/** Queue from which buffer selected*/
//  wait_queue_head_t waiting;  /** Processes waiting on free bufs */
/[EMAIL PROTECTED] */

Is this a left over from gamma maybe?

 leading to the null dereference.
 
   I assume these lines need to be wrapped in an
 if(drm_core_check_feature(dev, DRIVER_HAVE_DMA)) { } or something, but
 I'll leave that to someone who actually understands this code.
 
 Robert
 
 
 


-- 
Jon Smirl
[EMAIL PROTECTED]


---
This SF.net email is sponsored by: IT Product Guide on ITManagersJournal
Use IT products in your business? Tell us what you think of them. Give us
Your Opinions, Get Free ThinkGeek Gift Certificates! Click to find out more
http://productguide.itmanagersjournal.com/guidepromo.tmpl
--
___
Dri-devel mailing list
[EMAIL PROTECTED]
https://lists.sourceforge.net/lists/listinfo/dri-devel