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 Received: from lists.ozlabs.org (lists.ozlabs.org [112.213.38.117]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id B7C4BF364CD for ; Fri, 10 Apr 2026 10:50:22 +0000 (UTC) Received: from boromir.ozlabs.org (localhost [127.0.0.1]) by lists.ozlabs.org (Postfix) with ESMTP id 4fsYVP32smz2yZ6; Fri, 10 Apr 2026 20:50:21 +1000 (AEST) Authentication-Results: lists.ozlabs.org; arc=none smtp.remote-ip=198.175.65.14 ARC-Seal: i=1; a=rsa-sha256; d=lists.ozlabs.org; s=201707; t=1775818221; cv=none; b=fmB8hA/b4osn4HTTIr7W4LfmkFfMLS+pLCxcoM4eR6/fw91ZpTMM+73AMwoLWQD0hkPbb0tj9Fx1GIx8Vv+b6Gm7Iy2orGHzgBN0bSpeRNxnJdE3WBLCSbdM1Ua7uzl7ZNaZlWSlJxLCPF4NwibaeosUo7v4/IMWa2Q2ipWOAt1BbtEH441wJ9KUb1qZY565sTreVZN9uP+3jDKVz624U1SXLy6yMPkUaPVfaIU4s4wgiiMg1jTgtw4OAi3/m8WnwD8AompJ1ireMkxkZs9y96vBccp/8vgpjuYpZu16YlqG+Wxqbfk4/64r4ncgnbBj853Y7uyJwq5fVkzL/8gPsQ== ARC-Message-Signature: i=1; a=rsa-sha256; d=lists.ozlabs.org; s=201707; t=1775818221; c=relaxed/relaxed; bh=fVXtGxEZhlalRdWn+jmflJwiiZlyyFnI18hmNwAuUks=; h=From:Date:To:cc:Subject:In-Reply-To:Message-ID:References: MIME-Version:Content-Type; b=SgJEnxRvvqNmGD6k0rpPiwbnBQotChl92DWUKzLAo9GaO4y3hX4qFxQAnZwN1CRFf4Uo+5oFOFiH+XagzZD+nLqIEcKvkpIXstCU5BJY7Orl8de7F4kWQTIh7SwcCihC94YITvYZ8l7LEUpDbpnbSTzxPkgLTh/syYiHLHj7J1WFkjRNOjqOj0Xdyju0fPaUcuQz4oLCfPdUM8pKmud4JCYpQFUBWNk9bYC2xgvyYi38/Iu89S/kDlyRCM8CyYP3H4/puvmRfZB9x7L2cxIt8wlsQrJRKVvTanbrDOO12g+mr0BMbsaG3vJ3L2VLBoz89V4ZjsY5efuOEr694YcAyw== ARC-Authentication-Results: i=1; lists.ozlabs.org; dmarc=pass (p=none dis=none) header.from=linux.intel.com; dkim=pass (2048-bit key; unprotected) header.d=intel.com header.i=@intel.com header.a=rsa-sha256 header.s=Intel header.b=btD35F4y; dkim-atps=neutral; spf=pass (client-ip=198.175.65.14; helo=mgamail.intel.com; envelope-from=ilpo.jarvinen@linux.intel.com; receiver=lists.ozlabs.org) smtp.mailfrom=linux.intel.com Authentication-Results: lists.ozlabs.org; dmarc=pass (p=none dis=none) header.from=linux.intel.com Authentication-Results: lists.ozlabs.org; dkim=pass (2048-bit key; unprotected) header.d=intel.com header.i=@intel.com header.a=rsa-sha256 header.s=Intel header.b=btD35F4y; dkim-atps=neutral Authentication-Results: lists.ozlabs.org; spf=pass (sender SPF authorized) smtp.mailfrom=linux.intel.com (client-ip=198.175.65.14; helo=mgamail.intel.com; envelope-from=ilpo.jarvinen@linux.intel.com; receiver=lists.ozlabs.org) Received: from mgamail.intel.com (mgamail.intel.com [198.175.65.14]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by lists.ozlabs.org (Postfix) with ESMTPS id 4fsYVJ3mgBz2xT6 for ; Fri, 10 Apr 2026 20:50:13 +1000 (AEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1775818217; x=1807354217; h=from:date:to:cc:subject:in-reply-to:message-id: references:mime-version:content-id; bh=htUsvW9HcAW+4Tu7OtqXPagEiVPdCGWQYuS/yVUGtOU=; b=btD35F4yDx5CDzuEkadRELIvyWoH2qR4v22kDrUcKWHj6R8zFxTpfHnA H7VWPZJUNmhbA4TDYTLfd4R4EErX/IRwgbOnsRqio9vmyVO+TTx7N9e2t JF0Xb+iB8FDlmBSbT4NlfAyb4ToCgQ8lcYv+X42p9wYW1n+KDNuhEyFpd FAZJSYidvQaHKJntrlkYU9YPfd5YpD3pPXvJ1/bVyjmDsPFID8xHewNxu /H2qcCRZ9XOmKCIDh/tx0aaEmEx5XiohcLwOOxvhF/cXg92HvkQuRaYg4 Zz6MmxBVnPeIRQFf+WNg0aPy6F8XaPTUqtBRuaozddqdCOg5kRpG3aPFD w==; X-CSE-ConnectionGUID: W4Lgb10qRUy6AwqxZTx36w== X-CSE-MsgGUID: lg0PEXffQh6apP3eDAf+Ug== X-IronPort-AV: E=McAfee;i="6800,10657,11754"; a="80694945" X-IronPort-AV: E=Sophos;i="6.23,171,1770624000"; d="scan'208";a="80694945" Received: from orviesa001.jf.intel.com ([10.64.159.141]) by orvoesa106.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 10 Apr 2026 03:50:10 -0700 X-CSE-ConnectionGUID: qz+PkvWOTiCtwSASfDIxpg== X-CSE-MsgGUID: QFr1YiJQRoO439xKOPHsHA== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="6.23,171,1770624000"; d="scan'208";a="267020190" Received: from ijarvine-mobl1.ger.corp.intel.com (HELO localhost) ([10.245.244.118]) by smtpauth.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 10 Apr 2026 03:49:58 -0700 From: =?UTF-8?q?Ilpo=20J=C3=A4rvinen?= Date: Fri, 10 Apr 2026 13:49:54 +0300 (EEST) To: =?ISO-8859-2?Q?Krzysztof_Wilczy=F1ski?= cc: Bjorn Helgaas , Bjorn Helgaas , Manivannan Sadhasivam , Lorenzo Pieralisi , Magnus Lindholm , Matt Turner , Richard Henderson , Christophe Leroy , Madhavan Srinivasan , Michael Ellerman , Nicholas Piggin , Dexuan Cui , =?ISO-8859-2?Q?Krzysztof_Ha=B3asa?= , Lukas Wunner , Oliver O'Halloran , Saurabh Singh Sengar , Shuan He , Srivatsa Bhat , linux-pci@vger.kernel.org, linux-alpha@vger.kernel.org, linuxppc-dev@lists.ozlabs.org Subject: Re: [PATCH 06/20] PCI/sysfs: Convert PCI resource files to static attributes In-Reply-To: <20260410055040.39233-7-kwilczynski@kernel.org> Message-ID: <4fc23ce0-7103-545b-bc11-230b52c2de94@linux.intel.com> References: <20260410055040.39233-1-kwilczynski@kernel.org> <20260410055040.39233-7-kwilczynski@kernel.org> X-Mailing-List: linuxppc-dev@lists.ozlabs.org List-Id: List-Help: List-Owner: List-Post: List-Archive: , List-Subscribe: , , List-Unsubscribe: Precedence: list MIME-Version: 1.0 Content-Type: multipart/mixed; BOUNDARY="8323328-2006288344-1775817599=:1195" Content-ID: This message is in MIME format. The first part should be readable text, while the remaining parts are likely unreadable without MIME-aware tools. --8323328-2006288344-1775817599=:1195 Content-Type: text/plain; CHARSET=ISO-8859-2 Content-Transfer-Encoding: QUOTED-PRINTABLE Content-ID: <6f453f32-23d3-1885-ab29-9bf4b9993389@linux.intel.com> On Fri, 10 Apr 2026, Krzysztof Wilczy=F1ski wrote: > Currently, the PCI resource files (resourceN, resourceN_wc) are > dynamically created by pci_create_sysfs_dev_files(), called from > both pci_bus_add_device() and the pci_sysfs_init() late_initcall, > with only a sysfs_initialized flag for synchronisation. This has > caused "duplicate filename" warnings and boot panics when both > paths race on the same device. >=20 > This is especially likely on Devicetree-based platforms, where the > PCI host controllers are platform drivers that probe via the driver > model, which can happen during or after the late_initcall. As such, > pci_bus_add_device() and pci_sysfs_init() are more likely to overlap. >=20 > Thus, convert to static const attributes with three attribute groups > (I/O, UC, WC), each with an .is_bin_visible callback that checks > resource flags, BAR length, and non_mappable_bars. A .bin_size > callback provides pci_resource_len() to the kernfs node for correct > stat and lseek behaviour. >=20 > As part of this conversion: >=20 > - Rename pci_read_resource_io() and pci_write_resource_io() to > pci_read_resource() and pci_write_resource() since the callbacks > are no longer I/O-specific in the static attribute context. >=20 > - Remove pci_create_resource_files(), pci_remove_resource_files(), > and pci_create_attr() which are no longer needed. >=20 > - Move the __weak stubs outside the #if guard so they remain > available for callers converted in subsequent commits. >=20 > Platforms that do not define the HAVE_PCI_MMAP macro or the > ARCH_GENERIC_PCI_MMAP_RESOURCE macro, such as Alpha architecture, > continue using their platform-specific resource file creation. >=20 > For reference, the dynamic creation dates back to the pre-Git era: >=20 > https://git.kernel.org/pub/scm/linux/kernel/git/tglx/history.git/commit= /drivers/pci/pci-sysfs.c?id=3D42298be0eeb5ae98453b3374c36161b05a46c5dc >=20 > The write-combine support was added in commit 45aec1ae72fc ("x86: PAT > export resource_wc in pci sysfs"). >=20 > Signed-off-by: Krzysztof Wilczy=F1ski > --- > drivers/pci/pci-sysfs.c | 242 +++++++++++++++++++++------------------- > include/linux/pci.h | 2 - > 2 files changed, 127 insertions(+), 117 deletions(-) >=20 > diff --git a/drivers/pci/pci-sysfs.c b/drivers/pci/pci-sysfs.c > index d29d79be8ee5..e56fddbe7914 100644 > --- a/drivers/pci/pci-sysfs.c > +++ b/drivers/pci/pci-sysfs.c > @@ -1200,14 +1200,14 @@ static ssize_t pci_resource_io(struct file *filp,= struct kobject *kobj, > #endif > } > =20 > -static ssize_t pci_read_resource_io(struct file *filp, struct kobject *k= obj, > +static ssize_t pci_read_resource(struct file *filp, struct kobject *kobj= , > =09=09=09=09 const struct bin_attribute *attr, char *buf, > =09=09=09=09 loff_t off, size_t count) > { > =09return pci_resource_io(filp, kobj, attr, buf, off, count, false); > } > =20 > -static ssize_t pci_write_resource_io(struct file *filp, struct kobject *= kobj, > +static ssize_t pci_write_resource(struct file *filp, struct kobject *kob= j, > =09=09=09=09 const struct bin_attribute *attr, char *buf, > =09=09=09=09 loff_t off, size_t count) > { > @@ -1261,129 +1261,136 @@ static const struct bin_attribute dev_resource#= #_bar##_wc_attr =3D {=09=09\ > =09.mmap =3D pci_mmap_resource_wc,=09=09=09=09=09=09\ > } > =20 > -/** > - * pci_remove_resource_files - cleanup resource files > - * @pdev: dev to cleanup > - * > - * If we created resource files for @pdev, remove them from sysfs and > - * free their resources. > - */ > -static void pci_remove_resource_files(struct pci_dev *pdev) > +static inline umode_t > +__pci_resource_attr_is_visible(struct kobject *kobj, > +=09=09=09 const struct bin_attribute *a, > +=09=09=09 int bar, bool write_combine, > +=09=09=09 unsigned long flags) > { > -=09int i; > +=09struct pci_dev *pdev =3D to_pci_dev(kobj_to_dev(kobj)); > =20 > -=09for (i =3D 0; i < PCI_STD_NUM_BARS; i++) { > -=09=09struct bin_attribute *res_attr; > - > -=09=09res_attr =3D pdev->res_attr[i]; > -=09=09if (res_attr) { > -=09=09=09sysfs_remove_bin_file(&pdev->dev.kobj, res_attr); > -=09=09=09kfree(res_attr); > -=09=09} > - > -=09=09res_attr =3D pdev->res_attr_wc[i]; > -=09=09if (res_attr) { > -=09=09=09sysfs_remove_bin_file(&pdev->dev.kobj, res_attr); > -=09=09=09kfree(res_attr); > -=09=09} > -=09} > -} > - > -static int pci_create_attr(struct pci_dev *pdev, int num, int write_comb= ine) > -{ > -=09/* allocate attribute structure, piggyback attribute name */ > -=09int name_len =3D write_combine ? 13 : 10; > -=09struct bin_attribute *res_attr; > -=09char *res_attr_name; > -=09int retval; > - > -=09res_attr =3D kzalloc(sizeof(*res_attr) + name_len, GFP_ATOMIC); > -=09if (!res_attr) > -=09=09return -ENOMEM; > - > -=09res_attr_name =3D (char *)(res_attr + 1); > - > -=09sysfs_bin_attr_init(res_attr); > -=09if (write_combine) { > -=09=09sprintf(res_attr_name, "resource%d_wc", num); > -=09=09res_attr->mmap =3D pci_mmap_resource_wc; > -=09} else { > -=09=09sprintf(res_attr_name, "resource%d", num); > -=09=09if (pci_resource_flags(pdev, num) & IORESOURCE_IO) { > -=09=09=09res_attr->read =3D pci_read_resource_io; > -=09=09=09res_attr->write =3D pci_write_resource_io; > -=09=09=09if (arch_can_pci_mmap_io()) > -=09=09=09=09res_attr->mmap =3D pci_mmap_resource_uc; > -=09=09} else { > -=09=09=09res_attr->mmap =3D pci_mmap_resource_uc; > -=09=09} > -=09} > -=09if (res_attr->mmap) { > -=09=09res_attr->f_mapping =3D iomem_get_mapping; > -=09=09/* > -=09=09 * generic_file_llseek() consults f_mapping->host to determine > -=09=09 * the file size. As iomem_inode knows nothing about the > -=09=09 * attribute, it's not going to work, so override it as well. > -=09=09 */ > -=09=09res_attr->llseek =3D pci_llseek_resource; > -=09} > -=09res_attr->attr.name =3D res_attr_name; > -=09res_attr->attr.mode =3D 0600; > -=09res_attr->size =3D pci_resource_len(pdev, num); > -=09res_attr->private =3D (void *)(unsigned long)num; > -=09retval =3D sysfs_create_bin_file(&pdev->dev.kobj, res_attr); > -=09if (retval) { > -=09=09kfree(res_attr); > -=09=09return retval; > -=09} > - > -=09if (write_combine) > -=09=09pdev->res_attr_wc[num] =3D res_attr; > -=09else > -=09=09pdev->res_attr[num] =3D res_attr; > - > -=09return 0; > -} > - > -/** > - * pci_create_resource_files - create resource files in sysfs for @dev > - * @pdev: dev in question > - * > - * Walk the resources in @pdev creating files for each resource availabl= e. > - */ > -static int pci_create_resource_files(struct pci_dev *pdev) > -{ > -=09int i; > -=09int retval; > - > -=09/* Skip devices with non-mappable BARs */ > =09if (pdev->non_mappable_bars) > =09=09return 0; > =20 > -=09/* Expose the PCI resources from this device as files */ > -=09for (i =3D 0; i < PCI_STD_NUM_BARS; i++) { > +=09if (!pci_resource_len(pdev, bar)) > +=09=09return 0; I know it's same as in the previous code but I dislike assuming len !=3D 0= =20 implies resource has been assigned. While it currently holds, I'd want to= =20 change that eventually. The current behavior causes issue e.g. if IOV resource fails to assign, it= =20 is reset (making its len 0 among other thing) and since IOV resource are=20 optional that is fine from kernel's perspective. But resetting the=20 resource means we also lose access to that resource because its type gets= =20 cleared so from kernel perspective the VF BAR stops to exist. Losing it=20 means the user cannot solve the issue by e.g. resizing some other BAR=20 smaller to make space to allow the VF BARs to assign successfully. So I think this code would actually want to check resource_assigned()=20 which implies also non-zero size. AFAICT, this change looks fine (despite the diff being very messy). > -=09=09/* skip empty resources */ > -=09=09if (!pci_resource_len(pdev, i)) > -=09=09=09continue; > +=09if ((pci_resource_flags(pdev, bar) & flags) !=3D flags) > +=09=09return 0; > =20 > -=09=09retval =3D pci_create_attr(pdev, i, 0); > -=09=09/* for prefetchable resources, create a WC mappable file */ > -=09=09if (!retval && arch_can_pci_mmap_wc() && > -=09=09 pci_resource_flags(pdev, i) & IORESOURCE_PREFETCH) > -=09=09=09retval =3D pci_create_attr(pdev, i, 1); > -=09=09if (retval) { > -=09=09=09pci_remove_resource_files(pdev); > -=09=09=09return retval; > -=09=09} > -=09} > -=09return 0; > +=09if (write_combine && !arch_can_pci_mmap_wc()) > +=09=09return 0; > + > +=09return a->attr.mode; > } > -#else /* !(defined(HAVE_PCI_MMAP) || defined(ARCH_GENERIC_PCI_MMAP_RESOU= RCE)) */ > -int __weak pci_create_resource_files(struct pci_dev *dev) { return 0; } > -void __weak pci_remove_resource_files(struct pci_dev *dev) { return; } > + > +static umode_t pci_dev_resource_io_is_visible(struct kobject *kobj, > +=09=09=09=09=09 const struct bin_attribute *a, > +=09=09=09=09=09 int n) > +{ > +=09return __pci_resource_attr_is_visible(kobj, a, n, false, > +=09=09=09=09=09 IORESOURCE_IO); > +} > + > +static umode_t pci_dev_resource_uc_is_visible(struct kobject *kobj, > +=09=09=09=09=09 const struct bin_attribute *a, > +=09=09=09=09=09 int n) > +{ > +=09return __pci_resource_attr_is_visible(kobj, a, n, false, > +=09=09=09=09=09 IORESOURCE_MEM); > +} > + > +static umode_t pci_dev_resource_wc_is_visible(struct kobject *kobj, > +=09=09=09=09=09 const struct bin_attribute *a, > +=09=09=09=09=09 int n) > +{ > +=09return __pci_resource_attr_is_visible(kobj, a, n, true, > +=09=09=09=09=09 IORESOURCE_MEM | IORESOURCE_PREFETCH); > +} > + > +static size_t pci_dev_resource_bin_size(struct kobject *kobj, > +=09=09=09=09=09const struct bin_attribute *a, > +=09=09=09=09=09int n) > +{ > +=09struct pci_dev *pdev =3D to_pci_dev(kobj_to_dev(kobj)); > + > +=09return pci_resource_len(pdev, n); > +} > + > +pci_dev_resource_io_attr(0); > +pci_dev_resource_io_attr(1); > +pci_dev_resource_io_attr(2); > +pci_dev_resource_io_attr(3); > +pci_dev_resource_io_attr(4); > +pci_dev_resource_io_attr(5); > + > +pci_dev_resource_uc_attr(0); > +pci_dev_resource_uc_attr(1); > +pci_dev_resource_uc_attr(2); > +pci_dev_resource_uc_attr(3); > +pci_dev_resource_uc_attr(4); > +pci_dev_resource_uc_attr(5); > + > +pci_dev_resource_wc_attr(0); > +pci_dev_resource_wc_attr(1); > +pci_dev_resource_wc_attr(2); > +pci_dev_resource_wc_attr(3); > +pci_dev_resource_wc_attr(4); > +pci_dev_resource_wc_attr(5); > + > +static const struct bin_attribute *const pci_dev_resource_io_attrs[] =3D= { > +=09&dev_resource0_io_attr, > +=09&dev_resource1_io_attr, > +=09&dev_resource2_io_attr, > +=09&dev_resource3_io_attr, > +=09&dev_resource4_io_attr, > +=09&dev_resource5_io_attr, > +=09NULL, > +}; > + > +static const struct bin_attribute *const pci_dev_resource_uc_attrs[] =3D= { > +=09&dev_resource0_uc_attr, > +=09&dev_resource1_uc_attr, > +=09&dev_resource2_uc_attr, > +=09&dev_resource3_uc_attr, > +=09&dev_resource4_uc_attr, > +=09&dev_resource5_uc_attr, > +=09NULL, > +}; > + > +static const struct bin_attribute *const pci_dev_resource_wc_attrs[] =3D= { > +=09&dev_resource0_wc_attr, > +=09&dev_resource1_wc_attr, > +=09&dev_resource2_wc_attr, > +=09&dev_resource3_wc_attr, > +=09&dev_resource4_wc_attr, > +=09&dev_resource5_wc_attr, > +=09NULL, > +}; > + > +static const struct attribute_group pci_dev_resource_io_attr_group =3D { > +=09.bin_attrs =3D pci_dev_resource_io_attrs, > +=09.is_bin_visible =3D pci_dev_resource_io_is_visible, > +=09.bin_size =3D pci_dev_resource_bin_size, > +}; > + > +static const struct attribute_group pci_dev_resource_uc_attr_group =3D { > +=09.bin_attrs =3D pci_dev_resource_uc_attrs, > +=09.is_bin_visible =3D pci_dev_resource_uc_is_visible, > +=09.bin_size =3D pci_dev_resource_bin_size, > +}; > + > +static const struct attribute_group pci_dev_resource_wc_attr_group =3D { > +=09.bin_attrs =3D pci_dev_resource_wc_attrs, > +=09.is_bin_visible =3D pci_dev_resource_wc_is_visible, > +=09.bin_size =3D pci_dev_resource_bin_size, > +}; > + > #endif > =20 > +int __weak pci_create_resource_files(struct pci_dev *dev) { return 0; } > +void __weak pci_remove_resource_files(struct pci_dev *dev) { } > + > /** > * pci_write_rom - used to enable access to the PCI ROM display > * @filp: sysfs file > @@ -1861,6 +1868,11 @@ static const struct attribute_group pci_dev_group = =3D { > =20 > const struct attribute_group *pci_dev_groups[] =3D { > =09&pci_dev_group, > +#if defined(HAVE_PCI_MMAP) || defined(ARCH_GENERIC_PCI_MMAP_RESOURCE) > +=09&pci_dev_resource_io_attr_group, > +=09&pci_dev_resource_uc_attr_group, > +=09&pci_dev_resource_wc_attr_group, > +#endif > =09&pci_dev_config_attr_group, > =09&pci_dev_rom_attr_group, > =09&pci_dev_reset_attr_group, > diff --git a/include/linux/pci.h b/include/linux/pci.h > index 1c270f1d5123..a7a104427b07 100644 > --- a/include/linux/pci.h > +++ b/include/linux/pci.h > @@ -2501,10 +2501,8 @@ int pcibios_alloc_irq(struct pci_dev *dev); > void pcibios_free_irq(struct pci_dev *dev); > resource_size_t pcibios_default_alignment(void); > =20 > -#if !defined(HAVE_PCI_MMAP) && !defined(ARCH_GENERIC_PCI_MMAP_RESOURCE) > extern int pci_create_resource_files(struct pci_dev *dev); > extern void pci_remove_resource_files(struct pci_dev *dev); > -#endif > =20 > #if defined(CONFIG_PCI_MMCONFIG) || defined(CONFIG_ACPI_MCFG) > void __init pci_mmcfg_early_init(void); >=20 --=20 i. --8323328-2006288344-1775817599=:1195--