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

2016-08-16 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: >>

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

2016-08-16 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

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

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

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

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,

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

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

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.

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. >>> >>

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'

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

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

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

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

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

2016-08-09 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

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

2016-08-09 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

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

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

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

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

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

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

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

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

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?

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?

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

2016-07-29 Thread Shaun Tancheff
Hi Jens, This series is based on linus' current tip after the merge of 'for-4.8/core' As Host Aware drives are becoming available we would like to be able to make use of such drives. This series is also intended to be suitable for use by Host Managed drives. ZAC/ZBC drives add new commands for

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

2016-07-29 Thread Shaun Tancheff
Hi Jens, This series is based on linus' current tip after the merge of 'for-4.8/core' As Host Aware drives are becoming available we would like to be able to make use of such drives. This series is also intended to be suitable for use by Host Managed drives. ZAC/ZBC drives add new commands for