Direct Rendering Infrastructure

Email Archive: dri-devel (read-only)

[Patch VIA UniChrome DRM][0/3 Ver2] Enhance DRM robustness

From: <BruceChang@vi...> - 2009-10-30 06:04

Hello Sirs:
    Based on the comment for my vious submission for VIA UniChrome DRM, we modify our code as the following patch. They are
1. Add support for ACPI suspend/resume issue.
2. Add an interfaces for communication with ddmpeg and v4l.
3. Fix video hang issue caused by verification function.
These patches are based on 2.6.32-rc5.
Please kindly help to check. And please kindly integrate it into mainline if it's OK.

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: BruceChang@via.com.tw

Thread View

Thread	Author	Date
[Patch VIA UniChrome DRM][3/5 Ver1] Enhance DRM robustness	<BruceChang@vi...>

Re: [Patch VIA UniChrome DRM][3/5 Ver1] Enhance DRM robustness

From: Thomas Hellstrom <thomas@sh...> - 2009-10-08 13:23

Bruce,
This looks OK, except for one modification below. Could you comment on
that one?

BruceChang@via.com.tw wrote:
> 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-08 10:17:39.000000000 +0800
> +++ linux-2.6.32-rc3-new/drivers/gpu/drm/via/via_mm.c 2009-10-08 10:18:08.000000000 +0800
> @@ -91,6 +91,17 @@ /> > int via_final_context(struct drm_device *dev, int context)
> {
> drm_via_private_t *dev_priv = (drm_via_private_t *) dev->dev_private;
> + struct drm_ctx_list *pos, *n;
> + bool context_valid = 0;
> +
> + list_for_each_entry_safe(pos, n, &dev->ctxlist, head)
> + if (pos->handle == context) {
> + context_valid = 1;
> + break;
> + }
> +
> + if (!context_valid)
> + return 0;
>
> via_release_futex(dev_priv, context);
>
When is this function called with an invalid context? Could you describe
what error you are seeing.

Thanks,

Thomas

RE: [Patch VIA UniChrome DRM][3/5 Ver1] Enhance DRM robustness

From: <BruceChang@vi...> - 2009-10-09 09:25

Hello Thomas
>> + if (!context_valid)
>> + return 0;
>>
>> via_release_futex(dev_priv, context);
> When is this function called with an invalid context? Could you describe
> what error you are seeing.

There are some bugs in the DDMPEG(a user mode library realizing decode driver) before, which destroy his context more than once, it was the behavior causes DRM segment fault. We add this patch for enhancing DRM robustness, although it is DDMPEG's fault.

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: BruceChang@via.com.tw

[Patch VIA UniChrome DRM][0/3 Ver2] Enhance DRM robustness

From: <BruceChang@vi...> - 2009-10-30 06:04

Hello Sirs:
Based on the comment for my vious submission for VIA UniChrome DRM, we modify our code as the following patch. They are
1. Add support for ACPI suspend/resume issue.
2. Add an interfaces for communication with ddmpeg and v4l.
3. Fix video hang issue caused by verification function.
These patches are based on 2.6.32-rc5.
Please kindly help to check. And please kindly integrate it into mainline if it's OK.

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: BruceChang@via.com.tw

RE: [Patch VIA UniChrome DRM][0/3 Ver2] Enhance DRM robustness

From: <BruceChang@vi...> - 2009-11-03 13:02