Merge lp:~cemil-azizoglu/mir/split-mesa-into-common-and-kms into lp:mir
- split-mesa-into-common-and-kms
- Merge into development-branch
Status: | Merged |
---|---|
Approved by: | Alan Griffiths |
Approved revision: | no longer in the source branch. |
Merged at revision: | 2615 |
Proposed branch: | lp:~cemil-azizoglu/mir/split-mesa-into-common-and-kms |
Merge into: | lp:mir |
Diff against target: |
586 lines (+125/-56) 31 files modified
src/platforms/mesa/server/CMakeLists.txt (+5/-0) src/platforms/mesa/server/common/CMakeLists.txt (+17/-0) src/platforms/mesa/server/common/buffer_allocator.h (+1/-1) src/platforms/mesa/server/common/platform_common.h (+38/-0) src/platforms/mesa/server/kms/CMakeLists.txt (+9/-15) src/platforms/mesa/server/kms/platform.h (+1/-5) tests/include/mir_test_doubles/mock_virtual_terminal.h (+1/-1) tests/include/mir_test_doubles/null_virtual_terminal.h (+1/-1) tests/include/mir_test_doubles/stub_gbm_native_buffer.h (+1/-1) tests/integration-tests/CMakeLists.txt (+6/-0) tests/mir_test_doubles/CMakeLists.txt (+6/-1) tests/mir_test_doubles/platform_factory.cpp (+1/-1) tests/unit-tests/CMakeLists.txt (+7/-0) tests/unit-tests/graphics/mesa/mock_kms_output.h (+1/-1) tests/unit-tests/graphics/mesa/test_anonymous_shm_file.cpp (+1/-1) tests/unit-tests/graphics/mesa/test_buffer_allocator.cpp (+2/-2) tests/unit-tests/graphics/mesa/test_bypass.cpp (+1/-1) tests/unit-tests/graphics/mesa/test_cursor.cpp (+4/-4) tests/unit-tests/graphics/mesa/test_display.cpp (+3/-3) tests/unit-tests/graphics/mesa/test_display_buffer.cpp (+2/-2) tests/unit-tests/graphics/mesa/test_display_configuration.cpp (+2/-2) tests/unit-tests/graphics/mesa/test_drm_helper.cpp (+1/-1) tests/unit-tests/graphics/mesa/test_gbm_buffer.cpp (+3/-2) tests/unit-tests/graphics/mesa/test_guest_platform.cpp (+1/-1) tests/unit-tests/graphics/mesa/test_ipc_operations.cpp (+2/-2) tests/unit-tests/graphics/mesa/test_kms_page_flipper.cpp (+1/-1) tests/unit-tests/graphics/mesa/test_linux_virtual_terminal.cpp (+1/-1) tests/unit-tests/graphics/mesa/test_nested_authentication.cpp (+1/-1) tests/unit-tests/graphics/mesa/test_platform.cpp (+1/-1) tests/unit-tests/graphics/mesa/test_real_kms_output.cpp (+2/-2) tests/unit-tests/graphics/mesa/test_shm_buffer.cpp (+2/-2) |
To merge this branch: | bzr merge lp:~cemil-azizoglu/mir/split-mesa-into-common-and-kms |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Daniel van Vugt | Abstain | ||
Alexandros Frantzis (community) | Approve | ||
PS Jenkins bot (community) | continuous-integration | Approve | |
Alan Griffiths | Approve | ||
Chris Halse Rogers | Approve | ||
Review via email:
|
Commit message
Split Mesa server side into common and KMS.
There are other platforms (like X) using the Mesa driver. Some of the code is common. This MP splits the server side of the Mesa driver into common/ and KMS/ subdirectories.
To see how this new directory structure might be used, you can take a look at :
https:/
Subsequent MP(s) will rename relevant build or run time #defines to make things more clear.
Description of the change
Split Mesa server side into common and KMS.
There are other platforms (like X) using the Mesa driver. Some of the code is common. This MP splits the server side of the Mesa driver into common/ and KMS/ subdirectories.
To see how this new directory structure might be used, you can take a look at :
https:/
Subsequent MP(s) will rename relevant build or run time #defines to make things more clear.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
PS Jenkins bot (ps-jenkins) wrote : | # |
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
Chris Halse Rogers (raof) wrote : | # |
Hm. Why is bypass-option in platform_common.h? That's a KMS-specific option.
Otherwise seems to match what we agreed in Dallas, so +1.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
Daniel van Vugt (vanvugt) wrote : | # |
(1) SHM is generic and not specific to Mesa. It needs to separated completely for use in any driver...
213 === renamed file 'src/platforms/
(2) Reconsider naming "KMS" vs "DRM" per:
https:/
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
Chris Halse Rogers (raof) wrote : | # |
While (1) is correct, in practice we've got no non-mesa drivers using it; android has something better (for them). While it might be nice to separate it out, I don't think it should be a prerequisite for landing this branch.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
Alexandros Frantzis (afrantzis) wrote : | # |
10+ add_subdirector
Would prefer lowercase for 'kms' directory.
164=== renamed file 'src/platforms/
165=== renamed file 'src/platforms/
These are KMS specific, does X11 need these? If not, we should place these into kms/ along with the DRMHelper class which needs to split out from src/platforms/
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
Cemil Azizoglu (cemil-azizoglu) wrote : | # |
> Hm. Why is bypass-option in platform_common.h? That's a KMS-specific option.
>
> Otherwise seems to match what we agreed in Dallas, so +1.
Bypass option is used in BufferAllocator which is (99.9%) common code except X doesn't use GBM_BO_USE_SCANOUT.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
Cemil Azizoglu (cemil-azizoglu) wrote : | # |
> 10+ add_subdirector
>
> Would prefer lowercase for 'kms' directory.
KMS is an acronym, and I'd prefer keeping the name in capitals. I agree changing it to lowercase for the package name though.
>
> 164=== renamed file 'src/platforms/
> 'src/platforms/
> 165=== renamed file 'src/platforms/
> 'src/platforms/
>
> These are KMS specific, does X11 need these? If not, we should place these
> into kms/ along with the DRMHelper class which needs to split out from
> src/platforms/
Strictly speaking X11 does not need it. DRMHelper _is_ used in the X version to open a 'render' node to avoid the authentication dance. But this could change and we might end up using authentication for X too in the final version. In the current version of DRMHelper that supports both KMS and X, I chose to introduce a flag indicating whether the DRMHelper object is instantiated for KMS or X and use it to simply bypass some code not needed for X. This allows me to support X with very little change in code and keeps the door open for potential authentication for other Mesa platforms in the future.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
Alexandros Frantzis (afrantzis) wrote : | # |
>> Would prefer lowercase for 'kms' directory.
>
> KMS is an acronym, and I'd prefer keeping the name in capitals.
> I agree changing it to lowercase for the package name though.
We have at least a few other acronyms in the codebase (LTTng, GBM, HWC), for which we use the lowercase version in directory and file names. The reasons are:
1. Our namespaces, which generally follow our directory structure and names, our always lowercase (e.g. mir::mesa::kms for mesa/kms)
2. All files and directories are lowercase, so having a few uppercase names just confuses things and doesn't offer any real benefits
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
Cemil Azizoglu (cemil-azizoglu) wrote : | # |
> >> Would prefer lowercase for 'kms' directory.
> >
> > KMS is an acronym, and I'd prefer keeping the name in capitals.
> > I agree changing it to lowercase for the package name though.
>
> We have at least a few other acronyms in the codebase (LTTng, GBM, HWC), for
> which we use the lowercase version in directory and file names. The reasons
> are:
>
> 1. Our namespaces, which generally follow our directory structure and names,
> our always lowercase (e.g. mir::mesa::kms for mesa/kms)
> 2. All files and directories are lowercase, so having a few uppercase names
> just confuses things and doesn't offer any real benefits
Ok. Done.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
PS Jenkins bot (ps-jenkins) wrote : | # |
PASSED: Continuous integration, rev:2604
http://
Executed test runs:
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
SUCCESS: http://
Click here to trigger a rebuild:
http://
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
Chris Halse Rogers (raof) wrote : | # |
Hm. I guess BufferAllocator should have a little gentle reworking to not take the BypassOption (and instead use MayScanout, or something similar internally), but I'm happy for that to be done later at our leisure.
Likewise, splitting out some of the display helpers.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
Alan Griffiths (alan-griffiths) wrote : | # |
Nits:
5 +if (MIR_BUILD_
6 + add_subdirector
7 +endif()
8 +
9 +if (MIR_BUILD_
10 + add_subdirector
11 +endif()
The test for MIR_BUILD_
~~~~
158 + mirsharedmesase
we shouldn't be linking against mirplatformgrap
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
Cemil Azizoglu (cemil-azizoglu) wrote : | # |
> Nits:
>
> 5 +if (MIR_BUILD_
> 6 + add_subdirector
> 7 +endif()
> 8 +
> 9 +if (MIR_BUILD_
> 10 + add_subdirector
> 11 +endif()
>
> The test for MIR_BUILD_
> src/platforms/
You're right the first one is not wanted here. The second one is though (it's more clear why when you consider presence of an X folder at the same level as kms/ and/or if you take a look at the renaming branch). This is now fixed.
>
> ~~~~
>
> 158 + mirsharedmesase
>
> we shouldn't be linking against mirplatformgrap
> this ought to be PRIVATE in target_
> dependency on a static library.)
Fixed.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
PS Jenkins bot (ps-jenkins) wrote : | # |
PASSED: Continuous integration, rev:2605
http://
Executed test runs:
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
SUCCESS: http://
Click here to trigger a rebuild:
http://
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
Daniel van Vugt (vanvugt) wrote : | # |
Pre-existing issues, but try not to make them worse...
(3) Pointless enum just makes the code bigger:
94 +enum class BypassOption
95 +{
96 + allowed,
97 + prohibited
98 +};
Should be replaced by "bool bypass_allowed" instead of a new 38-line file.
(4) If you see anything named "Helper" that's a hack which should be replaced with something more concrete, if not simply a better class name.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
Daniel van Vugt (vanvugt) wrote : | # |
(5) If presently:
"android" == gralloc + hwc
"mesa" == gbm + drm
and you propose:
"kms" == gbm + drm
then how do we fit in (soon):
"nvidia" = eglstreams + kms
? Seems like "kms" then is the wrong name for the open driver because nvidia et all will use KMS too.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
Daniel van Vugt (vanvugt) wrote : | # |
Feels like "mesa" is actually the right driver name and we just want to move some stuff into a separate library for other drivers to use.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
Alan Griffiths (alan-griffiths) wrote : | # |
> Pre-existing issues, but try not to make them worse...
>
> (3) Pointless enum just makes the code bigger:
> 94 +enum class BypassOption
> 95 +{
> 96 + allowed,
> 97 + prohibited
> 98 +};
> Should be replaced by "bool bypass_allowed" instead of a new 38-line file.
I've not looked into the reason this is a new file. (Shouldn't it be declared with the interface that uses it?) But I prefer to read code that says:
foo-
over code that says:
foo-
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
Alan Griffiths (alan-griffiths) wrote : | # |
> > Nits:
> >
> > 5 +if (MIR_BUILD_
> > 6 + add_subdirector
> > 7 +endif()
> > 8 +
> > 9 +if (MIR_BUILD_
> > 10 + add_subdirector
> > 11 +endif()
> >
> > The test for MIR_BUILD_
> > src/platforms/
>
> You're right the first one is not wanted here. The second one is though (it's
> more clear why when you consider presence of an X folder at the same level as
> kms/ and/or if you take a look at the renaming branch). This is now fixed.
The second one might be wanted after moving to src/platforms/
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
Alexandros Frantzis (afrantzis) wrote : | # |
OK for now. We will get a better idea of how to split/refactor further when we have the X platform code.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
Cemil Azizoglu (cemil-azizoglu) wrote : | # |
> Feels like "mesa" is actually the right driver name and we just want to move
> some stuff into a separate library for other drivers to use.
Mesa is the name of the driver and X driver uses it also, so I don't think that's the right name.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
Daniel van Vugt (vanvugt) wrote : | # |
X does use Mesa. That's always been true and I don't see any issue with that.
I'm mostly concerned about proprietary drivers that use KMS and their own buffer sharing. In that case to say "kms" is a mutually exclusive driver to "nvidia" is very confusing.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
Daniel van Vugt (vanvugt) wrote : | # |
BTW, I'm not talking about "foo->bypass(true)" but the more flexible and expressive form:
foo-
Hence no bespoke non-bool-boolean enum and no 38-line file to contain it is required.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
Daniel van Vugt (vanvugt) wrote : | # |
"Suggest fixing", but otherwise abstain.
I'm more concerned with this code stabilizing (to avoid conflicts in the near term) than the syntax itself.
![](/+icing/build/overlay/assets/skins/sam/images/close.gif)
Alan Griffiths (alan-griffiths) wrote : | # |
> BTW, I'm not talking about "foo->bypass(true)" but the more flexible and
> expressive form:
> foo->allow_
let's look at the real code:
std::shared_
{
return std::make_
}
vs
std::shared_
{
return std::make_
}
...
auto bypass_option = mgm::BypassOpti
if (!options-
vs
auto bypass_option = true;
if (!options-
(I do realize that neither of us would actually write something as verbose as either of those last options.)
> Hence no bespoke non-bool-boolean enum and no 38-line file to contain it is
> required.
I still see no reason for platform_common.h to be a separate file in this MP. (If fact I don't see how it is meaningful without the mgm::Platform interface.)
Preview Diff
1 | === added file 'src/platforms/mesa/server/CMakeLists.txt' |
2 | --- src/platforms/mesa/server/CMakeLists.txt 1970-01-01 00:00:00 +0000 |
3 | +++ src/platforms/mesa/server/CMakeLists.txt 2015-06-01 18:31:25 +0000 |
4 | @@ -0,0 +1,5 @@ |
5 | +add_subdirectory(common/) |
6 | + |
7 | +if (MIR_BUILD_PLATFORM_MESA) |
8 | + add_subdirectory(kms/) |
9 | +endif() |
10 | |
11 | === added directory 'src/platforms/mesa/server/common' |
12 | === added file 'src/platforms/mesa/server/common/CMakeLists.txt' |
13 | --- src/platforms/mesa/server/common/CMakeLists.txt 1970-01-01 00:00:00 +0000 |
14 | +++ src/platforms/mesa/server/common/CMakeLists.txt 2015-06-01 18:31:25 +0000 |
15 | @@ -0,0 +1,17 @@ |
16 | +include_directories( |
17 | + ${server_common_include_dirs} |
18 | + ${DRM_INCLUDE_DIRS} |
19 | +) |
20 | + |
21 | +add_library( |
22 | + mirsharedmesaservercommon-static STATIC |
23 | + |
24 | + anonymous_shm_file.cpp |
25 | + buffer_allocator.cpp |
26 | + display_helpers.cpp |
27 | + drm_close_threadsafe.cpp |
28 | + drm_mode_resources.cpp |
29 | + gbm_buffer.cpp |
30 | + ipc_operations.cpp |
31 | + shm_buffer.cpp |
32 | +) |
33 | |
34 | === renamed file 'src/platforms/mesa/server/anonymous_shm_file.cpp' => 'src/platforms/mesa/server/common/anonymous_shm_file.cpp' |
35 | === renamed file 'src/platforms/mesa/server/anonymous_shm_file.h' => 'src/platforms/mesa/server/common/anonymous_shm_file.h' |
36 | === renamed file 'src/platforms/mesa/server/buffer_allocator.cpp' => 'src/platforms/mesa/server/common/buffer_allocator.cpp' |
37 | === renamed file 'src/platforms/mesa/server/buffer_allocator.h' => 'src/platforms/mesa/server/common/buffer_allocator.h' |
38 | --- src/platforms/mesa/server/buffer_allocator.h 2015-05-07 09:06:20 +0000 |
39 | +++ src/platforms/mesa/server/common/buffer_allocator.h 2015-06-01 18:31:25 +0000 |
40 | @@ -19,7 +19,7 @@ |
41 | #ifndef MIR_GRAPHICS_MESA_BUFFER_ALLOCATOR_H_ |
42 | #define MIR_GRAPHICS_MESA_BUFFER_ALLOCATOR_H_ |
43 | |
44 | -#include "platform.h" |
45 | +#include "platform_common.h" |
46 | #include "mir/graphics/graphic_buffer_allocator.h" |
47 | #include "mir/graphics/buffer_id.h" |
48 | |
49 | |
50 | === renamed file 'src/platforms/mesa/server/buffer_texture_binder.h' => 'src/platforms/mesa/server/common/buffer_texture_binder.h' |
51 | === renamed file 'src/platforms/mesa/server/display_helpers.cpp' => 'src/platforms/mesa/server/common/display_helpers.cpp' |
52 | === renamed file 'src/platforms/mesa/server/display_helpers.h' => 'src/platforms/mesa/server/common/display_helpers.h' |
53 | === renamed file 'src/platforms/mesa/server/drm_authentication.h' => 'src/platforms/mesa/server/common/drm_authentication.h' |
54 | === renamed file 'src/platforms/mesa/server/drm_close_threadsafe.cpp' => 'src/platforms/mesa/server/common/drm_close_threadsafe.cpp' |
55 | === renamed file 'src/platforms/mesa/server/drm_close_threadsafe.h' => 'src/platforms/mesa/server/common/drm_close_threadsafe.h' |
56 | === renamed file 'src/platforms/mesa/server/drm_mode_resources.cpp' => 'src/platforms/mesa/server/common/drm_mode_resources.cpp' |
57 | === renamed file 'src/platforms/mesa/server/drm_mode_resources.h' => 'src/platforms/mesa/server/common/drm_mode_resources.h' |
58 | === renamed file 'src/platforms/mesa/server/gbm_buffer.cpp' => 'src/platforms/mesa/server/common/gbm_buffer.cpp' |
59 | === renamed file 'src/platforms/mesa/server/gbm_buffer.h' => 'src/platforms/mesa/server/common/gbm_buffer.h' |
60 | === renamed file 'src/platforms/mesa/server/ipc_operations.cpp' => 'src/platforms/mesa/server/common/ipc_operations.cpp' |
61 | === renamed file 'src/platforms/mesa/server/ipc_operations.h' => 'src/platforms/mesa/server/common/ipc_operations.h' |
62 | === added file 'src/platforms/mesa/server/common/platform_common.h' |
63 | --- src/platforms/mesa/server/common/platform_common.h 1970-01-01 00:00:00 +0000 |
64 | +++ src/platforms/mesa/server/common/platform_common.h 2015-06-01 18:31:25 +0000 |
65 | @@ -0,0 +1,38 @@ |
66 | +/* |
67 | + * Copyright © 2015 Canonical Ltd. |
68 | + * |
69 | + * This program is free software: you can redistribute it and/or modify it |
70 | + * under the terms of the GNU Lesser General Public License version 3, |
71 | + * as published by the Free Software Foundation. |
72 | + * |
73 | + * This program is distributed in the hope that it will be useful, |
74 | + * but WITHOUT ANY WARRANTY; without even the implied warranty of |
75 | + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the |
76 | + * GNU Lesser General Public License for more details. |
77 | + * |
78 | + * You should have received a copy of the GNU Lesser General Public License |
79 | + * along with this program. If not, see <http://www.gnu.org/licenses/>. |
80 | + * |
81 | + * Authored by: Cemil Azizoglu <cemil.azizoglu@canonical.com> |
82 | + */ |
83 | + |
84 | +#ifndef MIR_GRAPHICS_MESA_PLATFORM_COMMON_H_ |
85 | +#define MIR_GRAPHICS_MESA_PLATFORM_COMMON_H_ |
86 | + |
87 | +namespace mir |
88 | +{ |
89 | +namespace graphics |
90 | +{ |
91 | +namespace mesa |
92 | +{ |
93 | + |
94 | +enum class BypassOption |
95 | +{ |
96 | + allowed, |
97 | + prohibited |
98 | +}; |
99 | + |
100 | +} |
101 | +} |
102 | +} |
103 | +#endif /* MIR_GRAPHICS_MESA_PLATFORM_COMMON_H_ */ |
104 | |
105 | === renamed file 'src/platforms/mesa/server/shm_buffer.cpp' => 'src/platforms/mesa/server/common/shm_buffer.cpp' |
106 | === renamed file 'src/platforms/mesa/server/shm_buffer.h' => 'src/platforms/mesa/server/common/shm_buffer.h' |
107 | === renamed file 'src/platforms/mesa/server/shm_file.h' => 'src/platforms/mesa/server/common/shm_file.h' |
108 | === added directory 'src/platforms/mesa/server/kms' |
109 | === renamed file 'src/platforms/mesa/server/CMakeLists.txt' => 'src/platforms/mesa/server/kms/CMakeLists.txt' |
110 | --- src/platforms/mesa/server/CMakeLists.txt 2015-05-19 21:34:34 +0000 |
111 | +++ src/platforms/mesa/server/kms/CMakeLists.txt 2015-06-01 18:31:25 +0000 |
112 | @@ -1,5 +1,6 @@ |
113 | include_directories( |
114 | ${server_common_include_dirs} |
115 | + ${PROJECT_SOURCE_DIR}/src/platforms/mesa/server/common |
116 | ) |
117 | |
118 | include_directories( |
119 | @@ -18,26 +19,18 @@ |
120 | add_library( |
121 | mirplatformgraphicsmesaobjects OBJECT |
122 | |
123 | - platform.cpp |
124 | - buffer_allocator.cpp |
125 | - gbm_buffer.cpp |
126 | + bypass.cpp |
127 | cursor.cpp |
128 | - display_helpers.cpp |
129 | display.cpp |
130 | - real_kms_display_configuration.cpp |
131 | - drm_mode_resources.cpp |
132 | display_buffer.cpp |
133 | - real_kms_output.cpp |
134 | - real_kms_output_container.cpp |
135 | + guest_platform.cpp |
136 | kms_page_flipper.cpp |
137 | linux_virtual_terminal.cpp |
138 | - drm_close_threadsafe.cpp |
139 | - guest_platform.cpp |
140 | - anonymous_shm_file.cpp |
141 | - shm_buffer.cpp |
142 | - bypass.cpp |
143 | - ipc_operations.cpp |
144 | nested_authentication.cpp |
145 | + platform.cpp |
146 | + real_kms_display_configuration.cpp |
147 | + real_kms_output.cpp |
148 | + real_kms_output_container.cpp |
149 | ) |
150 | |
151 | add_library( |
152 | @@ -48,8 +41,9 @@ |
153 | |
154 | target_link_libraries( |
155 | mirplatformgraphicsmesa |
156 | - |
157 | + PRIVATE |
158 | mirplatform |
159 | + mirsharedmesaservercommon-static |
160 | ${Boost_PROGRAM_OPTIONS_LIBRARY} |
161 | ${DRM_LDFLAGS} ${DRM_LIBRARIES} |
162 | ${GBM_LDFLAGS} ${GBM_LIBRARIES} |
163 | |
164 | === renamed file 'src/platforms/mesa/server/bypass.cpp' => 'src/platforms/mesa/server/kms/bypass.cpp' |
165 | === renamed file 'src/platforms/mesa/server/bypass.h' => 'src/platforms/mesa/server/kms/bypass.h' |
166 | === renamed file 'src/platforms/mesa/server/cursor.cpp' => 'src/platforms/mesa/server/kms/cursor.cpp' |
167 | === renamed file 'src/platforms/mesa/server/cursor.h' => 'src/platforms/mesa/server/kms/cursor.h' |
168 | === renamed file 'src/platforms/mesa/server/display.cpp' => 'src/platforms/mesa/server/kms/display.cpp' |
169 | === renamed file 'src/platforms/mesa/server/display.h' => 'src/platforms/mesa/server/kms/display.h' |
170 | === renamed file 'src/platforms/mesa/server/display_buffer.cpp' => 'src/platforms/mesa/server/kms/display_buffer.cpp' |
171 | === renamed file 'src/platforms/mesa/server/display_buffer.h' => 'src/platforms/mesa/server/kms/display_buffer.h' |
172 | === renamed file 'src/platforms/mesa/server/guest_platform.cpp' => 'src/platforms/mesa/server/kms/guest_platform.cpp' |
173 | === renamed file 'src/platforms/mesa/server/guest_platform.h' => 'src/platforms/mesa/server/kms/guest_platform.h' |
174 | === renamed file 'src/platforms/mesa/server/kms_display_configuration.h' => 'src/platforms/mesa/server/kms/kms_display_configuration.h' |
175 | === renamed file 'src/platforms/mesa/server/kms_output.h' => 'src/platforms/mesa/server/kms/kms_output.h' |
176 | === renamed file 'src/platforms/mesa/server/kms_output_container.h' => 'src/platforms/mesa/server/kms/kms_output_container.h' |
177 | === renamed file 'src/platforms/mesa/server/kms_page_flipper.cpp' => 'src/platforms/mesa/server/kms/kms_page_flipper.cpp' |
178 | === renamed file 'src/platforms/mesa/server/kms_page_flipper.h' => 'src/platforms/mesa/server/kms/kms_page_flipper.h' |
179 | === renamed file 'src/platforms/mesa/server/linux_virtual_terminal.cpp' => 'src/platforms/mesa/server/kms/linux_virtual_terminal.cpp' |
180 | === renamed file 'src/platforms/mesa/server/linux_virtual_terminal.h' => 'src/platforms/mesa/server/kms/linux_virtual_terminal.h' |
181 | === renamed file 'src/platforms/mesa/server/nested_authentication.cpp' => 'src/platforms/mesa/server/kms/nested_authentication.cpp' |
182 | === renamed file 'src/platforms/mesa/server/nested_authentication.h' => 'src/platforms/mesa/server/kms/nested_authentication.h' |
183 | === renamed file 'src/platforms/mesa/server/page_flipper.h' => 'src/platforms/mesa/server/kms/page_flipper.h' |
184 | === renamed file 'src/platforms/mesa/server/platform.cpp' => 'src/platforms/mesa/server/kms/platform.cpp' |
185 | === renamed file 'src/platforms/mesa/server/platform.h' => 'src/platforms/mesa/server/kms/platform.h' |
186 | --- src/platforms/mesa/server/platform.h 2015-03-31 02:35:42 +0000 |
187 | +++ src/platforms/mesa/server/kms/platform.h 2015-06-01 18:31:25 +0000 |
188 | @@ -20,6 +20,7 @@ |
189 | #define MIR_GRAPHICS_MESA_PLATFORM_H_ |
190 | |
191 | #include "mir/graphics/platform.h" |
192 | +#include "platform_common.h" |
193 | #include "display_helpers.h" |
194 | |
195 | namespace mir |
196 | @@ -29,11 +30,6 @@ |
197 | { |
198 | namespace mesa |
199 | { |
200 | -enum class BypassOption |
201 | -{ |
202 | - allowed, |
203 | - prohibited |
204 | -}; |
205 | |
206 | class VirtualTerminal; |
207 | class InternalNativeDisplay; |
208 | |
209 | === renamed file 'src/platforms/mesa/server/real_kms_display_configuration.cpp' => 'src/platforms/mesa/server/kms/real_kms_display_configuration.cpp' |
210 | === renamed file 'src/platforms/mesa/server/real_kms_display_configuration.h' => 'src/platforms/mesa/server/kms/real_kms_display_configuration.h' |
211 | === renamed file 'src/platforms/mesa/server/real_kms_output.cpp' => 'src/platforms/mesa/server/kms/real_kms_output.cpp' |
212 | === renamed file 'src/platforms/mesa/server/real_kms_output.h' => 'src/platforms/mesa/server/kms/real_kms_output.h' |
213 | === renamed file 'src/platforms/mesa/server/real_kms_output_container.cpp' => 'src/platforms/mesa/server/kms/real_kms_output_container.cpp' |
214 | === renamed file 'src/platforms/mesa/server/real_kms_output_container.h' => 'src/platforms/mesa/server/kms/real_kms_output_container.h' |
215 | === renamed file 'src/platforms/mesa/server/symbols.map' => 'src/platforms/mesa/server/kms/symbols.map' |
216 | === renamed file 'src/platforms/mesa/server/virtual_terminal.h' => 'src/platforms/mesa/server/kms/virtual_terminal.h' |
217 | === modified file 'tests/include/mir_test_doubles/mock_virtual_terminal.h' |
218 | --- tests/include/mir_test_doubles/mock_virtual_terminal.h 2015-01-22 09:00:14 +0000 |
219 | +++ tests/include/mir_test_doubles/mock_virtual_terminal.h 2015-06-01 18:31:25 +0000 |
220 | @@ -19,7 +19,7 @@ |
221 | #ifndef MIR_TEST_DOUBLES_MOCK_VIRTUAL_TERMINAL_H_ |
222 | #define MIR_TEST_DOUBLES_MOCK_VIRTUAL_TERMINAL_H_ |
223 | |
224 | -#include "src/platforms/mesa/server/virtual_terminal.h" |
225 | +#include "src/platforms/mesa/server/kms/virtual_terminal.h" |
226 | |
227 | #include <gmock/gmock.h> |
228 | |
229 | |
230 | === modified file 'tests/include/mir_test_doubles/null_virtual_terminal.h' |
231 | --- tests/include/mir_test_doubles/null_virtual_terminal.h 2015-01-22 09:00:14 +0000 |
232 | +++ tests/include/mir_test_doubles/null_virtual_terminal.h 2015-06-01 18:31:25 +0000 |
233 | @@ -19,7 +19,7 @@ |
234 | #ifndef MIR_TEST_DOUBLES_NULL_VIRTUAL_TERMINAL_H_ |
235 | #define MIR_TEST_DOUBLES_NULL_VIRTUAL_TERMINAL_H_ |
236 | |
237 | -#include "src/platforms/mesa/server/virtual_terminal.h" |
238 | +#include "src/platforms/mesa/server/kms/virtual_terminal.h" |
239 | |
240 | namespace mir |
241 | { |
242 | |
243 | === modified file 'tests/include/mir_test_doubles/stub_gbm_native_buffer.h' |
244 | --- tests/include/mir_test_doubles/stub_gbm_native_buffer.h 2015-03-31 02:35:42 +0000 |
245 | +++ tests/include/mir_test_doubles/stub_gbm_native_buffer.h 2015-06-01 18:31:25 +0000 |
246 | @@ -19,7 +19,7 @@ |
247 | #ifndef MIR_TEST_DOUBLES_STUB_GBM_NATIVE_BUFFER_H_ |
248 | #define MIR_TEST_DOUBLES_STUB_GBM_NATIVE_BUFFER_H_ |
249 | |
250 | -#include "src/platforms/mesa/server/gbm_buffer.h" |
251 | +#include "src/platforms/mesa/server/common/gbm_buffer.h" |
252 | #include "mir/geometry/size.h" |
253 | #include <unistd.h> |
254 | #include <fcntl.h> |
255 | |
256 | === modified file 'tests/integration-tests/CMakeLists.txt' |
257 | --- tests/integration-tests/CMakeLists.txt 2015-05-19 21:34:34 +0000 |
258 | +++ tests/integration-tests/CMakeLists.txt 2015-06-01 18:31:25 +0000 |
259 | @@ -117,6 +117,12 @@ |
260 | ${MIR_SERVER_REFERENCES} |
261 | ) |
262 | |
263 | +if (MIR_BUILD_PLATFORM_MESA) |
264 | +target_link_libraries(mir_integration_tests |
265 | + mirsharedmesaservercommon-static |
266 | +) |
267 | +endif() |
268 | + |
269 | if (MIR_BUILD_PLATFORM_ANDROID) |
270 | target_link_libraries(mir_integration_tests |
271 | mirsharedandroid-static |
272 | |
273 | === modified file 'tests/mir_test_doubles/CMakeLists.txt' |
274 | --- tests/mir_test_doubles/CMakeLists.txt 2015-05-19 21:34:34 +0000 |
275 | +++ tests/mir_test_doubles/CMakeLists.txt 2015-06-01 18:31:25 +0000 |
276 | @@ -33,7 +33,12 @@ |
277 | ) |
278 | |
279 | if (MIR_BUILD_PLATFORM_MESA) |
280 | - include_directories(${DRM_INCLUDE_DIRS} ${GBM_INCLUDE_DIRS} ${CMAKE_SOURCE_DIR}) |
281 | + include_directories( |
282 | + ${PROJECT_SOURCE_DIR}/src/platforms/mesa/server/common |
283 | + ${DRM_INCLUDE_DIRS} |
284 | + ${GBM_INCLUDE_DIRS} |
285 | + ${CMAKE_SOURCE_DIR} |
286 | + ) |
287 | list(APPEND MIR_TEST_DOUBLES_PLATFORM_SRCS |
288 | ${CMAKE_CURRENT_SOURCE_DIR}/mock_drm.cpp |
289 | ${CMAKE_CURRENT_SOURCE_DIR}/mock_gbm.cpp |
290 | |
291 | === modified file 'tests/mir_test_doubles/platform_factory.cpp' |
292 | --- tests/mir_test_doubles/platform_factory.cpp 2015-04-09 08:57:24 +0000 |
293 | +++ tests/mir_test_doubles/platform_factory.cpp 2015-06-01 18:31:25 +0000 |
294 | @@ -21,7 +21,7 @@ |
295 | #include "mir/graphics/platform.h" |
296 | |
297 | #ifndef ANDROID |
298 | -#include "src/platforms/mesa/server/platform.h" |
299 | +#include "src/platforms/mesa/server/kms/platform.h" |
300 | #endif |
301 | |
302 | #include "src/server/report/null_report_factory.h" |
303 | |
304 | === modified file 'tests/unit-tests/CMakeLists.txt' |
305 | --- tests/unit-tests/CMakeLists.txt 2015-05-19 21:34:34 +0000 |
306 | +++ tests/unit-tests/CMakeLists.txt 2015-06-01 18:31:25 +0000 |
307 | @@ -23,6 +23,7 @@ |
308 | ${PROJECT_SOURCE_DIR}/src/include/client |
309 | ${PROJECT_SOURCE_DIR}/src/include/common |
310 | ${PROJECT_SOURCE_DIR}/src/platforms/common/client |
311 | + ${PROJECT_SOURCE_DIR}/src/platforms/mesa/server/common |
312 | ${GLIB_INCLUDE_DIRS} |
313 | ) |
314 | |
315 | @@ -134,6 +135,12 @@ |
316 | ${XKBCOMMON_LIBRARIES} |
317 | ) |
318 | |
319 | +if (MIR_BUILD_PLATFORM_MESA) |
320 | +target_link_libraries(mir_unit_tests |
321 | + mirsharedmesaservercommon-static |
322 | +) |
323 | +endif() |
324 | + |
325 | if (MIR_BUILD_PLATFORM_ANDROID) |
326 | target_link_libraries(mir_unit_tests |
327 | mirsharedandroid-static |
328 | |
329 | === modified file 'tests/unit-tests/graphics/mesa/mock_kms_output.h' |
330 | --- tests/unit-tests/graphics/mesa/mock_kms_output.h 2015-01-22 09:00:14 +0000 |
331 | +++ tests/unit-tests/graphics/mesa/mock_kms_output.h 2015-06-01 18:31:25 +0000 |
332 | @@ -19,7 +19,7 @@ |
333 | #ifndef MOCK_KMS_OUTPUT_H_ |
334 | #define MOCK_KMS_OUTPUT_H_ |
335 | |
336 | -#include "src/platforms/mesa/server/kms_output.h" |
337 | +#include "src/platforms/mesa/server/kms/kms_output.h" |
338 | #include <gmock/gmock.h> |
339 | |
340 | namespace mir |
341 | |
342 | === modified file 'tests/unit-tests/graphics/mesa/test_anonymous_shm_file.cpp' |
343 | --- tests/unit-tests/graphics/mesa/test_anonymous_shm_file.cpp 2015-01-22 09:00:14 +0000 |
344 | +++ tests/unit-tests/graphics/mesa/test_anonymous_shm_file.cpp 2015-06-01 18:31:25 +0000 |
345 | @@ -16,7 +16,7 @@ |
346 | * Authored by: Alexandros Frantzis <alexandros.frantzis@canonical.com> |
347 | */ |
348 | |
349 | -#include "src/platforms/mesa/server/anonymous_shm_file.h" |
350 | +#include "src/platforms/mesa/server/common/anonymous_shm_file.h" |
351 | #include <gtest/gtest.h> |
352 | |
353 | namespace mgm = mir::graphics::mesa; |
354 | |
355 | === modified file 'tests/unit-tests/graphics/mesa/test_buffer_allocator.cpp' |
356 | --- tests/unit-tests/graphics/mesa/test_buffer_allocator.cpp 2015-05-07 09:06:20 +0000 |
357 | +++ tests/unit-tests/graphics/mesa/test_buffer_allocator.cpp 2015-06-01 18:31:25 +0000 |
358 | @@ -16,9 +16,9 @@ |
359 | * Authored by: Christopher James Halse Rogers <christopher.halse.rogers@canonical.com> |
360 | */ |
361 | |
362 | -#include "src/platforms/mesa/server/platform.h" |
363 | +#include "src/platforms/mesa/server/kms/platform.h" |
364 | #include "mir/graphics/graphic_buffer_allocator.h" |
365 | -#include "src/platforms/mesa/server/buffer_allocator.h" |
366 | +#include "src/platforms/mesa/server/common/buffer_allocator.h" |
367 | #include "mir/graphics/buffer_properties.h" |
368 | |
369 | #include "mir_test_doubles/mock_drm.h" |
370 | |
371 | === modified file 'tests/unit-tests/graphics/mesa/test_bypass.cpp' |
372 | --- tests/unit-tests/graphics/mesa/test_bypass.cpp 2015-01-22 09:00:14 +0000 |
373 | +++ tests/unit-tests/graphics/mesa/test_bypass.cpp 2015-06-01 18:31:25 +0000 |
374 | @@ -16,7 +16,7 @@ |
375 | * Authored by: Daniel van Vugt <daniel.van.vugt@canonical.com> |
376 | */ |
377 | |
378 | -#include "src/platforms/mesa/server/bypass.h" |
379 | +#include "src/platforms/mesa/server/kms/bypass.h" |
380 | #include "mir_test_doubles/mock_display_buffer.h" |
381 | #include "mir_test_doubles/fake_renderable.h" |
382 | |
383 | |
384 | === modified file 'tests/unit-tests/graphics/mesa/test_cursor.cpp' |
385 | --- tests/unit-tests/graphics/mesa/test_cursor.cpp 2015-03-31 02:35:42 +0000 |
386 | +++ tests/unit-tests/graphics/mesa/test_cursor.cpp 2015-06-01 18:31:25 +0000 |
387 | @@ -16,10 +16,10 @@ |
388 | * Authored by: Alexandros Frantzis <alexandros.frantzis@canonical.com> |
389 | */ |
390 | |
391 | -#include "src/platforms/mesa/server/cursor.h" |
392 | -#include "src/platforms/mesa/server/kms_output.h" |
393 | -#include "src/platforms/mesa/server/kms_output_container.h" |
394 | -#include "src/platforms/mesa/server/kms_display_configuration.h" |
395 | +#include "src/platforms/mesa/server/kms/cursor.h" |
396 | +#include "src/platforms/mesa/server/kms/kms_output.h" |
397 | +#include "src/platforms/mesa/server/kms/kms_output_container.h" |
398 | +#include "src/platforms/mesa/server/kms/kms_display_configuration.h" |
399 | |
400 | #include "mir/graphics/cursor_image.h" |
401 | |
402 | |
403 | === modified file 'tests/unit-tests/graphics/mesa/test_display.cpp' |
404 | --- tests/unit-tests/graphics/mesa/test_display.cpp 2015-03-31 02:35:42 +0000 |
405 | +++ tests/unit-tests/graphics/mesa/test_display.cpp 2015-06-01 18:31:25 +0000 |
406 | @@ -16,9 +16,9 @@ |
407 | * Authored by: Alexandros Frantzis <alexandros.frantzis@canonical.com> |
408 | */ |
409 | #include <boost/throw_exception.hpp> |
410 | -#include "src/platforms/mesa/server/platform.h" |
411 | -#include "src/platforms/mesa/server/display.h" |
412 | -#include "src/platforms/mesa/server/virtual_terminal.h" |
413 | +#include "src/platforms/mesa/server/kms/platform.h" |
414 | +#include "src/platforms/mesa/server/kms/display.h" |
415 | +#include "src/platforms/mesa/server/kms/virtual_terminal.h" |
416 | #include "src/server/report/logging/display_report.h" |
417 | #include "mir/logging/logger.h" |
418 | #include "mir/graphics/display_buffer.h" |
419 | |
420 | === modified file 'tests/unit-tests/graphics/mesa/test_display_buffer.cpp' |
421 | --- tests/unit-tests/graphics/mesa/test_display_buffer.cpp 2015-05-28 10:01:07 +0000 |
422 | +++ tests/unit-tests/graphics/mesa/test_display_buffer.cpp 2015-06-01 18:31:25 +0000 |
423 | @@ -15,8 +15,8 @@ |
424 | * |
425 | * Authored by: Daniel van Vugt <daniel.van.vugt@canonical.com> |
426 | */ |
427 | -#include "src/platforms/mesa/server/platform.h" |
428 | -#include "src/platforms/mesa/server/display_buffer.h" |
429 | +#include "src/platforms/mesa/server/kms/platform.h" |
430 | +#include "src/platforms/mesa/server/kms/display_buffer.h" |
431 | #include "src/server/report/null_report_factory.h" |
432 | #include "mir_test_doubles/mock_egl.h" |
433 | #include "mir_test_doubles/mock_gl.h" |
434 | |
435 | === modified file 'tests/unit-tests/graphics/mesa/test_display_configuration.cpp' |
436 | --- tests/unit-tests/graphics/mesa/test_display_configuration.cpp 2015-01-22 09:00:14 +0000 |
437 | +++ tests/unit-tests/graphics/mesa/test_display_configuration.cpp 2015-06-01 18:31:25 +0000 |
438 | @@ -20,8 +20,8 @@ |
439 | #include "mir/graphics/display_configuration.h" |
440 | #include "mir/graphics/display.h" |
441 | #include "src/server/graphics/default_display_configuration_policy.h" |
442 | -#include "src/platforms/mesa/server/platform.h" |
443 | -#include "src/platforms/mesa/server/kms_display_configuration.h" |
444 | +#include "src/platforms/mesa/server/kms/platform.h" |
445 | +#include "src/platforms/mesa/server/kms/kms_display_configuration.h" |
446 | |
447 | #include "mir_test_doubles/mock_egl.h" |
448 | #include "mir_test_doubles/mock_gl.h" |
449 | |
450 | === modified file 'tests/unit-tests/graphics/mesa/test_drm_helper.cpp' |
451 | --- tests/unit-tests/graphics/mesa/test_drm_helper.cpp 2015-02-02 12:18:18 +0000 |
452 | +++ tests/unit-tests/graphics/mesa/test_drm_helper.cpp 2015-06-01 18:31:25 +0000 |
453 | @@ -16,7 +16,7 @@ |
454 | * Authored by: Alexandros Frantzis <alexandros.frantzis@canonical.com> |
455 | */ |
456 | |
457 | -#include "src/platforms/mesa/server/display_helpers.h" |
458 | +#include "src/platforms/mesa/server/common/display_helpers.h" |
459 | #include "mir/udev/wrapper.h" |
460 | |
461 | #include "mir_test_framework/udev_environment.h" |
462 | |
463 | === modified file 'tests/unit-tests/graphics/mesa/test_gbm_buffer.cpp' |
464 | --- tests/unit-tests/graphics/mesa/test_gbm_buffer.cpp 2015-01-22 09:00:14 +0000 |
465 | +++ tests/unit-tests/graphics/mesa/test_gbm_buffer.cpp 2015-06-01 18:31:25 +0000 |
466 | @@ -23,8 +23,9 @@ |
467 | |
468 | #include "mir_test_framework/udev_environment.h" |
469 | |
470 | -#include "src/platforms/mesa/server/gbm_buffer.h" |
471 | -#include "src/platforms/mesa/server/buffer_allocator.h" |
472 | +#include "src/platforms/mesa/server/kms/platform.h" |
473 | +#include "src/platforms/mesa/server/common/gbm_buffer.h" |
474 | +#include "src/platforms/mesa/server/common/buffer_allocator.h" |
475 | #include "mir/graphics/buffer_properties.h" |
476 | #include "mir_test_doubles/platform_factory.h" |
477 | |
478 | |
479 | === modified file 'tests/unit-tests/graphics/mesa/test_guest_platform.cpp' |
480 | --- tests/unit-tests/graphics/mesa/test_guest_platform.cpp 2015-03-31 02:35:42 +0000 |
481 | +++ tests/unit-tests/graphics/mesa/test_guest_platform.cpp 2015-06-01 18:31:25 +0000 |
482 | @@ -17,7 +17,7 @@ |
483 | */ |
484 | |
485 | #include "mir/graphics/nested_context.h" |
486 | -#include "src/platforms/mesa/server/guest_platform.h" |
487 | +#include "src/platforms/mesa/server/kms/guest_platform.h" |
488 | #include "mir/graphics/buffer_properties.h" |
489 | #include "mir/graphics/platform_ipc_operations.h" |
490 | #include "mir_toolkit/mesa/platform_operation.h" |
491 | |
492 | === modified file 'tests/unit-tests/graphics/mesa/test_ipc_operations.cpp' |
493 | --- tests/unit-tests/graphics/mesa/test_ipc_operations.cpp 2015-03-31 02:35:42 +0000 |
494 | +++ tests/unit-tests/graphics/mesa/test_ipc_operations.cpp 2015-06-01 18:31:25 +0000 |
495 | @@ -16,8 +16,8 @@ |
496 | * Authored by: Kevin DuBois <kevin.dubois@canonical.com> |
497 | */ |
498 | |
499 | -#include "src/platforms/mesa/server/ipc_operations.h" |
500 | -#include "src/platforms/mesa/server/drm_authentication.h" |
501 | +#include "src/platforms/mesa/server/common/ipc_operations.h" |
502 | +#include "src/platforms/mesa/server/common/drm_authentication.h" |
503 | #include "mir/graphics/platform_ipc_package.h" |
504 | #include "mir/graphics/platform_operation_message.h" |
505 | #include "mir_toolkit/mesa/platform_operation.h" |
506 | |
507 | === modified file 'tests/unit-tests/graphics/mesa/test_kms_page_flipper.cpp' |
508 | --- tests/unit-tests/graphics/mesa/test_kms_page_flipper.cpp 2015-03-31 02:35:42 +0000 |
509 | +++ tests/unit-tests/graphics/mesa/test_kms_page_flipper.cpp 2015-06-01 18:31:25 +0000 |
510 | @@ -16,7 +16,7 @@ |
511 | * Authored by: Alexandros Frantzis <alexandros.frantzis@canonical.com> |
512 | */ |
513 | |
514 | -#include "src/platforms/mesa/server/kms_page_flipper.h" |
515 | +#include "src/platforms/mesa/server/kms/kms_page_flipper.h" |
516 | |
517 | #include "mir_test_doubles/mock_drm.h" |
518 | #include "mir_test_doubles/mock_display_report.h" |
519 | |
520 | === modified file 'tests/unit-tests/graphics/mesa/test_linux_virtual_terminal.cpp' |
521 | --- tests/unit-tests/graphics/mesa/test_linux_virtual_terminal.cpp 2015-01-22 09:00:14 +0000 |
522 | +++ tests/unit-tests/graphics/mesa/test_linux_virtual_terminal.cpp 2015-06-01 18:31:25 +0000 |
523 | @@ -16,7 +16,7 @@ |
524 | * Authored by: Alexandros Frantzis <alexandros.frantzis@canonical.com> |
525 | */ |
526 | |
527 | -#include "src/platforms/mesa/server/linux_virtual_terminal.h" |
528 | +#include "src/platforms/mesa/server/kms/linux_virtual_terminal.h" |
529 | #include "src/server/report/null_report_factory.h" |
530 | #include "mir/graphics/event_handler_register.h" |
531 | |
532 | |
533 | === modified file 'tests/unit-tests/graphics/mesa/test_nested_authentication.cpp' |
534 | --- tests/unit-tests/graphics/mesa/test_nested_authentication.cpp 2015-03-31 02:35:42 +0000 |
535 | +++ tests/unit-tests/graphics/mesa/test_nested_authentication.cpp 2015-06-01 18:31:25 +0000 |
536 | @@ -18,7 +18,7 @@ |
537 | |
538 | #include "mir/graphics/nested_context.h" |
539 | #include "mir/graphics/platform_operation_message.h" |
540 | -#include "src/platforms/mesa/server/nested_authentication.h" |
541 | +#include "src/platforms/mesa/server/kms/nested_authentication.h" |
542 | #include "mir_toolkit/mesa/platform_operation.h" |
543 | #include "mir_test_doubles/mock_drm.h" |
544 | #include "mir_test_doubles/mock_nested_context.h" |
545 | |
546 | === modified file 'tests/unit-tests/graphics/mesa/test_platform.cpp' |
547 | --- tests/unit-tests/graphics/mesa/test_platform.cpp 2015-05-07 09:06:20 +0000 |
548 | +++ tests/unit-tests/graphics/mesa/test_platform.cpp 2015-06-01 18:31:25 +0000 |
549 | @@ -20,7 +20,7 @@ |
550 | #include "mir/graphics/event_handler_register.h" |
551 | #include "mir/graphics/platform_ipc_operations.h" |
552 | #include "mir/graphics/platform_operation_message.h" |
553 | -#include "src/platforms/mesa/server/platform.h" |
554 | +#include "src/platforms/mesa/server/kms/platform.h" |
555 | #include "src/server/report/null_report_factory.h" |
556 | #include "mir/emergency_cleanup_registry.h" |
557 | #include "mir/shared_library.h" |
558 | |
559 | === modified file 'tests/unit-tests/graphics/mesa/test_real_kms_output.cpp' |
560 | --- tests/unit-tests/graphics/mesa/test_real_kms_output.cpp 2015-03-31 02:35:42 +0000 |
561 | +++ tests/unit-tests/graphics/mesa/test_real_kms_output.cpp 2015-06-01 18:31:25 +0000 |
562 | @@ -16,8 +16,8 @@ |
563 | * Authored by: Alexandros Frantzis <alexandros.frantzis@canonical.com> |
564 | */ |
565 | |
566 | -#include "src/platforms/mesa/server/real_kms_output.h" |
567 | -#include "src/platforms/mesa/server/page_flipper.h" |
568 | +#include "src/platforms/mesa/server/kms/real_kms_output.h" |
569 | +#include "src/platforms/mesa/server/kms/page_flipper.h" |
570 | |
571 | #include "mir_test/fake_shared.h" |
572 | |
573 | |
574 | === modified file 'tests/unit-tests/graphics/mesa/test_shm_buffer.cpp' |
575 | --- tests/unit-tests/graphics/mesa/test_shm_buffer.cpp 2015-03-31 02:35:42 +0000 |
576 | +++ tests/unit-tests/graphics/mesa/test_shm_buffer.cpp 2015-06-01 18:31:25 +0000 |
577 | @@ -16,8 +16,8 @@ |
578 | * Authored by: Alexandros Frantzis <alexandros.frantzis@canonical.com> |
579 | */ |
580 | |
581 | -#include "src/platforms/mesa/server/shm_buffer.h" |
582 | -#include "src/platforms/mesa/server/shm_file.h" |
583 | +#include "src/platforms/mesa/server/common/shm_buffer.h" |
584 | +#include "src/platforms/mesa/server/common/shm_file.h" |
585 | |
586 | #include "mir_test_doubles/mock_gl.h" |
587 |
PASSED: Continuous integration, rev:2603 jenkins. qa.ubuntu. com/job/ mir-ci/ 3939/ jenkins. qa.ubuntu. com/job/ mir-android- vivid-i386- build/2623 jenkins. qa.ubuntu. com/job/ mir-clang- wily-amd64- build/134 jenkins. qa.ubuntu. com/job/ mir-mediumtests -vivid- touch/2571 jenkins. qa.ubuntu. com/job/ mir-wily- amd64-ci/ 95 jenkins. qa.ubuntu. com/job/ mir-wily- amd64-ci/ 95/artifact/ work/output/ *zip*/output. zip jenkins. qa.ubuntu. com/job/ mir-mediumtests -builder- vivid-armhf/ 2571 jenkins. qa.ubuntu. com/job/ mir-mediumtests -builder- vivid-armhf/ 2571/artifact/ work/output/ *zip*/output. zip jenkins. qa.ubuntu. com/job/ mir-mediumtests -runner- mako/5423 s-jenkins. ubuntu- ci:8080/ job/touch- flash-device/ 20831
http://
Executed test runs:
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
SUCCESS: http://
Click here to trigger a rebuild: s-jenkins. ubuntu- ci:8080/ job/mir- ci/3939/ rebuild
http://