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.3 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 72EFCC432BE for ; Thu, 26 Aug 2021 16:34:26 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 4BB7760FD9 for ; Thu, 26 Aug 2021 16:34:26 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S233368AbhHZQfM (ORCPT ); Thu, 26 Aug 2021 12:35:12 -0400 Received: from verein.lst.de ([213.95.11.211]:59817 "EHLO verein.lst.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229787AbhHZQfM (ORCPT ); Thu, 26 Aug 2021 12:35:12 -0400 Received: by verein.lst.de (Postfix, from userid 2407) id 4E9226736F; Thu, 26 Aug 2021 18:34:22 +0200 (CEST) Date: Thu, 26 Aug 2021 18:34:22 +0200 From: Christoph Hellwig To: Milan Broz Cc: Christoph Hellwig , Jens Axboe , Hillf Danton , Tetsuo Handa , Pavel Tatashin , "Reviewed-by : Tyler Hicks" , linux-block@vger.kernel.org Subject: Re: [PATCH 5/8] loop: merge the cryptoloop module into the main loop module Message-ID: <20210826163422.GA28718@lst.de> References: <20210826133810.3700-1-hch@lst.de> <20210826133810.3700-6-hch@lst.de> <977860f6-efc4-a55e-50e3-c5204fc762c5@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <977860f6-efc4-a55e-50e3-c5204fc762c5@gmail.com> User-Agent: Mutt/1.5.17 (2007-11-01) Precedence: bulk List-ID: X-Mailing-List: linux-block@vger.kernel.org On Thu, Aug 26, 2021 at 06:31:50PM +0200, Milan Broz wrote: > the cryptoloop is insecure, most of the encryption modes are deprecated > (and known to be problematic); util-linux no longer support cryptoloop > options in losetup. > > Isn't the better way to go just to remove cryptoloop completely? I'm not sure if we're going to break existing users. So maybe for now get rid of the horrible registration interface, an add a patch to deprecate it with a warning when people actually use it, and then eventually remove it.