Re: [Xen-devel] [PATCH v6 15/18] tools/libxl: adjust the indentation

[Konrad Rzeszutek Wilk <konrad.wilk@xxxxxxxxxx>]

On Wed, Dec 30, 2015 at 10:29:05AM +0800, Wen Congyang wrote:
> This is just tidying up after the previous automatic renaming.

s/previous/ tools/libxl: rename remus device to checkpoint device patch/

> 
> Signed-off-by: Yang Hongyang <hongyang.yang@xxxxxxxxxxxx>
> Signed-off-by: Wen Congyang <wency@xxxxxxxxxxxxxx>
> Acked-by: Ian Campbell <ian.campbell@xxxxxxxxxx>

Reviewed-by: Konrad Rzeszutek Wilk <konrad.wilk@xxxxxxxxxx>
> ---
>  tools/libxl/libxl_checkpoint_device.c | 21 +++++++++++----------
>  tools/libxl/libxl_internal.h          | 19 +++++++++++--------
>  2 files changed, 22 insertions(+), 18 deletions(-)
> 
> diff --git a/tools/libxl/libxl_checkpoint_device.c 
> b/tools/libxl/libxl_checkpoint_device.c
> index 109cd23..226f159 100644
> --- a/tools/libxl/libxl_checkpoint_device.c
> +++ b/tools/libxl/libxl_checkpoint_device.c
> @@ -73,9 +73,9 @@ static void devices_teardown_cb(libxl__egc *egc,
>  /* checkpoint device setup and teardown */
>  
>  static libxl__checkpoint_device* checkpoint_device_init(libxl__egc *egc,
> -                                              
> libxl__checkpoint_devices_state *cds,
> -                                              libxl__device_kind kind,
> -                                              void *libxl_dev)
> +                                        libxl__checkpoint_devices_state *cds,
> +                                        libxl__device_kind kind,
> +                                        void *libxl_dev)
>  {
>      libxl__checkpoint_device *dev = NULL;
>  
> @@ -89,9 +89,10 @@ static libxl__checkpoint_device* 
> checkpoint_device_init(libxl__egc *egc,
>  }
>  
>  static void checkpoint_devices_setup(libxl__egc *egc,
> -                                libxl__checkpoint_devices_state *cds);
> +                                     libxl__checkpoint_devices_state *cds);
>  
> -void libxl__checkpoint_devices_setup(libxl__egc *egc, 
> libxl__checkpoint_devices_state *cds)
> +void libxl__checkpoint_devices_setup(libxl__egc *egc,
> +                                     libxl__checkpoint_devices_state *cds)
>  {
>      int i, rc;
>  
> @@ -137,7 +138,7 @@ out:
>  }
>  
>  static void checkpoint_devices_setup(libxl__egc *egc,
> -                                libxl__checkpoint_devices_state *cds)
> +                                     libxl__checkpoint_devices_state *cds)
>  {
>      int i, rc;
>  
> @@ -285,12 +286,12 @@ static void devices_checkpoint_cb(libxl__egc *egc,
>  
>  /* API implementations */
>  
> -#define define_checkpoint_api(api)                                \
> -void libxl__checkpoint_devices_##api(libxl__egc *egc,                        
> \
> -                                libxl__checkpoint_devices_state *cds)        
> \
> +#define define_checkpoint_api(api)                                      \
> +void libxl__checkpoint_devices_##api(libxl__egc *egc,                   \
> +                                libxl__checkpoint_devices_state *cds)   \
>  {                                                                       \
>      int i;                                                              \
> -    libxl__checkpoint_device *dev;                                           
> \
> +    libxl__checkpoint_device *dev;                                      \
>                                                                          \
>      STATE_AO_GC(cds->ao);                                               \
>                                                                          \
> diff --git a/tools/libxl/libxl_internal.h b/tools/libxl/libxl_internal.h
> index 7f80ec5..5b99d6e 100644
> --- a/tools/libxl/libxl_internal.h
> +++ b/tools/libxl/libxl_internal.h
> @@ -2818,7 +2818,8 @@ typedef struct libxl__save_helper_state {
>   * Each device type needs to implement the interfaces specified in
>   * the libxl__checkpoint_device_instance_ops if it wishes to support Remus.
>   *
> - * The high-level control flow through the checkpoint device layer is shown 
> below:
> + * The high-level control flow through the checkpoint device layer is shown
> + * below:
>   *
>   * xl remus
>   *  |->  libxl_domain_remus_start
> @@ -2879,7 +2880,8 @@ int 
> init_subkind_drbd_disk(libxl__checkpoint_devices_state *cds);
>  void cleanup_subkind_drbd_disk(libxl__checkpoint_devices_state *cds);
>  
>  typedef void libxl__checkpoint_callback(libxl__egc *,
> -                                   libxl__checkpoint_devices_state *, int 
> rc);
> +                                        libxl__checkpoint_devices_state *,
> +                                        int rc);
>  
>  /*
>   * State associated with a checkpoint invocation, including parameters
> @@ -2887,7 +2889,7 @@ typedef void libxl__checkpoint_callback(libxl__egc *,
>   * save/restore machinery.
>   */
>  struct libxl__checkpoint_devices_state {
> -    /*---- must be set by caller of 
> libxl__checkpoint_device_(setup|teardown) ----*/
> +    /*-- must be set by caller of libxl__checkpoint_device_(setup|teardown) 
> --*/
>  
>      libxl__ao *ao;
>      uint32_t domid;
> @@ -2900,7 +2902,8 @@ struct libxl__checkpoint_devices_state {
>      /*
>       * this array is allocated before setup the checkpoint devices by the
>       * checkpoint abstract layer.
> -     * devs may be NULL, means there's no checkpoint devices that has been 
> set up.
> +     * devs may be NULL, means there's no checkpoint devices that has been
> +     * set up.
>       * the size of this array is 'num_devices', which is the total number
>       * of libxl nic devices and disk devices(num_nics + num_disks).
>       */
> @@ -2962,13 +2965,13 @@ struct libxl__checkpoint_device {
>  _hidden void libxl__checkpoint_devices_setup(libxl__egc *egc,
>                                          libxl__checkpoint_devices_state 
> *cds);
>  _hidden void libxl__checkpoint_devices_teardown(libxl__egc *egc,
> -                                           libxl__checkpoint_devices_state 
> *cds);
> +                                        libxl__checkpoint_devices_state 
> *cds);
>  _hidden void libxl__checkpoint_devices_postsuspend(libxl__egc *egc,
> -                                              
> libxl__checkpoint_devices_state *cds);
> +                                        libxl__checkpoint_devices_state 
> *cds);
>  _hidden void libxl__checkpoint_devices_preresume(libxl__egc *egc,
> -                                            libxl__checkpoint_devices_state 
> *cds);
> +                                        libxl__checkpoint_devices_state 
> *cds);
>  _hidden void libxl__checkpoint_devices_commit(libxl__egc *egc,
> -                                         libxl__checkpoint_devices_state 
> *cds);
> +                                        libxl__checkpoint_devices_state 
> *cds);
>  _hidden int libxl__netbuffer_enabled(libxl__gc *gc);
>  
>  /*----- Legacy conversion helper -----*/
> -- 
> 2.5.0
> 
> 
> 
> 
> _______________________________________________
> Xen-devel mailing list
> Xen-devel@xxxxxxxxxxxxx
> http://lists.xen.org/xen-devel

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel