From mboxrd@z Thu Jan 1 00:00:00 1970 From: Zhang Rui Subject: Re: v4.10-rc4 to v4.10-rc5: battery regression on Nokia N900 Date: Fri, 27 Jan 2017 11:39:42 +0800 Message-ID: <1485488382.2469.27.camel@intel.com> References: <20170123144031.GA7870@amd> <20170123232654.GA19342@amd> <20170123234912.GA2460@roeck-us.net> <20170124070639.GA5068@rzhang1-surface> <20170124073720.GB5603@amd> <88c94ea6-abe2-0f20-337e-e9ee00c883d8@roeck-us.net> <20170124175800.GA15070@amd> <20170124184526.GA25056@roeck-us.net> <20170125111233.GB3912@amd> <20170125120918.GA7936@pali> <1485481030.2469.15.camel@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: In-Reply-To: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=m.gmane.org@lists.infradead.org To: Guenter Roeck , Pali =?ISO-8859-1?Q?Roh=E1r?= , Pavel Machek Cc: ivo.g.dimitrov.75@gmail.com, khilman@kernel.org, tony@atomide.com, aaro.koskinen@iki.fi, sre@kernel.org, kernel list , abcloriens@gmail.com, fabio.estevam@nxp.com, linux-omap@vger.kernel.org, patrikbachan@gmail.com, linux-arm-kernel , serge@hallyn.com List-Id: linux-omap@vger.kernel.org T24gVGh1LCAyMDE3LTAxLTI2IGF0IDE4OjAzIC0wODAwLCBHdWVudGVyIFJvZWNrIHdyb3RlOgo+ IE9uIDAxLzI2LzIwMTcgMDU6MzcgUE0sIFpoYW5nIFJ1aSB3cm90ZToKPiA+IAo+ID4gT24gV2Vk LCAyMDE3LTAxLTI1IGF0IDEzOjA5ICswMTAwLCBQYWxpIFJvaMOhciB3cm90ZToKPiA+ID4gCj4g PiA+IE9uIFdlZG5lc2RheSAyNSBKYW51YXJ5IDIwMTcgMTI6MTI6MzMgUGF2ZWwgTWFjaGVrIHdy b3RlOgo+ID4gPiA+IAo+ID4gPiA+IAo+ID4gPiA+IEhpIQo+ID4gPiA+IAo+ID4gPiA+ID4gCj4g PiA+ID4gPiAKPiA+ID4gPiA+ID4gCj4gPiA+ID4gPiA+IAo+ID4gPiA+ID4gPiA+IAo+ID4gPiA+ ID4gPiA+IAo+ID4gPiA+ID4gPiA+ID4gCj4gPiA+ID4gPiA+ID4gPiAKPiA+ID4gPiA+ID4gPiA+ ID4gCj4gPiA+ID4gPiA+ID4gPiA+IAo+ID4gPiA+ID4gPiA+ID4gPiBSaWdodC4KPiA+ID4gPiA+ ID4gPiA+ID4gCj4gPiA+ID4gPiA+ID4gPiA+IEJlZm9yZSByZXZlcnRpbmcsIGNhbiB5b3UgcGxl YXNlIHRyeSBpZiB0aGlzIHBhdGNoCj4gPiA+ID4gPiA+ID4gPiA+IHdvcmtzCj4gPiA+ID4gPiA+ ID4gPiA+IG9yIG5vdD8KPiA+ID4gPiA+ID4gPiA+IE5vdCByZWFsbHkuIFJldmVydCBub3cuIFNv cnJ5Lgo+ID4gPiA+ID4gPiA+ID4gCj4gPiA+ID4gPiA+ID4gPiBBcmUgeW91IHN1cmU/IFRoaXMg ZG9lcyBub3QgbG9vayBlcXVpdmFsZW50IHRvIG1lIGF0Cj4gPiA+ID4gPiA+ID4gPiBhbGwuCj4g PiA+ID4gPiA+ID4gPiAKPiA+ID4gPiA+ID4gPiA+ICJuYW1lIiBmaWxlIGhhbmRsaW5nIG1vdmVk IGZyb20gZHJpdmVycyB0byB0aGUgY29yZSwKPiA+ID4gPiA+ID4gPiA+IHdoaWNoCj4gPiA+ID4g PiA+ID4gPiBhZGRlZCBzb21lCj4gPiA+ID4gPiA+ID4gPiBjcmF6eSBjaGVja3Mgd2hhdCBuYW1l IGNhbiBjb250YWluLiBFdmVuIGlmIHRoaXMKPiA+ID4gPiA+ID4gPiA+ICJ3b3JrcyIsCj4gPiA+ ID4gPiA+ID4gPiB3aGF0IGlzIHRoZQo+ID4gPiA+ID4gPiA+ID4gZXhwZWN0ZWQgZWZmZWN0IG9u IHRoZSAibmFtZSIgZmlsZT8KPiA+ID4gPiA+ID4gPiA+IAo+ID4gPiA+ID4gPiA+IFRoZSBod21v biBuYW1lIGF0dHJpYnV0ZSBtdXN0IG5vdCBpbmNsdWRlICctJywgYXMKPiA+ID4gPiA+ID4gPiBk b2N1bWVudGVkCj4gPiA+ID4gPiA+ID4gaW4KPiA+ID4gPiA+ID4gPiBEb2N1bWVudGF0aW9uL2h3 bW9uL3N5c2ZzLWludGVyZmFjZS4gSXMgZW5mb3JjaW5nIHRoYXQKPiA+ID4gPiA+ID4gPiAnY3Jh enknID8KPiA+ID4gPiA+ID4gPiBNYXliZSBpbiB5b3VyIHdvcmxkLCBidXQgbm90IGluIG1pbmUu Cj4gPiA+ID4gPiA+IFdlbGwsIGxldHMgcmV2ZXJ0IHRoZSBwYXRjaCBhbmQgdGhlbiB3ZSBjYW4g ZGlzY3VzcyB3aGF0IHRvCj4gPiA+ID4gPiA+IGRvCj4gPiA+ID4gPiA+IHdpdGgKPiA+ID4gPiA+ ID4gdGhlICJuYW1lIiBwcm9ibGVtLgo+ID4gPiA+IE9rLCBzbyB0aGUgcGF0Y2ggaXMgb24gdGhl IHdheSBpbi4gV2hhdCB0byBkbyBuZXh0Pwo+ID4gPiA+IAo+ID4gPiA+IHBhdmVsQG45MDA6L3N5 cy9jbGFzcy9od21vbiQgY2F0IGh3bW9uMC9uYW1lCj4gPiA+ID4gYnEyNzIwMC0wCj4gPiA+ID4g cGF2ZWxAbjkwMDovc3lzL2NsYXNzL2h3bW9uJCBjYXQgaHdtb24xL25hbWUKPiA+ID4gPiByeDUx LWJhdHRlcnkKPiA+ID4gPiAKPiA+ID4gPiA+IAo+ID4gPiA+ID4gCj4gPiA+ID4gPiBUbyBwcm92 aWRlIHNvbWUgZGV0YWlsOiBsaWJzZW5zb3JzIGdldHMganVzdCBhcyBjb25mdXNlZCB3aXRoCj4g PiA+ID4gPiB3aWxkY2FyZHMKPiA+ID4gPiA+IGFuZCB3aGl0ZXNwYWNlL25ld2xpbmUgYXMgaXQg ZG9lcyB3aXRoICctJyBpbiB0aGUgcmVwb3J0ZWQKPiA+ID4gPiA+IG5hbWUsCj4gPiA+ID4gPiB3 aGljaAo+ID4gPiA+ID4gaXMgd2h5IHRob3NlIGFyZSBibG9ja2VkIGJ5IHRoZSBuZXcgQVBJLgo+ ID4gPiA+IE9rLi4uIFF1ZXN0aW9uIGlzICJkb2VzIHNvbWVvbmUgYWN0dWFsbHkgdXNlIGh3bW9u Ki9uYW1lIG9uCj4gPiA+ID4gTjkwMCI/Cj4gPiA+ID4gSWYKPiA+ID4gPiBzbywgd2UgY2FuJ3Qg Y2hhbmdlIGl0LCBidXQgaXQgaXMgd2VsbCBwb3NzaWJsZSB0aGF0IG5vb25lIGlzLgo+ID4gPiBJ SVJDIGh3bW9uIGlzIHVzZWQgb24gTm9raWEgTjkwMC4KPiA+ID4gCj4gPiA+IEJ1dCBJIGhhdmUg bm90IHNlZW4gaHdtb24gZGV2aWNlcyBmb3IgYnEyNzIwMCBhbmQgcng1MS1iYXR0ZXJ5Cj4gPiA+ IHlldC4KPiA+ID4gVGhvc2UgYXJlIHBvd2VyIHN1cHBseSBkcml2ZXIgYW5kIGF1dG8tZXhwb3J0 aW5nIHRoZW0gYWxzbyB2aWEKPiA+ID4gaHdtb24KPiA+ID4gaXMKPiA+ID4gc29tZXRoaW5nIG5l dywgcmlnaHQ/IElmIHllcywgdGhlbiB3ZSBjYW4gdXNlIGFueSBuYW1lIGZvciB0aG9zZQo+ID4g PiBuZXcKPiA+ID4gaHdtb24gZGV2aWNlcyBhcyB0aGV5IGNhbm5vdCBicmVhayB1c2Vyc3BhY2Uu Li4gYXMgdGhlcmUgaXMgbm8KPiA+ID4gdXNlcnNwYWNlCj4gPiA+IGFwcGxpY2F0aW9uIGZvciB0 aGVtLgo+ID4gPiAKPiA+IElmIHRoaXMgaXMgdGhlIGNhc2UsIHlvdSdkIGJldHRlciBzZXQKPiA+ IChzdHJ1Y3QgdGhlcm1hbF96b25lX3BhcmFtcyktPm5vX2h3bW9uIHdoZW4gcmVnaXN0ZXJpbmcg dGhlIHRoZXJtYWwKPiA+IHpvbmUgZGV2aWNlLCBpbiB3aGljaCBjYXNlLCB0aGUgaHdtb24gZGV2 aWNlIHdpbGwgbm90IGJlIGNyZWF0ZWQuCj4gPiAKPiA+IEluIGZhY3QsIEknZCBwcmVmZXIgdG8g Y2hhbmdlIHR6cC0+bm9faHdtb24gdG8gdHpwLT5od21vbiB0byBub3QKPiA+IGNyZWF0ZQo+ID4g aHdtb24gSS9GIGJ5IGRlZmF1bHQsIGFuZCBzZWUgaWYgdGhlcmUgaXMgYW55b25lIHVzaW5nIGl0 LiBJZiB5ZXMsCj4gPiB3ZQo+ID4gY2FuIHNldCB0aGUgZmxhZyBpbiBzb2MgdGhlcm1hbCBkcml2 ZXIsIGV4cGxpY2l0bHksIGF0IG1lYW50aW1lLCBhCj4gPiBod21vbiBjb21wYXRpYmxlIG5hbWUg aXMgcmVxdWlyZWQuCj4gPiAKPiA+IEJ1dCBvbmUgZm9yZXNlZWFibGUgcmVzdWx0IGlzIHRoYXQg d2UgbWF5IGdldCBidWcgcmVwb3J0cyBmcm9tIGVuZAo+ID4gdXNlcgo+ID4gdGhhdCBzb21lIHNl bnNvcnMgKGFjcGl0eiwgZXRjKSBhcmUgZ29uZSBpbiAnc2Vuc29ycycgb3V0cHV0LiBBbmQKPiA+ IFRCSCwKPiA+IEknbSBub3QgcXVpdGUgc3VyZSBpZiB0aGlzIGNhbiBiZSBjb3VudGVkIGFzIGEg cmVncmVzc2lvbiBvciBub3QuCj4gPiAKPiBUaGF0IHNvdW5kcyBsaWtlIGZ1bi4gQ2hhbmdpbmcg YnEyNzIwMC0wIHRvIGJxMjcyMDBfMCBpcyBGb3JiaWRkZW4gYnkKPiB0aGUgQUJJIFBvbGljZSwg YnV0IHRha2luZyB0aGUgZW50aXJlIGRldmljZSBhd2F5IGlzIG9rLgo+IApOby4gSU1PLCBpdCBk ZXBlbmRzIG9uIGlmIHRoZSBpbnRlcmZhY2UgaXMgdXNlZCBvciBub3QuCklmIGh3bW9uIEkvRiBp cyB1c2VkLCB3ZSBjYW4gbm90IHRha2UgaXQgYXdheSwgbm9yIGNoYW5nZSBpdHMgbmFtZS4KSWYg dGhlcm1hbCB6b25lIEkvRiBpcyB1c2VkLCB3ZSBjYW4gbm90IGNoYW5nZSBpdCdzICd0eXBlJyBu YW1lIHRvIGJlCmNvbXBhdGlibGUgd2l0aCBuZXcgaHdtb24gQVBJLgoKPiBBbnl3YXksIHNvdW5k cyBnb29kIHRvIG1lLiBObyBvbmUgd2lsbCB1c2Ugc29tZXRoaW5nIHRoYXQgaXNuJ3QKPiB0aGVy ZSwKPiBhbmQgbm8gb25lIHdpbGwgcmVhbGl6ZSB0aGF0IGl0IGNvdWxkIGhhdmUgYmVlbiB0aGVy ZSwgc28gSSBkb24ndAo+IGV4cGVjdAo+IGFueW9uZSB0byBjb21wbGFpbi4KClllcywgSSBhZ3Jl ZS4KCnRoYW5rcywKcnVpCgpfX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fXwpsaW51eC1hcm0ta2VybmVsIG1haWxpbmcgbGlzdApsaW51eC1hcm0ta2VybmVsQGxp c3RzLmluZnJhZGVhZC5vcmcKaHR0cDovL2xpc3RzLmluZnJhZGVhZC5vcmcvbWFpbG1hbi9saXN0 aW5mby9saW51eC1hcm0ta2VybmVsCg== From mboxrd@z Thu Jan 1 00:00:00 1970 From: rui.zhang@intel.com (Zhang Rui) Date: Fri, 27 Jan 2017 11:39:42 +0800 Subject: v4.10-rc4 to v4.10-rc5: battery regression on Nokia N900 In-Reply-To: References: <20170123144031.GA7870@amd> <20170123232654.GA19342@amd> <20170123234912.GA2460@roeck-us.net> <20170124070639.GA5068@rzhang1-surface> <20170124073720.GB5603@amd> <88c94ea6-abe2-0f20-337e-e9ee00c883d8@roeck-us.net> <20170124175800.GA15070@amd> <20170124184526.GA25056@roeck-us.net> <20170125111233.GB3912@amd> <20170125120918.GA7936@pali> <1485481030.2469.15.camel@intel.com> Message-ID: <1485488382.2469.27.camel@intel.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Thu, 2017-01-26 at 18:03 -0800, Guenter Roeck wrote: > On 01/26/2017 05:37 PM, Zhang Rui wrote: > > > > On Wed, 2017-01-25 at 13:09 +0100, Pali Roh?r wrote: > > > > > > On Wednesday 25 January 2017 12:12:33 Pavel Machek wrote: > > > > > > > > > > > > Hi! > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Right. > > > > > > > > > > > > > > > > > > Before reverting, can you please try if this patch > > > > > > > > > works > > > > > > > > > or not? > > > > > > > > Not really. Revert now. Sorry. > > > > > > > > > > > > > > > > Are you sure? This does not look equivalent to me at > > > > > > > > all. > > > > > > > > > > > > > > > > "name" file handling moved from drivers to the core, > > > > > > > > which > > > > > > > > added some > > > > > > > > crazy checks what name can contain. Even if this > > > > > > > > "works", > > > > > > > > what is the > > > > > > > > expected effect on the "name" file? > > > > > > > > > > > > > > > The hwmon name attribute must not include '-', as > > > > > > > documented > > > > > > > in > > > > > > > Documentation/hwmon/sysfs-interface. Is enforcing that > > > > > > > 'crazy' ? > > > > > > > Maybe in your world, but not in mine. > > > > > > Well, lets revert the patch and then we can discuss what to > > > > > > do > > > > > > with > > > > > > the "name" problem. > > > > Ok, so the patch is on the way in. What to do next? > > > > > > > > pavel at n900:/sys/class/hwmon$ cat hwmon0/name > > > > bq27200-0 > > > > pavel at n900:/sys/class/hwmon$ cat hwmon1/name > > > > rx51-battery > > > > > > > > > > > > > > > > > > > To provide some detail: libsensors gets just as confused with > > > > > wildcards > > > > > and whitespace/newline as it does with '-' in the reported > > > > > name, > > > > > which > > > > > is why those are blocked by the new API. > > > > Ok... Question is "does someone actually use hwmon*/name on > > > > N900"? > > > > If > > > > so, we can't change it, but it is well possible that noone is. > > > IIRC hwmon is used on Nokia N900. > > > > > > But I have not seen hwmon devices for bq27200 and rx51-battery > > > yet. > > > Those are power supply driver and auto-exporting them also via > > > hwmon > > > is > > > something new, right? If yes, then we can use any name for those > > > new > > > hwmon devices as they cannot break userspace... as there is no > > > userspace > > > application for them. > > > > > If this is the case, you'd better set > > (struct thermal_zone_params)->no_hwmon when registering the thermal > > zone device, in which case, the hwmon device will not be created. > > > > In fact, I'd prefer to change tzp->no_hwmon to tzp->hwmon to not > > create > > hwmon I/F by default, and see if there is anyone using it. If yes, > > we > > can set the flag in soc thermal driver, explicitly, at meantime, a > > hwmon compatible name is required. > > > > But one foreseeable result is that we may get bug reports from end > > user > > that some sensors (acpitz, etc) are gone in 'sensors' output. And > > TBH, > > I'm not quite sure if this can be counted as a regression or not. > > > That sounds like fun. Changing bq27200-0 to bq27200_0 is Forbidden by > the ABI Police, but taking the entire device away is ok. > No. IMO, it depends on if the interface is used or not. If hwmon I/F is used, we can not take it away, nor change its name. If thermal zone I/F is used, we can not change it's 'type' name to be compatible with new hwmon API. > Anyway, sounds good to me. No one will use something that isn't > there, > and no one will realize that it could have been there, so I don't > expect > anyone to complain. Yes, I agree. thanks, rui From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753822AbdA0Dkw (ORCPT ); Thu, 26 Jan 2017 22:40:52 -0500 Received: from mga06.intel.com ([134.134.136.31]:2766 "EHLO mga06.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753487AbdA0Dkt (ORCPT ); Thu, 26 Jan 2017 22:40:49 -0500 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.33,293,1477983600"; d="scan'208";a="1087798036" Message-ID: <1485488382.2469.27.camel@intel.com> Subject: Re: v4.10-rc4 to v4.10-rc5: battery regression on Nokia N900 From: Zhang Rui To: Guenter Roeck , Pali =?ISO-8859-1?Q?Roh=E1r?= , Pavel Machek Cc: sre@kernel.org, kernel list , linux-arm-kernel , linux-omap@vger.kernel.org, tony@atomide.com, khilman@kernel.org, aaro.koskinen@iki.fi, ivo.g.dimitrov.75@gmail.com, patrikbachan@gmail.com, serge@hallyn.com, abcloriens@gmail.com, fabio.estevam@nxp.com Date: Fri, 27 Jan 2017 11:39:42 +0800 In-Reply-To: References: <20170123144031.GA7870@amd> <20170123232654.GA19342@amd> <20170123234912.GA2460@roeck-us.net> <20170124070639.GA5068@rzhang1-surface> <20170124073720.GB5603@amd> <88c94ea6-abe2-0f20-337e-e9ee00c883d8@roeck-us.net> <20170124175800.GA15070@amd> <20170124184526.GA25056@roeck-us.net> <20170125111233.GB3912@amd> <20170125120918.GA7936@pali> <1485481030.2469.15.camel@intel.com> Content-Type: text/plain; charset="UTF-8" X-Mailer: Evolution 3.18.5.2-0ubuntu3 Mime-Version: 1.0 Content-Transfer-Encoding: 8bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thu, 2017-01-26 at 18:03 -0800, Guenter Roeck wrote: > On 01/26/2017 05:37 PM, Zhang Rui wrote: > > > > On Wed, 2017-01-25 at 13:09 +0100, Pali Rohár wrote: > > > > > > On Wednesday 25 January 2017 12:12:33 Pavel Machek wrote: > > > > > > > > > > > > Hi! > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Right. > > > > > > > > > > > > > > > > > > Before reverting, can you please try if this patch > > > > > > > > > works > > > > > > > > > or not? > > > > > > > > Not really. Revert now. Sorry. > > > > > > > > > > > > > > > > Are you sure? This does not look equivalent to me at > > > > > > > > all. > > > > > > > > > > > > > > > > "name" file handling moved from drivers to the core, > > > > > > > > which > > > > > > > > added some > > > > > > > > crazy checks what name can contain. Even if this > > > > > > > > "works", > > > > > > > > what is the > > > > > > > > expected effect on the "name" file? > > > > > > > > > > > > > > > The hwmon name attribute must not include '-', as > > > > > > > documented > > > > > > > in > > > > > > > Documentation/hwmon/sysfs-interface. Is enforcing that > > > > > > > 'crazy' ? > > > > > > > Maybe in your world, but not in mine. > > > > > > Well, lets revert the patch and then we can discuss what to > > > > > > do > > > > > > with > > > > > > the "name" problem. > > > > Ok, so the patch is on the way in. What to do next? > > > > > > > > pavel@n900:/sys/class/hwmon$ cat hwmon0/name > > > > bq27200-0 > > > > pavel@n900:/sys/class/hwmon$ cat hwmon1/name > > > > rx51-battery > > > > > > > > > > > > > > > > > > > To provide some detail: libsensors gets just as confused with > > > > > wildcards > > > > > and whitespace/newline as it does with '-' in the reported > > > > > name, > > > > > which > > > > > is why those are blocked by the new API. > > > > Ok... Question is "does someone actually use hwmon*/name on > > > > N900"? > > > > If > > > > so, we can't change it, but it is well possible that noone is. > > > IIRC hwmon is used on Nokia N900. > > > > > > But I have not seen hwmon devices for bq27200 and rx51-battery > > > yet. > > > Those are power supply driver and auto-exporting them also via > > > hwmon > > > is > > > something new, right? If yes, then we can use any name for those > > > new > > > hwmon devices as they cannot break userspace... as there is no > > > userspace > > > application for them. > > > > > If this is the case, you'd better set > > (struct thermal_zone_params)->no_hwmon when registering the thermal > > zone device, in which case, the hwmon device will not be created. > > > > In fact, I'd prefer to change tzp->no_hwmon to tzp->hwmon to not > > create > > hwmon I/F by default, and see if there is anyone using it. If yes, > > we > > can set the flag in soc thermal driver, explicitly, at meantime, a > > hwmon compatible name is required. > > > > But one foreseeable result is that we may get bug reports from end > > user > > that some sensors (acpitz, etc) are gone in 'sensors' output. And > > TBH, > > I'm not quite sure if this can be counted as a regression or not. > > > That sounds like fun. Changing bq27200-0 to bq27200_0 is Forbidden by > the ABI Police, but taking the entire device away is ok. > No. IMO, it depends on if the interface is used or not. If hwmon I/F is used, we can not take it away, nor change its name. If thermal zone I/F is used, we can not change it's 'type' name to be compatible with new hwmon API. > Anyway, sounds good to me. No one will use something that isn't > there, > and no one will realize that it could have been there, so I don't > expect > anyone to complain. Yes, I agree. thanks, rui