Re: [PATCH v6 0/2] Block layer support ZAC/ZBC commands

2016-08-15 Thread Shaun Tancheff
On Tue, Aug 9, 2016 at 11:38 PM, Damien Le Moal wrote: > Shaun, > > On 8/10/16 12:58, Shaun Tancheff wrote: >> >> On Tue, Aug 9, 2016 at 3:09 AM, Damien Le Moal >> wrote: On Aug 9, 2016, at 15:47, Hannes Reinecke wrote: >> >> >> [trim] >> > Since disk type == 0 for everything that

Re: [PATCH v6 0/2] Block layer support ZAC/ZBC commands

2016-08-15 Thread Shaun Tancheff
On Mon, Aug 15, 2016 at 11:00 PM, Damien Le Moal wrote: > > Shaun, > >> On Aug 14, 2016, at 09:09, Shaun Tancheff wrote: > […] >>> No, surely not. >>> But one of the _big_ advantages for the RB tree is blkdev_discard(). >>> Without the RB tree any mkfs program will issue a 'discard' for ever

Re: [PATCH v6 0/2] Block layer support ZAC/ZBC commands

2016-08-15 Thread Damien Le Moal
Shaun, > On Aug 14, 2016, at 09:09, Shaun Tancheff wrote: […] >>> >> No, surely not. >> But one of the _big_ advantages for the RB tree is blkdev_discard(). >> Without the RB tree any mkfs program will issue a 'discard' for every >> sector. We will be able to coalesce those into one discard per

Re: [PATCH v6 0/2] Block layer support ZAC/ZBC commands

2016-08-14 Thread Shaun Tancheff
On Tue, Aug 9, 2016 at 1:47 AM, Hannes Reinecke wrote: > On 08/05/2016 10:35 PM, Shaun Tancheff wrote: >> On Tue, Aug 2, 2016 at 8:29 PM, Damien Le Moal >> wrote: >>> Hannes, Shaun, >>> >>> Let me add some more comments. >>> On Aug 2, 2016, at 23:35, Hannes Reinecke wrote: On 08/

Re: [PATCH v6 0/2] Block layer support ZAC/ZBC commands

2016-08-09 Thread Damien Le Moal
Shaun, On 8/10/16 12:58, Shaun Tancheff wrote: On Tue, Aug 9, 2016 at 3:09 AM, Damien Le Moal wrote: On Aug 9, 2016, at 15:47, Hannes Reinecke wrote: [trim] Since disk type == 0 for everything that isn't HM so I would prefer the sysfs 'zoned' file just report if the drive is HA or HM. O

Re: [PATCH v6 0/2] Block layer support ZAC/ZBC commands

2016-08-09 Thread Shaun Tancheff
On Tue, Aug 9, 2016 at 3:09 AM, Damien Le Moal wrote: >> On Aug 9, 2016, at 15:47, Hannes Reinecke wrote: [trim] >>> Since disk type == 0 for everything that isn't HM so I would prefer the >>> sysfs 'zoned' file just report if the drive is HA or HM. >>> >> Okay. So let's put in the 'zoned' attr

Re: [PATCH v6 0/2] Block layer support ZAC/ZBC commands

2016-08-09 Thread Shaun Tancheff
On Tue, Aug 9, 2016 at 1:47 AM, Hannes Reinecke wrote: > On 08/05/2016 10:35 PM, Shaun Tancheff wrote: >> On Tue, Aug 2, 2016 at 8:29 PM, Damien Le Moal >> wrote: On Aug 2, 2016, at 23:35, Hannes Reinecke wrote: On 08/01/2016 07:07 PM, Shaun Tancheff wrote: > On Mon, Aug 1, 2016 a

Re: [PATCH v6 0/2] Block layer support ZAC/ZBC commands

2016-08-09 Thread Damien Le Moal
Hannes, > On Aug 9, 2016, at 15:47, Hannes Reinecke wrote: [...] >>> >>> Can we agree on an interface ? >>> Summarizing all the discussions we had, I am all in favor of the following: >>> >>> 1) A "zone_size" sysfs attribute to indicate that a drive is zoned: >>> The already existing device typ

Re: [PATCH v6 0/2] Block layer support ZAC/ZBC commands

2016-08-08 Thread Hannes Reinecke
On 08/05/2016 10:35 PM, Shaun Tancheff wrote: > On Tue, Aug 2, 2016 at 8:29 PM, Damien Le Moal wrote: >> Hannes, Shaun, >> >> Let me add some more comments. >> >>> On Aug 2, 2016, at 23:35, Hannes Reinecke wrote: >>> >>> On 08/01/2016 07:07 PM, Shaun Tancheff wrote: On Mon, Aug 1, 2016 at 4:

Re: [PATCH v6 0/2] Block layer support ZAC/ZBC commands

2016-08-05 Thread Shaun Tancheff
On Tue, Aug 2, 2016 at 8:29 PM, Damien Le Moal wrote: > Hannes, Shaun, > > Let me add some more comments. > >> On Aug 2, 2016, at 23:35, Hannes Reinecke wrote: >> >> On 08/01/2016 07:07 PM, Shaun Tancheff wrote: >>> On Mon, Aug 1, 2016 at 4:41 AM, Christoph Hellwig wrote: Can you pleas

Re: [PATCH v6 0/2] Block layer support ZAC/ZBC commands

2016-08-02 Thread Damien Le Moal
Hannes, Shaun, Let me add some more comments. > On Aug 2, 2016, at 23:35, Hannes Reinecke wrote: > > On 08/01/2016 07:07 PM, Shaun Tancheff wrote: >> On Mon, Aug 1, 2016 at 4:41 AM, Christoph Hellwig wrote: >>> >>> Can you please integrate this with Hannes series so that it uses >>> his cache

Re: [PATCH v6 0/2] Block layer support ZAC/ZBC commands

2016-08-02 Thread Hannes Reinecke
On 08/01/2016 07:07 PM, Shaun Tancheff wrote: > On Mon, Aug 1, 2016 at 4:41 AM, Christoph Hellwig wrote: >> >> Can you please integrate this with Hannes series so that it uses >> his cache of the zone information? > > Adding Hannes and Damien to Cc. > > Christoph, > > I can make a patch the mar

Re: [PATCH v6 0/2] Block layer support ZAC/ZBC commands

2016-08-01 Thread Shaun Tancheff
On Mon, Aug 1, 2016 at 4:41 AM, Christoph Hellwig wrote: > > Can you please integrate this with Hannes series so that it uses > his cache of the zone information? Adding Hannes and Damien to Cc. Christoph, I can make a patch the marshal Hannes' RB-Tree into to a block report, that is quite simp

Re: [PATCH v6 0/2] Block layer support ZAC/ZBC commands

2016-08-01 Thread Christoph Hellwig
Can you please integrate this with Hannes series so that it uses his cache of the zone information?