Received: by 2002:a05:6a10:1287:0:0:0:0 with SMTP id d7csp2049760pxv; Sat, 17 Jul 2021 02:37:25 -0700 (PDT) X-Google-Smtp-Source: ABdhPJyE+7i3OPv4GmFDE977AW3+KwK7WY/KuhDXvmLWsOdjKDdOkAQvy6/x+Dy0kXtFk38IqoIW X-Received: by 2002:a17:906:1704:: with SMTP id c4mr16863295eje.182.1626514645154; Sat, 17 Jul 2021 02:37:25 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1626514645; cv=none; d=google.com; s=arc-20160816; b=n580V0SriNJowYQ6QiwYXnvjblK9je37/hj+iTN9dhOD+SOQQtwmNrCL8lCggMBxui mypSI5aOqPBEu0zfzCEP1KiMjsZhD4HID6svpKViwnPcQistox/eGdPf5LfiOagWLJI8 pBAzfvycB1XaDVa+qoo4J73AjwHjYsJvs8joNsqpuKZ8+j0b4bs/23Vkgec/r1qLdKqM +YuNf6wz9c6EWQKDmSbK8W25z9h0Fu6a81A78vqsPVCIBQPVEY88pE0Yax6qPRA7awbI D698OhqPlM3fAwHh48FajMKhKsH8TwQ9ws/U7tiEmmeWZAIFEVoNHIldufL1QR8ZbZDT 3Bgw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:in-reply-to:content-transfer-encoding :content-disposition:mime-version:references:message-id:subject:cc :to:from:date:dkim-signature; bh=vMfm6Ul5FP8FBIaSxrSqUf9N4MLQ8vm57kxlps+k2qU=; b=cdP+dE7GrGb58CFGve2rgDC5t/0QJEJzyAHN9G12yg36RUGFl/o2rpk4PLEZw7H7CO b4WGjFqNZBhUg6kzEY5nZOREb6dtPy5np05Ml6UopbgSIpBqb3vt+QCLyk3v4inoBaJy wvC+9EEGQm3cyVGStamDsuwUsIFvTeZCJ5+22xTWEyYGnoJJR+VayWr50NlueDf+kK0m 8vMidyPGS7+sTNVjV64rRa2QZ29/iGSjQbl6LcYrQgwakOWfwrb6o5gV7aMucdjGvMpc X6I2K4Q1aXpotYS1GK9dsgN47tCtRTL8SmKEQc34sCtjBLThH9Gye1beCU7sJo4jneEv 7X+Q== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@redhat.com header.s=mimecast20190719 header.b=U1De0HZg; 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; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=redhat.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id cs17si14233082ejc.65.2021.07.17.02.37.01; Sat, 17 Jul 2021 02:37:25 -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=@redhat.com header.s=mimecast20190719 header.b=U1De0HZg; 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; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=redhat.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S232615AbhGQJit (ORCPT + 99 others); Sat, 17 Jul 2021 05:38:49 -0400 Received: from us-smtp-delivery-124.mimecast.com ([216.205.24.124]:60705 "EHLO us-smtp-delivery-124.mimecast.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S231862AbhGQJit (ORCPT ); Sat, 17 Jul 2021 05:38:49 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1626514552; h=from:from: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:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=vMfm6Ul5FP8FBIaSxrSqUf9N4MLQ8vm57kxlps+k2qU=; b=U1De0HZg3C0tzFp3neO9BzMfNyvJKIQAbC5+PDXA4+yo4yA8isDXD0wkT42095+CA63zDT 5IrulCNpvoaAyE+TqwPTKDcgJy0VsimokREuWHPeDbT0LC1FsT1AUrDE2XeTTEIQghBHe4 LoOXnblXpbKVaU6xjecLulxJe2zftRc= Received: from mimecast-mx01.redhat.com (mimecast-mx01.redhat.com [209.132.183.4]) (Using TLS) by relay.mimecast.com with ESMTP id us-mta-3-br8uF12bMaujpXW8N-2Tmg-1; Sat, 17 Jul 2021 05:35:48 -0400 X-MC-Unique: br8uF12bMaujpXW8N-2Tmg-1 Received: from smtp.corp.redhat.com (int-mx03.intmail.prod.int.phx2.redhat.com [10.5.11.13]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mimecast-mx01.redhat.com (Postfix) with ESMTPS id 5762A1084F53; Sat, 17 Jul 2021 09:35:46 +0000 (UTC) Received: from T590 (ovpn-12-83.pek2.redhat.com [10.72.12.83]) by smtp.corp.redhat.com (Postfix) with ESMTPS id C433260862; Sat, 17 Jul 2021 09:35:36 +0000 (UTC) Date: Sat, 17 Jul 2021 17:35:32 +0800 From: Ming Lei To: Oleksandr Natalenko Cc: linux-kernel@vger.kernel.org, Jens Axboe , Christoph Hellwig , Sagi Grimberg , linux-nvme@lists.infradead.org, David Jeffery , Laurence Oberman , Paolo Valente , Jan Kara , Sasha Levin , Greg Kroah-Hartman , Keith Busch Subject: Re: New warning in nvme_setup_discard Message-ID: References: <4729812.CpyZKHjjVO@natalenko.name> <2455133.St5lIfLNcX@natalenko.name> <1687461.6WkTYu5mUM@natalenko.name> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <1687461.6WkTYu5mUM@natalenko.name> X-Scanned-By: MIMEDefang 2.79 on 10.5.11.13 Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, Jul 16, 2021 at 02:56:22PM +0200, Oleksandr Natalenko wrote: > On pátek 16. července 2021 12:41:52 CEST Ming Lei wrote: > > > Do I understand correctly that this will be something like: > > > > > > Fixes: 2705dfb209 ("block: fix discard request merge") > > > > > > ? > > > > > > Because as the bisection progresses, I've bumped into this commit only. > > > Without it the issue is not reproducible, at least so far. > > > > It could be. > > > > So can you just test v5.14-rc1? > > Doing it right now, but I've got another issue. Why BFQ is not listed here: > > ``` > /sys/class/block/nvme0n1/queue/scheduler:[mq-deadline] kyber none > /sys/class/block/nvme1n1/queue/scheduler:[mq-deadline] kyber none > ``` Maybe you need to check if the build is OK, I can't reproduce it in my VM, and BFQ is still builtin: [root@ktest-01 ~]# uname -a Linux ktest-01 5.14.0-rc1+ #52 SMP Fri Jul 16 18:56:36 CST 2021 x86_64 x86_64 x86_64 GNU/Linux [root@ktest-01 ~]# cat /sys/block/nvme0n1/queue/scheduler [none] mq-deadline kyber bfq > > ? > > It is a built-in, FWIW: > > ``` > $ modinfo bfq > name: bfq > filename: (builtin) > description: MQ Budget Fair Queueing I/O Scheduler > license: GPL > file: block/bfq > author: Paolo Valente > alias: bfq-iosched > ``` > > So far the issue is not reproducible with your patch + 5.13.2 as well as 5.14- > rc1 (but I don't have BFQ either with v5.14-rc1). You have to verify it with BFQ applied, :-) Thanks, Ming