From: Thomas H. <th...@sh...> - 2009-10-08 13:40:51
|
Bruce, Bru...@vi... wrote: > Hello All: > This is patch 1 of the 5 patches. It's based on 2.6.32-rc3 and for the support of ACPI suspend/resume. > To support suspend / resume, some information located in video memory need to be saved because > 3D sync mechanism will use these information after resume. 6 files has been modified for this patch. > > Sign-off by Bruce c. Chang <bru...@vi...> > With the non-modesetting drm model, this is the task of the X server, mainly because there might be an fbdev driver like viafb that owns the device at suspend time. If the drm driver were to do the same, there would be a clash. For example, the openChrome X driver handles (or at least should handle) this in the leavevt / entervt functions that are called before suspend and after resume like so: leavevt { dri_lock(); copy_all_dri_vram_to_system_using_pci_dma(); } entervt { copy_saved_dri_vram_back(); init_3d_engine(); dri_unlock(); } It might be that one should add functionality to take down agp and restore agp command submission as well. Could you describe why you don't use this scheme? Also, could you describe in what situations INIT_JUDGE is needed? Thanks, Thomas > diff -Nur linux-2.6.32-rc3-old/drivers/gpu/drm/via/via_dma.c linux-2.6.32-rc3-new/drivers/gpu/drm/via/via_dma.c > --- linux-2.6.32-rc3-old/drivers/gpu/drm/via/via_dma.c 2009-10-05 08:12:30.000000000 +0800 > +++ linux-2.6.32-rc3-new/drivers/gpu/drm/via/via_dma.c 2009-10-08 10:11:59.000000000 +0800 > @@ -68,6 +68,8 @@ > *vb++ = (w2); \ > dev_priv->dma_low += 8; > > +static void via_cmdbuf_flush(struct drm_via_private *dev_priv, > + uint32_t cmd_type); > static void via_cmdbuf_start(drm_via_private_t * dev_priv); > static void via_cmdbuf_pause(drm_via_private_t * dev_priv); > static void via_cmdbuf_reset(drm_via_private_t * dev_priv); > @@ -155,6 +157,11 @@ > > int via_dma_cleanup(struct drm_device * dev) > { > + struct drm_via_video_save_head *pnode; > + > + for (pnode = via_video_save_head; pnode; pnode = > + (struct drm_via_video_save_head *)pnode->next) > + memcpy(pnode->psystemmem, pnode->pvideomem, pnode->size); > if (dev->dev_private) { > drm_via_private_t *dev_priv = > (drm_via_private_t *) dev->dev_private; > @@ -175,6 +182,7 @@ > drm_via_private_t * dev_priv, > drm_via_dma_init_t * init) > { > + struct drm_via_video_save_head *pnode; > if (!dev_priv || !dev_priv->mmio) { > DRM_ERROR("via_dma_init called before via_map_init\n"); > return -EFAULT; > @@ -195,6 +203,9 @@ > return -EINVAL; > } > > + for (pnode = via_video_save_head; pnode; pnode = pnode->next) > + memcpy(pnode->pvideomem, pnode->psystemmem, pnode->size); > + > dev_priv->ring.map.offset = dev->agp->base + init->offset; > dev_priv->ring.map.size = init->size; > dev_priv->ring.map.type = 0; > @@ -737,6 +748,155 @@ > return ret; > } > > +/*The following functions are for ACPI*/ > + > +static void initialize3Dengine(drm_via_private_t *dev_priv) > +{ > + int i = 0; > + > + VIA_WRITE(0x43C, 0x00010000); > + > + for (i = 0; i <= 0x7D; i++) > + VIA_WRITE(0x440, (unsigned long) i << 24); > + > + VIA_WRITE(0x43C, 0x00020000); > + > + for (i = 0; i <= 0x94; i++) > + VIA_WRITE(0x440, (unsigned long) i << 24); > + > + VIA_WRITE(0x440, 0x82400000); > + VIA_WRITE(0x43C, 0x01020000); > + > + for (i = 0; i <= 0x94; i++) > + VIA_WRITE(0x440, (unsigned long) i << 24); > + > + VIA_WRITE(0x440, 0x82400000); > + VIA_WRITE(0x43C, 0xfe020000); > + > + for (i = 0; i <= 0x03; i++) > + VIA_WRITE(0x440, (unsigned long) i << 24); > + > + VIA_WRITE(0x43C, 0x00030000); > + > + for (i = 0; i <= 0xff; i++) > + VIA_WRITE(0x440, 0); > + > + VIA_WRITE(0x43C, 0x00100000); > + VIA_WRITE(0x440, 0x00333004); > + VIA_WRITE(0x440, 0x10000002); > + VIA_WRITE(0x440, 0x60000000); > + VIA_WRITE(0x440, 0x61000000); > + VIA_WRITE(0x440, 0x62000000); > + VIA_WRITE(0x440, 0x63000000); > + VIA_WRITE(0x440, 0x64000000); > + > + VIA_WRITE(0x43C, 0x00fe0000); > + VIA_WRITE(0x440, 0x40008c0f); > + VIA_WRITE(0x440, 0x44000000); > + VIA_WRITE(0x440, 0x45080C04); > + VIA_WRITE(0x440, 0x46800408); > + VIA_WRITE(0x440, 0x50000000); > + VIA_WRITE(0x440, 0x51000000); > + VIA_WRITE(0x440, 0x52000000); > + VIA_WRITE(0x440, 0x53000000); > + > + > + VIA_WRITE(0x43C, 0x00fe0000); > + VIA_WRITE(0x440, 0x08000001); > + VIA_WRITE(0x440, 0x0A000183); > + VIA_WRITE(0x440, 0x0B00019F); > + VIA_WRITE(0x440, 0x0C00018B); > + VIA_WRITE(0x440, 0x0D00019B); > + VIA_WRITE(0x440, 0x0E000000); > + VIA_WRITE(0x440, 0x0F000000); > + VIA_WRITE(0x440, 0x10000000); > + VIA_WRITE(0x440, 0x11000000); > + VIA_WRITE(0x440, 0x20000000); > +} > +/* For acpi case, when system resume from suspend or hibernate, > + * need to re-initialize dma info into HW > + */ > +int via_drm_resume(struct drm_device *drm_dev) > +{ > + struct pci_dev *pci = drm_dev->pdev; > + drm_via_private_t *dev_priv = > + (drm_via_private_t *) drm_dev->dev_private; > + struct drm_via_video_save_head *pnode = 0; > + > + pci_set_power_state(pci, PCI_D0); > + pci_restore_state(pci); > + if (pci_enable_device(pci)) > + return -1; > + /*pci_set_master(pci);*/ > + if (!dev_priv->initialize) > + return 0; > + /* when resume, initialize 3d registers */ > + initialize3Dengine(dev_priv); > + > + /* here we need to restore some video memory content */ > + for (pnode = via_video_save_head; pnode; pnode = pnode->next) > + memcpy(pnode->pvideomem, pnode->psystemmem, pnode->size); > + > + /* if pci path, return */ > + if (!dev_priv->ring.virtual_start) > + return 0; > + > + dev_priv->dma_ptr = dev_priv->ring.virtual_start; > + dev_priv->dma_low = 0; > + dev_priv->dma_high = 0x1000000; > + dev_priv->dma_wrap = 0x1000000; > + dev_priv->dma_offset = 0x0; > + dev_priv->last_pause_ptr = NULL; > + dev_priv->hw_addr_ptr = dev_priv->mmio->handle + 0x418; > + > + via_cmdbuf_start(dev_priv); > + > + return 0; > +} > + > +int via_drm_suspend(struct drm_device *drm_dev, pm_message_t state) > +{ > + struct pci_dev *pci = drm_dev->pdev; > + drm_via_private_t *dev_priv = > + (drm_via_private_t *) drm_dev->dev_private; > + > + struct drm_via_video_save_head *pnode = 0; > + > + pci_save_state(pci); > + > + if (!dev_priv->initialize) > + return 0; > + /*here we need to save some video mem information into system memory, > + to keep the system consistent between suspend *before* and *after* > + 1.save only necessary */ > + for (pnode = via_video_save_head; pnode; > + pnode = (struct drm_via_video_save_head *)pnode->next) > + memcpy(pnode->psystemmem, pnode->pvideomem, pnode->size); > + > + /* Only agp path need to flush the cmd */ > + if (dev_priv->ring.virtual_start) > + via_cmdbuf_reset(dev_priv); > + > + if (state.event == PM_EVENT_SUSPEND) { > + /* Shut down the device */ > + pci_disable_device(pci); > + pci_set_power_state(pci, PCI_D3hot); > + } > + return 0; > +} > + > +/* For user mode to query whether the drm has been initialized or not */ > +int via_drm_init_judge(struct drm_device *dev, void *data, > + struct drm_file *file_priv) > +{ > + struct drm_via_private *dev_priv = dev->dev_private; > + > + if (dev_priv->initialize) > + *(int *)data = 1; > + else > + *(int *)data = -1; > + return 0; > +} > struct drm_ioctl_desc via_ioctls[] = { > DRM_IOCTL_DEF(DRM_VIA_ALLOCMEM, via_mem_alloc, DRM_AUTH), > DRM_IOCTL_DEF(DRM_VIA_FREEMEM, via_mem_free, DRM_AUTH), > @@ -751,7 +911,8 @@ > DRM_IOCTL_DEF(DRM_VIA_CMDBUF_SIZE, via_cmdbuf_size, DRM_AUTH), > DRM_IOCTL_DEF(DRM_VIA_WAIT_IRQ, via_wait_irq, DRM_AUTH), > DRM_IOCTL_DEF(DRM_VIA_DMA_BLIT, via_dma_blit, DRM_AUTH), > - DRM_IOCTL_DEF(DRM_VIA_BLIT_SYNC, via_dma_blit_sync, DRM_AUTH) > + DRM_IOCTL_DEF(DRM_VIA_BLIT_SYNC, via_dma_blit_sync, DRM_AUTH), > + DRM_IOCTL_DEF(DRM_VIA_INIT_JUDGE, via_drm_init_judge, 0) > }; > > int via_max_ioctl = DRM_ARRAY_SIZE(via_ioctls); > diff -Nur linux-2.6.32-rc3-old/drivers/gpu/drm/via/via_drv.c linux-2.6.32-rc3-new/drivers/gpu/drm/via/via_drv.c > --- linux-2.6.32-rc3-old/drivers/gpu/drm/via/via_drv.c 2009-10-05 08:12:30.000000000 +0800 > +++ linux-2.6.32-rc3-new/drivers/gpu/drm/via/via_drv.c 2009-10-08 10:11:59.000000000 +0800 > @@ -38,6 +38,8 @@ > DRIVER_IRQ_SHARED, > .load = via_driver_load, > .unload = via_driver_unload, > + .suspend = via_drm_suspend, > + .resume = via_drm_resume, > .context_dtor = via_final_context, > .get_vblank_counter = via_get_vblank_counter, > .enable_vblank = via_enable_vblank, > diff -Nur linux-2.6.32-rc3-old/drivers/gpu/drm/via/via_drv.h linux-2.6.32-rc3-new/drivers/gpu/drm/via/via_drv.h > --- linux-2.6.32-rc3-old/drivers/gpu/drm/via/via_drv.h 2009-10-05 08:12:30.000000000 +0800 > +++ linux-2.6.32-rc3-new/drivers/gpu/drm/via/via_drv.h 2009-10-08 10:11:59.000000000 +0800 > @@ -95,6 +95,7 @@ > unsigned long agp_offset; > drm_via_blitq_t blit_queues[VIA_NUM_BLIT_ENGINES]; > uint32_t dma_diff; > + int initialize; > } drm_via_private_t; > > enum via_family { > @@ -154,4 +155,7 @@ > extern void via_dmablit_handler(struct drm_device *dev, int engine, int from_irq); > extern void via_init_dmablit(struct drm_device *dev); > > +extern int via_drm_resume(struct drm_device *dev); > +extern int via_drm_suspend(struct drm_device *dev, pm_message_t state); > + > #endif > diff -Nur linux-2.6.32-rc3-old/drivers/gpu/drm/via/via_map.c linux-2.6.32-rc3-new/drivers/gpu/drm/via/via_map.c > --- linux-2.6.32-rc3-old/drivers/gpu/drm/via/via_map.c 2009-10-05 08:12:30.000000000 +0800 > +++ linux-2.6.32-rc3-new/drivers/gpu/drm/via/via_map.c 2009-10-08 10:11:59.000000000 +0800 > @@ -65,12 +65,24 @@ > via_init_dmablit(dev); > > dev->dev_private = (void *)dev_priv; > + > + /* from doing this, we has the stuff in prev data > + * structure to manage agp > + */ > + dev->agp_buffer_map = drm_core_findmap(dev, dev_priv->agpAddr); > + if (!dev->agp_buffer_map) > + DRM_INFO("Dma buffer region not exist!\n"); > + > + /* end */ > + dev_priv->initialize = 1; > return 0; > } > > int via_do_cleanup_map(struct drm_device * dev) > { > + drm_via_private_t *dev_priv = dev->dev_private; > via_dma_cleanup(dev); > + dev_priv->initialize = 0; > > return 0; > } > diff -Nur linux-2.6.32-rc3-old/drivers/gpu/drm/via/via_mm.c linux-2.6.32-rc3-new/drivers/gpu/drm/via/via_mm.c > --- linux-2.6.32-rc3-old/drivers/gpu/drm/via/via_mm.c 2009-10-05 08:12:30.000000000 +0800 > +++ linux-2.6.32-rc3-new/drivers/gpu/drm/via/via_mm.c 2009-10-08 10:11:59.000000000 +0800 > @@ -30,6 +30,7 @@ > #include "via_drv.h" > #include "drm_sman.h" > > +struct drm_via_video_save_head *via_video_save_head; > #define VIA_MM_ALIGN_SHIFT 4 > #define VIA_MM_ALIGN_MASK ( (1 << VIA_MM_ALIGN_SHIFT) - 1) > > @@ -114,6 +115,90 @@ > mutex_unlock(&dev->struct_mutex); > } > > +/* This function is specially for saving some information > + * about video memory allocation. > + * when the video allocated needed to be saved automatically in acpi, > + * we need to call this function to record it for later use. > + */ > +static int via_videomem_presave_ok(drm_via_private_t *dev_priv, > + drm_via_mem_t *mem) > +{ > + void *pvideomem = 0, *psystemmem = 0; > + struct drm_via_video_save_head *pnode = 0; > + > + if (!mem || !mem->size || (mem->type != VIA_MEM_VIDEO_SAVE)) > + return 0; > + > + /* here the mem->offset is the absolute address, > + * not the offset within videomem > + */ > + pvideomem = (void *)ioremap(dev_priv->fb->offset + mem->offset, > + mem->size); > + if (!pvideomem) > + return 0; > + > + psystemmem = kmalloc(mem->size, GFP_KERNEL); > + if (!psystemmem) { > + iounmap(pvideomem); > + return 0; > + } > + > + /* map success, then save this information into > + * a data structure for later saving usage > + */ > + pnode = kmalloc( > + sizeof(struct drm_via_video_save_head), GFP_KERNEL); > + if (!pnode) { > + iounmap(pvideomem); > + kfree(psystemmem); > + return 0; > + } > + > + pnode->next = 0; > + pnode->psystemmem = psystemmem; > + pnode->pvideomem = pvideomem; > + pnode->size = mem->size; > + pnode->token = mem->offset; > + > + /* insert this node into list */ > + if (!via_video_save_head) { > + via_video_save_head = pnode; > + } else { > + pnode->next = via_video_save_head; > + via_video_save_head = pnode; > + } > + > + return 1; > +} > + > +/* please refer to function via_videomem_presave_ok */ > +static int via_videomem_housekeep_ok(drm_via_mem_t *mem) > +{ > + struct drm_via_video_save_head **ppnode = 0; > + struct drm_via_video_save_head *tmpnode = 0; > + /* if this mem's token match with one node of the list */ > + for (ppnode = &via_video_save_head; *ppnode; > + ppnode = (struct drm_via_video_save_head **)(&((*ppnode)->next))) { > + if ((*ppnode)->token == mem->offset) > + break; > + } > + > + if (*ppnode == 0) { > + /* not found, the user may specify the wrong mem node to free */ > + return 0; > + } > + > + /* delete this node from the list and then > + *free all the mem to avoid memory leak > + */ > + tmpnode = *ppnode; > + *ppnode = (*ppnode)->next; > + iounmap(tmpnode->pvideomem); > + kfree(tmpnode->psystemmem); > + kfree(tmpnode); > + > + return 1; > +} > int via_mem_alloc(struct drm_device *dev, void *data, > struct drm_file *file_priv) > { > @@ -123,12 +208,13 @@ > drm_via_private_t *dev_priv = (drm_via_private_t *) dev->dev_private; > unsigned long tmpSize; > > - if (mem->type > VIA_MEM_AGP) { > + if (mem->type > VIA_MEM_VIDEO_SAVE) { > DRM_ERROR("Unknown memory type allocation\n"); > return -EINVAL; > } > mutex_lock(&dev->struct_mutex); > - if (0 == ((mem->type == VIA_MEM_VIDEO) ? dev_priv->vram_initialized : > + if (0 == ((mem->type == VIA_MEM_VIDEO || > + mem->type == VIA_MEM_VIDEO_SAVE) ? dev_priv->vram_initialized : > dev_priv->agp_initialized)) { > DRM_ERROR > ("Attempt to allocate from uninitialized memory manager.\n"); > @@ -137,15 +223,25 @@ > } > > tmpSize = (mem->size + VIA_MM_ALIGN_MASK) >> VIA_MM_ALIGN_SHIFT; > - item = drm_sman_alloc(&dev_priv->sman, mem->type, tmpSize, 0, > - (unsigned long)file_priv); > + item = drm_sman_alloc(&dev_priv->sman, > + (mem->type == VIA_MEM_VIDEO_SAVE ? VIA_MEM_VIDEO : mem->type), > + tmpSize, 0, (unsigned long)file_priv); > mutex_unlock(&dev->struct_mutex); > if (item) { > - mem->offset = ((mem->type == VIA_MEM_VIDEO) ? > - dev_priv->vram_offset : dev_priv->agp_offset) + > - (item->mm-> > - offset(item->mm, item->mm_info) << VIA_MM_ALIGN_SHIFT); > + mem->offset = ((mem->type == VIA_MEM_VIDEO || > + mem->type == VIA_MEM_VIDEO_SAVE) ? > + dev_priv->vram_offset : dev_priv->agp_offset) + > + (item->mm->offset(item->mm, item->mm_info) << > + VIA_MM_ALIGN_SHIFT); > mem->index = item->user_hash.key; > + if (mem->type == VIA_MEM_VIDEO_SAVE) { > + if (!via_videomem_presave_ok(dev_priv, mem)) { > + mutex_lock(&dev->struct_mutex); > + drm_sman_free_key(&dev_priv->sman, mem->index); > + mutex_unlock(&dev->struct_mutex); > + retval = -ENOMEM; > + } > + } > } else { > mem->offset = 0; > mem->size = 0; > @@ -165,6 +261,10 @@ > > mutex_lock(&dev->struct_mutex); > ret = drm_sman_free_key(&dev_priv->sman, mem->index); > + if (mem->type == VIA_MEM_VIDEO_SAVE) { > + if (!via_videomem_housekeep_ok(mem)) > + ret = -EINVAL; > + } > mutex_unlock(&dev->struct_mutex); > DRM_DEBUG("free = 0x%lx\n", mem->index); > > diff -Nur linux-2.6.32-rc3-old/include/drm/via_drm.h linux-2.6.32-rc3-new/include/drm/via_drm.h > --- linux-2.6.32-rc3-old/include/drm/via_drm.h 2009-10-05 08:12:30.000000000 +0800 > +++ linux-2.6.32-rc3-new/include/drm/via_drm.h 2009-10-08 10:11:59.000000000 +0800 > @@ -79,6 +79,7 @@ > #define DRM_VIA_WAIT_IRQ 0x0d > #define DRM_VIA_DMA_BLIT 0x0e > #define DRM_VIA_BLIT_SYNC 0x0f > +#define DRM_VIA_INIT_JUDGE 0x16 > > #define DRM_IOCTL_VIA_ALLOCMEM DRM_IOWR(DRM_COMMAND_BASE + DRM_VIA_ALLOCMEM, drm_via_mem_t) > #define DRM_IOCTL_VIA_FREEMEM DRM_IOW( DRM_COMMAND_BASE + DRM_VIA_FREEMEM, drm_via_mem_t) > @@ -95,6 +96,8 @@ > #define DRM_IOCTL_VIA_WAIT_IRQ DRM_IOWR( DRM_COMMAND_BASE + DRM_VIA_WAIT_IRQ, drm_via_irqwait_t) > #define DRM_IOCTL_VIA_DMA_BLIT DRM_IOW(DRM_COMMAND_BASE + DRM_VIA_DMA_BLIT, drm_via_dmablit_t) > #define DRM_IOCTL_VIA_BLIT_SYNC DRM_IOW(DRM_COMMAND_BASE + DRM_VIA_BLIT_SYNC, drm_via_blitsync_t) > +#define DRM_IOCTL_VIA_INIT_JUDGE DRM_IOR(DRM_COMMAND_BASE + \ > + DRM_VIA_INIT_JUDGE, int) > > /* Indices into buf.Setup where various bits of state are mirrored per > * context and per buffer. These can be fired at the card as a unit, > @@ -114,6 +117,7 @@ > #define VIA_MEM_SYSTEM 2 > #define VIA_MEM_MIXED 3 > #define VIA_MEM_UNKNOWN 4 > +#define VIA_MEM_VIDEO_SAVE 2 /*For video memory need to be saved in ACPI */ > > typedef struct { > __u32 offset; > @@ -274,4 +278,13 @@ > drm_via_blitsync_t sync; > } drm_via_dmablit_t; > > +struct drm_via_video_save_head { > + void *pvideomem; > + void *psystemmem; > + int size; > + /* token used to identify this video memory */ > + unsigned long token; > + void *next; > +} ; > +extern struct drm_via_video_save_head *via_video_save_head; > #endif /* _VIA_DRM_H_ */ > > Thanks and Best regards > ================================================= > Bruce C. Chang(張祖明) > VIA Technologies, Inc. > Address: 1F, 531, Chung-Cheng Road, Hsin-Tien, 231 Taipei > Tel: +886-2-22185452 Ext 7323 > Mobile: +886-968343824 > Fax: +886-2-22186282 > Skype: Bruce.C.Chang > Email: Bru...@vi... > |