<html> <head> <meta http-equiv="Content-Type" content="text/html; charset=utf-8"> <meta name="Generator" content="Microsoft Exchange Server"> <style></style> </head> <body> <div> <div> <div style="font-family:Calibri,sans-serif; font-size:11pt">Looks OK <br> <br> Best,<br> Maxim Nestratov</div> </div> <div dir="ltr"> <hr> <span style="font-family:Calibri,sans-serif; font-size:11pt; font-weight:bold">От: </span><span style="font-family:Calibri,sans-serif; font-size:11pt"><a href="mailto:dguryanov@parallels.com">Dmitry Guryanov</a></span><br> <span style="font-family:Calibri,sans-serif; font-size:11pt; font-weight:bold">Отправлено: </span><span style="font-family:Calibri,sans-serif; font-size:11pt">‎10.‎12.‎2014 17:30</span><br> <span style="font-family:Calibri,sans-serif; font-size:11pt; font-weight:bold">Кому: </span><span style="font-family:Calibri,sans-serif; font-size:11pt"><a href="mailto:libvir-list@redhat.com">libvir-list@redhat.com</a></span><br> <span style="font-family:Calibri,sans-serif; font-size:11pt; font-weight:bold">Копия: </span><span style="font-family:Calibri,sans-serif; font-size:11pt"><a href="mailto:mipt-aburluka@parallels.com">Alexander Burluka</a>; <a href="mailto:mnestratov@parallels.com">Maxim Nestratov</a>; <a href="mailto:dguryanov@parallels.com"> Dmitry Guryanov</a></span><br> <span style="font-family:Calibri,sans-serif; font-size:11pt; font-weight:bold">Тема: </span><span style="font-family:Calibri,sans-serif; font-size:11pt">[PATCH 1/3] parallels: support NULL virDomainVideoAccelDefPtr</span><br> <br> </div> </div> <font size="2"><span style="font-size:10pt;"> <div class="PlainText">I support if virDomainVideoAccelDefPtr is NULL it means<br> default values for video acceleration. So we don't need<br> to report error.<br> <br> Signed-off-by: Dmitry Guryanov <dguryanov@parallels.com><br> ---<br> src/parallels/parallels_sdk.c | 2 +-<br> 1 file changed, 1 insertion(+), 1 deletion(-)<br> <br> diff --git a/src/parallels/parallels_sdk.c b/src/parallels/parallels_sdk.c<br> index 0b05bc1..0980f50 100644<br> --- a/src/parallels/parallels_sdk.c<br> +++ b/src/parallels/parallels_sdk.c<br> @@ -2029,7 +2029,7 @@ static int prlsdkCheckVideoUnsupportedParams(virDomainDefPtr def)<br> return -1;<br> }<br> <br> - if (v->accel == NULL || v->accel->support2d || v->accel->support3d) {<br> + if (v->accel != NULL && (v->accel->support2d || v->accel->support3d)) {<br> virReportError(VIR_ERR_CONFIG_UNSUPPORTED, "%s",<br> _("Parallels Cloud Server doesn't support "<br> "setting video acceleration parameters."));<br> -- <br> 1.9.3<br> <br> </div> </span></font> </body> </html>