From: Darren Hart <dvhart@infradead.org>
To: Vadim Pasternak <vadimp@mellanox.com>
Cc: andy.shevchenko@gmail.com, gregkh@linuxfoundation.org,
linux-kernel@vger.kernel.org,
platform-driver-x86@vger.kernel.org, jiri@resnulli.us,
michaelsh@mellanox.com, ivecera@redhat.com
Subject: Re: [PATCH v2 3/7] platform/mellanox: mlxreg-hotplug: add extra cycle for hotplug work queue
Date: Thu, 24 May 2018 16:52:14 -0700 [thread overview]
Message-ID: <20180524235214.GA10172@fury> (raw)
In-Reply-To: <1525675735-125547-1-git-send-email-vadimp@mellanox.com>
On Mon, May 07, 2018 at 06:48:51AM +0000, Vadim Pasternak wrote:
Hi Vadim,
> Add extra cycle for hotplug work queue to handle the case when a signal is
> It adds missed logic for signal acknowledge, by adding an extra run for
> received, but no specific signal assertion is detected. Such case
> theoretically can happen for example in case several units are removed or
> inserted at the same time. In this situation acknowledge for some signal
> can be missed at signal top aggreagation status level. The extra run will
> allow to handler to acknowledge the missed signal.
>
...
>
> Fixes: 07b89c2b2a5e ("platform/x86: Introduce support for Mellanox hotplug driver")
This sha doesn't exist in mainline, I believe you are referring to:
$ git l | grep "Introduce support for Mellanox hotplug"
304887041d95 platform/x86: Introduce support for Mellanox hotplug driver
While this was introduced in 4.10. There have been many changes since then, with
significant changes in the 4.17 cycle. I have not added the stable tag, please
let me know if you think this should be pushed to stable, and which of the
intervening patches you'd consider dependencies.
I've fixed the sha in the commit message, and queued for testing.
Thanks,
--
Darren Hart
VMware Open Source Technology Center
next prev parent reply other threads:[~2018-05-24 23:52 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-05-07 6:48 [PATCH v2 3/7] platform/mellanox: mlxreg-hotplug: add extra cycle for hotplug work queue Vadim Pasternak
2018-05-07 6:48 ` [PATCH v2 4/7] platform: mellanox: add new ODM system types to mlx-platform Vadim Pasternak
2018-05-07 6:48 ` [PATCH v2 5/7] platform/x86: mlx-platform: Add LED platform driver activation Vadim Pasternak
2018-05-24 23:57 ` Darren Hart
2018-05-07 6:48 ` [PATCH v2 6/7] platform/mellanox: Introduce support for Mellanox register access driver Vadim Pasternak
2018-05-25 0:31 ` Darren Hart
2018-05-26 11:15 ` Vadim Pasternak
2018-05-26 20:47 ` Darren Hart
2018-05-07 6:48 ` [PATCH v2 7/7] platform/x86: mlx-platform: Add mlxreg-io platform driver activation Vadim Pasternak
2018-05-24 23:52 ` Darren Hart [this message]
2018-05-26 11:11 ` [PATCH v2 3/7] platform/mellanox: mlxreg-hotplug: add extra cycle for hotplug work queue Vadim Pasternak
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=20180524235214.GA10172@fury \
--to=dvhart@infradead.org \
--cc=andy.shevchenko@gmail.com \
--cc=gregkh@linuxfoundation.org \
--cc=ivecera@redhat.com \
--cc=jiri@resnulli.us \
--cc=linux-kernel@vger.kernel.org \
--cc=michaelsh@mellanox.com \
--cc=platform-driver-x86@vger.kernel.org \
--cc=vadimp@mellanox.com \
/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).