Re: screen rotation flipped in 4.8-rc

[Matt Ranostay]

On Wed, Aug 31, 2016 at 1:18 PM, Pandruvada, Srinivas
<srinivas.pandruvada@xxxxxxxxx> wrote:
> On Tue, 2016-08-30 at 17:41 -0700, Matt Ranostay wrote:
>> On Tue, Aug 30, 2016 at 3:42 PM, Pandruvada, Srinivas
>> <srinivas.pandruvada@xxxxxxxxx> 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?
I would highly doubt if it was a issue with that dcache patchsets it
wouldn't be breaking a ton of things.

> Thanks,
> Srinivas
>>
>> Thanks,
>>
>> Matt
>>
>> >
>> >
>> >
>> > commit 703b5faf22fbddf984a361e6555f3a03fdba63d9
>> > Author: George Spelvin <linux@xxxxxxxxxxxxxxxxxxx>
>> > 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 <linux@xxxxxxxxxxxxxxxxxxx>
>> >     Signed-off-by: Linus Torvalds <torvalds@xxxxxxxxxxxxxxxxxxxx>
>> >
>> >
>> > Thanks,
>> > Srinivas