Re: [PATCH] staging: lustre: Avoid creating two variables

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 24 April 2015 at 15:46, Gujulan Elango, Hari Prasath (H.)
<hgujulan@xxxxxxxxxxx> wrote:
> This patch avoids creating two variables in the final assembly output
> i.e. a string and a pointer to the string.
>
> Signed-off-by: Hari Prasath Gujulan Elango <hgujulan@xxxxxxxxxxx>
> ---
>  drivers/staging/lustre/lustre/libcfs/linux/linux-tracefile.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/staging/lustre/lustre/libcfs/linux/linux-tracefile.c b/drivers/staging/lustre/lustre/libcfs/linux/linux-tracefile.c
> index c8e2930..8882707 100644
> --- a/drivers/staging/lustre/lustre/libcfs/linux/linux-tracefile.c
> +++ b/drivers/staging/lustre/lustre/libcfs/linux/linux-tracefile.c
> @@ -241,7 +241,7 @@ void cfs_print_to_console(struct ptldebug_header *hdr, int mask,
>                           const char *buf, int len, const char *file,
>                           const char *fn)
>  {
> -       char *prefix = "Lustre", *ptype = NULL;
> +       char prefix[] = "Lustre", *ptype = NULL;
>
>         if ((mask & D_EMERG) != 0) {
>                 prefix = dbghdr_to_err_string(hdr);
> --
> 1.9.1
> _______________________________________________
> devel mailing list
> devel@xxxxxxxxxxxxxxxxxxxxxx
> http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

Please discard this change as it breaks the build. I had build a
different copy of the source and assumed it succeeded. Sorry for the
mistake.

-- 
Regards,
G.E.Hari Prasath
_______________________________________________
devel mailing list
devel@xxxxxxxxxxxxxxxxxxxxxx
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel




[Index of Archives]     [Linux Driver Backports]     [DMA Engine]     [Linux GPIO]     [Linux SPI]     [Video for Linux]     [Linux USB Devel]     [Linux Coverity]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux