On Thu, Aug 10, 2017 at 5:29 AM, Jani Nikula wrote:
> Revert to a normal comment, as the enum isn't properly documented
> anyway.
>
> Signed-off-by: Jani Nikula
> ---
> drivers/gpu/drm/i915/i915_reg.h | 2 +-
> 1 file changed, 1 insertion(+), 1
== Series Details ==
Series: pm_rps: Extended testcases with checking PMINTRMSK register value
URL : https://patchwork.freedesktop.org/series/28625/
State : failure
== Summary ==
IGT patchset tested on top of latest successful build
c129026622accef6f54c0cfb0dc55e930cfa60b5 igt: add
Fixes: 3cb0514bf1ca ("dim: add checks for author and committer sign-off-by")
Cc: Benjamin Gaignard
Cc: Daniel Vetter
Signed-off-by: Jani Nikula
---
dim | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git
Thou shalt not push dim patches before passing 'make check'.
Cc: Daniel Vetter
Cc: Maarten Lankhorst
Cc: Benjamin Gaignard
Jani Nikula (4):
dim: Fix shellcheck SC1083
dim: Fix shellcheck SC2162 and
Fixes: 809748225c4e ("dim: Add add-missing-cc command")
Cc: Maarten Lankhorst
Signed-off-by: Jani Nikula
---
dim | 10 ++
1 file changed, 6 insertions(+), 4 deletions(-)
diff --git a/dim b/dim
index 28a137221535..070b199c699b
The fix is a bit artificial, but mute the warning.
Fixes: 809748225c4e ("dim: Add add-missing-cc command")
Cc: Maarten Lankhorst
Signed-off-by: Jani Nikula
---
dim | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/dim
Fixes: 7968b1c836f6 ("dim: Enforce review requirements")
Cc: Daniel Vetter
Signed-off-by: Jani Nikula
---
dim | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/dim b/dim
index f576fca0488a..28a137221535 100755
--- a/dim
+++ b/dim
On Thu, 10 Aug 2017, rodrigo.v...@gmail.com wrote:
>> On Aug 9, 2017, at 8:20 PM, Dhinakaran Pandiyan wrote:
>>
>> We currently assume port A is connected to a DP sink when VBT is absent,
>> instead assume it is connected to an eDP sink, which seems like a more
>> common
On Thu, 10 Aug 2017, "Navare, Manasi D" wrote:
> We currently assume port A is connected to a DP sink when VBT is absent,
> instead assume it is connected to an eDP sink, which seems like a more common
> configuration. Although I don't have data to back this up, it is
On Thu, Aug 10, 2017 at 12:22 AM, Jani Nikula wrote:
> On Thu, 10 Aug 2017, Rodrigo Vivi wrote:
>> On Wed, Aug 9, 2017 at 2:08 PM, Jani Nikula wrote:
>>> Matter of taste, but looks better than the default alabaster,
>>
>> not
On Thu, Aug 10, 2017 at 5:29 AM, Jani Nikula wrote:
> Remove old warning about docproc directive that's not supported in the
> Sphinx toolchain.
I got confused... the warning is not supported by sphinx or docproc?
>
> Signed-off-by: Jani Nikula
>
On 4 August 2017 at 12:20, Lionel Landwerlin
wrote:
> Filling rate of the buffer must discard context switch reports as they
> do not depend upon the periodicity, instead they're a factor on the
> amount of different applications concurrently running on the system.
On Fri, 2017-08-04 at 12:20 +0100, Lionel Landwerlin wrote:
> We want the absolute max the hardware can do, not the max value
> set by a previous application/user.
>
> Signed-off-by: Lionel Landwerlin
> ---
> tests/perf.c | 2 +-
> 1 file changed, 1 insertion(+),
I'm not sure if this is really the case and I don't believe
this is the real fix for the bug mentioned here, but since
I don't see a reliable path when mst_port is set and when
mode_valid is requested I believe it is worth to have this
protection here.
Bugzilla:
On Thu, Aug 10, 2017 at 2:40 AM, Chris Wilson
wrote:
> Quoting Jason Ekstrand (2017-08-10 06:35:43)
> > This adds both trivial error-checking tests as well as more complex
> > tests which actually test whether or not waits do what they're supposed
> > to do. They only
On Thu, Aug 10, 2017 at 05:56:52PM +0300, Jani Nikula wrote:
> On Thu, 10 Aug 2017, "Navare, Manasi D" wrote:
> > We currently assume port A is connected to a DP sink when VBT is absent,
> > instead assume it is connected to an eDP sink, which seems like a more
> >
On 4 August 2017 at 12:20, Lionel Landwerlin
wrote:
> New issues that were discovered while making the tests work on Gen8+ :
>
> - we need to measure timings between periodic reports and discard all
>other kind of reports
>
> - it seems periodicity of the
On Thu, 10 Aug 2017, Rodrigo Vivi wrote:
> On Thu, Aug 10, 2017 at 5:29 AM, Jani Nikula wrote:
>> Remove old warning about docproc directive that's not supported in the
>> Sphinx toolchain.
>
> I got confused... the warning is not supported by
On Wed, 09 Aug 2017, Jim Bride wrote:
> Some fixed resolution panels actually support more than one mode,
> with the only thing different being the refresh rate. Having this
> alternate mode available to us is desirable, because it allows us to
> test PSR on panels
On Thu, 10 Aug 2017, Rodrigo Vivi wrote:
> On Thu, Aug 10, 2017 at 12:39:49PM +0300, Jani Nikula wrote:
>> Based on Sphinx-quickstart, with existing and generated Makefiles merged
>> together.
>>
>> This makes the rst2html and Sphinx builds work side by side. Plain
>>
On Fri, 11 Aug 2017, Dhinakaran Pandiyan wrote:
> DPCD 600h - SET_POWER & SET_DP_PWR_VOLTAGE defines power state
>
> 101 = Set Main-Link for local Sink device and all downstream Sink
> devices to D3 (power-down mode), keep AUX block fully powered, ready to
> reply within a
On Thu, 10 Aug 2017, Rodrigo Vivi wrote:
> On Wed, Aug 9, 2017 at 2:08 PM, Jani Nikula wrote:
>> Based on Sphinx-quickstart, with existing and generated Makefiles merged
>> together.
>>
>> This makes the rst2html and Sphinx builds work side by side.
With this tweak the test pass:
--- a/tests/kms_sysfs_edid_timing.c
+++ b/tests/kms_sysfs_edid_timing.c
@@ -99,7 +99,7 @@ int main(int argc, char **argv)
edid_size = read(fd_edid, edid, 512);
threshold = THRESHOLD_PER_EDID_BLOCK * (edid_size / 128);
-
Sorry Clinton I fail the test again on my BDW NUCi5 with this V3.
Check: https://bugs.freedesktop.org/show_bug.cgi?id=100047
For details.
/Marta
> -Original Message-
> From: Taylor, Clinton A
> Sent: Thursday, August 10, 2017 1:52 AM
> To: intel-gfx@lists.freedesktop.org
> Cc: Taylor,
On Thu, 10 Aug 2017, Rodrigo Vivi wrote:
> On Wed, Aug 9, 2017 at 2:33 PM, Jani Nikula wrote:
>> Way back when the WaveDrom stuff was added, the scripts could only be
>> accessed over http. This caused issues with sites served over https and
>>
On 10/08/17 00:13, Chris Wilson wrote:
Quoting Lionel Landwerlin (2017-08-09 23:47:20)
On 09/08/17 16:38, Chris Wilson wrote:
This is called from execlist context init which we need to be unlocked.
Commit f89823c21224 ("drm/i915/perf: Implement
I915_PERF_ADD/REMOVE_CONFIG interface") added a
On 10/08/17 00:13, Chris Wilson wrote:
Quoting Lionel Landwerlin (2017-08-09 23:47:20)
On 09/08/17 16:38, Chris Wilson wrote:
This is called from execlist context init which we need to be unlocked.
Commit f89823c21224 ("drm/i915/perf: Implement
I915_PERF_ADD/REMOVE_CONFIG interface") added a
On Wed, Aug 09, 2017 at 10:00:16AM -0700, Belgaumkar, Vinay wrote:
>
>
> On 8/9/2017 7:32 AM, Arkadiusz Hiler wrote:
> > On Tue, Aug 08, 2017 at 03:09:00PM -0700, Vinay Belgaumkar wrote:
> > > This is an RFC for adding documentation to IGT subtests. Each subtest can
> > > have
> > > something
On Wed, Aug 09, 2017 at 01:18:55PM -0700, Belgaumkar, Vinay wrote:
> > @@ -710,6 +721,8 @@ static int common_init(int *argc, char **argv,
> > static struct option long_options[] = {
> > {"list-subtests", 0, 0, OPT_LIST_SUBTESTS},
> > {"run-subtest", 1, 0,
On Thu, 10 Aug 2017, Rodrigo Vivi wrote:
> I don't know WaveDrom much... but it works so
>
> Acked-by: Rodrigo Vivi
Thanks, pushed the first three.
BR,
Jani.
>
> On Wed, Aug 9, 2017 at 2:08 PM, Jani Nikula wrote:
>>
testdisplay has no subtests so hook them to existing subtest handling.
Signed-off-by: Petri Latvala
---
TODO: Make testdisplay use igt_simple_init_parse_opts...
tests/testdisplay.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/tests/testdisplay.c
Signed-off-by: Petri Latvala
---
tests/igt_command_line.sh | 12 +++-
1 file changed, 11 insertions(+), 1 deletion(-)
diff --git a/tests/igt_command_line.sh b/tests/igt_command_line.sh
index 7f80fc8..b3a794c 100755
--- a/tests/igt_command_line.sh
+++
A simple and naive format: Double newline denotes paragraph change,
otherwise insert subtest documentation into the generated docs as-is.
Signed-off-by: Petri Latvala
---
Still not sure whether inside is
kosher.
docs/reference/intel-gpu-tools/Makefile.am | 6 ++
Quoting Jason Ekstrand (2017-08-10 06:35:43)
> +igt_main
> +{
> + int fd;
> +
> + igt_fixture {
> + fd = drm_open_driver(DRIVER_ANY);
DRIVER_ANY | DRIVER_VGEM
(ANY really means KMS)
> + igt_require(has_syncobj_wait(fd));
> +
Way back when the WaveDrom stuff was added, the scripts could only be
accessed over http. This caused issues with sites served over https and
modern browsers rightly complaining about mixed content. This was worked
around by downloading the WaveDrom scripts over http at build time, and
bundling
On Thu, 10 Aug 2017, Rodrigo Vivi wrote:
> On Wed, Aug 9, 2017 at 2:08 PM, Jani Nikula wrote:
>> Matter of taste, but looks better than the default alabaster,
>
> not sure which one I preferred...
> alabaster goes better with 01.org... but it indeed
On mer., 2017-08-09 at 09:18 -0700, Matt Roper wrote:
> On Mon, Jul 31, 2017 at 10:36:05AM +0200, Jean Delvare wrote:
> >
> > Hi Matt, Mauro,
> >
> > On Thu, 17 Mar 2016 15:18:20 +0100, Jean Delvare wrote:
> > >
> > > On Tue, 8 Mar 2016 10:32:37 -0800, Matt Roper wrote:
> > > >
> > > > A
Since we do use the SW_SYNC in igt for validating dma-fence and
sync_file, and wish to expand usage to cover driver independent portions
of syncobj interaction, ensure SW_SYNC is included in our testing
Kconfig.
Signed-off-by: Chris Wilson
Cc: Jason Ekstrand
Quoting Jason Ekstrand (2017-08-10 06:35:43)
> This adds both trivial error-checking tests as well as more complex
> tests which actually test whether or not waits do what they're supposed
> to do. They only currently work on i915 but it should be simple to hook
> them up for other drivers by
Based on Sphinx-quickstart, with existing and generated Makefiles merged
together.
This makes the rst2html and Sphinx builds work side by side. Plain
'make' continues to use rst2html, and 'make html' and friends use
Sphinx. The intention is to keep both for a transition period so that we
can have
v2 of [1], with sphinx_rtd_theme and some tweaks to make it nicer.
BR,
Jani.
[1] cover.1502312576.git.jani.nikula@intel.com">http://mid.mail-archive.com/cover.1502312576.git.jani.nikula@intel.com
Jani Nikula (4):
doc: load WaveDrom scripts directly from CDN instead of bundling
doc:
I have no idea what I was thinking when I added them as raw html. Using
the image directive should allow browsers to handle the page layout
better.
Signed-off-by: Jani Nikula
---
drm-intel.rst | 4 +---
drm-misc.rst | 4 +---
2 files changed, 2 insertions(+), 6
Wrap wavedrom timelines in (idea copied
from the wavedrom sphinx extension) to provide scrollbars for
overflowing content. This is useful even with rst2html.
Signed-off-by: Jani Nikula
---
drm-intel-timeline.rst | 2 ++
drm-misc-timeline.rst | 2 ++
2 files changed, 4
On Wed, Aug 09, 2017 at 04:13:05PM -0700, Joe Kniss wrote:
> On Wed, Aug 9, 2017 at 12:14 PM, Noralf Trønnes wrote:
> >
> > Den 09.08.2017 01.42, skrev Joe Kniss:
> >>
> >> Because all drivers currently use gem objects for framebuffer planes,
> >> the virtual create_handle()
Op 10-08-17 om 10:56 schreef Mahesh Kumar:
> Hi,
>
>
> On Wednesday 09 August 2017 03:16 PM, Maarten Lankhorst wrote:
>> Op 18-07-17 om 14:49 schreef Mahesh Kumar:
>>> From: "Kumar, Mahesh"
>>>
>>> This patch creates an entry in debugfs to check the status of IPC.
>>>
Quoting Jason Ekstrand (2017-08-10 06:35:43)
> +static void
> +test_single_wait(int fd, uint32_t test_flags, int expect)
> +{
> + uint32_t syncobj = syncobj_create(fd);
> + uint32_t flags = 0;
> + int timeline;
> +
> + if (test_flags & WAIT_FOR_SUBMIT)
> +
The current documentation for tests is limited to a single string per
test binary. This patch adds support for documenting individual
subtests.
The syntax for subtest documentation is:
igt_document_subtest("Frob knobs to see if one of the "
"crossbeams will go out of
Signed-off-by: Petri Latvala
---
README | 5 +
1 file changed, 5 insertions(+)
diff --git a/README b/README
index d1ea952..5b6cb00 100644
--- a/README
+++ b/README
@@ -32,6 +32,11 @@ tests/
be run. Some tests have futher options and these are detailed by
New ever-growing series with some fixes and additions.
Petri Latvala (5):
Add support for subtest-specific documentation
README: Describe the subtest documentation command line flags
docs: Include subtest documentation
testdisplay: Handle subtest documentation flags
igt_command_line.sh:
Hi,
On Wednesday 09 August 2017 03:16 PM, Maarten Lankhorst wrote:
Op 18-07-17 om 14:49 schreef Mahesh Kumar:
From: "Kumar, Mahesh"
This patch creates an entry in debugfs to check the status of IPC.
This can also be used to enable/disable IPC in supported platforms.
== Series Details ==
Series: Subtest documentation
URL : https://patchwork.freedesktop.org/series/28605/
State : failure
== Summary ==
IGT patchset tested on top of latest successful build
c129026622accef6f54c0cfb0dc55e930cfa60b5 igt: add syncobj_basic.
with latest DRM-Tip kernel build
We need to look at crtc->state->active for the legacy crc to match
the drm crc api. Even though the drm api explictily checks it, there's
no harm in doing the same there so keep it for both.
Introduce a intel_crtc->ips_disabled flag for CRC, which is protected
by crtc->mutex and set it to true
On Thu, 10 Aug 2017, Jani Nikula wrote:
> v2 of [1], with sphinx_rtd_theme and some tweaks to make it nicer.
And sample output at https://people.freedesktop.org/~jani/html/
J.
>
> BR,
> Jani.
>
>
> [1]
Quoting Maarten Lankhorst (2017-08-10 13:41:56)
> The test shows the need for coherency through the dma-buf sync ioctl's,
> but forgets to call dirtyfb, without this the FB Is never updated and we
> will fail anyway.
>
> Solve this by adding a drmModeDirtyFB, I've confirmed by adding -n
> that
On 4 August 2017 at 12:20, Lionel Landwerlin
wrote:
> Signed-off-by: Lionel Landwerlin
Reviewed-by: Matthew Auld
___
Intel-gfx mailing list
On ke, 2017-07-26 at 14:25 +0100, Chris Wilson wrote:
> Since we occasionally stuff an error pointer into obj->mm.pages for a
> semi-permanent or even permanent failure, we have to be more careful and
> not just test against NULL when deciding if the object has a complete
> set of its concurrent
Temporary workaround for "make check" to pass despite the issue with
kms_ccs which do not list any subtests, even though it should.
TO BE REVERTED by Daniel Stone
Cc: Daniel Stone
Cc: Daniel Vetter
Signed-off-by: Arkadiusz Hiler
On Thu, Aug 10, 2017 at 5:39 AM, Jani Nikula wrote:
> I have no idea what I was thinking when I added them as raw html. Using
> the image directive should allow browsers to handle the page layout
> better.
>
> Signed-off-by: Jani Nikula
TIL! Thanks
On Thu, Aug 10, 2017 at 5:39 AM, Jani Nikula wrote:
> Wrap wavedrom timelines in (idea copied
> from the wavedrom sphinx extension) to provide scrollbars for
> overflowing content. This is useful even with rst2html.
>
> Signed-off-by: Jani Nikula
On Thu, Aug 10, 2017 at 04:13:44PM +0300, Jani Nikula wrote:
> On Wed, 09 Aug 2017, Jim Bride wrote:
> > Some fixed resolution panels actually support more than one mode,
> > with the only thing different being the refresh rate. Having this
> > alternate mode available
On Thu, 2017-08-10 at 09:37 -0700, Jim Bride wrote:
> On Thu, Aug 10, 2017 at 04:13:44PM +0300, Jani Nikula wrote:
> > On Wed, 09 Aug 2017, Jim Bride wrote:
> > > Some fixed resolution panels actually support more than one mode,
> > > with the only thing different being
Quoting Rodrigo Vivi (2017-08-10 15:50:43)
> I'm not sure if this is really the case and I don't believe
> this is the real fix for the bug mentioned here, but since
> I don't see a reliable path when mst_port is set and when
> mode_valid is requested I believe it is worth to have this
>
== Series Details ==
Series: drm/i915: Split obj->cache_coherent to track r/w
URL : https://patchwork.freedesktop.org/series/28641/
State : success
== Summary ==
Series 28641v1 drm/i915: Split obj->cache_coherent to track r/w
Quoting Mika Kuoppala (2017-08-10 12:04:51)
> Update gen9 renderstate to account the, long overdue, changes for
> igt commit 5c07135b7bd2 ("tools/null_state/gen9: Send all
> components in VF state").
>
> Signed-off-by: Mika Kuoppala
Does what it says on the tin,
This is called from execlist context init which we need to be unlocked.
Commit f89823c21224 ("drm/i915/perf: Implement
I915_PERF_ADD/REMOVE_CONFIG interface") added a lockdep assert to this
path for unclear reasons, remove it again!
Fixes: 701f8231a2fe ("drm/i915/perf: prune OA configs")
As we cannot have an exclusive stream set if the perf has not been
initialized, we only need to check for that exclusive stream.
Signed-off-by: Chris Wilson
Cc: Lionel Landwerlin
Cc: Matthew Auld
---
Another case where we need to call sysfs_attr_init() to setup the
internal lockdep class prior to use:
[9.325229] BUG: key 880168bc7bb0 not in .data!
[9.325240] DEBUG_LOCKS_WARN_ON(1)
[9.325250] [ cut here ]
[9.325280] WARNING: CPU: 1 PID: 275 at
On 4 August 2017 at 12:20, Lionel Landwerlin
wrote:
> Signed-off-by: Lionel Landwerlin
Reviewed-by: Matthew Auld
___
Intel-gfx mailing list
On 08/10/2017 12:00 AM, Lofstedt, Marta wrote:
With this tweak the test pass:
--- a/tests/kms_sysfs_edid_timing.c
+++ b/tests/kms_sysfs_edid_timing.c
@@ -99,7 +99,7 @@ int main(int argc, char **argv)
edid_size = read(fd_edid, edid, 512);
threshold =
On Thu, 10 Aug 2017, Patchwork wrote:
> == Series Details ==
>
> Series: drm/i915: add register macro definition style guide (rev2)
> URL : https://patchwork.freedesktop.org/series/28370/
> State : failure
>
> == Summary ==
>
> Series 28370v2 drm/i915: add
On Thu, Aug 10, 2017 at 12:39:46PM +0300, Jani Nikula wrote:
> Way back when the WaveDrom stuff was added, the scripts could only be
> accessed over http. This caused issues with sites served over https and
> modern browsers rightly complaining about mixed content. This was worked
> around by
On Thu, Aug 10, 2017 at 12:39:49PM +0300, Jani Nikula wrote:
> Based on Sphinx-quickstart, with existing and generated Makefiles merged
> together.
>
> This makes the rst2html and Sphinx builds work side by side. Plain
> 'make' continues to use rst2html, and 'make html' and friends use
> Sphinx.
Quoting Matthew Auld (2017-08-10 20:02:22)
> On 10 August 2017 at 18:57, Chris Wilson wrote:
> > As we cannot have an exclusive stream set if the perf has not been
> > initialized, we only need to check for that exclusive stream.
> >
> > Signed-off-by: Chris Wilson
On Thu, Aug 10, 2017 at 9:23 AM, Chris Wilson wrote:
> Quoting Rodrigo Vivi (2017-08-10 15:50:43)
>> I'm not sure if this is really the case and I don't believe
>> this is the real fix for the bug mentioned here, but since
>> I don't see a reliable path when mst_port is
On Thu, Aug 10, 2017 at 4:36 AM, Joonas Lahtinen
wrote:
> On ke, 2017-08-09 at 13:07 -0700, Rodrigo Vivi wrote:
>> A missing part to EU slice power gating is the
>> debugfs interface. This patch actually should have been
>> squashed to the initial EU slice power
On 08/08/17 05:30, Michal Wajdeczko wrote:
During debug we may want to investigate all communication
from the Guc. Add proper tracing macros in debug config.
v2: convert remaining DRM_DEBUG into new CT_DEBUG (Michal)
v3: use dedicated Kconfig (Daniele)
Signed-off-by: Michal Wajdeczko
On Wed, Aug 9, 2017 at 9:44 PM, Pandiyan, Dhinakaran
wrote:
> On Wed, 2017-08-02 at 12:26 -0700, Rodrigo Vivi wrote:
>> The idea is to have an unique place to decide the pin-port
>> per platform.
>>
>> So let's create this function now without any functional
>>
On 8/10/2017 3:26 AM, Petri Latvala wrote:
The current documentation for tests is limited to a single string per
test binary. This patch adds support for documenting individual
subtests.
The syntax for subtest documentation is:
igt_document_subtest("Frob knobs to see if one of the "
"Frequencies over 5.4 GHz only supported on certain
DDI ports and SKUs, and requires Vccio >= 0.95V."
More specifically, for current CNL SKUs available
(CNL-U and CNL-Y) we have:
DDI A - 5.4G eDP
DDI B - 8.1G DP
DDI C - 8.1G DP
DDI D - 5.4G DP
v2: Rebase on top of source_rates changes.
v3:
Different from SKL we don't need ctrl1 and cfgcr2, but
we need to dump cfgcr0 and cfgcr1 instead.
v2: rebase and commit message
Cc: Clint Taylor
Cc: Mika Kahola
Signed-off-by: Rodrigo Vivi
Reviewed-by: Mika Kahola
On 8/10/2017 3:26 AM, Petri Latvala wrote:
testdisplay has no subtests so hook them to existing subtest handling.
Signed-off-by: Petri Latvala
---
tests/testdisplay.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/tests/testdisplay.c b/tests/testdisplay.c
On 8/10/2017 3:26 AM, Petri Latvala wrote:
Signed-off-by: Petri Latvala
---
tests/igt_command_line.sh | 12 +++-
1 file changed, 11 insertions(+), 1 deletion(-)
diff --git a/tests/igt_command_line.sh b/tests/igt_command_line.sh
index 7f80fc8..b3a794c
On Thu, 2017-08-10 at 14:23 -0700, Rodrigo Vivi wrote:
> On Wed, Aug 9, 2017 at 9:44 PM, Pandiyan, Dhinakaran
> wrote:
> > On Wed, 2017-08-02 at 12:26 -0700, Rodrigo Vivi wrote:
> >> The idea is to have an unique place to decide the pin-port
> >> per platform.
> >>
Signed-off-by: Jason Ekstrand
---
tests/syncobj_basic.c | 77 +--
1 file changed, 19 insertions(+), 58 deletions(-)
diff --git a/tests/syncobj_basic.c b/tests/syncobj_basic.c
index a7a6742..b08dd2d 100644
---
Signed-off-by: Jason Ekstrand
---
lib/Makefile.sources | 2 +
lib/igt_syncobj.c| 186 +++
lib/igt_syncobj.h| 66 ++
3 files changed, 254 insertions(+)
create mode 100644 lib/igt_syncobj.c
create
Signed-off-by: Jason Ekstrand
---
lib/igt_vgem.c | 25 -
lib/igt_vgem.h | 1 +
2 files changed, 25 insertions(+), 1 deletion(-)
diff --git a/lib/igt_vgem.c b/lib/igt_vgem.c
index 21cccb3..069c6dd 100644
--- a/lib/igt_vgem.c
+++ b/lib/igt_vgem.c
@@
This adds both trivial error-checking tests as well as more complex
tests which actually test whether or not waits do what they're supposed
to do. They only currently work on i915 but it should be simple to hook
them up for other drivers by simply implementing the little function
pointer hook
On 10 August 2017 at 20:29, Joe Kniss wrote:
> On Wed, Aug 9, 2017 at 4:13 PM, Joe Kniss wrote:
>> On Wed, Aug 9, 2017 at 12:14 PM, Noralf Trønnes wrote:
>>>
>>> Den 09.08.2017 01.42, skrev Joe Kniss:
Because all drivers currently
Verified this as per the Bspec and it looks good.
Manasi
On Thu, Aug 10, 2017 at 03:40:08PM -0700, Rodrigo Vivi wrote:
> "Frequencies over 5.4 GHz only supported on certain
> DDI ports and SKUs, and requires Vccio >= 0.95V."
>
> More specifically, for current CNL SKUs available
> (CNL-U and
On ke, 2017-07-26 at 14:26 +0100, Chris Wilson wrote:
> Remove the struct_mutex requirement around dev_priv->mm.bound_list and
> dev_priv->mm.unbound_list by giving it its own spinlock. This reduces
> one more requirement for struct_mutex and in the process gives us
> slightly more accurate
== Series Details ==
Series: tests/igt_command_line: Ignore subtest list for kms_ccs
URL : https://patchwork.freedesktop.org/series/28642/
State : success
== Summary ==
IGT patchset tested on top of latest successful build
c129026622accef6f54c0cfb0dc55e930cfa60b5 igt: add syncobj_basic.
with
From: Clint Taylor
Current 50ms max threshold timing for an EDID read is very close to the
actual time for a 2 block HDMI EDID read. Adjust the timings base on
connector type as DP reads are at 1 MBit and HDMI at 100K bit. If an LSPcon
is connected to device under
When FBC is enabled for linear, legacy Y-tiled and Yf-tiled
surfaces on gen9, the cfb stride must be programmed by SW as
cfb_stride = ceiling[(at least plane width in pixels)/
(32 * compression limit factor)] * 8
v2: Minor fix for a build error
v3: Fixed subject, register
Cleanup the code. Map the pins in accordance to
individual platforms rather than according to ports.
Create separate functions for platforms.
v2:
- Add missing condition for CoffeeLake. Make platform
specific functions static. Add function
i915_ddc_pin_mapping().
Sugested-by Ville Syrjala
On 10 August 2017 at 18:57, Chris Wilson wrote:
> As we cannot have an exclusive stream set if the perf has not been
> initialized, we only need to check for that exclusive stream.
>
> Signed-off-by: Chris Wilson
> Cc: Lionel Landwerlin
== Series Details ==
Series: tests/kms_mmap_write_crc: Add drmModeDirtyFB after dirtying fb
URL : https://patchwork.freedesktop.org/series/28624/
State : warning
== Summary ==
IGT patchset tested on top of latest successful build
c129026622accef6f54c0cfb0dc55e930cfa60b5 igt: add
Quoting Katarzyna Dec (2017-08-10 14:06:15)
> In addition to checking whether the frequency is in correct range
> for certain scenario, we can also verify whether PM interrupts are
> masked correctly.
What does correctly mean? It was a recommendation not a requirement.
You are checking internal
== Series Details ==
Series: drm/i915: Avoid null dereference if mst_port is unset.
URL : https://patchwork.freedesktop.org/series/28633/
State : success
== Summary ==
Series 28633v1 drm/i915: Avoid null dereference if mst_port is unset.
It is indeed much better.
Reviewed-by: Rodrigo Vivi
On Thu, Aug 10, 2017 at 2:39 AM, Jani Nikula wrote:
> I have no idea what I was thinking when I added them as raw html. Using
> the image directive should allow browsers to handle the page layout
On Thu, 2017-08-10 at 19:45 +0200, Stefan Assmann wrote:
> On 2017-08-10 07:50, Rodrigo Vivi wrote:
> > I'm not sure if this is really the case and I don't believe
> > this is the real fix for the bug mentioned here, but since
> > I don't see a reliable path when mst_port is set and when
> >
1 - 100 of 142 matches
Mail list logo