Received: by 2002:a05:6a10:a852:0:0:0:0 with SMTP id d18csp2932218pxy; Mon, 3 May 2021 11:09:47 -0700 (PDT) X-Google-Smtp-Source: ABdhPJzTnSBX02fxIC9bXI0hqsiSx+1rEQWuSaxO+eq5OVKzVEzr6/axHQ/SRSlOagoenlkbgRdl X-Received: by 2002:a63:e706:: with SMTP id b6mr19566048pgi.302.1620065387715; Mon, 03 May 2021 11:09:47 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1620065387; cv=none; d=google.com; s=arc-20160816; b=KjPBvbaHfIlG0Psd1u+JQUmfGhNiq9ThoXWbn3YAMgcgpQPULaz48WTtoxSmqQK7lF 6Ik2QZP6L1aHpiGuM8Vmf1ToU55KMzdIKaPoEG9VzZA3CtLvBs5fnzw1N2dmOE9OhRla nQ65GGuxTamUtve/NLFN8E+czOTcaPa1f2M0lHtR4f4cz41sCSzfNIYHaAaqiR5aKb0I YGbPuQWV/4M68irFpySKwybrNx3bebdb1AfKPp0MTPJhyZwJx7t2LCW0xtLNN5IvAliY YmjAo7MGr96sWg44AxTex0UzdyBW/Wnwin6cxbKD1N0sIlzUt03x5q3NNBA4ZRbxNB3T jj4g== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:content-language :in-reply-to:mime-version:user-agent:date:message-id:from:references :to:subject:dkim-signature; bh=6egtxTN5q4QLPOJLRd3y+Z0fKzalIDR3UohhkwtOCM8=; b=qgxeoLBEntrQzzg24rRxLgZFBh1HRmPhlsDeuh8Oj4oWzH4RE06Tu2Q8h25srWMXsP Ujzth4kqEl/Y0bCdqVE2K7SPVv3Bh4KamGgjzAfN9lawHK61MkmZvJk6V0jhsTg3Z+bB geMJBKRZQhPv+A3wjHaH7cL9ITgjzVoTB/N8MTDZ/JZawDHf7efgQCPcUIpH3Fl8TJ39 UiIGUCkFkGlYEM+4Z5EwDOsZ2eHtSzk/HNvb2FWC98flTmVKDu3KjonDNobDLU39JEXd GwTa9Puu/LG5F7W6wFFo/9QK9bRviepIiwtzcp1+Khfl0WgqejDA3yUE+Zsb88J7Lyky cvPw== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@kernel-dk.20150623.gappssmtp.com header.s=20150623 header.b=rMzOccYn; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id g12si14773636pfi.75.2021.05.03.11.09.35; Mon, 03 May 2021 11:09:47 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=pass header.i=@kernel-dk.20150623.gappssmtp.com header.s=20150623 header.b=rMzOccYn; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229899AbhECPAh (ORCPT + 99 others); Mon, 3 May 2021 11:00:37 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:56238 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229687AbhECPAg (ORCPT ); Mon, 3 May 2021 11:00:36 -0400 Received: from mail-io1-xd34.google.com (mail-io1-xd34.google.com [IPv6:2607:f8b0:4864:20::d34]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 72FACC061761 for ; Mon, 3 May 2021 07:59:42 -0700 (PDT) Received: by mail-io1-xd34.google.com with SMTP id a11so4336211ioo.0 for ; Mon, 03 May 2021 07:59:42 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=kernel-dk.20150623.gappssmtp.com; s=20150623; h=subject:to:references:from:message-id:date:user-agent:mime-version :in-reply-to:content-language:content-transfer-encoding; bh=6egtxTN5q4QLPOJLRd3y+Z0fKzalIDR3UohhkwtOCM8=; b=rMzOccYnjQjJTQplZd4m5b4GYay4fmSvBEXQ0cyrH9TWKEEaxSW7ofFPCXRs6iV6ZV gn73VPb/MX5vg1btYyq2Gj38MRIouiIJ+j4GMlahaOcMMPBIAXqGfHAvFgbZPraXTR1Y +WcV4bAqr2poTCc3OMyAQpdso+VA272r2vwbSwj368IFnfgMBk7pUWjQ4YYZfWRO/7Xs H1rDmVrdkjETcygVtzVSBfcngngQLovheZLeGdIPvPJWR0sR+z2l52KuNSmNoMVppLNR AvjKs1NTEeLXCGjD0BIUOLk7xVt68zMFFQACW9XAzHC4MNaclyET0PVR8awjabSMexKU vVSg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:subject:to:references:from:message-id:date :user-agent:mime-version:in-reply-to:content-language :content-transfer-encoding; bh=6egtxTN5q4QLPOJLRd3y+Z0fKzalIDR3UohhkwtOCM8=; b=mF4Tn3Hxfev7EUm8Vt0fJLYzUzMjL6xAFxyxJyiy/a7jv9Gz8ntMK65wGmogPUDkjB eBZ/zQ4CkmNKOOCFBeK/h9ZfTTHVh/oVBWSuwqHeKctiMaQB6Plb+bD3T0Bg6Ww3exR0 JFSgetO2z9WFj7b0vNGZT+ksCfHpZtn+ECWKcJFPWQ3VW/Z9m+yKGJQiBEtVTSDwNAvQ 2nt5cpac2RG6kix7lm/nf+tErE6goz7Ux6pk1Jv4kxo98gQxDEYs2aM5sPL/tW9C5Wz0 NoI2xJ6SOkgMhRhNtxtS3v0kF36Yqjy5NGMak0aOnPFdDYthEvdqWxCUQ1W5hKg3tmpq dmAQ== X-Gm-Message-State: AOAM533o2kPNrKFRSRwuy7/8CpQ+x780yoqvCS6bd5GCo+0zXJ0iKhLj VOay+9uNtnvElArpkPhxTuuxzw== X-Received: by 2002:a05:6602:81e:: with SMTP id z30mr15264730iow.90.1620053981968; Mon, 03 May 2021 07:59:41 -0700 (PDT) Received: from [192.168.1.30] ([65.144.74.34]) by smtp.gmail.com with ESMTPSA id q26sm273508ioh.42.2021.05.03.07.59.41 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Mon, 03 May 2021 07:59:41 -0700 (PDT) Subject: Re: [PATCH v9] bio: limit bio max size To: Bart Van Assche , Changheun Lee , yi.zhang@redhat.com, ming.lei@redhat.com, bgoncalv@redhat.com, hch@lst.de, jaegeuk@kernel.org, linux-block@vger.kernel.org, linux-kernel@vger.kernel.org, patchwork-bot@kernel.org References: <20210503095203.29076-1-nanich.lee@samsung.com> <72a0b4e7-a374-97fb-ae73-ed24f719da98@acm.org> From: Jens Axboe Message-ID: Date: Mon, 3 May 2021 08:59:41 -0600 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.10.0 MIME-Version: 1.0 In-Reply-To: <72a0b4e7-a374-97fb-ae73-ed24f719da98@acm.org> Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 7bit Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 5/3/21 8:54 AM, Bart Van Assche wrote: > On 5/3/21 2:52 AM, Changheun Lee wrote: >> struct queue_limits { >> + unsigned int bio_max_bytes; >> + >> enum blk_bounce bounce; >> unsigned long seg_boundary_mask; >> unsigned long virt_boundary_mask; > > On April 23 Damien Le Moal posted the following comment about the > bio_max_bytes member: "Please move this below in the structure together > with all other fields that are unsigned int too." Making that change > sounds like a good idea to me. Why? It's next to an enum, so the patch (as it stands) actually fills a hole in the struct. Moving it would be a mistake and waste more space. If anything, we should move the bounce member below virt_boundary_mask, but since this one fills a hole, that's better left as a separate patch. -- Jens Axboe