Received: by 2002:a05:7412:8521:b0:e2:908c:2ebd with SMTP id t33csp545341rdf; Fri, 3 Nov 2023 08:05:06 -0700 (PDT) X-Google-Smtp-Source: AGHT+IH+INbf7SxzghhlWB/6WUoLpaufXfihhZ1r1o1j2f0ojzEmhnBEHyhuLkID59KHPjo34WE2 X-Received: by 2002:a17:902:ea10:b0:1cc:56b3:efc5 with SMTP id s16-20020a170902ea1000b001cc56b3efc5mr14604544plg.21.1699023906594; Fri, 03 Nov 2023 08:05:06 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1699023906; cv=none; d=google.com; s=arc-20160816; b=o++6SzR34jNMiaVQPVIFk/dJ2SLlxrlRpEUB+RvIkA0ruPkNTmtXVX8IlmfwnrST+5 jJ3AzwCbxLy62pWLqsAlH8H2D9AVNL+810ekWYQ3407Hl4XI28+O1mmoOh0fWsvbaUfQ 6FLjPPE13fp4j+qFPl6zNRLYAYhYQ17RHGcmRx9lEvlRrtJf34FVFQiU2vrh3LSPNut9 CiiaYuXYiQNbL/GxfL7/saPDYRYYtd6DPlCcMmZ0meuo/VfATg9700dKFlDRq99c9QuP aq7WcwaLcUDO+a6l3e4HFiCPOg+/INQo0OEopPqgPEQCSdoi63zTV0aq9tUYtpfbhrg7 9tIw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:in-reply-to:from :references:cc:to:content-language:subject:user-agent:mime-version :date:message-id; bh=VqsEhwQCgORTttRkJNMy0lsBkeGL10cvGzD/GgULHH4=; fh=Ivb8mFzz6syfyEW0rSHeYabcie3uMfW2+2B1xeKMc5k=; b=axVxcynJfATnnb973Xhve7m3PBcMTF85h3gLeD4NwpFNu2m8oPV0y3yU38FaeBACqQ cEYWATgLzqFBoqxj7DtESH7EI1kshGgMgCBGHyua8O2rdOM5wmuru5RjpnLxcV3jG4My kY9M3iTUbshKi1KAt0rc8EkZacHJ8+AQbK5OBv2nzm6as9vsmmqEOA/y59vQpy2R9kfZ GwOFlR7ojrEkdfWnSWfW0j9AlCNesSD4bYBPbLyrBU0ma1g47EnfOLW0uH6HoTe0v7Qp MB3I8K7mfpGdJaGiwe1PmB4V5HCtJ+5f4gTu0nveE3vcsb0k6hSPJ/n9SyQKMykTLsWo QFeQ== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:5 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=acm.org Return-Path: Received: from groat.vger.email (groat.vger.email. [2620:137:e000::3:5]) by mx.google.com with ESMTPS id e6-20020a17090301c600b001c5da8b630fsi1710794plh.421.2023.11.03.08.05.05 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 03 Nov 2023 08:05:06 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:5 as permitted sender) client-ip=2620:137:e000::3:5; Authentication-Results: mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:5 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=acm.org Received: from out1.vger.email (depot.vger.email [IPv6:2620:137:e000::3:0]) by groat.vger.email (Postfix) with ESMTP id CE6A18303B09; Fri, 3 Nov 2023 08:05:02 -0700 (PDT) X-Virus-Status: Clean X-Virus-Scanned: clamav-milter 0.103.10 at groat.vger.email Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S232488AbjKCPEj (ORCPT + 99 others); Fri, 3 Nov 2023 11:04:39 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:56702 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S231825AbjKCPEi (ORCPT ); Fri, 3 Nov 2023 11:04:38 -0400 Received: from mail-pl1-f170.google.com (mail-pl1-f170.google.com [209.85.214.170]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id C64D1D49; Fri, 3 Nov 2023 08:04:35 -0700 (PDT) Received: by mail-pl1-f170.google.com with SMTP id d9443c01a7336-1cc0d0a0355so17860395ad.3; Fri, 03 Nov 2023 08:04:35 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1699023875; x=1699628675; h=content-transfer-encoding:in-reply-to:from:references:cc:to :content-language:subject:user-agent:mime-version:date:message-id :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=VqsEhwQCgORTttRkJNMy0lsBkeGL10cvGzD/GgULHH4=; b=I/F1k0S9htvX0heUSEvR3enmyKicgyGY6TmaZNGbLr4Qvvd619F+OUQ8bA2Q0BCIKw bnASGBIGpoX3hWeYLzM09qgCKeG5UUTIkqdpT/h7A4aUVkirEpKTjI011OgX0KXs/ch1 7Jn/0ql6iKJawFQMfmgpwKaJrnjpctnSXmCY/aRwdtoSiX4Q+vRvxRySPsM9q9h0UPxp 1u7Y7OQeqC0co5B3sReNYhkeEVQZ1YYcmxzjwGqFI54ukyvje+en+yBVDcz+w1YhfQNn 41w+mI6X6ReBKqt44h5Wcin3+jzfjW9DQgwiDvMa55dgdFcYYZqVOgzPQGxnxCuqZkNS d+rQ== X-Gm-Message-State: AOJu0YwZinbbGNvYznMuN2C+SLRthQePxKoGvVSQUfLoh7QllVvmjAmL pYmLMNpeQCPb0SpFHfw/SHLI1XcRxOM= X-Received: by 2002:a17:902:cac1:b0:1ca:4092:7200 with SMTP id y1-20020a170902cac100b001ca40927200mr15951111pld.54.1699023875001; Fri, 03 Nov 2023 08:04:35 -0700 (PDT) Received: from [192.168.51.14] (c-73-231-117-72.hsd1.ca.comcast.net. [73.231.117.72]) by smtp.gmail.com with ESMTPSA id j5-20020a170902c08500b001cc0e3a29a8sm1517612pld.89.2023.11.03.08.04.23 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Fri, 03 Nov 2023 08:04:33 -0700 (PDT) Message-ID: Date: Fri, 3 Nov 2023 08:04:15 -0700 MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH RFC V2 6/6] RDMA/rxe: Support PAGE_SIZE aligned MR Content-Language: en-US To: Li Zhijian , zyjzyj2000@gmail.com, jgg@ziepe.ca, leon@kernel.org, linux-rdma@vger.kernel.org Cc: linux-kernel@vger.kernel.org, rpearsonhpe@gmail.com, matsuda-daisuke@fujitsu.com, yi.zhang@redhat.com References: <20231103095549.490744-1-lizhijian@fujitsu.com> <20231103095549.490744-7-lizhijian@fujitsu.com> From: Bart Van Assche In-Reply-To: <20231103095549.490744-7-lizhijian@fujitsu.com> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit X-Spam-Status: No, score=-0.8 required=5.0 tests=HEADER_FROM_DIFFERENT_DOMAINS, MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,T_SCC_BODY_TEXT_LINE autolearn=unavailable autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on groat.vger.email Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org X-Greylist: Sender passed SPF test, not delayed by milter-greylist-4.6.4 (groat.vger.email [0.0.0.0]); Fri, 03 Nov 2023 08:05:03 -0700 (PDT) On 11/3/23 02:55, Li Zhijian wrote: > - return ib_sg_to_pages(ibmr, sgl, sg_nents, sg_offset, rxe_set_page); > + for_each_sg(sgl, sg, sg_nents, i) { > + u64 dma_addr = sg_dma_address(sg) + sg_offset; > + unsigned int dma_len = sg_dma_len(sg) - sg_offset; > + u64 end_dma_addr = dma_addr + dma_len; > + u64 page_addr = dma_addr & PAGE_MASK; > + > + if (sg_dma_len(sg) == 0) { > + rxe_dbg_mr(mr, "empty SGE\n"); > + return -EINVAL; > + } > + do { > + int ret = rxe_store_page(mr, page_addr); > + if (ret) > + return ret; > + > + page_addr += PAGE_SIZE; > + } while (page_addr < end_dma_addr); > + sg_offset = 0; > + } > + > + return ib_sg_to_pages(ibmr, sgl, sg_nents, sg_offset_p, rxe_set_page); > } Is this change necessary? There is already a loop in ib_sg_to_pages() that splits SG entries that are larger than mr->page_size into entries with size mr->page_size. Bart.