From: Patrick Farrell
Currently, it's difficult to match brw RPCs to objects and
extents from client logs. This patch adds a D_RPCTRACE
debug message giving the necessary information.
Signed-off-by: Patrick Farrell
Intel-bug-id:
From: Lai Siyao
ll_intent_drop_lock() may sleep, which should not be called inside
spinlock.
Signed-off-by: Lai Siyao
Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-2272
Reviewed-on: http://review.whamcloud.com/9665
Reviewed-by: Fan Yong
From: John L. Hammond
Remove no longer used osc_packmd() and osc_getstripe().
Several ioctls cases that are no longer used are removed.
Remove no longer used adjust_kms() infrastructure.
Signed-off-by: John L. Hammond
Intel-bug-id:
From: Dmitry Eremin
Change the type accordant usage.
Signed-off-by: Dmitry Eremin
Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-5577
Reviewed-on: http://review.whamcloud.com/12384
Reviewed-by: John L. Hammond
From: Dmitry Eremin
ltd_idx is used as unsigned.
Signed-off-by: Dmitry Eremin
Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-5577
Reviewed-on: http://review.whamcloud.com/11879
Reviewed-by: John L. Hammond
From: John L. Hammond
Remove RCU2HANDLE macro from lustre_handles.h.
Signed-off-by: John L. Hammond
Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-2675
Reviewed-on: http://review.whamcloud.com/11498
Reviewed-by: Andreas Dilger
From: Vitaly Fertman
While clients will resend client->server RPCs, servers would not
resend server->client RPCs such as LDLM callbacks (blocking
or completion callbacks/ASTs). This could result in clients being
evicted from the server if blocking callbacks were
From: John L. Hammond
Remove some unused declarations from lustre_lib.h and move
some others to more natural headers.
Signed-off-by: John L. Hammond
Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-2675
Reviewed-on:
From: John L. Hammond
We are already touching dentry in CDEBUG macros so it
will crash long before these checks. Since this is the
case no need to do an additional check.
Signed-off-by: John L. Hammond
Intel-bug-id:
From: John L. Hammond
In the handler for LL_IOC_LMV_GETSTRIPE convert stripe FIDs from
little to CPU endian when unpacking lmv_user_md. In lmv_unpack_md_v1()
fix a double conversion of the stripe count.
Signed-off-by: John L. Hammond
From: Vitaly Fertman
otherwise, waiting lock may get granted as no BLOCKED_GRANTED
flag is returned
Signed-off-by: Vitaly Fertman
Xyratex-bug-id: MRP-1944
Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-5496
Reviewed-on:
From: John L. Hammond
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: John L. Hammond
Earlier a bunch of interval handling got removed since it wasn't
used by the upstream client. Now some of it is needed again for
the client code so this patch restores what is needed.
Signed-off-by: James Simmons
---
.../staging/lustre/lustre/include/interval_tree.h |
From: Li Xi
This patch enlarges OST_MAXREQSIZE so as to make the
request size large enough for 4MB RPC.
Signed-off-by: Li Xi
Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-4755
Reviewed-on: http://review.whamcloud.com/9599
Reviewed-by: Andreas Dilger
From: Dmitry Eremin
Comparison of unsigned value against 0 is always true.
Signed-off-by: Dmitry Eremin
Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-5200
Reviewed-on: http://review.whamcloud.com/11217
Reviewed-by: John L. Hammond
From: Wang Shilong
Previously, when running 'modprobe lustre', it hit the following
error message which is becaue of network initialisation failure:
modprobe: ERROR: could not insert 'lustre': Input/output error
However, error code is there, just let it return to caller,
after
From: Bobi Jam
When a read ahead does not reach the end of the region reserved from
ras, we'd set ras::ras_next_readahead back to where we left off; For
stride read ahead, it needs to make sure that the offset is no less
than ras_stride_offset, so that the stride read ahead
From: Andreas Dilger
Move the LUSTRE_VERSION_CODE checks to trigger on x.y.53 instead of
x.y.50, so that it is into the development cycle that they are hit
instead of right at the start.
In many cases, the #warning has been removed (to prevent build errors)
and instead
From: wang di
Return correct value (rc) in lmv_unlink. In lmv_unlink,
-ENOENT might be ingored for local directory unlink.
Signed-off-by: wang di
Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-5371
Reviewed-on: http://review.whamcloud.com/11170
Original linkea was only used for the lustre server code
so it was removed from the upstream client. Now it needs
to be restored for client work that uses this infrastructure.
Signed-off-by: James Simmons
---
drivers/staging/lustre/lustre/include/lu_object.h |7 +
From: wang di
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: wang di
Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-4691
From: Niu Yawei
In brw_interpret(), we forgot page offset when calculating
write offset, that leads to wrong kms for sync write.
Signed-off-by: Niu Yawei
Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-5463
Reviewed-on:
From: Niu Yawei
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)
From: Bobi Jam
'commit 77605e41a26f ("staging/lustre/clio: add pages into writeback cache
in batches")' adds a page to a list aggregate issuing them to writeback
cache; A page add is referenced in llite/vvp io scope, while writeback
cache commit de-refers it under osc sub
From: John L. Hammond
If lvbo initialization has failed then save the return status (from
lr_lvb_len) before putting the resource.
Signed-off-by: John L. Hammond
Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-5305
Reviewed-on:
On Sep 17, 2016, at 06:04, Baoyou Xie wrote:
>
> We get a few warnings when building kernel with W=1:
> drivers/staging/lustre/lustre/lmv/lmv_obd.c:1640:1: warning: no previous
> prototype for 'lmv_locate_target_for_name' [-Wmissing-prototypes]
>
On Sep 18, 2016, at 14:21, nayeem wrote:
> On Friday 16 September 2016 01:30 PM, Dilger, Andreas wrote:
>> On Sep 15, 2016, at 12:33, nayeem wrote:
>>> On Wednesday 14 September 2016 10:44 AM, Dilger, Andreas wrote:
On Sep 12, 2016, at 04:27,
On Friday 16 September 2016 01:30 PM, Dilger, Andreas wrote:
On Sep 15, 2016, at 12:33, nayeem wrote:
On Wednesday 14 September 2016 10:44 AM, Dilger, Andreas wrote:
On Sep 12, 2016, at 04:27, Greg KH wrote:
On Fri, Sep 09, 2016 at
> We get 1 warning when building kernel with W=1:
> drivers/staging/lustre/lustre/lmv/lmv_obd.c:2774:5: warning: no previous
> prototype for 'lmv_pack_md' [-Wmissing-prototypes]
>
> In fact, this function is not declared in any file,but should be
> declared in a header file, thus can be
Fixed several minor brace coding style issues.
Signed-off-by: Cathal Mullaney
---
drivers/staging/rts5208/rtsx_card.c | 26 +-
1 file changed, 13 insertions(+), 13 deletions(-)
diff --git a/drivers/staging/rts5208/rtsx_card.c
PTR_ALIGN is a bit shorter than N_BYTE_ALIGMENT.
Signed-off-by: Ivan Safonov
---
drivers/staging/rtl8188eu/core/rtw_recv.c | 2 +-
drivers/staging/rtl8188eu/core/rtw_xmit.c | 6 +++---
drivers/staging/rtl8188eu/include/basic_types.h | 4
To avoid unnecessary typecast.
Signed-off-by: Ivan Safonov
---
drivers/staging/rtl8188eu/include/drv_types.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/rtl8188eu/include/drv_types.h
b/drivers/staging/rtl8188eu/include/drv_types.h
GET_HAL_DATA replaced by its definition.
Signed-off-by: Ivan Safonov
---
drivers/staging/rtl8188eu/core/rtw_sreset.c | 9 +--
drivers/staging/rtl8188eu/hal/bb_cfg.c| 13 ++--
drivers/staging/rtl8188eu/hal/odm.c | 17 ++---
This macro does not used.
Signed-off-by: Ivan Safonov
---
drivers/staging/rtl8188eu/include/rtl8188e_hal.h | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/staging/rtl8188eu/include/rtl8188e_hal.h
b/drivers/staging/rtl8188eu/include/rtl8188e_hal.h
index
usb_hal.h is empty.
Signed-off-by: Ivan Safonov
---
drivers/staging/rtl8188eu/hal/hal_intf.c| 1 -
drivers/staging/rtl8188eu/hal/usb_halinit.c | 1 -
drivers/staging/rtl8188eu/include/usb_hal.h | 18 --
drivers/staging/rtl8188eu/os_dep/os_intfs.c | 2
rtl8188eu_set_hal_ops only allocates
HalData member of adapter structure.
Signed-off-by: Ivan Safonov
---
drivers/staging/rtl8188eu/hal/usb_halinit.c | 7 ---
drivers/staging/rtl8188eu/include/usb_hal.h | 2 --
drivers/staging/rtl8188eu/os_dep/usb_intf.c | 7 +--
3
xmitframe_direct is a simple wrapper around
rtw_xmitframe_coalesce and rtw_dump_xframe functions.
Many wrappers complicates code reading.
Signed-off-by: Ivan Safonov
---
drivers/staging/rtl8188eu/hal/rtl8188eu_xmit.c | 19 ++-
1 file changed, 6
rtw_hal_set_hwreg and rtw_hal_get_hwreg does not used
with HW_VAR_DM_FLAG parameter.
Signed-off-by: Ivan Safonov
---
drivers/staging/rtl8188eu/hal/usb_halinit.c | 7 ---
drivers/staging/rtl8188eu/include/hal_intf.h | 1 -
2 files changed, 8 deletions(-)
diff --git
usb_write_port writes only xmit_buf object data to device.
In addition, an appropriate name for this argument is used.
Signed-off-by: Ivan Safonov
---
drivers/staging/rtl8188eu/hal/rtl8188eu_xmit.c| 4 ++--
drivers/staging/rtl8188eu/include/usb_ops_linux.h | 2 +-
pkt_hdrlen has a constant value.
Signed-off-by: Ivan Safonov
---
drivers/staging/rtl8188eu/core/rtw_xmit.c| 4 +---
drivers/staging/rtl8188eu/include/rtw_xmit.h | 1 -
2 files changed, 1 insertion(+), 4 deletions(-)
diff --git
This function does not used.
Signed-off-by: Ivan Safonov
---
drivers/staging/rtl8188eu/include/usb_ops_linux.h | 15 ---
1 file changed, 15 deletions(-)
diff --git a/drivers/staging/rtl8188eu/include/usb_ops_linux.h
Kalle Valo writes:
> Jes Sorensen writes:
>
>> Joe Perches writes:
>>> I think it'd be nicer to use dev_dbg for all these cases
>>> and as well use some new macro that includes the test
>>>
>>> Something like:
>>>
>>> #define
Larry Finger writes:
> On 09/17/2016 03:59 PM, Jes Sorensen wrote:
>> Larry Finger writes:
>>> As soon as debugging is turned on, the logs are filled with messages
>>> reporting the interrupt status. As this quantity is usually zero, this
>>>
Fixes typo,
FAILUER -> FAILURE
Recieve -> Receive
Signed-off-by: Hariharan R
---
drivers/staging/ks7010/ks_hostif.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/staging/ks7010/ks_hostif.c
b/drivers/staging/ks7010/ks_hostif.c
We get 4 warnings when building kernel with W=1:
drivers/vme/bridges/vme_fake.c:374:6: warning: no previous prototype for
'fake_lm_check' [-Wmissing-prototypes]
drivers/vme/bridges/vme_fake.c:609:6: warning: no previous prototype for
'fake_vmewrite8' [-Wmissing-prototypes]
We get 1 warning when building kernel with W=1:
drivers/staging/lustre/lustre/lmv/lmv_obd.c:2774:5: warning: no previous
prototype for 'lmv_pack_md' [-Wmissing-prototypes]
In fact, this function is not declared in any file,but should be
declared in a header file, thus can be recognized in other
We get 1 warning when building kernel with W=1:
drivers/staging/ks7010/ks7010_sdio.c:363:5: warning: no previous prototype for
'ks_wlan_hw_tx' [-Wmissing-prototypes]
In fact, this function is declared in
drivers/staging/ks7010/ks_wlan_net.c and
drivers/staging/ks7010/ks_hostif.c, but should be
We get 2 warnings when building kernel with W=1:
drivers/staging/ks7010/ks7010_sdio.c:152:6: warning: no previous prototype for
'ks_wlan_hw_wakeup_request' [-Wmissing-prototypes]
drivers/staging/ks7010/ks7010_sdio.c:255:5: warning: no previous prototype for
'ks_wlan_hw_power_save'
We get 1 warning when building kernel with W=1:
drivers/staging/ks7010/ks_wlan_net.c:3392:6: warning: no previous prototype for
'send_packet_complete' [-Wmissing-prototypes]
In fact, this function is declared in
drivers/staging/ks7010/ks_hostif.c, but should be
declared in a header file. thus
Hi,
Thanks for the review. I will send a new patch.
Gwenhael
On 12/09/2016 14:02, Lars-Peter Clausen wrote:
Hi,
Thanks for the patch.
On 09/11/2016 12:52 PM, Gwenhael Goavec-Merou wrote:
+static struct ad9834_platform_data *ad9834_parse_dt(struct spi_device *spi)
+{
+ struct
ad9834 driver needs some default properties. Currently these parameters are
provided through platform_data.
This patch adds a function to create this pdata based on device-tree node.
Signed-off-by: Gwenhael Goavec-Merou
---
Changes v1 -> v2:
- use
Jes Sorensen writes:
> Joe Perches writes:
>> On Sat, 2016-09-17 at 12:09 -0500, Larry Finger wrote:
>>> As soon as debugging is turned on, the logs are filled with messages
>>> reporting the interrupt status. As this quantity is usually zero, this
>>>
101 - 152 of 152 matches
Mail list logo