- 19 Sep, 2016 10 commits
-
-
John L. Hammond authored
Define new constants LL_LEASE_{RD,WR,UN}LCK for use as the argument to and return value from the LL_IOC_{GET,SET}_LEASE ioctls. As arguments, these contants replace the use of F_{RD,WR,UN}LCK from fcntl.h. As return values they replace the use of FMODE_{READ,WRITE} which are internal to the Linux kernel source and not under the control of the Lustre ioctl interface. Signed-off-by:John L. Hammond <john.hammond@intel.com> Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-5013 Reviewed-on: http://review.whamcloud.com/10233Reviewed-by:
Jinshan Xiong <jinshan.xiong@intel.com> Reviewed-by:
Lai Siyao <lai.siyao@intel.com> Reviewed-by:
Oleg Drokin <oleg.drokin@intel.com> Signed-off-by:
James Simmons <jsimmons@infradead.org> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Niu Yawei authored
Umount process relies on lu_site_purge(-1) to purge all objects before umount, however, if there happen to have a cache shrinker which calls lu_site_purge(nr) in parallel, some objects may still being freed by cache shrinker even after the lu_site_purge(-1) called by umount done. This can be simply fixed by serializing purge threads, since it doesn't make any sense to have them in parallel. Signed-off-by:
Niu Yawei <yawei.niu@intel.com> Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-5331 Reviewed-on: http://review.whamcloud.com/11099Reviewed-by:
Mike Pershin <mike.pershin@intel.com> Reviewed-by:
-
John L. Hammond authored
Add the prefix rnb_ to the members of struct niobuf_remote. Delete the relevant compat macros from ofd_internal.h. Signed-off-by:
Andreas Dilger <andreas.dilger@intel.com> Reviewed-by:
-
John L. Hammond authored
Add the prefix lnb_ to the members of struct niobuf_local that do not already have it. Change the struct dentry *lnb_dentry member to void *lnb_data as it is not used to hold a pointer to struct dentry. Signed-off-by:
Alex Zhuravlev <alexey.zhuravlev@intel.com> Reviewed-by:
-
wang di authored
Return correct value (rc) in lmv_unlink. In lmv_unlink, -ENOENT might be ingored for local directory unlink. Signed-off-by:
wang di <di.wang@intel.com> Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-5371 Reviewed-on: http://review.whamcloud.com/11170Reviewed-by:
-
Alexander Zarochentsev authored
two optimizations in the busy loop wait in class_cleanup(): (1) remove unnecessary spinlocks protecting obd_conn_inprogress access (2) replace cond_resched by more suitable yield to unconditionally yield the CPU if the condition check fails. Signed-off-by:Alexander Zarochentsev <alexander_zarochentsev@xyratex.com> Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-5397 Xyratex-bug-id: MRP-1991 Reviewed-on: http://review.whamcloud.com/11198Reviewed-by:
Dmitry Eremin <dmitry.eremin@intel.com> Reviewed-by:
-
wang di authored
LMV should honor MDT index embedded in the name of volatile file, then during hsm restore, the file under striped dir can be restored to the right MDT. Signed-off-by:
Henri Doreau <henri.doreau@cea.fr> Reviewed-by:
-
John L. Hammond authored
If lvbo initialization has failed then save the return status (from lr_lvb_len) before putting the resource. Signed-off-by:
Emoly Liu <emoly.liu@intel.com> Reviewed-by:
-
Li Xi authored
This patch enlarges OST_MAXREQSIZE so as to make the request size large enough for 4MB RPC. Signed-off-by:
Li Xi <lixi@ddn.com> Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-4755 Reviewed-on: http://review.whamcloud.com/9599Reviewed-by:
James Simmons <uja.ornl@gmail.com> Reviewed-by:
-
Li Xi authored
When ll_prep_md_op_data() fails, ll_statahead_thread() does not release reference of sai and parent dentry. It does not wake up parent thread either. This patch fixes these problems. Signed-off-by:
-
- 18 Sep, 2016 21 commits
-
-
Greg Kroah-Hartman authored
This reverts commit ed88363e. It was incorrect :( Cc: Anchal Jain <anchalj109@gmail.com> Signed-off-by:
-
Namrata A Shettar authored
Replace NULL comparison to resolve checkpatch issue. - x == NULL => !x Signed-off-by:
Namrata A Shettar <namrataashettar@gmail.com> Signed-off-by:
-
Namrata A Shettar authored
Replace NULL comparison to resolve checkpatch issue. - x == NULL => !x Signed-off-by:
-
sayli karnik authored
This patch removes the typedef 'wlandevice_t' to the 'wlandevice' structure. Signed-off-by:
sayli karnik <karniksayli1995@gmail.com> Signed-off-by:
-
Sandhya Bankar authored
Drop cast on the result of kzalloc. The semantic patch that makes this change is as follows: @@ type T; @@ - (T *) (\(kmalloc\|kzalloc\|kcalloc\|kmem_cache_alloc\|kmem_cache_zalloc\| kmem_cache_alloc_node\|kmalloc_node\|kzalloc_node\)(...)) Signed-off-by:
Sandhya Bankar <bankarsandhya512@gmail.com> Signed-off-by:
-
Sandhya Bankar authored
Use !x instead of x == NULL. This patch was found by checkpatch. Signed-off-by:
-
Sandhya Bankar authored
Remove redundant if-statement. Signed-off-by:
-
Sandhya Bankar authored
Use !x instead of x == NULL. This patch was found by checkpatch. Signed-off-by:
-
Bhumika Goyal authored
This patch compresses the return logic into a single statement and removes the now unused return variable. Done using coccinelle: @@ type T; identifier r; @@ - T r; ... when !=r - r = + return ...; - return r; Signed-off-by:Bhumika Goyal <bhumirks@gmail.com> Signed-off-by:
-
Namrata A Shettar authored
Replace explicit NULL comparison with equivalent expression to resolve checkpatch issue. - x != NULL => x Signed-off-by:
-
Anchal Jain authored
Add spaces around operators {+, -, &, /, <<, &&} Signed-off-by:Anchal Jain <anchalj109@gmail.com> Signed-off-by:
-
Anchal Jain authored
Convert pointer comparisons to NULL to NULL pointer 'p' Signed-off-by:
-
sayli karnik authored
This patch removes excess round braces around expressions/variables. Signed-off-by:
-
sayli karnik authored
The patch removes an unnecessary space after a cast, as detected by checkpatch.pl. Signed-off-by:
-
sayli karnik authored
The patch removes unnecessary blank lines as detected by checkpatch.pl. Also, it removes a commented line of unrequired code. Signed-off-by:
-
sayli karnik authored
The patch fixes the following errors detected by checkpatch.pl: ERROR: space prohibited before that close parenthesis ')' ERROR: space prohibited after that open parenthesis '(' Signed-off-by: -
sayli karnik authored
The patch removes braces {} for single statement blocks, as suggested by checkpatch.pl. Signed-off-by: -
sayli karnik authored
This patch replaces <asm/atomic.h> with <linux/atomic.h>, as suggested by checkpatch.pl. Signed-off-by:
-
Cathal Mullaney authored
Fixed three minor coding style issues. Signed-off-by:
Cathal Mullaney <chuckleberryfinn@gmail.com> Signed-off-by:
-
Jon Eyolfson authored
Changed if statement bodies from 2 levels deeper than if statement to 1. Signed-off-by:
Jon Eyolfson <jon@eyl.io> Signed-off-by:
-
Anchal Jain authored
This patch replaces bit shifting on 1 with the BIT(x) macro as it's extensively used by other function in the file olpc_dcon.h . Signed-off-by:
-
- 17 Sep, 2016 9 commits
-
-
Cathal Mullaney authored
Fixed a coding style issue. Signed-off-by:
-
Namrata A Shettar authored
Correct the word 'success' being misspelt in the macro: - LED_BLINK_WPS_SUCESS_INTERVAL_ALPHA => LED_BLINK_WPS_SUCCESS_INTERVAL_ALPHA Signed-off-by:
-
sayli karnik authored
staging: rtl8188eu: core: Replace memcpy() with ether_addr_copy() if ethernet addresses are __aligned(2) This patch fixes the checkpatch warning: Prefer ether_addr_copy() over memcpy() if the Ethernet addresses are __aligned(2) Done using coccinelle: @@ expression exp1, exp2; @@ - memcpy(exp1, exp2, ETH_ALEN); + ether_addr_copy(exp1, exp2); The output of pahole shows that the addresses are aligned: struct ieee80211_hdr { __le16 frame_control; /* 0 2 */ __le16 duration_id; /* 2 2 */ u8 addr1[6]; /* 4 6 */ u8 addr2[6]; /* 10 6 */ u8 addr3[6]; /* 16 6 */ __le16 seq_ctrl; /* 22 2 */ u8 addr4[6]; /* 24 6 */ /* size: 30, cachelines: 1, members: 7 */ /* last cacheline: 30 bytes */ }; struct wlan_bssid_ex { u32 Length; /* 0 4 */ unsigned char MacAddress[6]; /* 4 6 */ u8 Reserved[2]; /* 10 2 */ struct ndis_802_11_ssid Ssid; /* 12 36 */ u32 Privacy; /* 48 4 */ long int Rssi; /* 52 8 */ enum NDIS_802_11_NETWORK_TYPE NetworkTypeInUse; /* 60 4 */ /* --- cacheline 1 boundary (64 bytes) --- */ struct ndis_802_11_config Configuration; /* 64 32 */ enum ndis_802_11_network_infra InfrastructureMode; /* 96 4 */ unsigned char SupportedRates[16]; /* 100 16 */ struct wlan_phy_info PhyInfo; /* 116 4 */ u32 IELength; /* 120 4 */ u8 IEs[768]; /* 124 768 */ /* --- cacheline 13 boundary (832 bytes) was 60 bytes ago --- */ /* size: 892, cachelines: 14, members: 13 */ /* last cacheline: 60 bytes */ }; struct stadel_event { unsigned char macaddr[6]; /* 0 6 */ unsigned char rsvd[2]; /* 6 2 */ int mac_id; /* 8 4 */ /* size: 12, cachelines: 1, members: 3 */ /* last cacheline: 12 bytes */ }; Signed-off-by: -
Andrew Kanner authored
Ran checkpatch.pl -f vme_pio2_core.c Fixed: WARNING: Symbolic permissions are not preferred. Consider using octal permissions (0444) Signed-off-by:
Andrew Kanner <andrew.kanner@gmail.com> Signed-off-by:
-
sayli karnik authored
This patch replaces type u_int8_t with u8 as suggested by checkpatch.pl. Signed-off-by:
-
sayli karnik authored
This patch replaces the explicit NULL comparisons with ! operator. It was found using Coccinelle: @disable is_null@ expression e; @@ - e == NULL + !e Signed-off-by:
Julia Lawall <julia.lawall@lip6.fr> Signed-off-by:
-
sayli karnik authored
This patch replaces left shifts on 1 with the BIT(x) macro, as suggested by checkpatch.pl. This was done using coccinelle script: @@ constant c; @@ - (1 << c) + BIT(c) Signed-off-by:
-
Bhumika Goyal authored
Remove extern specifier from function declaration as they have it by default. Also move extern declaration from .c files to their respective header file 'ks_hostif.h'. Coccinelle was used to remove extern and other changes were done by hand. Script: @@ identifier func; type T; @@ - extern T func(...); Signed-off-by:
-
Namrata A Shettar authored
Add space around binary operators to resolve checkpatch issue. Signed-off-by:
-
