lkml.org 
[lkml]   [2015]   [Jul]   [3]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH v7 1/3] sysfs: added sysfs_link_entry_to_kobj()


On 07/02/2015 08:04 PM, Tejun Heo wrote:
> Hello, Jason.
>
> On Tue, Jun 23, 2015 at 10:19:32AM -0600, Jason Gunthorpe wrote:
>> The concept was to move the sysfs files to the natural location on the
>> core's struct tpm.struct device and leave a symlink behind on the
>> platform_device for compat.
>
> I see. The only problem I see is that this might get abused. Can you
> clearly mark the function that it's not intended to be used in
> anything new, e.g. by renaming it to sth like
> __compat_only_sysfs_link_entry_to_kobj().


Great! We will do that. Thanks for taking time and understanding what
we are dealing with. We are trying to fix all the bad behavior from TPM
subsystem and bring up to the todays standards so that next gen TPM 2.0
devices will have a solid platform.

>> We could also just move the sysfs file and ignore the compat symlink,
>> but it is not clear to me if that is an OK UAPI break for sysfs?
>
> We shouldn't if it has actual chance of disturbing userland.
>
> Thanks.

/Jarkko


\
 
 \ /
  Last update: 2015-07-03 13:21    [W:0.049 / U:0.160 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site