Please do not reply to this email: if you want to comment on the bug, go to    
       
the URL shown below and enter yourcomments there.     
   
https://bugs.freedesktop.org/show_bug.cgi?id=3998          
     
           Summary: request for removing virt_to_bus from drm_bufs.c
           Product: DRI
           Version: DRI CVS
          Platform: PC
        OS/Version: Linux
            Status: NEW
          Severity: enhancement
          Priority: P2
         Component: DRM modules
        AssignedTo: [email protected]
        ReportedBy: [EMAIL PROTECTED]


Hi,

I just checked out DRM from CVS and got an unresolved symbol on my ppc64
machine, because virt_to_bus does not exist on this architecture.

This is the part from drm/linux-core/drm_bufs.c which uses this function (DRM 
CVS):

[...] (line 811)
for (offset = 0;
     offset + size <= total && entry->buf_count < count;
     offset += alignment, ++entry->buf_count) {
        buf = &entry->buflist[entry->buf_count];
        buf->idx = dma->buf_count + entry->buf_count;
        buf->total = alignment;
        buf->order = order;
        buf->used = 0;
        buf->offset = (dma->byte_count + byte_count + offset);
        buf->address = (void *)(page + offset);
        buf->bus_address = virt_to_bus(buf->address);           #### this line
        buf->next = NULL;
        buf->waiting = 0;
        buf->pending = 0;
        init_waitqueue_head(&buf->dma_wait);
        buf->filp = NULL;
[...]


and here is the according part from the linux kernel
(linux-2.6.12/drivers/char/drm/drm_bufs.c):

[...] (line 653)
for ( offset = 0 ;
      offset + size <= total && entry->buf_count < count ;
      offset += alignment, ++entry->buf_count ) {
        buf          = &entry->buflist[entry->buf_count];
        buf->idx     = dma->buf_count + entry->buf_count;
        buf->total   = alignment;
        buf->order   = order;
        buf->used    = 0;
        buf->offset  = (dma->byte_count + byte_count + offset);
        buf->address = (void *)(page + offset);
        buf->next    = NULL;
        buf->waiting = 0;
        buf->pending = 0;
        init_waitqueue_head( &buf->dma_wait );
        buf->filp    = NULL;
[...]

It seems like that line was just ommited. If it is not needed, wouldn't it be
possible to remove it from DRM CVS, too? That would resolve the unresolved 
symbol.

Regards,

Markus Rothe          
     
     
--           
Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email         
     
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.


-------------------------------------------------------
SF.Net email is Sponsored by the Better Software Conference & EXPO
September 19-22, 2005 * San Francisco, CA * Development Lifecycle Practices
Agile & Plan-Driven Development * Managing Projects & Teams * Testing & QA
Security * Process Improvement & Measurement * http://www.sqe.com/bsce5sf
--
_______________________________________________
Dri-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/dri-devel

Reply via email to