From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga12.intel.com ([192.55.52.136]:11704 "EHLO mga12.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751372AbeCORn1 (ORCPT ); Thu, 15 Mar 2018 13:43:27 -0400 Date: Thu, 15 Mar 2018 19:43:23 +0200 From: Ville =?iso-8859-1?Q?Syrj=E4l=E4?= To: Ulrich Hecht Cc: intel-gfx@lists.freedesktop.org, linux-renesas-soc@vger.kernel.org, laurent.pinchart@ideasonboard.com Subject: Re: [Intel-gfx] [PATCH igt 6/8] lib/igt_pm: turn absence of autosuspend_delay_ms from fail to skip Message-ID: <20180315174323.GY5453@intel.com> References: <1521125144-28614-1-git-send-email-ulrich.hecht+renesas@gmail.com> <1521125144-28614-7-git-send-email-ulrich.hecht+renesas@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <1521125144-28614-7-git-send-email-ulrich.hecht+renesas@gmail.com> Sender: linux-renesas-soc-owner@vger.kernel.org List-ID: On Thu, Mar 15, 2018 at 03:45:42PM +0100, Ulrich Hecht wrote: > Fixes false negatives on everything that doesn't happen to be at a > specific hard-coded sysfs path... > > Signed-off-by: Ulrich Hecht > --- > lib/igt_pm.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/lib/igt_pm.c b/lib/igt_pm.c > index 5bf5b2e..641157b 100644 > --- a/lib/igt_pm.c > +++ b/lib/igt_pm.c > @@ -262,7 +262,7 @@ bool igt_setup_runtime_pm(void) > * suite goes faster and we have a higher probability of triggering race > * conditions. */ > fd = open(POWER_DIR "/autosuspend_delay_ms", O_WRONLY); The hardocded path should probably go then. igt_sysfs_path() + "/device/power" looks like it should dtrt. Would need to plumb the fd down though. Hopefully every caller has it handy. > - igt_assert_f(fd >= 0, > + igt_require_f(fd >= 0, > "Can't open " POWER_DIR "/autosuspend_delay_ms\n"); > > /* If we fail to write to the file, it means this system doesn't support > -- > 2.7.4 > > _______________________________________________ > Intel-gfx mailing list > Intel-gfx@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/intel-gfx -- Ville Syrj�l� Intel OTC From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ville =?iso-8859-1?Q?Syrj=E4l=E4?= Subject: Re: [PATCH igt 6/8] lib/igt_pm: turn absence of autosuspend_delay_ms from fail to skip Date: Thu, 15 Mar 2018 19:43:23 +0200 Message-ID: <20180315174323.GY5453@intel.com> References: <1521125144-28614-1-git-send-email-ulrich.hecht+renesas@gmail.com> <1521125144-28614-7-git-send-email-ulrich.hecht+renesas@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: Received: from mga02.intel.com (mga02.intel.com [134.134.136.20]) by gabe.freedesktop.org (Postfix) with ESMTPS id DDD8C6E27C for ; Thu, 15 Mar 2018 17:43:26 +0000 (UTC) Content-Disposition: inline In-Reply-To: <1521125144-28614-7-git-send-email-ulrich.hecht+renesas@gmail.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" To: Ulrich Hecht Cc: linux-renesas-soc@vger.kernel.org, intel-gfx@lists.freedesktop.org, laurent.pinchart@ideasonboard.com List-Id: intel-gfx@lists.freedesktop.org T24gVGh1LCBNYXIgMTUsIDIwMTggYXQgMDM6NDU6NDJQTSArMDEwMCwgVWxyaWNoIEhlY2h0IHdy b3RlOgo+IEZpeGVzIGZhbHNlIG5lZ2F0aXZlcyBvbiBldmVyeXRoaW5nIHRoYXQgZG9lc24ndCBo YXBwZW4gdG8gYmUgYXQgYQo+IHNwZWNpZmljIGhhcmQtY29kZWQgc3lzZnMgcGF0aC4uLgo+IAo+ IFNpZ25lZC1vZmYtYnk6IFVscmljaCBIZWNodCA8dWxyaWNoLmhlY2h0K3JlbmVzYXNAZ21haWwu Y29tPgo+IC0tLQo+ICBsaWIvaWd0X3BtLmMgfCAyICstCj4gIDEgZmlsZSBjaGFuZ2VkLCAxIGlu c2VydGlvbigrKSwgMSBkZWxldGlvbigtKQo+IAo+IGRpZmYgLS1naXQgYS9saWIvaWd0X3BtLmMg Yi9saWIvaWd0X3BtLmMKPiBpbmRleCA1YmY1YjJlLi42NDExNTdiIDEwMDY0NAo+IC0tLSBhL2xp Yi9pZ3RfcG0uYwo+ICsrKyBiL2xpYi9pZ3RfcG0uYwo+IEBAIC0yNjIsNyArMjYyLDcgQEAgYm9v bCBpZ3Rfc2V0dXBfcnVudGltZV9wbSh2b2lkKQo+ICAJICogc3VpdGUgZ29lcyBmYXN0ZXIgYW5k IHdlIGhhdmUgYSBoaWdoZXIgcHJvYmFiaWxpdHkgb2YgdHJpZ2dlcmluZyByYWNlCj4gIAkgKiBj b25kaXRpb25zLiAqLwo+ICAJZmQgPSBvcGVuKFBPV0VSX0RJUiAiL2F1dG9zdXNwZW5kX2RlbGF5 X21zIiwgT19XUk9OTFkpOwoKVGhlIGhhcmRvY2RlZCBwYXRoIHNob3VsZCBwcm9iYWJseSBnbyB0 aGVuLgoKaWd0X3N5c2ZzX3BhdGgoKSArICIvZGV2aWNlL3Bvd2VyIiBsb29rcyBsaWtlIGl0IHNo b3VsZCBkdHJ0LgpXb3VsZCBuZWVkIHRvIHBsdW1iIHRoZSBmZCBkb3duIHRob3VnaC4gSG9wZWZ1 bGx5IGV2ZXJ5IGNhbGxlcgpoYXMgaXQgaGFuZHkuCgo+IC0JaWd0X2Fzc2VydF9mKGZkID49IDAs Cj4gKwlpZ3RfcmVxdWlyZV9mKGZkID49IDAsCj4gIAkJICAgICAiQ2FuJ3Qgb3BlbiAiIFBPV0VS X0RJUiAiL2F1dG9zdXNwZW5kX2RlbGF5X21zXG4iKTsKPiAgCj4gIAkvKiBJZiB3ZSBmYWlsIHRv IHdyaXRlIHRvIHRoZSBmaWxlLCBpdCBtZWFucyB0aGlzIHN5c3RlbSBkb2Vzbid0IHN1cHBvcnQK PiAtLSAKPiAyLjcuNAo+IAo+IF9fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fCj4gSW50ZWwtZ2Z4IG1haWxpbmcgbGlzdAo+IEludGVsLWdmeEBsaXN0cy5mcmVl ZGVza3RvcC5vcmcKPiBodHRwczovL2xpc3RzLmZyZWVkZXNrdG9wLm9yZy9tYWlsbWFuL2xpc3Rp bmZvL2ludGVsLWdmeAoKLS0gClZpbGxlIFN5cmrDpGzDpApJbnRlbCBPVEMKX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18KSW50ZWwtZ2Z4IG1haWxpbmcgbGlz dApJbnRlbC1nZnhAbGlzdHMuZnJlZWRlc2t0b3Aub3JnCmh0dHBzOi8vbGlzdHMuZnJlZWRlc2t0 b3Aub3JnL21haWxtYW4vbGlzdGluZm8vaW50ZWwtZ2Z4Cg==