All of lore.kernel.org
 help / color / mirror / Atom feed
From: Nilay Shroff <nilay@linux.ibm.com>
To: Shinichiro Kawasaki <shinichiro.kawasaki@wdc.com>
Cc: "linux-block@vger.kernel.org" <linux-block@vger.kernel.org>,
	"yukuai3@huawei.com" <yukuai3@huawei.com>,
	"yi.zhang@redhat.com" <yi.zhang@redhat.com>,
	"bvanassche@acm.org" <bvanassche@acm.org>,
	"gjoyce@ibm.com" <gjoyce@ibm.com>
Subject: Re: [PATCHv2 blktests 0/2] throtl: fix IO block-size and race while submitting IO
Date: Thu, 16 Jan 2025 16:52:08 +0530	[thread overview]
Message-ID: <5507bb17-b2aa-4df7-ac3b-73b7a8e4a096@linux.ibm.com> (raw)
In-Reply-To: <3bttr4zv2clzes6q4cuyy3l35ls7cm2r6ljzcgzc3cpvut7dmn@grxcm43s2yse>

Hi Shinichiro,

On 12/24/24 1:56 PM, Shinichiro Kawasaki wrote:
> On Dec 18, 2024 / 19:13, Nilay Shroff wrote:
>> Hi,
>>
>> The first patch fixes the IO block-size used for submitting IO depending
>> on the throttle device max-sectors settings.
>>
>> The second patch fixes the potential race condition between submitting IO
>> and setting PID of the background IO process to cgroup.procs.
>>
>> Changes from v1:
>>     - Use $BASHPID to write the child sub-shell PID into cgroup.procs
>>       (Shinichiro Kawasaki)
>>     - Link to v1: https://lore.kernel.org/all/20241217131440.1980151-1-nilay@linux.ibm.com/
> 
> Thanks for this v2 series. The patches looks good to me. In case anyone has
> some more comments after this vacation season, I will wait several days before
> applying them.

A gentle ping...

Just wanted to check with you whether did you get a chance to merge this change?

Thanks,
--Nilay

  reply	other threads:[~2025-01-16 11:22 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-12-18 13:43 [PATCHv2 blktests 0/2] throtl: fix IO block-size and race while submitting IO Nilay Shroff
2024-12-18 13:43 ` [PATCHv2 blktests 1/2] throtl/002: calculate block-size based on device max-sectors setting Nilay Shroff
2024-12-26  7:30   ` Yi Zhang
2024-12-18 13:43 ` [PATCHv2 blktests 2/2] throtl: fix the race between submitting IO and setting cgroup.procs Nilay Shroff
2024-12-25  2:55   ` Yu Kuai
2024-12-24  8:26 ` [PATCHv2 blktests 0/2] throtl: fix IO block-size and race while submitting IO Shinichiro Kawasaki
2025-01-16 11:22   ` Nilay Shroff [this message]
2025-01-17  0:35     ` Shinichiro Kawasaki

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=5507bb17-b2aa-4df7-ac3b-73b7a8e4a096@linux.ibm.com \
    --to=nilay@linux.ibm.com \
    --cc=bvanassche@acm.org \
    --cc=gjoyce@ibm.com \
    --cc=linux-block@vger.kernel.org \
    --cc=shinichiro.kawasaki@wdc.com \
    --cc=yi.zhang@redhat.com \
    --cc=yukuai3@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.