Received: by 2002:a05:6a10:d5a5:0:0:0:0 with SMTP id gn37csp670436pxb; Thu, 30 Sep 2021 14:36:41 -0700 (PDT) X-Google-Smtp-Source: ABdhPJxk5SPwxFbsCVl0HreK0YZWkvPc7ICyVANFGpwDAigc1Ps+Uou6VQHxTVOxWuQJ65ZWcL6H X-Received: by 2002:a17:906:a044:: with SMTP id bg4mr1792533ejb.312.1633037800936; Thu, 30 Sep 2021 14:36:40 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1633037800; cv=none; d=google.com; s=arc-20160816; b=QGfRECcvugXE4700tyRulhmSxB+/LgLci1K52tzDBGoFY6H81lt0yL3u7xzyT6UUT7 xC7u/fo89IPFjewCj6rU/TgEufVXDUuM0LUEQ47Wur1Ifu+T1dzG6GWb5iEQv3TY8woi d6Pd/D13mZiv2F4nYPoV19KNVfVgkHw2lkrjXTIgWNsYIpcjEWKe6lUGFwIilLPXz6ER K+i7idQDlV+wsmftfJcyEkQTy/TaGYT1Cl7OtvIq20lRND8BV5hCQNmt6rvpa4IG+4wf E1SjjiGlCqKpmEQ50skWxaM1/wjVWVAvWDGBoTQToZut7eKlJAQxr+xCCFR4uB7AzWfh EKuw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:cc:to:subject:message-id:date:from:in-reply-to :references:mime-version:dkim-signature; bh=YNu/YUTTEta1euczrqSgmEotnhGowg7gkC/LCBiwK4s=; b=JOmVzOSV4dvR9UjR1g24mhxDojLUUA03LOYj3Ub1a0zHrypjaXk2yxeg9/R0KrrHDT 2u12zD6ir4bGvslv8Ev24PjFnmO38mGWq5YAC1Im2Ck2PfC0wwFh3OrDHJKKWsJko5on uPQSZR+wAHVUy4s3AKQtmKTagzTwxZEqizCe2KP6Y2GdLKefd/f20lfS7QdAnRglY++F GsZzeS9uCB1+N69T/O8EveH48rM4PVAP6ODI+xELyxDS8vdlsktk/1bWGRGxQUutrbBh 4YGEXErdxR2TNT8E8TzVp5+U0CPrONE9zdnmRxc0u87IYkVbHZU8i9c5S/qtmt59BHkl wx+A== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@intel-com.20210112.gappssmtp.com header.s=20210112 header.b=xj9wCZap; 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=fail (p=NONE sp=NONE dis=NONE) header.from=intel.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id x8si7927891edd.386.2021.09.30.14.36.14; Thu, 30 Sep 2021 14:36:40 -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=@intel-com.20210112.gappssmtp.com header.s=20210112 header.b=xj9wCZap; 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=fail (p=NONE sp=NONE dis=NONE) header.from=intel.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229941AbhI3UZw (ORCPT + 99 others); Thu, 30 Sep 2021 16:25:52 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:34174 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1347637AbhI3UZv (ORCPT ); Thu, 30 Sep 2021 16:25:51 -0400 Received: from mail-pl1-x634.google.com (mail-pl1-x634.google.com [IPv6:2607:f8b0:4864:20::634]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 7BC15C06176A for ; Thu, 30 Sep 2021 13:24:08 -0700 (PDT) Received: by mail-pl1-x634.google.com with SMTP id t11so4837892plq.11 for ; Thu, 30 Sep 2021 13:24:08 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=intel-com.20210112.gappssmtp.com; s=20210112; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=YNu/YUTTEta1euczrqSgmEotnhGowg7gkC/LCBiwK4s=; b=xj9wCZapqmLIJztql+O6rs7uhLCij1deyqu/+aL8hw5KpdACRtEzLzlrczZPQazJf2 FupFdb9n864IVhhEWE2BFU2kLYdnzUmtFaxbjHm6lb//5fDkvE5CAaUcSQvsRG36P8PP eun2CY6L6A5F0Wlzc/fObt5myxz90grs1McqqY8m7v98HFHvToc+FDZTHTkWeW44HECM xCD+ovGiDADokJx7WJDuutLzRkMC4VORWMEfE6EY2d8Olt83Rs2NuhEFcPoVePw5SUH+ SZYrcirr0KsiRYZZIdz5IaTMK6SzVZ+teg347CmH3QnWX3NUvpnIOGpCQjLVm0yeaTnN DPGw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=YNu/YUTTEta1euczrqSgmEotnhGowg7gkC/LCBiwK4s=; b=2Zr5C7gtS1qyAqMmIOAGUThqO6/3zKs9tPcjxhi9JYRUPG52RexIROxkktYsuymrdg gNfJb8Ph2nak4lECNJMAbdsuDHHwQ/TLZMoMKBXU//0vwwqZuGoQElKo+70rgUltUVRL vbsoWYEXbExt8+aVrMgZQlKkjSO5IA5QczklIW7N5wJ+v4uJz6jqvzswBdIF3MBf1ZZQ Rm6OoZHvXC/XtU2+Hof1xKb9R/s7v9WK4RcT3zw9GNjDQeV4n5/rhr8Q0FCmmuiHQWbB Tqvnhp9zXa8psk71bIfG/nREWaB1um2JMwidESNl3n5wW4SWHVkoJYi4gEvxhH9KBASc V0nQ== X-Gm-Message-State: AOAM531Q8vGfU0ZCUU+tE5l67/Yjk3YrFqRG5VqwbnjAaH5H6HhDxQEj Q2BjzCCvSjENvC5e6X5hpPiGKDti8KyDyI1Mr5nILw== X-Received: by 2002:a17:90b:3ec3:: with SMTP id rm3mr7610144pjb.93.1633033447983; Thu, 30 Sep 2021 13:24:07 -0700 (PDT) MIME-Version: 1.0 References: <20210930010511.3387967-1-sathyanarayanan.kuppuswamy@linux.intel.com> <20210930010511.3387967-2-sathyanarayanan.kuppuswamy@linux.intel.com> In-Reply-To: From: Dan Williams Date: Thu, 30 Sep 2021 13:23:57 -0700 Message-ID: Subject: Re: [PATCH v2 1/6] driver core: Move the "authorized" attribute from USB/Thunderbolt to core To: "Kuppuswamy, Sathyanarayanan" Cc: Yehezkel Bernat , Greg Kroah-Hartman , Borislav Petkov , X86 ML , Bjorn Helgaas , Thomas Gleixner , Ingo Molnar , Andreas Noever , "Michael S . Tsirkin" , Michael Jamet , "Rafael J . Wysocki" , Mika Westerberg , Jonathan Corbet , Jason Wang , Andi Kleen , Kuppuswamy Sathyanarayanan , LKML , Linux PCI , USB list , virtualization@lists.linux-foundation.org Content-Type: text/plain; charset="UTF-8" Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thu, Sep 30, 2021 at 12:50 PM Kuppuswamy, Sathyanarayanan wrote: > > > > On 9/30/21 12:04 PM, Dan Williams wrote: > >>> That's why it was highlighted in the changelog. Hopefully a > >>> Thunderbolt developer can confirm if it is a non-issue. > >>> Documentation/ABI/testing/sysfs-bus-thunderbolt does not seem to > >>> answer this question about whether authorized_show and > >>> authorized_store need to be symmetric. > >> Apparently, Bolt does read it [1] and cares about it [2]. > > Ah, thank you! > > > > Yeah, looks like the conversion to bool was indeed too hopeful. > > > > IIUC, the end result of value "2" in authorized sysfs is to just > "authorize" or "de-authorize". In that case, can the user space > driver adapt to this int->bool change? Just want to know the > possibility. ABIs are forever. The kernel has to uphold its contract to bolt that it will return '2' and not '1' after '2' has been written.