Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
MIME-Version: 1.0
From:   "NeilBrown" <neilb@suse.de>
To:     "OGAWA Hirofumi" <hirofumi@mail.parknet.co.jp>
Cc:     linux-kernel@vger.kernel.org, linux-block@vger.kernel.org,
        "Christoph Hellwig" <hch@infradead.org>
Subject: Re: [PATCH v2] FAT: use schedule_timeout_uninterruptible() instead of
 congestion_wait()
In-reply-to: <87h7bfmtqc.fsf@mail.parknet.co.jp>
References: <163712349419.13692.2859038330142282757@noble.neil.brown.name>,
 <87ee79yiik.fsf@mail.parknet.co.jp>,
 <163754226639.13692.10449616189734943162@noble.neil.brown.name>,
 <874k84hi5q.fsf@mail.parknet.co.jp>,
 <163911011670.9928.6438117555175130033@noble.neil.brown.name>,
 <163911016975.9928.6568675782275129@noble.neil.brown.name>,
 <87h7bfmtqc.fsf@mail.parknet.co.jp>
Date:   Mon, 13 Dec 2021 13:28:43 +1100
Message-id: <163936252397.22433.9103044991910658320@noble.neil.brown.name>
Precedence: bulk

On Sat, 11 Dec 2021, OGAWA Hirofumi wrote:
> "NeilBrown" <neilb@suse.de> writes:
> 
> > congestion_wait() in this context is just a sleep - block devices do not
> > in general support congestion signalling any more.
> >
> > The goal here is to wait for any recently written data to get to
> > storage.  blkdev_issue_flush() is thought to be too expensive, so
> > replace congestion_wait() with an explicit timeout.
> 
> If just replace, the following looks better
> 
> 	set_current_state(TASK_UNINTERRUPTIBLE);
> 	io_schedule_timeout(HZ/10);
> 
> Otherwise,
> 
> Acked-by: OGAWA Hirofumi <hirofumi@mail.parknet.co.jp>

Thanks.
According to MAINTAINERS, I should send patches for this code to you,
with the implication (I assumed) that you would forwarded them upstream
if acceptable.
But the fact that you have send mt an Acked-By seems to suggest that you
won't be doing that.
To whom should I send this patch with your acked-by?

Thanks,
NeilBrown