From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753267AbbFLQEl (ORCPT ); Fri, 12 Jun 2015 12:04:41 -0400 Received: from mta02.ornl.gov ([128.219.177.12]:45875 "EHLO mta02.ornl.gov" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750898AbbFLQEi convert rfc822-to-8bit (ORCPT ); Fri, 12 Jun 2015 12:04:38 -0400 X-SG: RELAYLIST X-IronPort-AV: E=Sophos;i="5.13,602,1427774400"; d="scan'208";a="81841088" From: "Simmons, James A." To: "'Tolga Ceylan'" , Greg Kroah-Hartman CC: "devel@driverdev.osuosl.org" , "HPDD-discuss@lists.01.org" , Tina Johnson , Greg Donald , "linux-kernel@vger.kernel.org" , Julia Lawall , Al Viro , aybuke ozdemir , Joe Perches , "Ben Evans (bevans@cray.com)" Subject: RE: [HPDD-discuss] [PATCH 1/1] staging: lustre/lustre/llite: get rid of incorrect type warning Thread-Topic: [HPDD-discuss] [PATCH 1/1] staging: lustre/lustre/llite: get rid of incorrect type warning Thread-Index: AQHQo+BZYaQssMDJsk+9j1OMMiSJr52nNuMAgAHRLWA= Date: Fri, 12 Jun 2015 16:04:15 +0000 Message-ID: References: <1433804517-7709-1-git-send-email-tolga.ceylan@gmail.com> <20150611004817.GA26022@kroah.com> In-Reply-To: Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-ms-exchange-transport-fromentityheader: Hosted x-originating-ip: [128.219.12.132] Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 8BIT MIME-Version: 1.0 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org >>On Wed, Jun 10, 2015 at 5:48 PM, Greg Kroah-Hartman > wrote: >> >> Are you sure all of these are correct? The kernel/user api for lustre >> is a complex beast, and just casting away the pointer types isn't >> usually the proper thing to do in order to resolve the issues here. >> >> thanks, >> >> greg k-h > >I'm not 100% sure, but the pointers that I added the annotation to end >up being used as user memory. (eg. passed to copy_to_user, etc.) >Sometimes these pointers are passed to functions that already have >__user annotation in their signatures (eg. ll_getname, copy_and_ioctl, >ll_fid2path, etc.). Using these simple cast are not the proper fix. We had a lot of issues with user land tools breaking due to leakage of kernel space stuff and other problems. Some work went into cleaning that up in the OpenSFS branch but it is not totally complete yet. Evans you wanted something challenging to work on well this is up your alley. I would recommend looking at JIRA ticket LU-6401 and all its sub tickets. You could start the port of those to the upstream client. At the same time we can finish the cleanup in the OpenSFS branch as well.