From: Vincent Pelletier <plr.vincent@gmail.com>
To: Mika Westerberg <mika.westerberg@linux.intel.com>
Cc: linux-leds@vger.kernel.org, Simon Guinot <simon.guinot@sequanux.org>
Subject: Re: leds-gpio on x86
Date: Wed, 19 Aug 2015 00:56:35 +0200 [thread overview]
Message-ID: <20150819005635.1daee673@x2> (raw)
In-Reply-To: <20150818133850.42bb31cf@x2>
Hello,
I did a first pass on gpio-f7188x module, and pushed several commits in
my current working branch:
https://github.com/vpelletier/linux/commits/ts651
This commit contains the relevant part of my changes:
gpio: gpio-f7188x: Use mutex for access serialisation.
(currently:
https://github.com/vpelletier/linux/commit/0fecf37bcee67d577f7e1e8e58afc463f452bcff
)
but "of course", it does not work: kernel Oops as soon as I trigger the
concurrent access which caused the original issue. With a single active
core, it works fine of course.
Sadly, the traceback is longer than on-screen info, despite being at
1920x1080 resolution with default console font. I recall reading about
a project to display kernel errors as qr-codes, did it get somewhere ?
Also, watchdog kicks in a few seconds after, complaining about hard
lock on (at least) one core, and scrolling even more. I suspect both
cores are hard locked, and have so far no idea what I did wrong:
- each acquire is followed by a release, with no code path around it
- gpio_device.can_sleep is true (as was the case before my patch)
...would I have somehow poked at another bug ?
Regards,
--
Vincent Pelletier
prev parent reply other threads:[~2015-08-18 22:56 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-08-05 20:09 leds-gpio on x86 Vincent Pelletier
2015-08-06 16:59 ` Mika Westerberg
2015-08-06 17:17 ` Vincent Pelletier
2015-08-06 17:35 ` Mika Westerberg
2015-08-06 18:18 ` Vincent Pelletier
2015-08-07 10:57 ` Mika Westerberg
2015-08-08 12:06 ` Vincent Pelletier
2015-08-11 12:00 ` Mika Westerberg
2015-08-11 17:42 ` Vincent Pelletier
2015-08-12 12:53 ` Mika Westerberg
2015-08-15 10:36 ` Vincent Pelletier
2015-08-17 21:08 ` Vincent Pelletier
2015-08-18 7:32 ` Vincent Pelletier
2015-08-18 9:02 ` Mika Westerberg
2015-08-18 11:38 ` Vincent Pelletier
2015-08-18 22:56 ` Vincent Pelletier [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20150819005635.1daee673@x2 \
--to=plr.vincent@gmail.com \
--cc=linux-leds@vger.kernel.org \
--cc=mika.westerberg@linux.intel.com \
--cc=simon.guinot@sequanux.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).