From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from verein.lst.de (verein.lst.de [213.95.11.211]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 1F29F17FAC5; Fri, 14 Jun 2024 16:03:30 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=213.95.11.211 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1718381013; cv=none; b=mWS3PNHb7/EVqtscY/0wVWHCnBU6SODLZBKgJN1Bb5o+b0wBMTNYE25DQSd6Ujau7vQbbxooPJE+NugFevdKo5XRPcFeuPluVkXxId/VWWnBXiNhR5acnQcyqDil9vtF4m3JTzzalrqC0hcKd/Ca2UtSdFjssCadwS2B7LvPTMc= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1718381013; c=relaxed/simple; bh=y9KHOvoliQVhwac/rD6nLgSGHo9c8nOF6OxAfdCQZyc=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=Jc/YskjZOqzEFenvrlrFEwIIsHQv4l+PNgI02Mf0TKB4w2VDlNyFw/K2BR963wbJ//7YOqsIhxmkW/gNnPtiS7mCMBoSgFE99t32DHrZ5Vy2u3PTpyXqBga+gnsaYL23tzV0ys/xPdFe5PQD9arwhADT864TrEjNwFUOU9RdoTg= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=lst.de; spf=pass smtp.mailfrom=lst.de; arc=none smtp.client-ip=213.95.11.211 Authentication-Results: smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=lst.de Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=lst.de Received: by verein.lst.de (Postfix, from userid 2407) id 7D45068CFE; Fri, 14 Jun 2024 18:03:22 +0200 (CEST) Date: Fri, 14 Jun 2024 18:03:22 +0200 From: Christoph Hellwig To: Jens Axboe Cc: Christoph Hellwig , "Martin K. Petersen" , Mike Snitzer , Mikulas Patocka , Song Liu , Yu Kuai , Dan Williams , Vishal Verma , Dave Jiang , Ira Weiny , Keith Busch , Sagi Grimberg , Chaitanya Kulkarni , linux-block@vger.kernel.org, dm-devel@lists.linux.dev, linux-raid@vger.kernel.org, nvdimm@lists.linux.dev, linux-nvme@lists.infradead.org, linux-scsi@vger.kernel.org Subject: Re: move integrity settings to queue_limits v3 Message-ID: <20240614160322.GA16649@lst.de> References: <20240613084839.1044015-1-hch@lst.de> Precedence: bulk X-Mailing-List: linux-block@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: User-Agent: Mutt/1.5.17 (2007-11-01) On Fri, Jun 14, 2024 at 06:33:41AM -0600, Jens Axboe wrote: > > The series is based on top of my previously sent "convert the SCSI ULDs > > to the atomic queue limits API v2" API. > > I was going to queue this up, but: > > drivers/scsi/sd.c: In function ‘sd_revalidate_disk’: > drivers/scsi/sd.c:3658:45: error: ‘lim’ undeclared (first use in this function) > 3658 | sd_config_protection(sdkp, &lim); > | ^~~ > drivers/scsi/sd.c:3658:45: note: each undeclared identifier is reported only once for each function it appears in That sounds like you didn't apply the above mentioned "convert the SCSI ULDs to the atomic queue limits API v2" series before?