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 vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id E79ACC7618A for ; Mon, 20 Mar 2023 17:21:38 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S232501AbjCTRVh (ORCPT ); Mon, 20 Mar 2023 13:21:37 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:42570 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S232628AbjCTRVF (ORCPT ); Mon, 20 Mar 2023 13:21:05 -0400 Received: from mail-io1-xd2a.google.com (mail-io1-xd2a.google.com [IPv6:2607:f8b0:4864:20::d2a]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 941963430E for ; Mon, 20 Mar 2023 10:17:01 -0700 (PDT) Received: by mail-io1-xd2a.google.com with SMTP id h83so5730965iof.8 for ; Mon, 20 Mar 2023 10:17:00 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=kernel-dk.20210112.gappssmtp.com; s=20210112; t=1679332603; x=1681924603; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :from:to:cc:subject:date:message-id:reply-to; bh=AFbCUolBatTymQNgJp0wJPKSHgjza09h9Rdg1vRQ8dQ=; b=gPY90wHag5Nnr5j+QO3RXOqEIGdIIcl4SDuDDHILEF8mp8u4Z2NyhcOiz1SSoXkA1F vo6JHV50H1231KyGblXMhlAN6feDEBhRFpZSf7vgjMdM+jqsVguDpJytvpZsvWzkBX5h QaJOIY88/H2RErsl9TSBuI/4mrNGW08zhPsMIbLus+LU9t1a4zWnOCT/PFQPNZyXNQW1 VixrQgczflJ0c3+ImdK+axHWuR4yZBuexQMT9unb5+eLBrCaHIdqHW2U16aS9BvCXJKf dnrYUU81UStdNN+FMMgSEwotmRXmS6Ge2zg6D5JYlwsXPBGJ6nB8wPFDEgngWmuHeXga JMBQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; t=1679332603; x=1681924603; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=AFbCUolBatTymQNgJp0wJPKSHgjza09h9Rdg1vRQ8dQ=; b=08rZGNeGCI6EilPKkPnR0mfm3hqTAIelYdHHIRwd9RE9gRCW8TGAMoCPwTjmySQwDt osyNbsBtyjZrTLS0MWG8Kw+XFOyn6Duuc75IhlfPRuiMSlVoq0ZvlQAeGvyYWBTEpM40 +uGO9EZNFS3IHdjrOQb+Ce6FEA2O7fVytQ6O5i/dkdv1SmmPK5l0wvbihKQU5VfuEkCZ CXtesoGGUdir6SEDj2ThewcqilefZ4j4f2iCJ9M3hx83VhNYcUgkXoYvB2qzUbHNvs9t iz49OaqLHsZI3gGXpvGHPMcbnrffIaxFLYmuS6CGmle23uD5xxcXLIlQMhgkrfPGDf+K 2MSQ== X-Gm-Message-State: AO0yUKVl1QkmncUKq+pl5G5IpL+1Y8z0ONVJb0mD0bcgaJPZf8TTmbRc gGAiYGUMslaCmUuIyHavTMZ+xA== X-Google-Smtp-Source: AK7set/S5YoRkSon5TRgMJ8/iDveOb13i/31OKu3EHFKuusNZSdAgSOCjWAjwROE2tV0eiHnRtlJvw== X-Received: by 2002:a05:6602:2d82:b0:752:dcbc:9f12 with SMTP id k2-20020a0566022d8200b00752dcbc9f12mr248164iow.2.1679332602929; Mon, 20 Mar 2023 10:16:42 -0700 (PDT) Received: from [192.168.1.94] ([96.43.243.2]) by smtp.gmail.com with ESMTPSA id q15-20020a6bf20f000000b00704608527d1sm3030576ioh.37.2023.03.20.10.16.42 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Mon, 20 Mar 2023 10:16:42 -0700 (PDT) Message-ID: <5aecde5b-c709-c8b3-28cd-5a361bd492b9@kernel.dk> Date: Mon, 20 Mar 2023 11:16:40 -0600 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux aarch64; rv:102.0) Gecko/20100101 Thunderbird/102.8.0 Subject: Re: [PATCH] blk-mq: remove hybrid polling To: Keith Busch , linux-block@vger.kernel.org, io-uring@vger.kernel.org Cc: Pavel Begunkov , Keith Busch References: <20230320161205.1714865-1-kbusch@meta.com> Content-Language: en-US From: Jens Axboe In-Reply-To: <20230320161205.1714865-1-kbusch@meta.com> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Precedence: bulk List-ID: X-Mailing-List: linux-block@vger.kernel.org > diff --git a/block/blk-sysfs.c b/block/blk-sysfs.c > index f1fce1c7fa44b..c6c231f3d0f10 100644 > --- a/block/blk-sysfs.c > +++ b/block/blk-sysfs.c > @@ -408,36 +408,7 @@ queue_rq_affinity_store(struct request_queue *q, const char *page, size_t count) > > static ssize_t queue_poll_delay_show(struct request_queue *q, char *page) > { > - int val; > - > - if (q->poll_nsec == BLK_MQ_POLL_CLASSIC) > - val = BLK_MQ_POLL_CLASSIC; > - else > - val = q->poll_nsec / 1000; > - > - return sprintf(page, "%d\n", val); > -} > - > -static ssize_t queue_poll_delay_store(struct request_queue *q, const char *page, > - size_t count) > -{ > - int err, val; > - > - if (!q->mq_ops || !q->mq_ops->poll) > - return -EINVAL; > - > - err = kstrtoint(page, 10, &val); > - if (err < 0) > - return err; > - > - if (val == BLK_MQ_POLL_CLASSIC) > - q->poll_nsec = BLK_MQ_POLL_CLASSIC; > - else if (val >= 0) > - q->poll_nsec = val * 1000; > - else > - return -EINVAL; > - > - return count; > + return sprintf(page, "%d\n", -1); > } Do we want to retain the _store setting here to avoid breaking anything? Yes, it won't do anything, but it's not like hybrid or classic poll had differences that would be user visible (outside of perhaps using different amounts of CPU). Apart from that, not any major comments. Killing all of this (now) unused code is great! Thanks for doing it. -- Jens Axboe