Re: FRV-setup: Clarification for "source code clean-up"?

From	Julia Lawall <julia.lawall@lip6.fr>
Newsgroups	linux.kernel
Subject	Re: FRV-setup: Clarification for "source code clean-up"?
Date	2016-10-21 13:10 +0200
Message-ID	<suFS2-4IK-31@gated-at.bofh.it> (permalink)
References	(3 earlier) <suCUa-2xD-25@gated-at.bofh.it> <suE9B-3xe-69@gated-at.bofh.it> <suEjh-3AW-85@gated-at.bofh.it> <suEMh-3Mc-1@gated-at.bofh.it> <suFfj-4fQ-23@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

Markus,

If you want to do something that might be useful, you could look into the
problem of missing checks for dma mapping failure.  There is  nice slide
presenatation about the issue:

http://www.slideshare.net/SamsungOSG/shuah-khan-dmamaperror

The slides are from a few years ago, but I did a small test recently, and
the problem seems to persist.  This needs to be done slowly and carefully,
because when there is a failure, you need to figure out what to do to
clean up and what value to return.  There is no point to clean up all of
the other checkpatch errors in the file at the same time.  Just stay on
the problem at hand.

julia

Thread

[PATCH 0/6] FRV-setup: Fine-tuning for six function implementations SF Markus Elfring <elfring@users.sourceforge.net> - 2016-10-21 07:10 +0200
  [PATCH 2/6] FRV-setup: Use seq_putc() in show_cpuinfo() SF Markus Elfring <elfring@users.sourceforge.net> - 2016-10-21 07:10 +0200
  [PATCH 3/6] FRV-setup: Add some spaces for better code readability SF Markus Elfring <elfring@users.sourceforge.net> - 2016-10-21 07:10 +0200
  [PATCH 5/6] FRV-setup: Fix indentation in two lines SF Markus Elfring <elfring@users.sourceforge.net> - 2016-10-21 07:20 +0200
    Re: [PATCH 5/6] FRV-setup: Fix indentation in two lines Jiri Kosina <jikos@kernel.org> - 2016-10-21 09:30 +0200
      Re: [PATCH 5/6] FRV-setup: Fix indentation in two lines Dan Carpenter <dan.carpenter@oracle.com> - 2016-10-21 10:00 +0200
        Re: FRV-setup: Clarification for "source code clean-up"? SF Markus Elfring <elfring@users.sourceforge.net> - 2016-10-21 11:20 +0200
          Re: FRV-setup: Clarification for "source code clean-up"? Dan Carpenter <dan.carpenter@oracle.com> - 2016-10-21 11:30 +0200
            Re: FRV-setup: Clarification for "source code clean-up"? SF Markus Elfring <elfring@users.sourceforge.net> - 2016-10-21 12:00 +0200
              Re: FRV-setup: Clarification for "source code clean-up"? Geert Uytterhoeven <geert@linux-m68k.org> - 2016-10-21 12:30 +0200
              Re: FRV-setup: Clarification for "source code clean-up"? Dan Carpenter <dan.carpenter@oracle.com> - 2016-10-21 12:30 +0200
                Re: FRV-setup: Clarification for "source code clean-up"? Julia Lawall <julia.lawall@lip6.fr> - 2016-10-21 13:10 +0200
                Re: Improving software around DMA API usage? SF Markus Elfring <elfring@users.sourceforge.net> - 2016-10-21 13:40 +0200
  [PATCH 6/6] FRV-setup: Move statements for "case" to separate lines SF Markus Elfring <elfring@users.sourceforge.net> - 2016-10-21 07:20 +0200

csiph-web