All of lore.kernel.org
 help / color / mirror / Atom feed
From: Shiju Jose <shiju.jose@huawei.com>
To: Mauro Carvalho Chehab <mchehab@kernel.org>
Cc: "linux-edac@vger.kernel.org" <linux-edac@vger.kernel.org>,
	tanxiaofei <tanxiaofei@huawei.com>,
	fenglei <fenglei47@h-partners.com>,
	Jonathan Cameron <jonathan.cameron@huawei.com>,
	Linuxarm <linuxarm@huawei.com>
Subject: RE: [PATCH 1/1] rasdaemon: Fix for regression in ras_mc_create_table() if some cpus are offline at the system start
Date: Tue, 21 Mar 2023 09:26:11 +0000	[thread overview]
Message-ID: <ea2afdc810954b03bf272199fcefd87b@huawei.com> (raw)
In-Reply-To: <20230318093553.3e524594@coco.lan>

Hi Mauro,

>-----Original Message-----
>From: Mauro Carvalho Chehab <mchehab@kernel.org>
>Sent: 18 March 2023 08:36
>To: Shiju Jose <shiju.jose@huawei.com>
>Cc: linux-edac@vger.kernel.org; tanxiaofei <tanxiaofei@huawei.com>; fenglei
><fenglei47@h-partners.com>; Jonathan Cameron
><jonathan.cameron@huawei.com>; Linuxarm <linuxarm@huawei.com>
>Subject: Re: [PATCH 1/1] rasdaemon: Fix for regression in
>ras_mc_create_table() if some cpus are offline at the system start
>
>Hi Jose,
>
>Em Tue, 14 Mar 2023 10:57:25 +0000
><shiju.jose@huawei.com> escreveu:
>
>> From: Shiju Jose <shiju.jose@huawei.com>
>>
>> Issues:
>> Regression in the ras_mc_create_table() if some of the cpus are
>> offline at the system start when run the rasdaemon. This issue is
>> reproducible in ras_mc_create_table() with decode and record
>> non-standard events and reproducible sometimes with
>> ras_mc_create_table() for the standard events.
>> Also in the multi thread way, there is memory leak in
>> ras_mc_event_opendb() as struct sqlite3_priv *priv and sqlite3 *db
>> allocated/initialized per thread, but stored in the common struct
>> ras_events ras in pthread data, which is shared across the threads.
>
>Could you please submit rasdaemon patches via github pull requests?
>
>It is currently easier to me to pick patches from there. You can still send then to
>the ML in case you want others to take a look and review them, but I'll be
>applying them only after the PR at rasdaemon.

I  will  submit the pull request in the github.
 
>
>Regards,
>Mauro
>
Thanks,
Shiju

      reply	other threads:[~2023-03-21  9:26 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-03-14 10:57 [PATCH 1/1] rasdaemon: Fix for regression in ras_mc_create_table() if some cpus are offline at the system start shiju.jose
2023-03-18  8:35 ` Mauro Carvalho Chehab
2023-03-21  9:26   ` Shiju Jose [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=ea2afdc810954b03bf272199fcefd87b@huawei.com \
    --to=shiju.jose@huawei.com \
    --cc=fenglei47@h-partners.com \
    --cc=jonathan.cameron@huawei.com \
    --cc=linux-edac@vger.kernel.org \
    --cc=linuxarm@huawei.com \
    --cc=mchehab@kernel.org \
    --cc=tanxiaofei@huawei.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.