- 20 Nov, 2013 5 commits
-
-
Nandini Hanumanthagowda authored
indented all if-else statement blocks to remove checkpatch warnings and errors like indent code wherever possible and no spaces at the start of line Signed-off-by:
Nandini Hanumanthagowda <nandu.hgowda@gmail.com> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Nandini Hanumanthagowda authored
aligned the case statements of 'switch' so that the switch and case are on same indentation level to fix the following error. ERROR: switch and case should be at the same indent Signed-off-by:
-
Nandini Hanumanthagowda authored
removed the "unnecessary whitespace before quoted new line" warning reported by checkpatch script Signed-off-by:
-
Nandini Hanumanthagowda authored
fixed following checkpatch error: ERROR: else should follow close brace '}' Signed-off-by:
-
Nandini Hanumanthagowda authored
fixed the following checkpatch error: ERROR: that open brace { should be on the previous line Signed-off-by:
-
- 19 Nov, 2013 31 commits
-
-
Nandini Hanumanthagowda authored
removed unnecessary braces around if-else statement whenever the if-else construct was followed by just single line of code to comply with linux coding style Signed-off-by:
-
Ana Rey authored
Removed innecessary spaces and indentation errors in function definition (including some curly braces wrongly placed and broke up at 80-chars per line). I still see some errors if I run the checkpatch.pl script, but those are not my fault. I'll address them in follow up patches. Signed-off-by:
Ana Rey <anarey@gmail.com> Acked-by:
Pablo Neira Ayuso <pablo@gnumonks.org> Signed-off-by:
-
Eric Skoglund authored
This patch fixes whitespace coding style errors: func () --> func() Signed-of-by:
Eric Skoglund <eric@pagefault.se> Signed-off-by:
-
Eric Skoglund authored
This patch fixes a brace coding style issue. Signed-off-by:
-
Eric Skoglund authored
This patch converts spaces to tabs to conform to the coding style standards. Signed-off-by:
-
SeongJae Park authored
Signed-off-by:
SeongJae Park <sj38.park@gmail.com> Signed-off-by:
-
SeongJae Park authored
Signed-off-by:
-
SeongJae Park authored
Signed-off-by:
-
Eddie Kovsky authored
Reduced checkpatch.pl warnings from 46 to 17 Long strings on lines over 80 characters remain untouched Signed-off-by:
Eddie Kovsky <ewk@edkovsky.org> Acked-by:
Pavel Machek <pavel@ucw.cz> Signed-off-by:
-
Geert Uytterhoeven authored
platform_device.name is "const char *" Signed-off-by:
Geert Uytterhoeven <geert@linux-m68k.org> Signed-off-by:
-
Geert Uytterhoeven authored
usb_driver.name is "const char *" Signed-off-by:
-
Geert Uytterhoeven authored
device_driver.name is "const char *" Signed-off-by:
-
Peng Tao authored
legacy code used to support kernel where struct ctl_table has strategy member. Cc: Andreas Dilger <andreas.dilger@intel.com> Signed-off-by:
Peng Tao <bergwolf@gmail.com> Signed-off-by:
-
Peng Tao authored
legacy code used to support kernel where struct ctl_table has ctl_name member. Cc: Andreas Dilger <andreas.dilger@intel.com> Signed-off-by:
-
Peng Tao authored
Cc: Andreas Dilger <andreas.dilger@intel.com> Signed-off-by:
-
Peng Tao authored
Cc: Andreas Dilger <andreas.dilger@intel.com> Signed-off-by:
-
Peng Tao authored
Cc: Andreas Dilger <andreas.dilger@intel.com> Signed-off-by:
-
Peng Tao authored
Cc: Andreas Dilger <andreas.dilger@intel.com> Signed-off-by:
-
Peng Tao authored
No more references to it. Cc: Andreas Dilger <andreas.dilger@intel.com> Signed-off-by:
-
Peng Tao authored
It is dead code because we defined CFS_SYSFS_MODULE_PARM. Cc: Andreas Dilger <andreas.dilger@intel.com> Signed-off-by:
-
Peng Tao authored
we have #define CFS_SYSFS_MODULE_PARM 1 so it is indeed dead code. Cc: Andreas Dilger <andreas.dilger@intel.com> Signed-off-by:
-
Peng Tao authored
It is only used by server. Cc: Andreas Dilger <andreas.dilger@intel.com> Signed-off-by:
-
Peng Tao authored
It is only used by user space. Cc: Andreas Dilger <andreas.dilger@intel.com> Signed-off-by:
-
Peng Tao authored
It is only used by user space. Cc: Andreas Dilger <andreas.dilger@intel.com> Signed-off-by:
-
Peng Tao authored
Cc: Andreas Dilger <andreas.dilger@intel.com> Signed-off-by:
-
Amir Shehata authored
To make the function a bit easier to read. This is coding style fix part of original Lustre commit in external tree. Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-3093 Lustre-change: http://review.whamcloud.com/6092Signed-off-by:
Amir Shehata <amir.shehata@intel.com> Reviewed-by:
Isaac Huang <he.huang@intel.com> Reviewed-by:
Liang Zhen <liang.zhen@intel.com> Reviewed-by:
Oleg Drokin <oleg.drokin@intel.com> Signed-off-by:
Andreas Dilger <andreas.dilger@intel.com> Signed-off-by:
-
Amir Shehata authored
This is part of original Lustre commit in external tree. Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-3093 Lustre-change: http://review.whamcloud.com/6092Signed-off-by:
-
Amir Shehata authored
This is coding style fix part of original Lustre commit in external tree. Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-3093 Lustre-change: http://review.whamcloud.com/6092Signed-off-by:
-
Amir Shehata authored
To make the function a bit easier to read. This is coding style fix part of original Lustre commit in external tree. Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-3093 Lustre-change: http://review.whamcloud.com/6092Signed-off-by:
-
Peng Tao authored
To follow kernel's "no spaces at the start of a line" rule. Signed-off-by:
-
Jinshan Xiong authored
Proposed way to do exclusive open: 0. First of all, we have to perform most of the work in kernel space; 1. Client exclusively opens the file with IT_RELEASE_OPEN. 1.1 exclusive open means the open will fail if the file is being opened by somebody else; 2. the MDT will handle IT_RELEASE as follows: 2.1 Revoke OPEN_LOCK on this file, just request EX mode of MDS_INODELOCK_OPEN lock and release it; 2.2 Acquire a rwsem, say open_rwsem, with write mode before trying to the file; for the normal open, it should acquire read mode of open_rwsem; 2.3 Check if the file is already being opened by others, if not return with -EBUSY; 2.4 Check if the file can be released; 2.5 Set a special flag in struct mdt_file_data to mark this open is exclusive; 2.5 Release open rwsem. >From now on, if the file is opened by others, it will mark mdt_file_data that the exclusive open is broken. 3. Client: if IT_RELEASE_OPEN is finished successfully, and do followings: 3.1 Acquire full PW or EX extent lock to flush dirty cache; 3.2 Pack the handle of layout lock, data version and other attars; 3.3 Close the file with IT_RELEASE_CLOSE. 4. Back to MDT to handle IT_RELEASE CLOSE: 4.1 Grab the open_rwsem 4.2 Check if the exclusive open has ever been broken, in that case, the RELEASE process will fail; 4.3 Verify the data version matches archive; 4.4 Grab EX layout lock 4.5 Swap the layout 4.6 Release EX layout lock 4.7 Close the exclusive open Basically we avoid granting EX layout lock back to client and introduce exclusive open so that we know it if the file has ever being accessed. I hope this can simplify things a little bit. Also, exclusive open can be used to implement file lease. In this patch, a framework of lease is implemented. However, only exclusive lease is supported right now. To apply a lease, MDS_OPEN_LEASE must be set to open the file, EX mode open lock is returned to the client side to hold a lease. From that time on, if this file is opened again by other processes, the open lock will be revoked so the client who holds the lease will know the lease is already broken by checking that open lock. To release a lease, normal close is used. The client will revoke the open lock before sending CLOSE request. Lease can be applied in two ways. ll_lease_open()/close() can be called directly if the lease holder is in kernel space; or if the lease holder lives in user space, it has to open the file first and then use ioctl() with command LL_IOC_SET_LEASE to apply a lease. The lease holder has to poll the lease status itself. Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-2919 Lustre-change: http://review.whamcloud.com/6730Signed-off-by:
Jinshan Xiong <jinshan.xiong@intel.com> Signed-off-by:
John L. Hammond <john.hammond@intel.com> Reviewed-by:
-
- 15 Nov, 2013 4 commits
-
-
Himangi Saraogi authored
This patch takes up the task mentioned in the TODO file of dwc2 to cleanup the microframe scheduler code. The while(!done) loops have been replaced with appropriate for loops and unnecessary variables like done and ret have been removed. Signed-off-by:
Himangi Saraogi <himangi774@gmail.com> Acked-by:
Paul Zimmerman <paulz@synopsys.com> Signed-off-by:
-
John L. Hammond authored
In lov_dump_lmm(), convert the lmm_magic from little-endian to host-endian byte order before the switch statement, as the other lov_dump_xxx() and lov_verify_xxx() functions already do. Remove the unused macro LMM_ASSERT(). Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-3297 Lustre-change: http://review.whamcloud.com/6290Signed-off-by:
Li Wei <wei.g.li@intel.com> Reviewed-by:
-
Li Xi authored
llog_open() calls llog_alloc_handle() taking NULL as the error return value. But llog_alloc_handle() returns ERR_PTR(-ENOMEM) instead when error. Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-3470 Lustre-change: http://review.whamcloud.com/6644Signed-off-by:
Li Xi <pkuelelixi@gmail.com> Reviewed-by:
Mike Pershin <mike.pershin@intel.com> Reviewed-by:
-
James Simmons authored
is_compat_task has been defined on all arches since v2.6.29. We can remove the test and dead code. Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-2800 Lustre-change: http://review.whamcloud.com/5393Signed-off-by:
Jeff Mahoney <jeffm@suse.com> Signed-off-by:
James Simmons <uja.ornl@gmail.com> Reviewed-by:
Bob Glossman <bob.glossman@intel.com> Reviewed-by:
-
