Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932067AbbDOIxA (ORCPT ); Wed, 15 Apr 2015 04:53:00 -0400 Received: from mondschein.lichtvoll.de ([194.150.191.11]:37526 "EHLO mail.lichtvoll.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754971AbbDOIwk convert rfc822-to-8bit (ORCPT ); Wed, 15 Apr 2015 04:52:40 -0400 From: Martin Steigerwald To: Greg Kroah-Hartman Cc: Andy Lutomirski , Al Viro , "Eric W. Biederman" , Linus Torvalds , Andrew Morton , Arnd Bergmann , One Thousand Gnomes , Tom Gundersen , Jiri Kosina , "linux-kernel@vger.kernel.org" , Daniel Mack , David Herrmann , Djalal Harouni Subject: Re: [GIT PULL] kdbus for 4.1-rc1 Date: Wed, 15 Apr 2015 10:52:37 +0200 Message-ID: <2159452.NLBQbQOEH5@merkaba> User-Agent: KMail/4.14.7 (Linux/4.0.0-tp520-btrfs-trim+; KDE/4.14.2; x86_64; git-b05922a; 2015-04-12) In-Reply-To: <20150415083219.GB16381@kroah.com> References: <20150413190350.GA9485@kroah.com> <1831440.HjEAktTL7T@merkaba> <20150415083219.GB16381@kroah.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8BIT Content-Type: text/plain; charset="utf-8" Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 5086 Lines: 121 Am Mittwoch, 15. April 2015, 10:32:19 schrieb Greg Kroah-Hartman: > On Wed, Apr 15, 2015 at 10:18:46AM +0200, Martin Steigerwald wrote: > > Am Dienstag, 14. April 2015, 18:36:28 schrieb Andy Lutomirski: > > > On Mon, Apr 13, 2015 at 1:22 PM, Al Viro > > > > wrote: > > > > On Mon, Apr 13, 2015 at 09:42:17PM +0200, Greg Kroah-Hartman wrote: > > > >> > I remain opposed to this half thought out trash of an ABI for > > > >> > the > > > >> > meta-data. > > > >> > > > >> You don't have to enable the metadata if you don't want to use > > > >> it, > > > >> it's > > > >> an option :) > > > > > > > > OK, _that_ argument needs to be stomped out. It had been used > > > > before, > > > > and it was a deliberate scam. There is no such thing as optional > > > > kernel interface, especially when udev/dbus/systemd crowd is > > > > nearby. > > > > We'd been through that excuse before; remember how devtmpfs was > > > > pushed in as "optional"? > > > > > > > > This is a huge red flag. On the level of "I need your account > > > > information to transfer $200M you might have inherited from my > > > > deceased client". > > > > > > > > Just to recap how it went the last time around: Kay kept pushing > > > > his > > > > piece of code into the tree, claiming that it was optional, that > > > > nobody who doesn't like it has to enable it, so what's the > > > > problem? > > > > OK, in it went. And pretty soon udev (maintained by the same... > > > > meticulously honorable person) had stopped working on the kernels > > > > that didn't have that enabled. > > > > > > > > We had been there before. To paraphrase another... meticulously > > > > honorable person, "if you didn't want something relied upon, why > > > > have > > > > you put it into the kernel?" Said person is on the record as > > > > having > > > > no problem whatsoever with adding dependencies to the bottom of > > > > userland stack. > > > > > > It appears that, if kdbus is merged, upstream udev may end up > > > requiring > > > it: > > > > > > http://lists.freedesktop.org/archives/systemd-devel/2014-May/019657. > > > html > > > > > > Grumble. > > > > Honestly, I think that tightly coupling systemd and udev to certain > > kernel versions in lock step is crap. > > Where do you see that happening? > > > That you require some minimum version after some reasonable time, > > sure. > > But in lockstep? Seriously. > > Has that happened in the past? Look at the minimum requirements of > systemd/udev today, something like the 3.7 kernel release, many years > old. I refer to the linked mailing list post from Lennart as I quote here: > To make this clear, we expect that systemd and kernels are updated in > lockstep. We explicitly do not support really old kernels with really > (which means 3.4 right now), but even that should be taken with a grain > of salt, as we already made clear that soon after kdbus is merged into > the kernel we'll probably make a hard requirement on it from the systemd > side. Thats plenty clear, isn´t it? As soond as kdbus is merged into kernel, systemd will depend on it, and then… if I need to go back to older kernel, I have to downgrade systemd as well? > > I certainly do not want a broken system just cause I have to load an > > older kernel version for some reason. > > No one does. But, work with your distribution if you end up with > something like this. Remember, the goal is that you can always run > newer kernels on older userspace, as that is something that we kernel > developers can enforce. Userspace programs have other requirements / > communities, it's up to them to decide what their oldest kernel version > they wish to support. Hint, even glibc makes these kinds of > requirements, it's nothing new at all here, so why is this even an > issue? Its no issue for me that systemd required kernel 3.7. But… what Lennart announces above regarding kdbus reads quite differently. > > And yes, I think its good not to force just about any userspace idea > > into the kernel. > > Do you have any technical objections to the patch as proposed? If I had, I would have written it. I explained already that I see that kernel developers have strong technical objections with kdbus. And that I think it is important to acknowledge it, instead of telling them, that the API is required from userspace, userspace people know what they do, and they should just go away with their concerns. Thats at least how I received quite some of your responses. Well and I raised an eyebrow on the busname matching rules and the capability stuff. Yet, I didn´t comment on it, cause I didn´t look at it in-depth. I just ask you to take those seriously who did. -- Martin 'Helios' Steigerwald - http://www.Lichtvoll.de GPG: 03B0 0D6C 0040 0710 4AFA B82F 991B EAAC A599 84C7 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/