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 X-Spam-Level: X-Spam-Status: No, score=-5.2 required=3.0 tests=BAYES_00, HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS, USER_AGENT_SANE_1 autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id C3034C433E2 for ; Fri, 11 Sep 2020 06:48:50 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 954E2221ED for ; Fri, 11 Sep 2020 06:48:50 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1725747AbgIKGsu (ORCPT ); Fri, 11 Sep 2020 02:48:50 -0400 Received: from verein.lst.de ([213.95.11.211]:35691 "EHLO verein.lst.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1725468AbgIKGsr (ORCPT ); Fri, 11 Sep 2020 02:48:47 -0400 Received: by verein.lst.de (Postfix, from userid 2407) id 2702967373; Fri, 11 Sep 2020 08:48:45 +0200 (CEST) Date: Fri, 11 Sep 2020 08:48:44 +0200 From: Christoph Hellwig To: Tom Yan Cc: Christoph Hellwig , linux-scsi@vger.kernel.org, dgilbert@interlog.com, Bart Van Assche , Alan Stern , akinobu.mita@gmail.com, linux-api@vger.kernel.org Subject: Re: [PATCH RESEND 2/4] scsi: sg: implement BLKSSZGET Message-ID: <20200911064844.GA22190@lst.de> References: <20200906012716.1553-1-tom.ty89@gmail.com> <20200906012716.1553-2-tom.ty89@gmail.com> <20200907060927.GA18909@lst.de> <20200908084258.GA17030@lst.de> <20200910052835.GB18283@lst.de> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.17 (2007-11-01) Sender: linux-api-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-api@vger.kernel.org On Fri, Sep 11, 2020 at 10:52:19AM +0800, Tom Yan wrote: > > How is that an advantage? Applications that works with block devices > > don't really work with a magic passthrough character device. > > You must assume that there are applications already assuming that > work. (And it will, at least in some cases, if this series get > merged.) Why "must" I assume that? > And you have not been giving me a solid point anyway, as I said, it's > just queue_*() at the end of the day; regardless of whether those > would work in all sg cases, we have been using them in the sg driver > anyway. > > And it's not like we have to guarantee that (the) ioctls can work in > every case anyway, right? (Especially when they aren't named SG_*). No. While it is unfortunte we have all kinds of cases of ioctls working differnetly on different devices. > > I mean, what's even your point? How do you propose we fix this? I propose to not "fix" anything, because nothing is broken except for maybe a lack of documentation.