lkml.org 
[lkml]   [2018]   [Jan]   [10]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
SubjectRE: [PATCH] CIFS: SMBD: work around gcc -Wmaybe-uninitialized warning
Date
> GCC versions from 4.9 to 6.3 produce a false-positive warning when dealing
> with a conditional spin_lock_irqsave():
>
> fs/cifs/smbdirect.c: In function 'smbd_recv_buf':
> include/linux/spinlock.h:260:3: warning: 'flags' may be used uninitialized in
> this function [-Wmaybe-uninitialized]
>
> This function calls some sleeping interfaces, so it is clear that it does not get
> called with interrupts disabled and there is no need to save the irq state
> before taking the spinlock. This lets us remove the variable, which makes the
> function slightly more efficient and avoids the warning.
>
> A further cleanup could do the same change for other functions in this file,
> but I did not want to take this too far for now.
>
> Fixes: ac69f66e54ca ("CIFS: SMBD: Implement function to receive data via
> RDMA receive")
> Signed-off-by: Arnd Bergmann <arnd@arndb.de>

Reviewed-by: Long Li <longli@microsoft.com>

> ---
> fs/cifs/smbdirect.c | 15 ++++++---------
> 1 file changed, 6 insertions(+), 9 deletions(-)
>
> diff --git a/fs/cifs/smbdirect.c b/fs/cifs/smbdirect.c index
> f527e22650f5..f9234ed83a60 100644
> --- a/fs/cifs/smbdirect.c
> +++ b/fs/cifs/smbdirect.c
> @@ -1862,7 +1862,6 @@ int smbd_recv_buf(struct smbd_connection *info,
> char *buf, unsigned int size)
> int to_copy, to_read, data_read, offset;
> u32 data_length, remaining_data_length, data_offset;
> int rc;
> - unsigned long flags;
>
> again:
> if (info->transport_status != SMBD_CONNECTED) { @@ -1935,15
> +1934,13 @@ int smbd_recv_buf(struct smbd_connection *info, char *buf,
> unsigned int size)
> * end of the queue
> */
> if (!queue_length)
> - spin_lock_irqsave(
> - &info-
> >reassembly_queue_lock,
> - flags);
> + spin_lock_irq(
> + &info-
> >reassembly_queue_lock);
> list_del(&response->list);
> queue_removed++;
> if (!queue_length)
> - spin_unlock_irqrestore(
> - &info-
> >reassembly_queue_lock,
> - flags);
> + spin_unlock_irq(
> + &info-
> >reassembly_queue_lock);
>
> info->count_reassembly_queue--;
> info-
> >count_dequeue_reassembly_queue++;
> @@ -1963,10 +1960,10 @@ int smbd_recv_buf(struct smbd_connection
> *info, char *buf, unsigned int size)
> to_read, data_read, offset);
> }
>
> - spin_lock_irqsave(&info->reassembly_queue_lock, flags);
> + spin_lock_irq(&info->reassembly_queue_lock);
> info->reassembly_data_length -= data_read;
> info->reassembly_queue_length -= queue_removed;
> - spin_unlock_irqrestore(&info->reassembly_queue_lock,
> flags);
> + spin_unlock_irq(&info->reassembly_queue_lock);
>
> info->first_entry_offset = offset;
> log_read(INFO, "returning to thread data_read=%d "
> --
> 2.9.0

\
 
 \ /
  Last update: 2018-01-14 23:22    [W:0.120 / U:0.068 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site