* [PATCH] drm/drm_vma_manager.c: Remove useless goto statement @ 2017-11-01 14:04 Liviu Dudau 2017-11-01 14:11 ` Chris Wilson ` (2 more replies) 0 siblings, 3 replies; 10+ messages in thread From: Liviu Dudau @ 2017-11-01 14:04 UTC (permalink / raw) To: Daniel Vetter; +Cc: Liviu Dudau, DRI-devel Commit db2395eccf08i ("drm: Convert drm_vma_manager to embedded interval-tree in drm_mm") removed a line in drm_vma_offset_add() function that makes checking the result of calling drm_mm_insert_node() and the goto call redundant. Remove them. Fixes: db2395eccf08i ("drm: Convert drm_vma_manager to embedded interval-tree in drm_mm") Cc: Chris Wilson <chris@chris-wilson.co.uk> Signed-off-by: Liviu Dudau <liviu.dudau@arm.com> --- drivers/gpu/drm/drm_vma_manager.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/gpu/drm/drm_vma_manager.c b/drivers/gpu/drm/drm_vma_manager.c index 28f1226576f8c..e545a7b2bcafc 100644 --- a/drivers/gpu/drm/drm_vma_manager.c +++ b/drivers/gpu/drm/drm_vma_manager.c @@ -213,8 +213,6 @@ int drm_vma_offset_add(struct drm_vma_offset_manager *mgr, } ret = drm_mm_insert_node(&mgr->vm_addr_space_mm, &node->vm_node, pages); - if (ret) - goto out_unlock; out_unlock: write_unlock(&mgr->vm_lock); -- 2.14.3 _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply related [flat|nested] 10+ messages in thread
* Re: [PATCH] drm/drm_vma_manager.c: Remove useless goto statement 2017-11-01 14:04 [PATCH] drm/drm_vma_manager.c: Remove useless goto statement Liviu Dudau @ 2017-11-01 14:11 ` Chris Wilson 2017-11-01 14:18 ` Liviu Dudau 2017-11-01 14:20 ` Liviu Dudau 2017-11-01 16:55 ` ✓ Fi.CI.BAT: success for " Patchwork 2017-11-01 17:37 ` ✓ Fi.CI.IGT: " Patchwork 2 siblings, 2 replies; 10+ messages in thread From: Chris Wilson @ 2017-11-01 14:11 UTC (permalink / raw) To: Liviu Dudau, Daniel Vetter; +Cc: Liviu Dudau, DRI-devel Quoting Liviu Dudau (2017-11-01 14:04:24) > Commit db2395eccf08i ("drm: Convert drm_vma_manager to embedded > interval-tree in drm_mm") removed a line in drm_vma_offset_add() function > that makes checking the result of calling drm_mm_insert_node() and the > goto call redundant. Remove them. > > Fixes: db2395eccf08i ("drm: Convert drm_vma_manager to embedded interval-tree in drm_mm") > Cc: Chris Wilson <chris@chris-wilson.co.uk> > Signed-off-by: Liviu Dudau <liviu.dudau@arm.com> > --- > drivers/gpu/drm/drm_vma_manager.c | 2 -- > 1 file changed, 2 deletions(-) > > diff --git a/drivers/gpu/drm/drm_vma_manager.c b/drivers/gpu/drm/drm_vma_manager.c > index 28f1226576f8c..e545a7b2bcafc 100644 > --- a/drivers/gpu/drm/drm_vma_manager.c > +++ b/drivers/gpu/drm/drm_vma_manager.c > @@ -213,8 +213,6 @@ int drm_vma_offset_add(struct drm_vma_offset_manager *mgr, > } > > ret = drm_mm_insert_node(&mgr->vm_addr_space_mm, &node->vm_node, pages); > - if (ret) > - goto out_unlock; Whilst you are here, you might consider this function too simple to even require the out_unlock... -Chris _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] drm/drm_vma_manager.c: Remove useless goto statement 2017-11-01 14:11 ` Chris Wilson @ 2017-11-01 14:18 ` Liviu Dudau 2017-11-01 14:20 ` Liviu Dudau 1 sibling, 0 replies; 10+ messages in thread From: Liviu Dudau @ 2017-11-01 14:18 UTC (permalink / raw) To: Chris Wilson; +Cc: Daniel Vetter, DRI-devel On Wed, Nov 01, 2017 at 02:11:44PM +0000, Chris Wilson wrote: > Quoting Liviu Dudau (2017-11-01 14:04:24) > > Commit db2395eccf08i ("drm: Convert drm_vma_manager to embedded > > interval-tree in drm_mm") removed a line in drm_vma_offset_add() function > > that makes checking the result of calling drm_mm_insert_node() and the > > goto call redundant. Remove them. > > > > Fixes: db2395eccf08i ("drm: Convert drm_vma_manager to embedded interval-tree in drm_mm") > > Cc: Chris Wilson <chris@chris-wilson.co.uk> > > Signed-off-by: Liviu Dudau <liviu.dudau@arm.com> > > --- > > drivers/gpu/drm/drm_vma_manager.c | 2 -- > > 1 file changed, 2 deletions(-) > > > > diff --git a/drivers/gpu/drm/drm_vma_manager.c b/drivers/gpu/drm/drm_vma_manager.c > > index 28f1226576f8c..e545a7b2bcafc 100644 > > --- a/drivers/gpu/drm/drm_vma_manager.c > > +++ b/drivers/gpu/drm/drm_vma_manager.c > > @@ -213,8 +213,6 @@ int drm_vma_offset_add(struct drm_vma_offset_manager *mgr, > > } > > > > ret = drm_mm_insert_node(&mgr->vm_addr_space_mm, &node->vm_node, pages); > > - if (ret) > > - goto out_unlock; > > Whilst you are here, you might consider this function too simple to even > require the out_unlock... Bah, you are right, should've removed that one too :) Will send an update shortly. Thanks! Liviu > -Chris -- ==================== | I would like to | | fix the world, | | but they're not | | giving me the | \ source code! / --------------- ¯\_(ツ)_/¯ _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] drm/drm_vma_manager.c: Remove useless goto statement 2017-11-01 14:11 ` Chris Wilson 2017-11-01 14:18 ` Liviu Dudau @ 2017-11-01 14:20 ` Liviu Dudau 2017-11-01 14:25 ` Chris Wilson 1 sibling, 1 reply; 10+ messages in thread From: Liviu Dudau @ 2017-11-01 14:20 UTC (permalink / raw) To: Chris Wilson; +Cc: Daniel Vetter, DRI-devel On Wed, Nov 01, 2017 at 02:11:44PM +0000, Chris Wilson wrote: > Quoting Liviu Dudau (2017-11-01 14:04:24) > > Commit db2395eccf08i ("drm: Convert drm_vma_manager to embedded > > interval-tree in drm_mm") removed a line in drm_vma_offset_add() function > > that makes checking the result of calling drm_mm_insert_node() and the > > goto call redundant. Remove them. > > > > Fixes: db2395eccf08i ("drm: Convert drm_vma_manager to embedded interval-tree in drm_mm") > > Cc: Chris Wilson <chris@chris-wilson.co.uk> > > Signed-off-by: Liviu Dudau <liviu.dudau@arm.com> > > --- > > drivers/gpu/drm/drm_vma_manager.c | 2 -- > > 1 file changed, 2 deletions(-) > > > > diff --git a/drivers/gpu/drm/drm_vma_manager.c b/drivers/gpu/drm/drm_vma_manager.c > > index 28f1226576f8c..e545a7b2bcafc 100644 > > --- a/drivers/gpu/drm/drm_vma_manager.c > > +++ b/drivers/gpu/drm/drm_vma_manager.c > > @@ -213,8 +213,6 @@ int drm_vma_offset_add(struct drm_vma_offset_manager *mgr, > > } > > > > ret = drm_mm_insert_node(&mgr->vm_addr_space_mm, &node->vm_node, pages); > > - if (ret) > > - goto out_unlock; > > Whilst you are here, you might consider this function too simple to even > require the out_unlock... Actually, looking at the function as it stands after the patch, if I want to remove the out_unlock label I need to re-write the whole function. Are you OK with me doing that? Best regards, Liviu > -Chris -- ==================== | I would like to | | fix the world, | | but they're not | | giving me the | \ source code! / --------------- ¯\_(ツ)_/¯ _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] drm/drm_vma_manager.c: Remove useless goto statement 2017-11-01 14:20 ` Liviu Dudau @ 2017-11-01 14:25 ` Chris Wilson 2017-11-01 14:44 ` [PATCH v2] " Liviu Dudau 0 siblings, 1 reply; 10+ messages in thread From: Chris Wilson @ 2017-11-01 14:25 UTC (permalink / raw) To: Liviu Dudau; +Cc: Daniel Vetter, DRI-devel Quoting Liviu Dudau (2017-11-01 14:20:53) > On Wed, Nov 01, 2017 at 02:11:44PM +0000, Chris Wilson wrote: > > Quoting Liviu Dudau (2017-11-01 14:04:24) > > > Commit db2395eccf08i ("drm: Convert drm_vma_manager to embedded > > > interval-tree in drm_mm") removed a line in drm_vma_offset_add() function > > > that makes checking the result of calling drm_mm_insert_node() and the > > > goto call redundant. Remove them. > > > > > > Fixes: db2395eccf08i ("drm: Convert drm_vma_manager to embedded interval-tree in drm_mm") > > > Cc: Chris Wilson <chris@chris-wilson.co.uk> > > > Signed-off-by: Liviu Dudau <liviu.dudau@arm.com> > > > --- > > > drivers/gpu/drm/drm_vma_manager.c | 2 -- > > > 1 file changed, 2 deletions(-) > > > > > > diff --git a/drivers/gpu/drm/drm_vma_manager.c b/drivers/gpu/drm/drm_vma_manager.c > > > index 28f1226576f8c..e545a7b2bcafc 100644 > > > --- a/drivers/gpu/drm/drm_vma_manager.c > > > +++ b/drivers/gpu/drm/drm_vma_manager.c > > > @@ -213,8 +213,6 @@ int drm_vma_offset_add(struct drm_vma_offset_manager *mgr, > > > } > > > > > > ret = drm_mm_insert_node(&mgr->vm_addr_space_mm, &node->vm_node, pages); > > > - if (ret) > > > - goto out_unlock; > > > > Whilst you are here, you might consider this function too simple to even > > require the out_unlock... > > Actually, looking at the function as it stands after the patch, if I > want to remove the out_unlock label I need to re-write the whole > function. Are you OK with me doing that? Yes. My suggestion is along the lines of diff --git a/drivers/gpu/drm/drm_vma_manager.c b/drivers/gpu/drm/drm_vma_manager.c index 28f1226576f8..ba7857ed4361 100644 --- a/drivers/gpu/drm/drm_vma_manager.c +++ b/drivers/gpu/drm/drm_vma_manager.c @@ -207,17 +207,13 @@ int drm_vma_offset_add(struct drm_vma_offset_manager *mgr, write_lock(&mgr->vm_lock); - if (drm_mm_node_allocated(&node->vm_node)) { - ret = 0; - goto out_unlock; - } + ret = 0; + if (!drm_mm_node_allocated(&node->vm_node)) + ret = drm_mm_insert_node(&mgr->vm_addr_space_mm, + &node->vm_node, pages); - ret = drm_mm_insert_node(&mgr->vm_addr_space_mm, &node->vm_node, pages); - if (ret) - goto out_unlock; - -out_unlock: write_unlock(&mgr->vm_lock); + return ret; } Can you please cc functional patches like this to intel-gfx@lists.freedesktop.org? That will give us some reassurance that the change isn't outright horrible. ;) -Chris _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply related [flat|nested] 10+ messages in thread
* [PATCH v2] drm/drm_vma_manager.c: Remove useless goto statement 2017-11-01 14:25 ` Chris Wilson @ 2017-11-01 14:44 ` Liviu Dudau 2017-11-01 14:47 ` Chris Wilson 0 siblings, 1 reply; 10+ messages in thread From: Liviu Dudau @ 2017-11-01 14:44 UTC (permalink / raw) To: Daniel Vetter; +Cc: intel-gfx, Liviu Dudau, DRI-devel Commit db2395eccf08i ("drm: Convert drm_vma_manager to embedded interval-tree in drm_mm") removed a line in drm_vma_offset_add() function that makes checking the result of calling drm_mm_insert_node() and the goto call redundant. Rework the function (as suggested by Chris Wilson) to eliminate the need for the goto and associated label. v2: rewrite function to remove all goto statements. Fixes: db2395eccf08i ("drm: Convert drm_vma_manager to embedded interval-tree in drm_mm") Cc: Chris Wilson <chris@chris-wilson.co.uk> Signed-off-by: Liviu Dudau <liviu.dudau@arm.com> --- drivers/gpu/drm/drm_vma_manager.c | 15 +++++---------- 1 file changed, 5 insertions(+), 10 deletions(-) diff --git a/drivers/gpu/drm/drm_vma_manager.c b/drivers/gpu/drm/drm_vma_manager.c index 28f1226576f8c..23c749c05b5aa 100644 --- a/drivers/gpu/drm/drm_vma_manager.c +++ b/drivers/gpu/drm/drm_vma_manager.c @@ -203,21 +203,16 @@ EXPORT_SYMBOL(drm_vma_offset_lookup_locked); int drm_vma_offset_add(struct drm_vma_offset_manager *mgr, struct drm_vma_offset_node *node, unsigned long pages) { - int ret; + int ret = 0; write_lock(&mgr->vm_lock); - if (drm_mm_node_allocated(&node->vm_node)) { - ret = 0; - goto out_unlock; - } + if (!drm_mm_node_allocated(&node->vm_node)) + ret = drm_mm_insert_node(&mgr->vm_addr_space_mm, + &node->vm_node, pages); - ret = drm_mm_insert_node(&mgr->vm_addr_space_mm, &node->vm_node, pages); - if (ret) - goto out_unlock; - -out_unlock: write_unlock(&mgr->vm_lock); + return ret; } EXPORT_SYMBOL(drm_vma_offset_add); -- 2.14.3 _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply related [flat|nested] 10+ messages in thread
* Re: [PATCH v2] drm/drm_vma_manager.c: Remove useless goto statement 2017-11-01 14:44 ` [PATCH v2] " Liviu Dudau @ 2017-11-01 14:47 ` Chris Wilson 2017-11-02 9:43 ` Daniel Vetter 0 siblings, 1 reply; 10+ messages in thread From: Chris Wilson @ 2017-11-01 14:47 UTC (permalink / raw) To: Liviu Dudau, Daniel Vetter; +Cc: intel-gfx, DRI-devel Quoting Liviu Dudau (2017-11-01 14:44:58) > Commit db2395eccf08i ("drm: Convert drm_vma_manager to embedded > interval-tree in drm_mm") removed a line in drm_vma_offset_add() function that > makes checking the result of calling drm_mm_insert_node() and the goto > call redundant. Rework the function (as suggested by Chris Wilson) to > eliminate the need for the goto and associated label. > > v2: rewrite function to remove all goto statements. > > Fixes: db2395eccf08i ("drm: Convert drm_vma_manager to embedded interval-tree in drm_mm") > Cc: Chris Wilson <chris@chris-wilson.co.uk> > Signed-off-by: Liviu Dudau <liviu.dudau@arm.com> Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk> -Chris _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH v2] drm/drm_vma_manager.c: Remove useless goto statement 2017-11-01 14:47 ` Chris Wilson @ 2017-11-02 9:43 ` Daniel Vetter 0 siblings, 0 replies; 10+ messages in thread From: Daniel Vetter @ 2017-11-02 9:43 UTC (permalink / raw) To: Chris Wilson; +Cc: Daniel Vetter, intel-gfx, DRI-devel On Wed, Nov 01, 2017 at 02:47:34PM +0000, Chris Wilson wrote: > Quoting Liviu Dudau (2017-11-01 14:44:58) > > Commit db2395eccf08i ("drm: Convert drm_vma_manager to embedded > > interval-tree in drm_mm") removed a line in drm_vma_offset_add() function that > > makes checking the result of calling drm_mm_insert_node() and the goto > > call redundant. Rework the function (as suggested by Chris Wilson) to > > eliminate the need for the goto and associated label. > > > > v2: rewrite function to remove all goto statements. > > > > Fixes: db2395eccf08i ("drm: Convert drm_vma_manager to embedded interval-tree in drm_mm") > > Cc: Chris Wilson <chris@chris-wilson.co.uk> > > Signed-off-by: Liviu Dudau <liviu.dudau@arm.com> > Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk> Applied to drm-misc-next. -Daniel -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 10+ messages in thread
* ✓ Fi.CI.BAT: success for drm/drm_vma_manager.c: Remove useless goto statement 2017-11-01 14:04 [PATCH] drm/drm_vma_manager.c: Remove useless goto statement Liviu Dudau 2017-11-01 14:11 ` Chris Wilson @ 2017-11-01 16:55 ` Patchwork 2017-11-01 17:37 ` ✓ Fi.CI.IGT: " Patchwork 2 siblings, 0 replies; 10+ messages in thread From: Patchwork @ 2017-11-01 16:55 UTC (permalink / raw) To: Liviu Dudau; +Cc: intel-gfx == Series Details == Series: drm/drm_vma_manager.c: Remove useless goto statement URL : https://patchwork.freedesktop.org/series/32987/ State : success == Summary == Series 32987v1 drm/drm_vma_manager.c: Remove useless goto statement https://patchwork.freedesktop.org/api/1.0/series/32987/revisions/1/mbox/ Test gem_ctx_switch: Subgroup basic-default-heavy: incomplete -> PASS (fi-glk-dsi) fi-bdw-5557u total:289 pass:268 dwarn:0 dfail:0 fail:0 skip:21 time:447s fi-blb-e6850 total:289 pass:223 dwarn:1 dfail:0 fail:0 skip:65 time:379s fi-bsw-n3050 total:289 pass:243 dwarn:0 dfail:0 fail:0 skip:46 time:539s fi-bwr-2160 total:289 pass:183 dwarn:0 dfail:0 fail:0 skip:106 time:276s fi-bxt-dsi total:289 pass:259 dwarn:0 dfail:0 fail:0 skip:30 time:508s fi-bxt-j4205 total:289 pass:260 dwarn:0 dfail:0 fail:0 skip:29 time:503s fi-byt-j1900 total:289 pass:253 dwarn:1 dfail:0 fail:0 skip:35 time:510s fi-byt-n2820 total:289 pass:249 dwarn:1 dfail:0 fail:0 skip:39 time:494s fi-cfl-s total:289 pass:253 dwarn:4 dfail:0 fail:0 skip:32 time:569s fi-cnl-y total:217 pass:196 dwarn:0 dfail:0 fail:0 skip:20 fi-elk-e7500 total:289 pass:229 dwarn:0 dfail:0 fail:0 skip:60 time:431s fi-gdg-551 total:289 pass:178 dwarn:1 dfail:0 fail:1 skip:109 time:258s fi-glk-1 total:289 pass:261 dwarn:0 dfail:0 fail:0 skip:28 time:586s fi-glk-dsi total:289 pass:257 dwarn:0 dfail:0 fail:2 skip:30 time:491s fi-hsw-4770 total:289 pass:262 dwarn:0 dfail:0 fail:0 skip:27 time:438s fi-hsw-4770r total:289 pass:262 dwarn:0 dfail:0 fail:0 skip:27 time:430s fi-ilk-650 total:289 pass:228 dwarn:0 dfail:0 fail:0 skip:61 time:429s fi-ivb-3520m total:289 pass:260 dwarn:0 dfail:0 fail:0 skip:29 time:506s fi-ivb-3770 total:289 pass:260 dwarn:0 dfail:0 fail:0 skip:29 time:470s fi-kbl-7500u total:289 pass:264 dwarn:1 dfail:0 fail:0 skip:24 time:486s fi-kbl-7560u total:289 pass:270 dwarn:0 dfail:0 fail:0 skip:19 time:572s fi-kbl-7567u total:289 pass:269 dwarn:0 dfail:0 fail:0 skip:20 time:481s fi-kbl-r total:289 pass:262 dwarn:0 dfail:0 fail:0 skip:27 time:584s fi-pnv-d510 total:289 pass:222 dwarn:1 dfail:0 fail:0 skip:66 time:571s fi-skl-6260u total:289 pass:269 dwarn:0 dfail:0 fail:0 skip:20 time:454s fi-skl-6600u total:289 pass:262 dwarn:0 dfail:0 fail:0 skip:27 time:598s fi-skl-6700hq total:289 pass:263 dwarn:0 dfail:0 fail:0 skip:26 time:649s fi-skl-6700k total:289 pass:265 dwarn:0 dfail:0 fail:0 skip:24 time:521s fi-skl-6770hq total:289 pass:269 dwarn:0 dfail:0 fail:0 skip:20 time:500s fi-snb-2520m total:289 pass:250 dwarn:0 dfail:0 fail:0 skip:39 time:572s fi-snb-2600 total:289 pass:249 dwarn:0 dfail:0 fail:0 skip:40 time:420s fff06da054b4752eabf7d4103d30a281f8de96d9 drm-tip: 2017y-11m-01d-13h-47m-20s UTC integration manifest c78001f8a807 drm/drm_vma_manager.c: Remove useless goto statement == Logs == For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_6296/ _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 10+ messages in thread
* ✓ Fi.CI.IGT: success for drm/drm_vma_manager.c: Remove useless goto statement 2017-11-01 14:04 [PATCH] drm/drm_vma_manager.c: Remove useless goto statement Liviu Dudau 2017-11-01 14:11 ` Chris Wilson 2017-11-01 16:55 ` ✓ Fi.CI.BAT: success for " Patchwork @ 2017-11-01 17:37 ` Patchwork 2 siblings, 0 replies; 10+ messages in thread From: Patchwork @ 2017-11-01 17:37 UTC (permalink / raw) To: Liviu Dudau; +Cc: intel-gfx == Series Details == Series: drm/drm_vma_manager.c: Remove useless goto statement URL : https://patchwork.freedesktop.org/series/32987/ State : success == Summary == Test kms_flip: Subgroup flip-vs-blocking-wf-vblank: fail -> PASS (shard-hsw) Subgroup wf_vblank-vs-modeset-interruptible: dmesg-warn -> PASS (shard-hsw) fdo#102614 Test gem_exec_suspend: Subgroup basic-S4: fail -> SKIP (shard-hsw) Test drv_module_reload: Subgroup basic-reload: dmesg-warn -> PASS (shard-hsw) fdo#102707 fdo#102614 https://bugs.freedesktop.org/show_bug.cgi?id=102614 fdo#102707 https://bugs.freedesktop.org/show_bug.cgi?id=102707 shard-hsw total:2539 pass:1432 dwarn:1 dfail:0 fail:8 skip:1098 time:9224s == Logs == For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_6296/shards.html _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 10+ messages in thread
end of thread, other threads:[~2017-11-02 9:43 UTC | newest] Thread overview: 10+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2017-11-01 14:04 [PATCH] drm/drm_vma_manager.c: Remove useless goto statement Liviu Dudau 2017-11-01 14:11 ` Chris Wilson 2017-11-01 14:18 ` Liviu Dudau 2017-11-01 14:20 ` Liviu Dudau 2017-11-01 14:25 ` Chris Wilson 2017-11-01 14:44 ` [PATCH v2] " Liviu Dudau 2017-11-01 14:47 ` Chris Wilson 2017-11-02 9:43 ` Daniel Vetter 2017-11-01 16:55 ` ✓ Fi.CI.BAT: success for " Patchwork 2017-11-01 17:37 ` ✓ Fi.CI.IGT: " Patchwork
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.