All of lore.kernel.org
 help / color / mirror / Atom feed
From: jeffy <jeffy.chen@rock-chips.com>
To: Rohit Vaswani <rvaswani@nvidia.com>,
	"linux-bluetooth@vger.kernel.org"
	<linux-bluetooth@vger.kernel.org>
Cc: Brian Norris <briannorris@chromium.org>,
	Douglas Anderson <dianders@chromium.org>,
	Johan Hedberg <johan.hedberg@intel.com>,
	Peter Hurley <peter@hurleysoftware.com>,
	Johan Hedberg <johan.hedberg@gmail.com>,
	"netdev@vger.kernel.org" <netdev@vger.kernel.org>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	"David S. Miller" <davem@davemloft.net>,
	Marcel Holtmann <marcel@holtmann.org>,
	Gustavo Padovan <gustavo@padovan.org>
Subject: Re: Bluetooth: might sleep error in hidp_session_thread
Date: Tue, 27 Jun 2017 17:43:51 +0800	[thread overview]
Message-ID: <595228D7.1070807@rock-chips.com> (raw)
In-Reply-To: <cc75d14021724c2d8b74a92482d67b7b@HQMAIL105.nvidia.com>

Hi Rohit,

On 06/24/2017 02:00 AM, Rohit Vaswani wrote:
> I don't have a way to reply back to the older message; but you can use by tested-by for the below patch and re-send:
>
ok, i've resent it, thanks for your test by~

and for replying to an old message, i've figured it out when i tried to 
do that recently:

1/ download the mbox file from the patchwork 
link(https://patchwork.kernel.org/patch/***)
2/ rename it to .mbox
3/ import into thunerbird(using ImportExportTools)

hope that helps ;)

