From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933252AbcHaUaB (ORCPT ); Wed, 31 Aug 2016 16:30:01 -0400 Received: from mga01.intel.com ([192.55.52.88]:2492 "EHLO mga01.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932393AbcHaU36 (ORCPT ); Wed, 31 Aug 2016 16:29:58 -0400 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.30,263,1470726000"; d="scan'208";a="873326433" From: "Pandruvada, Srinivas" To: "mranostay@gmail.com" CC: "linux-kernel@vger.kernel.org" , "linux@sciencehorizons.net" , "linux-iio@vger.kernel.org" , "jic23@kernel.org" , "hadess@hadess.net" Subject: Re: screen rotation flipped in 4.8-rc Thread-Topic: screen rotation flipped in 4.8-rc Thread-Index: AQHSAw+/cxID8hdQrk+fxxVEMaF2HKBisB+AgAFI44CAAAGPAIAAAYaA Date: Wed, 31 Aug 2016 20:29:56 +0000 Message-ID: <1472675395.5025.31.camel@intel.com> References: <1472596930.23886.13.camel@intel.com> <1472674733.5025.28.camel@intel.com> In-Reply-To: Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [10.7.199.80] Content-Type: text/plain; charset="utf-8" Content-ID: <1283D13F338209408E6357C5BD55AC1B@intel.com> MIME-Version: 1.0 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Transfer-Encoding: 8bit X-MIME-Autoconverted: from base64 to 8bit by mail.home.local id u7VKU5NN007432 On Wed, 2016-08-31 at 13:24 -0700, Matt Ranostay wrote: > On Wed, Aug 31, 2016 at 1:18 PM, Pandruvada, Srinivas > wrote: > > > > On Tue, 2016-08-30 at 17:41 -0700, Matt Ranostay wrote: > > > > > > On Tue, Aug 30, 2016 at 3:42 PM, Pandruvada, Srinivas > > > wrote: > > > > > > > > > > > > Hi All, > > > > > > > > I observed that using iio-sensor-proxy.service, the auto screen > > > > rotation flipped on my laptop (Normal -> vertical, vertical- > > > > > > > > > > normal) > > > > using kernel v4.8. > > > > > > > > Anyone else has seen this? > > > > > > > > I did a bisect and found a commit, which I am not sure how can > > > > it > > > > impact. > > > Could you post the results of CONFIG_TEST_HASH enabled? > > Attached dmesg.txt with the CONFIG_TEST_HASH=y. > > > > I see > > [    4.276138] test_hash: __hash_32() has no arch implementation to > > test. > > [    4.276138] test_hash: hash_32() has no arch implementation to > > test. > > [    4.276139] test_hash: hash_64() has no arch implementation to > > test. > > [    4.276141] test_hash: 33152 tests passed. > > > Ok have you tested with that patchset reverted? And if so does the > regression disappear? Yes, but not after adding CONFIG_TEST_HASH=y. regression disappears. > I would highly doubt if it was a issue with that dcache patchsets it > wouldn't be breaking a ton of things. I also think that. Thanks, Srinivas > > > > > Thanks, > > Srinivas > > > > > > > > > Thanks, > > > > > > Matt > > > > > > > > > > > > > > > > > > > > > > > commit 703b5faf22fbddf984a361e6555f3a03fdba63d9 > > > > Author: George Spelvin > > > > Date:   Fri Jun 10 00:22:12 2016 -0400 > > > > > > > >     fs/dcache.c: Save one 32-bit multiply in dcache lookup > > > > > > > >     Noe that we're mixing in the parent pointer earlier, we > > > >     don't need to use hash_32() to mix its bits.  Instead, we > > > > can > > > >     just take the msbits of the hash value directly. > > > > > > > >     For those applications which use the partial_name_hash(), > > > >     move the multiply to end_name_hash. > > > > > > > >     Signed-off-by: George Spelvin > > > >     Signed-off-by: Linus Torvalds > > > g> > > > > > > > > > > > > Thanks, > > > > Srinivas