linux-kernel - linux-next: build failure after merge of the v4l-dvb tree

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [day] [month] [year] [list]

Date:	Fri, 19 Jun 2015 14:47:26 +1000
From:	Michael Ellerman <mpe@...erman.id.au>
To:	Mauro Carvalho Chehab <mchehab@...radead.org>
Cc:	linux-next@...r.kernel.org,
	Linux Kernel Mailing List <linux-kernel@...r.kernel.org>
Subject: linux-next: build failure after merge of the v4l-dvb tree

Hi all,

After merging the v4l-dvb tree, today's linux-next build (x86_64 allmodconfig)
failed like this:

  drivers/media/platform/sti/bdisp/bdisp-debug.c: In function 'bdisp_dbg_perf':
  drivers/media/platform/sti/bdisp/bdisp-debug.c:600:50: error: expected ';' before ')' token
     min_fps = SECOND / (s32)bdisp->dbg.min_duration);
                                                    ^
  drivers/media/platform/sti/bdisp/bdisp-debug.c:600:50: error: expected statement before ')' token


Caused by commit fdf11845cbc1 "bdisp-debug: don't try to divide by s64":

  @@ -585,16 +587,27 @@ static int bdisp_dbg_perf(struct seq_file *s, void *data)
          }

          avg_time_us = bdisp->dbg.tot_duration;
  -       do_div(avg_time_us, request->nb_req);
  -
  -       avg_fps = 1000000;
  -       min_fps = 1000000;
  -       max_fps = 1000000;
  -       last_fps = 1000000;
  -       do_div(avg_fps, avg_time_us);
  -       do_div(min_fps, bdisp->dbg.min_duration);
  -       do_div(max_fps, bdisp->dbg.max_duration);
  -       do_div(last_fps, bdisp->dbg.last_duration);
  +       div64_s64(avg_time_us, request->nb_req);
  +
  +       if (avg_time_us > SECOND)
  +               avg_fps = 0;
  +       else
  +               avg_fps = SECOND / (s32)avg_time_us;
  +
  +       if (bdisp->dbg.min_duration > SECOND)
  +               min_fps = 0;
  +       else
  +               min_fps = SECOND / (s32)bdisp->dbg.min_duration);
                                                                 ^

I fixed it up for today.

cheers



--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/