From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751772AbeBWOvH (ORCPT ); Fri, 23 Feb 2018 09:51:07 -0500 Received: from mail-lf0-f67.google.com ([209.85.215.67]:41064 "EHLO mail-lf0-f67.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751351AbeBWOvG (ORCPT ); Fri, 23 Feb 2018 09:51:06 -0500 X-Google-Smtp-Source: AG47ELvYBa/q1IsxmghHh6m0zxyCHIPtbbmBhgX8S6mJkfACaIlYd7/5lYhTQ/LUneSVIkTXz64H5g== Subject: Re: [PATCH 1/9] drm/xen-front: Introduce Xen para-virtualized frontend driver To: Boris Ostrovsky , xen-devel@lists.xenproject.org, linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org, airlied@linux.ie, daniel.vetter@intel.com, seanpaul@chromium.org, gustavo@padovan.org, jgross@suse.com, konrad.wilk@oracle.com Cc: Oleksandr Andrushchenko References: <1519200222-20623-1-git-send-email-andr2000@gmail.com> <1519200222-20623-2-git-send-email-andr2000@gmail.com> <1c569896-e60c-bed3-fda8-c72b8d0a349d@gmail.com> <3740a458-35d7-b51d-5cd0-be3bebd87574@oracle.com> From: Oleksandr Andrushchenko Message-ID: Date: Fri, 23 Feb 2018 16:51:03 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.6.0 MIME-Version: 1.0 In-Reply-To: <3740a458-35d7-b51d-5cd0-be3bebd87574@oracle.com> Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit Content-Language: en-US Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 02/23/2018 04:39 PM, Boris Ostrovsky wrote: > On 02/23/2018 01:37 AM, Oleksandr Andrushchenko wrote: >> On 02/23/2018 12:23 AM, Boris Ostrovsky wrote: >>> On 02/21/2018 03:03 AM, Oleksandr Andrushchenko wrote: >>>> +static struct xenbus_driver xen_driver = { >>>> + .ids = xen_drv_ids, >>>> + .probe = xen_drv_probe, >>>> + .remove = xen_drv_remove, >>>> + .otherend_changed = backend_on_changed, >>> What does "_on_" stand for? >> Well, it is somewhat like a hint that this is called "on" event, >> e.g. event when the other end state has changed, backend in this >> case. It could be something like "backend_on_state_changed" > If you look at other xenbus_drivers none of the uses this so I think we > should stick to conventional naming. (and the same applies to other > backend_on_* routines). ok, no problem. will rename to be aligned with other frontends > > -boris From mboxrd@z Thu Jan 1 00:00:00 1970 From: Oleksandr Andrushchenko Subject: Re: [PATCH 1/9] drm/xen-front: Introduce Xen para-virtualized frontend driver Date: Fri, 23 Feb 2018 16:51:03 +0200 Message-ID: References: <1519200222-20623-1-git-send-email-andr2000@gmail.com> <1519200222-20623-2-git-send-email-andr2000@gmail.com> <1c569896-e60c-bed3-fda8-c72b8d0a349d@gmail.com> <3740a458-35d7-b51d-5cd0-be3bebd87574@oracle.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8"; Format="flowed" Content-Transfer-Encoding: base64 Return-path: Received: from mail-lf0-x243.google.com (mail-lf0-x243.google.com [IPv6:2a00:1450:4010:c07::243]) by gabe.freedesktop.org (Postfix) with ESMTPS id ADB106F191 for ; Fri, 23 Feb 2018 14:51:06 +0000 (UTC) Received: by mail-lf0-x243.google.com with SMTP id f75so7694434lfg.6 for ; Fri, 23 Feb 2018 06:51:06 -0800 (PST) In-Reply-To: <3740a458-35d7-b51d-5cd0-be3bebd87574@oracle.com> Content-Language: en-US List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" To: Boris Ostrovsky , xen-devel@lists.xenproject.org, linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org, airlied@linux.ie, daniel.vetter@intel.com, seanpaul@chromium.org, gustavo@padovan.org, jgross@suse.com, konrad.wilk@oracle.com Cc: Oleksandr Andrushchenko List-Id: dri-devel@lists.freedesktop.org T24gMDIvMjMvMjAxOCAwNDozOSBQTSwgQm9yaXMgT3N0cm92c2t5IHdyb3RlOgo+IE9uIDAyLzIz LzIwMTggMDE6MzcgQU0sIE9sZWtzYW5kciBBbmRydXNoY2hlbmtvIHdyb3RlOgo+PiBPbiAwMi8y My8yMDE4IDEyOjIzIEFNLCBCb3JpcyBPc3Ryb3Zza3kgd3JvdGU6Cj4+PiBPbiAwMi8yMS8yMDE4 IDAzOjAzIEFNLCBPbGVrc2FuZHIgQW5kcnVzaGNoZW5rbyB3cm90ZToKPj4+PiArc3RhdGljIHN0 cnVjdCB4ZW5idXNfZHJpdmVyIHhlbl9kcml2ZXIgPSB7Cj4+Pj4gKyAgICAuaWRzID0geGVuX2Ry dl9pZHMsCj4+Pj4gKyAgICAucHJvYmUgPSB4ZW5fZHJ2X3Byb2JlLAo+Pj4+ICsgICAgLnJlbW92 ZSA9IHhlbl9kcnZfcmVtb3ZlLAo+Pj4+ICsgICAgLm90aGVyZW5kX2NoYW5nZWQgPSBiYWNrZW5k X29uX2NoYW5nZWQsCj4+PiBXaGF0IGRvZXMgIl9vbl8iIHN0YW5kIGZvcj8KPj4gV2VsbCwgaXQg aXMgc29tZXdoYXQgbGlrZSBhIGhpbnQgdGhhdCB0aGlzIGlzIGNhbGxlZCAib24iIGV2ZW50LAo+ PiBlLmcuIGV2ZW50IHdoZW4gdGhlIG90aGVyIGVuZCBzdGF0ZSBoYXMgY2hhbmdlZCwgYmFja2Vu ZCBpbiB0aGlzCj4+IGNhc2UuIEl0IGNvdWxkIGJlIHNvbWV0aGluZyBsaWtlICJiYWNrZW5kX29u X3N0YXRlX2NoYW5nZWQiCj4gSWYgeW91IGxvb2sgYXQgb3RoZXIgeGVuYnVzX2RyaXZlcnMgbm9u ZSBvZiB0aGUgdXNlcyB0aGlzIHNvIEkgdGhpbmsgd2UKPiBzaG91bGQgc3RpY2sgdG8gY29udmVu dGlvbmFsIG5hbWluZy4gKGFuZCB0aGUgc2FtZSBhcHBsaWVzIHRvIG90aGVyCj4gYmFja2VuZF9v bl8qIHJvdXRpbmVzKS4Kb2ssIG5vIHByb2JsZW0uIHdpbGwgcmVuYW1lIHRvIGJlIGFsaWduZWQg d2l0aCBvdGhlciBmcm9udGVuZHMKPgo+IC1ib3JpcwoKX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX18KZHJpLWRldmVsIG1haWxpbmcgbGlzdApkcmktZGV2ZWxA bGlzdHMuZnJlZWRlc2t0b3Aub3JnCmh0dHBzOi8vbGlzdHMuZnJlZWRlc2t0b3Aub3JnL21haWxt YW4vbGlzdGluZm8vZHJpLWRldmVsCg==