From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-5.2 required=3.0 tests=BAYES_00, HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS, USER_AGENT_SANE_1 autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 20BD3C4708B for ; Thu, 27 May 2021 11:43:53 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 003C16113B for ; Thu, 27 May 2021 11:43:52 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S233771AbhE0LpY (ORCPT ); Thu, 27 May 2021 07:45:24 -0400 Received: from verein.lst.de ([213.95.11.211]:38471 "EHLO verein.lst.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S233490AbhE0LpX (ORCPT ); Thu, 27 May 2021 07:45:23 -0400 Received: by verein.lst.de (Postfix, from userid 2407) id 9B82868AFE; Thu, 27 May 2021 13:43:47 +0200 (CEST) Date: Thu, 27 May 2021 13:43:47 +0200 From: Christoph Hellwig To: Chaitanya Kulkarni Cc: Christoph Hellwig , Matthew Wilcox , "linux-block@vger.kernel.org" , "linux-scsi@vger.kernel.org" , "target-devel@vger.kernel.org" , "linux-btrfs@vger.kernel.org" , "axboe@kernel.dk" , "mb@lightnvm.io" , "martin.petersen@oracle.com" , "clm@fb.com" , "josef@toxicpanda.com" , "dsterba@suse.com" , Johannes Thumshirn , "ming.lei@redhat.com" , "osandov@fb.com" , "jefflexu@linux.alibaba.com" Subject: Re: [RFC PATCH 0/8] block: fix bio_add_XXX_page() return type Message-ID: <20210527114347.GA18214@lst.de> References: <20210520062255.4908-1-chaitanya.kulkarni@wdc.com> <20210524073527.GA24302@lst.de> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.17 (2007-11-01) Precedence: bulk List-ID: X-Mailing-List: linux-block@vger.kernel.org On Wed, May 26, 2021 at 02:55:57AM +0000, Chaitanya Kulkarni wrote: > Is above comment is on this series or on the API present in the folio > patches [1] ? All of the above. > Since if we change the return type to bool for the functions in > question [2] in this series we also need to modify the callers, I'm not sure > that is worth it though. Yes, all the caller would need to change as well.