From: Mauro Carvalho Chehab <mchehab+huawei@kernel.org>
To: Matthew Emerson <Matthew.Emerson@ibm.com>
Cc: "linux-edac@vger.kernel.org" <linux-edac@vger.kernel.org>,
"mchehab@kernel.org" <mchehab@kernel.org>
Subject: Re: RASDaemon patch
Date: Tue, 28 Jan 2025 15:10:58 +0100 [thread overview]
Message-ID: <20250128151058.0bcf49a4@foz.lan> (raw)
In-Reply-To: <BL1PR15MB5337303B199FE97F5B005884E6EF2@BL1PR15MB5337.namprd15.prod.outlook.com>
Em Tue, 28 Jan 2025 11:19:45 +0000
Matthew Emerson <Matthew.Emerson@ibm.com> escreveu:
> Hi,
>
> I’m a developer at IBM working on the storage systems range, I am working on using RASDaemon as a method to help manage correctable errors in DIMMs, and I was hoping to talk to you about a patch I was intending to submit.
>
> Essentially, I was hoping to submit a patch which will have the trigger feature work when the threshold is exceeded rather than when any event occurs. As I am looking for more to happen than offline the page via soft or hard offline when the threshold is exceeded, and it would be beneficial to have a custom script trigger.
Triggering with thresholds makes sense. Yet, as the event trigger starts
an script, it would be possible to use it to check for thresholds inside
the script itself. Not sure what would be the benefit to embed the event
threshold calculus inside rasdaemon. Granted, it can query the database to
better detect thresholds.
On that sense, one thing that I'm considering is to extend the DB interface
for it to work with other databases (MariaDB/MySQL, Postgres,...) and have
a better data model to help taking such policy decisions. Patches to
take such approach are welcomed.
Yet, if you have something already coded, please send a PR to github.
I'll review there. You may also open an issue for us to discuss it there
and keep track of its progress.
> I wanted to know whether this would be something you would consider accepting into the codebase? As I have something working and tested right now. If you were I also wanted to know roughly how long the process would take until merging?
It very much depends on how much time I have to take a look on userspace
code. I'm intending to review a bunch of pending stuff maybe next week
or at the week after that.
> Please get back to me as soon as possible.
>
> Kind regards,
>
> Matt
>
> Unless otherwise stated above:
>
> IBM United Kingdom Limited
> Registered in England and Wales with number 741598
> Registered office: Building C, IBM Hursley Office, Hursley Park Road, Winchester, Hampshire SO21 2JN
Thanks,
Mauro
parent reply other threads:[~2025-01-28 14:11 UTC|newest]
Thread overview: expand[flat|nested] mbox.gz Atom feed
[parent not found: <BL1PR15MB5337303B199FE97F5B005884E6EF2@BL1PR15MB5337.namprd15.prod.outlook.com>]
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=20250128151058.0bcf49a4@foz.lan \
--to=mchehab+huawei@kernel.org \
--cc=Matthew.Emerson@ibm.com \
--cc=linux-edac@vger.kernel.org \
--cc=mchehab@kernel.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