> For patch: [v4,3/3] Bluetooth: hidp: fix possible might sleep error in hidp_session_thread
>
> Tested-by: Rohit Vaswani <rvaswani@nvidia.com>
>
> -Rohit
>
> -----Original Message-----
> From: jeffy [mailto:jeffy.chen@rock-chips.com]
> Sent: Friday, June 23, 2017 05:39
> To: Rohit Vaswani; linux-bluetooth@vger.kernel.org
> Cc: Brian Norris; Douglas Anderson; Johan Hedberg; Peter Hurley; Johan Hedberg; netdev@vger.kernel.org; linux-kernel@vger.kernel.org; David S. Miller; Marcel Holtmann; Gustavo Padovan
> Subject: Re: Bluetooth: might sleep error in hidp_session_thread
>
> Hi Rohit,
>
> Thanx for your reply, and sorry to the delay, somehow your mail was marked as spam by the mail server :(
>
> On 06/13/2017 02:31 AM, Rohit Vaswani wrote:
>> Hi Jeffy,
>>
>> I was looking into the patch from Jeffy Chen from February 14 2017  :
>>    [v4,3/3] Bluetooth: hidp: fix possible might sleep error in
>> hidp_session_thread: https://patchwork.kernel.org/patch/9570931/
>>
>> We faced a similar issue and this patch seems to fix the problem in our preliminary test.
>> I am trying to check if there was a reason this wasn't merged earlier ?
>
> hmm, i'm not sure why, but please feel free to add your test-by~
>>
>> Thanks,
>> Rohit
>>
>> nvpublic
>>
>>
>>
>
>

WARNING: multiple messages have this Message-ID (diff)
From: jeffy <jeffy.chen-TNX95d0MmH7DzftRWevZcw@public.gmane.org>
To: Rohit Vaswani <rvaswani-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>,
	"linux-bluetooth-u79uwXL29TY76Z2rM5mHXA@public.gmane.org"
	<linux-bluetooth-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>
Cc: Brian Norris
	<briannorris-F7+t8E8rja9g9hUCZPvPmw@public.gmane.org>,
	Douglas Anderson
	<dianders-F7+t8E8rja9g9hUCZPvPmw@public.gmane.org>,
	Johan Hedberg
	<johan.hedberg-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>,
	Peter Hurley
	<peter-WaGBZJeGNqdsbIuE7sb01tBPR1lH4CV8@public.gmane.org>,
	Johan Hedberg
	<johan.hedberg-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>,
	"netdev-u79uwXL29TY76Z2rM5mHXA@public.gmane.org"
	<netdev-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>,
	"linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org"
	<linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>,
	"David S. Miller" <davem-fT/PcQaiUtIeIZ0/mPfg9Q@public.gmane.org>,
	Marcel Holtmann <marcel-kz+m5ild9QBg9hUCZPvPmw@public.gmane.org>,
	Gustavo Padovan <gustavo-THi1TnShQwVAfugRpC6u6w@public.gmane.org>
Subject: Re: Bluetooth: might sleep error in hidp_session_thread
Date: Tue, 27 Jun 2017 17:43:51 +0800	[thread overview]
Message-ID: <595228D7.1070807@rock-chips.com> (raw)
In-Reply-To: <cc75d14021724c2d8b74a92482d67b7b-wO81nVYWzR5AOoK4uxnQ11aTQe2KTcn/@public.gmane.org>

Hi Rohit,

On 06/24/2017 02:00 AM, Rohit Vaswani wrote:
> I don't have a way to reply back to the older message; but you can use by tested-by for the below patch and re-send:
>
ok, i've resent it, thanks for your test by~

and for replying to an old message, i've figured it out when i tried to 
do that recently:

1/ download the mbox file from the patchwork 
link(https://patchwork.kernel.org/patch/***)
2/ rename it to .mbox
3/ import into thunerbird(using ImportExportTools)

hope that helps ;)

> For patch: [v4,3/3] Bluetooth: hidp: fix possible might sleep error in hidp_session_thread
>
> Tested-by: Rohit Vaswani <rvaswani-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
>
> -Rohit
>
> -----Original Message-----
> From: jeffy [mailto:jeffy.chen-TNX95d0MmH7DzftRWevZcw@public.gmane.org]
> Sent: Friday, June 23, 2017 05:39
> To: Rohit Vaswani; linux-bluetooth-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
> Cc: Brian Norris; Douglas Anderson; Johan Hedberg; Peter Hurley; Johan Hedberg; netdev-u79uwXL29TY76Z2rM5mHXA@public.gmane.org; linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org; David S. Miller; Marcel Holtmann; Gustavo Padovan
> Subject: Re: Bluetooth: might sleep error in hidp_session_thread
>
> Hi Rohit,
>
> Thanx for your reply, and sorry to the delay, somehow your mail was marked as spam by the mail server :(
>
> On 06/13/2017 02:31 AM, Rohit Vaswani wrote:
>> Hi Jeffy,
>>
>> I was looking into the patch from Jeffy Chen from February 14 2017  :
>>    [v4,3/3] Bluetooth: hidp: fix possible might sleep error in
>> hidp_session_thread: https://patchwork.kernel.org/patch/9570931/
>>
>> We faced a similar issue and this patch seems to fix the problem in our preliminary test.
>> I am trying to check if there was a reason this wasn't merged earlier ?
>
> hmm, i'm not sure why, but please feel free to add your test-by~
>>
>> Thanks,
>> Rohit
>>
>> nvpublic
>>
>>
>>
>
>

  reply	other threads:[~2017-06-27  9:43 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-06-12 18:31 Bluetooth: might sleep error in hidp_session_thread Rohit Vaswani
2017-06-12 18:31 ` Rohit Vaswani
2017-06-23 12:38 ` jeffy
2017-06-23 18:00   ` Rohit Vaswani
2017-06-23 18:00     ` Rohit Vaswani
2017-06-27  9:43     ` jeffy [this message]
2017-06-27  9:43       ` jeffy

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=595228D7.1070807@rock-chips.com \
    --to=jeffy.chen@rock-chips.com \
    --cc=briannorris@chromium.org \
    --cc=davem@davemloft.net \
    --cc=dianders@chromium.org \
    --cc=gustavo@padovan.org \
    --cc=johan.hedberg@gmail.com \
    --cc=johan.hedberg@intel.com \
    --cc=linux-bluetooth@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=marcel@holtmann.org \
    --cc=netdev@vger.kernel.org \
    --cc=peter@hurleysoftware.com \
    --cc=rvaswani@nvidia.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.