Re: [PATCH wayland-protocols v2] Introduce xdg-foreign protocol

2017-09-12 Thread Marco Martin
On Tue, Aug 22, 2017 at 6:07 PM, Marco Martin  wrote:
>> You'll need to create an "unstable v2" version, as this is a
>> non-backward compatible change. To do that, copy the XML file, changing
>
> attached a patch that adds the new v2 file, comments adressed

ping? do i still need to do something on it?

--
Marco Martin
___
wayland-devel mailing list
wayland-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/wayland-devel


Re: How to support mixed DPI in Xwayland?

2017-09-12 Thread Adam Jackson
On Tue, 2017-09-12 at 19:34 +0200, Joseph Burt wrote:
> On Mon, Sep 11, 2017 at 3:44 PM, Adam Jackson  wrote:
> > 
> > Root window size is only ever sent during the initial connection
> > handshake, and the client extension libraries don't update it when the
> > root window is reconfigured [2]. So we have a bootstrapping problem:
> > how is the X server supposed to know which set of lies to give the
> > client when it connects? If you have multiple displays (either logical
> > views or whole processes) then you decide this when you connect, and
> > remote X apps [3] have an obvious way to pick the right one.
> 
> Could this be done with one server listening on two sockets? This
> could work for X servers in general, has it been discussed in that
> context?

"Two sockets" is what I meant by "logical views", there. Depending
which socket you connect to, you'd get different connection handshake
data and different coordinate scalings, but would otherwise interact
with the same set of objects. Again, easy enough to describe, not so
easy to actually implement.

- ajax
___
wayland-devel mailing list
wayland-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/wayland-devel


Re: How to support mixed DPI in Xwayland?

2017-09-12 Thread Joseph Burt
On Mon, Sep 11, 2017 at 3:44 PM, Adam Jackson  wrote:
> On Sun, 2017-09-10 at 22:25 +0200, Joseph Burt wrote:
>
>> What about always running the X server at hardware resolution,
>
> This isn't a fixed number. Outputs can be hotplugged.

Oh yeah, and they can have distinct DPIs. That means downscaling or
upscaling even DPI-aware clients for some outputs. Maintaining the
whole X space scaled in 96 DPI logical pixels is looking better and
better. Xwayland is for legacy support after all...

On Thu, Sep 7, 2017 at 10:15 PM, Adam Jackson  wrote:
> On Thu, 2017-09-07 at 12:17 -0400, Olivier Fourdan wrote:
>
>> The other solution would be to have the same screen, but have Xwayland to
>> give different scaling conversions for root window size, screen size, events
>> coordinates, etc. depending on the client, if it's HiDPI aware or not,
>> some sort of a "hidden" screen.
>
> Root window size is only ever sent during the initial connection
> handshake, and the client extension libraries don't update it when the
> root window is reconfigured [2]. So we have a bootstrapping problem:
> how is the X server supposed to know which set of lies to give the
> client when it connects? If you have multiple displays (either logical
> views or whole processes) then you decide this when you connect, and
> remote X apps [3] have an obvious way to pick the right one.

Could this be done with one server listening on two sockets? This
could work for X servers in general, has it been discussed in that
context?

Cheers,
Joseph
___
wayland-devel mailing list
wayland-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/wayland-devel


Re: [PATCH] Add aspect-ratio support in wayland layer.

2017-09-12 Thread Pekka Paalanen
On Mon, 11 Sep 2017 14:36:13 +0530
"Sharma, Shashank"  wrote:

> Hello Pekka
> 
> Thanks for your review comments and constructive discussions with Ankit 
> on IRC.
> I am Shashank from Intel, I am guiding Ankit on this activity.  I have 
> few comments (inline),
> it would be great if you can let us know your view on this.

Hi,

nice to meet you!

Please bear with my ignorance below.

> On 9/8/2017 7:43 PM, Pekka Paalanen wrote:
> > On Wed,  6 Sep 2017 17:50:00 +0530
> > Nautiyal Ankit  wrote:
> >  
> >> From: Ankit Nautiyal 
> >>
> >> This patch series adds video mode aspect ratio support for
> >> wayland protocol and weston drm compositor. This patch series
> >> adds two patches:
> >> 1- Adds aspect ratio flag bits bits in wl_output_mode flags
> >> as per DRM layer implementation.
> >> 2- Preserves and uses the aspect ratio flags while selecting
> >> the mode for modeset in weston drm compositor.
> >>
> >> The corresponding kernel implementation for the aspect ratio support
> >> in DRM Layer can be found here:
> >> https://patchwork.freedesktop.org/series/10850/
> >>
> >> This is a four patch series, in which two patches are already
> >> merged, and two are waiting for user space implementation.  
> > Hi,
> >
> > we discussed these patches on #wayland in IRC, and I believe we came to
> > the following conclusion:
> >
> > - Let's not add to wayland.xml, and not advertise the aspect ratio to
> >clients. There is no need to do this, and clients are currently not
> >able to take advantage of it either.  
> - In this case, how to decide what should be the aspect ratio of the 
> output mode and based on what ?
>For example If a CEA mode is available for 16:9 and 4:3 aspects, 
> which one to pick and how ? If we don't
>get inputs from the client, we would be picking the first mode in the 
> list which might not be the best practice.

Clients have no part in picking the video mode in general. Like I said,
there is no standard Wayland protocol to tell the compositor to pick
a video mode at this time.

I'm also starting to think that exposing the list of possible video
modes was not well thought through and would have been better omitted
completely. It already breaks apart on nested compositors where instead
of a few discrete modes one can pick freely from a range of widths and
heights, for instance.

The policy and configuration to choose the appropriate video mode lies
with each compositor. In Weston's case those are weston.ini for
configuration and the DRM-backend has the policy. Weston's policy is
simple and probably inadequate as it is. The way to fix that is to
patch Weston.

How to pick the right mode is a question that has no answer without a
specific use case and the environment. I envision moving the policy
from the DRM-backend to the users of libweston (into e.g. Weston) in
the future.

For example for a desktop, I struggle to imagine why anyone would want
to pick a video mode with non-square pixels. All generic software
assumes pixels are square. I would assume that also display panels are
built with square pixels, no?

For TV and such uses, I know nothing at all except for the feeling that
the video standards are still crippled by the designs from the 1960s.

If a compositor is being used for a special, non-desktop purpose, I
would also imagine it encodes its own configuration and policy for
choosing video modes. The choice requires case specific knowledge.
Clients could also be involved via domain specific Wayland protocol
extensions.

> - While supporting advance display outputs like HDMI 2.0 / UHD, when the 
> content can be in super wide aspect
> like 64:27 for movies/game, wouldn't it be under utilization of 
> display capabilities not to pick it ?

I'm afraid you will need to explain those use cases in much more detail
before I could make any comment on what is better or suggest a solution.

My naive thinking is, if a desktop compositor chooses a video mode with
non-square pixels, then it needs to scale all application content such
that applications will effectively have square pixels since that is
what they expect. I assume reality is different, but I have no
knowledge of it.

If display panels are manufactured with square pixels anyway, the above
configuration would result in scaling twice for no benefit that I can
see.

What is it that makes these non-square pixel video modes desireable?


Thanks,
pq


pgpPQzVszOgYs.pgp
Description: OpenPGP digital signature
___
wayland-devel mailing list
wayland-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/wayland-devel