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 Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by smtp.lore.kernel.org (Postfix) with ESMTP id 2DED6C54EAA for ; Mon, 30 Jan 2023 18:20:37 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id BEE906B0080; Mon, 30 Jan 2023 13:20:36 -0500 (EST) Received: by kanga.kvack.org (Postfix, from userid 40) id B77FD6B0081; Mon, 30 Jan 2023 13:20:36 -0500 (EST) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id A19216B0082; Mon, 30 Jan 2023 13:20:36 -0500 (EST) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0016.hostedemail.com [216.40.44.16]) by kanga.kvack.org (Postfix) with ESMTP id 920176B0080 for ; Mon, 30 Jan 2023 13:20:36 -0500 (EST) Received: from smtpin24.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay03.hostedemail.com (Postfix) with ESMTP id 2B7F2A0AEA for ; Mon, 30 Jan 2023 18:20:36 +0000 (UTC) X-FDA: 80412280872.24.8E45E87 Received: from mail-ej1-f41.google.com (mail-ej1-f41.google.com [209.85.218.41]) by imf19.hostedemail.com (Postfix) with ESMTP id 5BA211A0023 for ; Mon, 30 Jan 2023 18:20:33 +0000 (UTC) Authentication-Results: imf19.hostedemail.com; dkim=pass header.d=gmail.com header.s=20210112 header.b=nKCKI2eU; spf=pass (imf19.hostedemail.com: domain of idryomov@gmail.com designates 209.85.218.41 as permitted sender) smtp.mailfrom=idryomov@gmail.com; dmarc=pass (policy=none) header.from=gmail.com ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1675102833; h=from:from:sender:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:cc:mime-version:mime-version: content-type:content-type:content-transfer-encoding: in-reply-to:in-reply-to:references:references:dkim-signature; bh=ESqBBPYpgVN5uZcCivO5DaR4zZN15VVDdNsOZiTKtrU=; b=mkj8nzuDhoxm9WpuChYBgJs5RpXPcF0NIu9IUR25Ex/o0ErYEwA3hU0GoRjWfwU1c7f2wM 1HmkbH1fOwmGmwq1oPqJVWHXc1lIapNXX7OhI4BOFUVLRTInQA+hGAGtDN3oP68g5UPO6r VaucPz7fYxzIdWcg5LD+PntmXFx0AfI= ARC-Authentication-Results: i=1; imf19.hostedemail.com; dkim=pass header.d=gmail.com header.s=20210112 header.b=nKCKI2eU; spf=pass (imf19.hostedemail.com: domain of idryomov@gmail.com designates 209.85.218.41 as permitted sender) smtp.mailfrom=idryomov@gmail.com; dmarc=pass (policy=none) header.from=gmail.com ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1675102833; a=rsa-sha256; cv=none; b=ZCsvv3zNJUzkc8C2Tx2iD8JlxB8KzznE1EyBVweijQRuei2iZDG6c0xAC6I8hr80oIO7I5 pWtXF1e26UyaWL1qFTUG24YmpOXWGAU1QbL3deORapkNOlhYTljpiEKniXzoztDz966I21 69bFnSOSC/ag2S+JY5lfb9WKrfg579w= Received: by mail-ej1-f41.google.com with SMTP id ml19so10809236ejb.0 for ; Mon, 30 Jan 2023 10:20:32 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20210112; h=cc:to:subject:message-id:date:from:in-reply-to:references :mime-version:from:to:cc:subject:date:message-id:reply-to; bh=ESqBBPYpgVN5uZcCivO5DaR4zZN15VVDdNsOZiTKtrU=; b=nKCKI2eUxGRcaSZgTmFPehKr3qVxRp0sSTX6As0CJA6cEO7S5tDGHwTtxcWOBvjG+5 ClZltX4Y3POVohJyazbmQlAgJeqcXDECvGEvMfxGmFMz3/p/ZhQ+I4K6gGF2MU13LJ4z 19Axk50kaCOy1pUa9HjxYbHCoGtn4Ea+kFtIk141HMca3CQJQn9DvmK96fWyEakjLuFq iS3N2a/s6O/Q2gCI7ySoWzJQ5GPGtaTalwh8TJ4L/KQFPakIBY+KAO+Np+5mMe0R40Ay cBp8FloBIDKo0FIkdpo6fBsfxBdHgDQ9NaLb0nfuj9pZ+SEcS+GIfKSiynUsG/BBJ7OX +CfQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=cc:to:subject:message-id:date:from:in-reply-to:references :mime-version:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=ESqBBPYpgVN5uZcCivO5DaR4zZN15VVDdNsOZiTKtrU=; b=11dQ/Lhwc22xXm7HxEp7vMZQLUxoBQPlUyJPPTfj7J4QsRRYmtppfrWeP+/maHJeG3 /bbTavMnmVgs4N1dJ5ubpkvA0adEIfdpsk5ox7mwL9xucGp1wuJSDFHiBgHPsAQbRhX6 nW1bMFy1Lv52zQk9GgXX8SMnEK7SQOCGc/aBPhrMdgAwBXYOT++UiZj/DBVFXt12IQ6W gXW5cgZlTVORuiTHZq4Z7/zwD19Mdi0AcxsKIyB+wT/3rFbmAi5w41dbDp9OwzHpMAU5 ONksOPzalOrE6owiQ/G0sSMNMgG/D+ORCerKu+4grbtuoeJOLsy4gKvtpslygfdSxLZc JHdw== X-Gm-Message-State: AFqh2kopt5e9EDNipeV8Wy1ktzwF1uAHoNQKWGXJpUdKYobpPiSP/GdD cTMBeHKCNrQnrbucmykh56jxjyQgp+BwttRvi6U= X-Google-Smtp-Source: AMrXdXunXSZTlv6MlRv2Hm0y6kSEWFLV2akgLcFsz+yoVqsfzKFvu004fWkS1yRIAW4Rk97VhQXcJdOpL6Dcz/TIiuw= X-Received: by 2002:a17:906:7754:b0:86f:2cc2:7028 with SMTP id o20-20020a170906775400b0086f2cc27028mr7705802ejn.133.1675102831692; Mon, 30 Jan 2023 10:20:31 -0800 (PST) MIME-Version: 1.0 References: <20230130092157.1759539-1-hch@lst.de> <20230130092157.1759539-24-hch@lst.de> In-Reply-To: <20230130092157.1759539-24-hch@lst.de> From: Ilya Dryomov Date: Mon, 30 Jan 2023 19:20:15 +0100 Message-ID: Subject: Re: [PATCH 23/23] net-ceph: use bvec_set_page to initialize bvecs To: Christoph Hellwig Cc: Jens Axboe , "Michael S. Tsirkin" , Jason Wang , Minchan Kim , Sergey Senozhatsky , Keith Busch , Sagi Grimberg , Chaitanya Kulkarni , "Martin K. Petersen" , David Howells , Marc Dionne , Xiubo Li , Steve French , Trond Myklebust , Anna Schumaker , Mike Marshall , Andrew Morton , "David S. Miller" , Eric Dumazet , Jakub Kicinski , Paolo Abeni , Chuck Lever , linux-block@vger.kernel.org, ceph-devel@vger.kernel.org, virtualization@lists.linux-foundation.org, linux-nvme@lists.infradead.org, linux-scsi@vger.kernel.org, target-devel@vger.kernel.org, kvm@vger.kernel.org, netdev@vger.kernel.org, linux-afs@lists.infradead.org, linux-cifs@vger.kernel.org, samba-technical@lists.samba.org, linux-fsdevel@vger.kernel.org, linux-nfs@vger.kernel.org, devel@lists.orangefs.org, io-uring@vger.kernel.org, linux-mm@kvack.org Content-Type: text/plain; charset="UTF-8" X-Rspam-User: X-Rspamd-Server: rspam04 X-Rspamd-Queue-Id: 5BA211A0023 X-Stat-Signature: ted1dg5kbrzspskkc6pigucn19y316kr X-HE-Tag: 1675102833-324859 X-HE-Meta: U2FsdGVkX1976izME4f7Toup4ia6Y6OQFNs6ALTs8JfN1HJrEEEt732HnwvXXb4PbKgDIn93UTPPnYO00IizAE43THtRHazESN0BVMa/9leK6pNKfepU/KQ0mRZjPZqHou2fgzSXEtTOje+/mFc89dr0GL8xcqbZh1wcnl1hcpOrMzZ3uexpgTTPDj5ND0qf5TwEUi0oiST9zz/ZKU/wePOvZlQ0Ult3yQccOUseLqjm5VUbOB8m/F4TZOQsUHvFIvsxaRC9Am0n6LdwN8IIN8NNQ35WT3pHRltzBj4yP5SUJ9o/RGg716Ge0fbqxZlUx1WEwvOoBOwGHjErNxMzCh+nBa2LZyVrT01BR6SgtMISssKqVDVfXlu4thninSVbDZtwDSKADnBoEnFZ+KLtZhQ9JrK9TXjv+Awde7XXfqN7zQtUhib43usHS8XOxp7LDU5Hr7e1B50bU9h0yrvfXJXbRXdeB000/CeAJ8Mc/5/Dl51FAADnvr9zASXQK+VCIWemQpJIy5UiCtTG5GrcMNyyV8MI7AKDtI+TkjyCjDWZGYklKYhvCLh6qYGHRiEx5qw87mR6vQv7ehsxEw2c1TMcR4y4QIWn03dhdbNDm70+Y/sMnvt75KqZKz89LVHqIZqk4unwY8Ykx8e2qTCkMLlguOk7neyLx5LZs/gFQSDHgR6+zpDVqLCu5ewQqHYxnWpqI5k0Rv10rnR8DfAtYUwYfI2+Q0+OVX8CtqT8ydstVgJPIqCh0q2iXBEseaYQxUIUx8CGVp8nU1nJpkWktkHreBGcyluE/Dvl+o9PUd7t7+iS4G1Jg3neRzAwWFEgCIzNNp3TqQP8/eEvIk7mWSL7lXwn5B2I/KKEV6SvzN0rJyBJ8BPZ1lxwr5ClOAj0FmLyQgyyg884gdJyFs5WczV6YPJI/GdFx5EmdRm9quX01kj2izjn/qrEvK9Wja7WajHyqXmUIEoi6Ozlf1z EHp+NpPg y6RrrU6fktT0pg5s96JZt5QXIJrpxL0dEq9WX4f7tXgpqyq/15B4oHcUoIZUUkxS+HlVdujYPpRfc0YbXuDL82wyxk5TkFJPevocnjEUj6ncDzyJkw1z9HNTAyU7wgGCPCPyg/vmIwdTuw+LFfCvLcmRe+A4wvGk/Rr+yqXBqh/LTvrZ/hltja+P5K0FIBdf33E6xd7qMli1KV27RZ6YJIrsdciYM9c1yp8ysoHcGh6aXylMO8W7lfcQxF1DiFYAJ2YjCEi/CV81NzCwDJR02uPbiAkn698e6iCZZ/AtefEqwoF/G5aqvx1mtwT84cBwQ2PGnDpJ0tVpk8ZGfxcGHtbTuiaa5V8YlDRzZpLKSN4CIVpdniVWbAq61YoXuPWmhD594R2gCQQrtTiRDDv94zR14gO7j4R9UsR2Rz/hB5XK1HZxdrKbnH6Wdug== X-Bogosity: Ham, tests=bogofilter, spamicity=0.000000, version=1.2.4 Sender: owner-linux-mm@kvack.org Precedence: bulk X-Loop: owner-majordomo@kvack.org List-ID: On Mon, Jan 30, 2023 at 10:23 AM Christoph Hellwig wrote: > > Use the bvec_set_page helper to initialize bvecs. > > Signed-off-by: Christoph Hellwig > --- > net/ceph/messenger_v1.c | 7 ++----- > net/ceph/messenger_v2.c | 28 +++++++++++----------------- > 2 files changed, 13 insertions(+), 22 deletions(-) > > diff --git a/net/ceph/messenger_v1.c b/net/ceph/messenger_v1.c > index d1787d7d33ef9a..d664cb1593a777 100644 > --- a/net/ceph/messenger_v1.c > +++ b/net/ceph/messenger_v1.c > @@ -40,15 +40,12 @@ static int ceph_tcp_recvmsg(struct socket *sock, void *buf, size_t len) > static int ceph_tcp_recvpage(struct socket *sock, struct page *page, > int page_offset, size_t length) > { > - struct bio_vec bvec = { > - .bv_page = page, > - .bv_offset = page_offset, > - .bv_len = length > - }; > + struct bio_vec bvec; > struct msghdr msg = { .msg_flags = MSG_DONTWAIT | MSG_NOSIGNAL }; > int r; > > BUG_ON(page_offset + length > PAGE_SIZE); > + bvec_set_page(&bvec, page, length, page_offset); > iov_iter_bvec(&msg.msg_iter, ITER_DEST, &bvec, 1, length); > r = sock_recvmsg(sock, &msg, msg.msg_flags); > if (r == -EAGAIN) > diff --git a/net/ceph/messenger_v2.c b/net/ceph/messenger_v2.c > index 3009028c4fa28f..301a991dc6a68e 100644 > --- a/net/ceph/messenger_v2.c > +++ b/net/ceph/messenger_v2.c > @@ -149,10 +149,10 @@ static int do_try_sendpage(struct socket *sock, struct iov_iter *it) > > while (iov_iter_count(it)) { > /* iov_iter_iovec() for ITER_BVEC */ > - bv.bv_page = it->bvec->bv_page; > - bv.bv_offset = it->bvec->bv_offset + it->iov_offset; > - bv.bv_len = min(iov_iter_count(it), > - it->bvec->bv_len - it->iov_offset); > + bvec_set_page(&bv, it->bvec->bv_page, > + min(iov_iter_count(it), > + it->bvec->bv_len - it->iov_offset), > + it->bvec->bv_offset + it->iov_offset); > > /* > * sendpage cannot properly handle pages with > @@ -286,9 +286,8 @@ static void set_out_bvec_zero(struct ceph_connection *con) > WARN_ON(iov_iter_count(&con->v2.out_iter)); > WARN_ON(!con->v2.out_zero); > > - con->v2.out_bvec.bv_page = ceph_zero_page; > - con->v2.out_bvec.bv_offset = 0; > - con->v2.out_bvec.bv_len = min(con->v2.out_zero, (int)PAGE_SIZE); > + bvec_set_page(&con->v2.out_bvec, ceph_zero_page, > + min(con->v2.out_zero, (int)PAGE_SIZE), 0); > con->v2.out_iter_sendpage = true; > iov_iter_bvec(&con->v2.out_iter, ITER_SOURCE, &con->v2.out_bvec, 1, > con->v2.out_bvec.bv_len); > @@ -863,10 +862,7 @@ static void get_bvec_at(struct ceph_msg_data_cursor *cursor, > > /* get a piece of data, cursor isn't advanced */ > page = ceph_msg_data_next(cursor, &off, &len); > - > - bv->bv_page = page; > - bv->bv_offset = off; > - bv->bv_len = len; > + bvec_set_page(bv, page, len, off); > } > > static int calc_sg_cnt(void *buf, int buf_len) > @@ -1855,9 +1851,8 @@ static void prepare_read_enc_page(struct ceph_connection *con) > con->v2.in_enc_resid); > WARN_ON(!con->v2.in_enc_resid); > > - bv.bv_page = con->v2.in_enc_pages[con->v2.in_enc_i]; > - bv.bv_offset = 0; > - bv.bv_len = min(con->v2.in_enc_resid, (int)PAGE_SIZE); > + bvec_set_page(&bv, con->v2.in_enc_pages[con->v2.in_enc_i], > + min(con->v2.in_enc_resid, (int)PAGE_SIZE), 0); > > set_in_bvec(con, &bv); > con->v2.in_enc_i++; > @@ -2998,9 +2993,8 @@ static void queue_enc_page(struct ceph_connection *con) > con->v2.out_enc_resid); > WARN_ON(!con->v2.out_enc_resid); > > - bv.bv_page = con->v2.out_enc_pages[con->v2.out_enc_i]; > - bv.bv_offset = 0; > - bv.bv_len = min(con->v2.out_enc_resid, (int)PAGE_SIZE); > + bvec_set_page(&bv, con->v2.out_enc_pages[con->v2.out_enc_i], > + min(con->v2.out_enc_resid, (int)PAGE_SIZE), 0); > > set_out_bvec(con, &bv, false); > con->v2.out_enc_i++; > -- > 2.39.0 > Hi Christoph, Nit on the patch title: this subsystem should be referred to as libceph instead of net-ceph or similar, see "git log -- net/ceph" or MAINTAINERS. Reviewed-by: Ilya Dryomov Thanks, Ilya